diff options
| author |   Raghu Vatsavayi <rvatsavayi@caviumnetworks.com> | 2016-06-14 16:54:43 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2016-06-15 21:44:31 -0700 |
| commit | ddc173a688a1ffef8b2a6547faaf68940d4dae0d (patch) | |
| tree | 2d8c35a044f064f4dd6f2b3953193c1685caeff8 /drivers/net/ethernet/cavium/liquidio/octeon_nic.c | |
| parent | ila: Fix checksum neutral mapping (diff) | |
| download | linux-dev-ddc173a688a1ffef8b2a6547faaf68940d4dae0d.tar.xz linux-dev-ddc173a688a1ffef8b2a6547faaf68940d4dae0d.zip | |
liquidio: Avoid double free during soft command
This patch is to resolve the double free issue by checking proper return
values from soft command.
Signed-off-by: Derek Chickles <derek.chickles@caviumnetworks.com>
Signed-off-by: Satanand Burla <satananda.burla@caviumnetworks.com>
Signed-off-by: Felix Manlunas <felix.manlunas@caviumnetworks.com>
Signed-off-by: Raghu Vatsavayi <raghu.vatsavayi@caviumnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/cavium/liquidio/octeon_nic.c')
| -rw-r--r-- | drivers/net/ethernet/cavium/liquidio/octeon_nic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/cavium/liquidio/octeon_nic.c b/drivers/net/ethernet/cavium/liquidio/octeon_nic.c index 1a0191549cb3..aacabe4c5443 100644 --- a/drivers/net/ethernet/cavium/liquidio/octeon_nic.c +++ b/drivers/net/ethernet/cavium/liquidio/octeon_nic.c @@ -178,7 +178,7 @@ octnet_send_nic_ctrl_pkt(struct octeon_device *oct, } retval = octeon_send_soft_command(oct, sc); - if (retval) { + if (retval == IQ_SEND_FAILED) { octeon_free_soft_command(oct, sc); dev_err(&oct->pci_dev->dev, "%s soft command send failed status: %x\n", __func__, retval); |