af_packet: fix crash on initialization failure
authorBruce Richardson <bruce.richardson@intel.com>
Tue, 16 Dec 2014 16:30:22 +0000 (16:30 +0000)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Wed, 17 Dec 2014 00:04:06 +0000 (01:04 +0100)
The cleanup code on error checks for *internals being NULL only after
using the pointer to perform other cleanup. Fix this by moving the
clean-up based on the pointer inside the check for NULL.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
lib/librte_pmd_af_packet/rte_eth_af_packet.c

index d0fb3eb..ad7242c 100644 (file)
@@ -676,14 +676,15 @@ error:
                rte_free(data);
        if (pci_dev)
                rte_free(pci_dev);
                rte_free(data);
        if (pci_dev)
                rte_free(pci_dev);
-       for (q = 0; q < nb_queues; q++) {
-               if ((*internals)->rx_queue[q].rd)
-                       rte_free((*internals)->rx_queue[q].rd);
-               if ((*internals)->tx_queue[q].rd)
-                       rte_free((*internals)->tx_queue[q].rd);
-       }
-       if (*internals)
+       if (*internals) {
+               for (q = 0; q < nb_queues; q++) {
+                       if ((*internals)->rx_queue[q].rd)
+                               rte_free((*internals)->rx_queue[q].rd);
+                       if ((*internals)->tx_queue[q].rd)
+                               rte_free((*internals)->tx_queue[q].rd);
+               }
                rte_free(*internals);
                rte_free(*internals);
+       }
        return -1;
 }
 
        return -1;
 }