Uninitialized timer in ip_mc_down

Hi,

if you do the following on a downed interface on 2.6, you get two "uninitialized timer" messages from the kernel, along with a stack trace.

	ip addr add 1.2.3.4 dev eth1
	ip addr del 1.2.3.4 dev eth1

The reason is that inetdev_create calls ip_mc_up if and only if the device is up. However, inetdev_destroy calls ip_mc_destroy_dev unconditionally, so that del_timer gets called for the two multicast timers which are uninitialized.

The attached patch seems to fix this. An alternative patch would be to add a check to ip_mc_down to see whether the timers are actually active before deleting them.

Olaf

Olaf Kirch     |  Stop wasting entropy - start using predictable
[email protected]   |  tempfile names today!
---------------+ 

@@ -183,7 +183,10 @@
 
 	in_dev->dead = 1;
 
-	ip_mc_destroy_dev(in_dev);
+	/* If the device isn't up, the multicast info isn't
+	 * initialized. */
+	if (in_dev->dev->flags & IFF_UP)
+		ip_mc_destroy_dev(in_dev);
 
 	while ((ifa = in_dev->ifa_list) != NULL) {
 		inet_del_ifa(in_dev, &in_dev->ifa_list, 0);