liquidio: lowmem: init allocated memory to 0
authorRick Farrington <ricardo.farrington@cavium.com>
Tue, 18 Jul 2017 00:51:37 +0000 (17:51 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 19 Jul 2017 20:24:46 +0000 (13:24 -0700)
For defensive programming, zero the allocated block 'oct->droq[0]' in
octeon_setup_output_queues() and 'oct->instr_queue[0]' in
octeon_setup_instr_queues().

Signed-off-by: Rick Farrington <ricardo.farrington@cavium.com>
Signed-off-by: Satanand Burla <satananda.burla@cavium.com>
Signed-off-by: Raghu Vatsavayi <raghu.vatsavayi@cavium.com>
Signed-off-by: Felix Manlunas <felix.manlunas@cavium.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/cavium/liquidio/octeon_device.c

index 623e28ca736e10d612c293992dd959cc0a34f891..f10014f7ae884b7236dbc4a58866d2ebc7a75cf6 100644 (file)
@@ -876,11 +876,11 @@ int octeon_setup_instr_queues(struct octeon_device *oct)
 
        oct->num_iqs = 0;
 
-       oct->instr_queue[0] = vmalloc_node(sizeof(*oct->instr_queue[0]),
+       oct->instr_queue[0] = vzalloc_node(sizeof(*oct->instr_queue[0]),
                                numa_node);
        if (!oct->instr_queue[0])
                oct->instr_queue[0] =
-                       vmalloc(sizeof(struct octeon_instr_queue));
+                       vzalloc(sizeof(struct octeon_instr_queue));
        if (!oct->instr_queue[0])
                return 1;
        memset(oct->instr_queue[0], 0, sizeof(struct octeon_instr_queue));
@@ -923,9 +923,9 @@ int octeon_setup_output_queues(struct octeon_device *oct)
                desc_size = CFG_GET_DEF_RX_BUF_SIZE(CHIP_CONF(oct, cn23xx_vf));
        }
        oct->num_oqs = 0;
-       oct->droq[0] = vmalloc_node(sizeof(*oct->droq[0]), numa_node);
+       oct->droq[0] = vzalloc_node(sizeof(*oct->droq[0]), numa_node);
        if (!oct->droq[0])
-               oct->droq[0] = vmalloc(sizeof(*oct->droq[0]));
+               oct->droq[0] = vzalloc(sizeof(*oct->droq[0]));
        if (!oct->droq[0])
                return 1;