net: caif: check return value of alloc_netdev

I don't know if dev can actually be NULL here, but the test should be
above alloc_netdev(), to avoid leaking the struct net_device in case
dev is actually NULL. And of course the return value from alloc_netdev
should be tested.

Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 5 insertions, 2 deletions

diff --git a/drivers/net/caif/caif_spi.c b/drivers/net/caif/caif_spi.c
index de3962014af7..4721948a92f6 100644
--- a/drivers/net/caif/caif_spi.c
+++ b/drivers/net/caif/caif_spi.c
@@ -730,11 +730,14 @@ int cfspi_spi_probe(struct platform_device *pdev)
         int res;
         dev = (struct cfspi_dev *)pdev->dev.platform_data;
 
-        ndev = alloc_netdev(sizeof(struct cfspi), "cfspi%d",
-                            NET_NAME_UNKNOWN, cfspi_setup);
         if (!dev)
                 return -ENODEV;
 
+        ndev = alloc_netdev(sizeof(struct cfspi), "cfspi%d",
+                            NET_NAME_UNKNOWN, cfspi_setup);
+        if (!ndev)
+                return -ENOMEM;
+
         cfspi = netdev_priv(ndev);
         netif_stop_queue(ndev);
         cfspi->ndev = ndev;