net: hns3: fix return of random stack value
authorGustavo A. R. Silva <gustavoars@kernel.org>
Wed, 10 Feb 2021 19:28:41 +0000 (13:28 -0600)
committerDavid S. Miller <davem@davemloft.net>
Thu, 11 Feb 2021 22:22:13 +0000 (14:22 -0800)
Currently, a random stack value is being returned because variable
_ret_ is not properly initialized. This variable is actually not
used anymore and it should be removed.

Fix this by removing all instances of variable ret and return 0.

Fixes: 64749c9c38a9 ("net: hns3: remove redundant return value of hns3_uninit_all_ring()")
Addresses-Coverity-ID: 1501700 ("Uninitialized scalar variable")
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/hisilicon/hns3/hns3_enet.c

index 9565b79..bf4302a 100644 (file)
@@ -4640,7 +4640,6 @@ static int hns3_reset_notify_uninit_enet(struct hnae3_handle *handle)
 {
        struct net_device *netdev = handle->kinfo.netdev;
        struct hns3_nic_priv *priv = netdev_priv(netdev);
-       int ret;
 
        if (!test_and_clear_bit(HNS3_NIC_STATE_INITED, &priv->state)) {
                netdev_warn(netdev, "already uninitialized\n");
@@ -4662,7 +4661,7 @@ static int hns3_reset_notify_uninit_enet(struct hnae3_handle *handle)
 
        hns3_put_ring_config(priv);
 
-       return ret;
+       return 0;
 }
 
 static int hns3_reset_notify(struct hnae3_handle *handle,