depca: fix leaks in depca_module_init()
authorKulikov Vasiliy <segooon@gmail.com>
Mon, 12 Jul 2010 04:52:33 +0000 (04:52 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 13 Jul 2010 03:21:40 +0000 (20:21 -0700)
Since some of xxx_register_driver() can return error we must unregister
already registered drivers.

Signed-off-by: Kulikov Vasiliy <segooon@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/depca.c

index bf66e9b3b19ed1c461ceb124d81f24b110171e54..44c0694c1f4eb7e9614910970c36d288ce63c6fa 100644 (file)
@@ -2061,18 +2061,35 @@ static int depca_ioctl(struct net_device *dev, struct ifreq *rq, int cmd)
 
 static int __init depca_module_init (void)
 {
-        int err = 0;
+       int err = 0;
 
 #ifdef CONFIG_MCA
-        err = mca_register_driver (&depca_mca_driver);
+       err = mca_register_driver(&depca_mca_driver);
+       if (err)
+               goto err;
 #endif
 #ifdef CONFIG_EISA
-        err |= eisa_driver_register (&depca_eisa_driver);
+       err = eisa_driver_register(&depca_eisa_driver);
+       if (err)
+               goto err_mca;
 #endif
-       err |= platform_driver_register (&depca_isa_driver);
-       depca_platform_probe ();
+       err = platform_driver_register(&depca_isa_driver);
+       if (err)
+               goto err_eisa;
 
-        return err;
+       depca_platform_probe();
+       return 0;
+
+err_eisa:
+#ifdef CONFIG_EISA
+       eisa_driver_unregister(&depca_eisa_driver);
+err_mca:
+#endif
+#ifdef CONFIG_MCA
+       mca_unregister_driver(&depca_mca_driver);
+err:
+#endif
+       return err;
 }
 
 static void __exit depca_module_exit (void)