Commit 00587f2f authored Jul 17, 2017 by Rick Farrington Committed by David S. Miller Jul 19, 2017

liquidio: lowmem: init allocated memory to 0

Fix GPF in octeon_init_droq(); zero the allocated block 'recv_buf_list'.
This prevents a GPF trying to access an invalid 'recv_buf_list[i]' entry
in octeon_droq_destroy_ring_buffers() if init didn't alloc all entries.

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>

parent 741912c5

drivers/net/ethernet/cavium/liquidio/octeon_droq.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -275,12 +275,12 @@ int octeon_init_droq(struct octeon_device *oct,
		droq->max_count);

		droq->recv_buf_list = (struct octeon_recv_buffer *)
		vmalloc_node(droq->max_count *
		vzalloc_node(droq->max_count *
		OCT_DROQ_RECVBUF_SIZE,
		numa_node);
		if (!droq->recv_buf_list)
		droq->recv_buf_list = (struct octeon_recv_buffer *)
		vmalloc(droq->max_count *
		vzalloc(droq->max_count *
		OCT_DROQ_RECVBUF_SIZE);
		if (!droq->recv_buf_list) {
		dev_err(&oct->pci_dev->dev, "Output queue recv buf list alloc failed\n");