]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
net: thunderx: fix double free error
authorAleksey Makarov <aleksey.makarov@cavium.com>
Thu, 9 Nov 2017 11:58:57 +0000 (14:58 +0300)
committerDavid S. Miller <davem@davemloft.net>
Sat, 11 Nov 2017 10:22:34 +0000 (19:22 +0900)
commit3d67a5075295982fb055be6a5d5c78b0e0be3591
treee7e3d3d84d5eec74682d804ed3e0adb881aa5bf4
parent86dabda426ac33f4782854ef71f0c9adafd24134
net: thunderx: fix double free error

This patch fixes an error in memory allocation/freeing in
ThunderX PF driver.

I moved the allocation to the probe() function and made it managed.

>From the Colin's email:

While running static analysis on linux-next with CoverityScan I found 3
double free errors in the Cavium thunder driver.

The issue occurs on the err_disable_device: label of function nic_probe
when nic_free_lmacmem(nic) is called and a double free occurs on
nic->duplex, nic->link and nic->speed.  This occurs when nic_init_hw()
fails:

        /* Initialize hardware */
        err = nic_init_hw(nic);
        if (err)
                goto err_release_regions;

nic_init_hw() calls nic_get_hw_info() and this calls nic_free_lmacmem()
if any of the allocations fail. This free'ing occurs again by the call
to nic_free_lmacmem() on the err_release_regions exit path in nic_probe().

Reported-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Aleksey Makarov <aleksey.makarov@cavium.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/cavium/thunder/nic_main.c