From: Pavel Belous <pavel.bel...@aquantia.com>

We should check for a null pointer for aq_nic_ndev_alloc
instead netdev_priv.

Signed-off-by: Pavel Belous <pavel.bel...@aquantia.com>
---
 drivers/net/ethernet/aquantia/atlantic/aq_nic.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/net/ethernet/aquantia/atlantic/aq_nic.c 
b/drivers/net/ethernet/aquantia/atlantic/aq_nic.c
index 4cf633c..e50fba2 100644
--- a/drivers/net/ethernet/aquantia/atlantic/aq_nic.c
+++ b/drivers/net/ethernet/aquantia/atlantic/aq_nic.c
@@ -202,12 +202,13 @@ struct aq_nic_s *aq_nic_alloc_cold(const struct 
net_device_ops *ndev_ops,
        int err = 0;
 
        ndev = aq_nic_ndev_alloc();
-       self = netdev_priv(ndev);
-       if (!self) {
-               err = -EINVAL;
+       if (!ndev) {
+               err = -ENOMEM;
                goto err_exit;
        }
 
+       self = netdev_priv(ndev);
+
        ndev->netdev_ops = ndev_ops;
        ndev->ethtool_ops = et_ops;
 
-- 
2.7.4

Reply via email to