ieee1394: fix oops on "modprobe -r ohci1394" after network class_device conversion
authorStefan Richter <stefanr@s5r6.in-berlin.de>
Tue, 20 Mar 2007 21:43:22 +0000 (22:43 +0100)
committerStefan Richter <stefanr@s5r6.in-berlin.de>
Fri, 23 Mar 2007 09:55:25 +0000 (10:55 +0100)
The networking subsystem has been converted from class_device to device
but ieee1394 hasn't.  This results in a 100% reproducible NULL pointer
dereference if the ohci1394 driver module is unloaded while the eth1394
module is still loaded.
http://lkml.org/lkml/2006/11/16/147
http://lkml.org/lkml/2007/3/14/4

This is a regression in 2.6.21-rc1.

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Tested-by: Ismail Dönmez <ismail@pardus.org.tr>
drivers/ieee1394/eth1394.c

index a953135219851292b3803f3b3799ac00be810ac0..03e44b337eb018c488073aa015c6dd63d481ed2e 100644 (file)
@@ -584,7 +584,10 @@ static void ether1394_add_host (struct hpsb_host *host)
         }
 
        SET_MODULE_OWNER(dev);
+#if 0
+       /* FIXME - Is this the correct parent device anyway? */
        SET_NETDEV_DEV(dev, &host->device);
+#endif
 
        priv = netdev_priv(dev);