From: "shaojie.dong" <shaojie.d...@isrc.iscas.ac.cn>

Function register_netdev() can fail, so we should check it's return value

Signed-off-by: shaojie.dong <shaojie.d...@isrc.iscas.ac.cn>
---
 drivers/staging/rtl8712/hal_init.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/rtl8712/hal_init.c 
b/drivers/staging/rtl8712/hal_init.c
index 715f1fe8b..38a3e3d44 100644
--- a/drivers/staging/rtl8712/hal_init.c
+++ b/drivers/staging/rtl8712/hal_init.c
@@ -45,7 +45,10 @@ static void rtl871x_load_fw_cb(const struct firmware 
*firmware, void *context)
        }
        adapter->fw = firmware;
        /* firmware available - start netdev */
-       register_netdev(adapter->pnetdev);
+       if (register_netdev(adapter->pnetdev) != 0) {
+               netdev_err(adapter->pnetdev, "register_netdev() failed\n");
+               free_netdev(adapter->pnetdev);
+       }
        complete(&adapter->rtl8712_fw_ready);
 }
 
-- 
2.17.1

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to