ibmvnic: Clean up resources on probe failure
authorNathan Fontenot <nfont@linux.vnet.ibm.com>
Tue, 8 Aug 2017 19:28:45 +0000 (14:28 -0500)
committerDavid S. Miller <davem@davemloft.net>
Thu, 10 Aug 2017 05:30:52 +0000 (22:30 -0700)
Ensure that any resources allocated during probe are released if the
probe of the driver fails.

Signed-off-by: Nathan Fontenot <nfont@linux.vnet.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/ibm/ibmvnic.c

index 32c1166527554dfd17a37e7c6658d03918d4c79a..cf3cf921eb35baf111979ae93d4ade7906cf497d 100644 (file)
@@ -3841,31 +3841,35 @@ static int ibmvnic_probe(struct vio_dev *dev, const struct vio_device_id *id)
 
        do {
                rc = ibmvnic_init(adapter);
-               if (rc && rc != EAGAIN) {
-                       free_netdev(netdev);
-                       return rc;
-               }
+               if (rc && rc != EAGAIN)
+                       goto ibmvnic_init_fail;
        } while (rc == EAGAIN);
 
        netdev->mtu = adapter->req_mtu - ETH_HLEN;
 
        rc = device_create_file(&dev->dev, &dev_attr_failover);
-       if (rc) {
-               free_netdev(netdev);
-               return rc;
-       }
+       if (rc)
+               goto ibmvnic_init_fail;
 
        rc = register_netdev(netdev);
        if (rc) {
                dev_err(&dev->dev, "failed to register netdev rc=%d\n", rc);
-               device_remove_file(&dev->dev, &dev_attr_failover);
-               free_netdev(netdev);
-               return rc;
+               goto ibmvnic_register_fail;
        }
        dev_info(&dev->dev, "ibmvnic registered\n");
 
        adapter->state = VNIC_PROBED;
        return 0;
+
+ibmvnic_register_fail:
+       device_remove_file(&dev->dev, &dev_attr_failover);
+
+ibmvnic_init_fail:
+       release_sub_crqs(adapter);
+       release_crq_queue(adapter);
+       free_netdev(netdev);
+
+       return rc;
 }
 
 static int ibmvnic_remove(struct vio_dev *dev)