diff options
| author | Raghu Vatsavayi | 2016-06-15 01:54:43 +0200 |
|---|---|---|
| committer | David S. Miller | 2016-06-16 06:44:31 +0200 |
| commit | ddc173a688a1ffef8b2a6547faaf68940d4dae0d (patch) | |
| tree | 2d8c35a044f064f4dd6f2b3953193c1685caeff8 /drivers/net/ethernet/cavium/liquidio/lio_ethtool.c | |
| parent | ila: Fix checksum neutral mapping (diff) | |
| download | kernel-qcow2-linux-ddc173a688a1ffef8b2a6547faaf68940d4dae0d.tar.gz kernel-qcow2-linux-ddc173a688a1ffef8b2a6547faaf68940d4dae0d.tar.xz kernel-qcow2-linux-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/lio_ethtool.c')
| -rw-r--r-- | drivers/net/ethernet/cavium/liquidio/lio_ethtool.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/cavium/liquidio/lio_ethtool.c b/drivers/net/ethernet/cavium/liquidio/lio_ethtool.c index 245c063ed4db..1096cdb36512 100644 --- a/drivers/net/ethernet/cavium/liquidio/lio_ethtool.c +++ b/drivers/net/ethernet/cavium/liquidio/lio_ethtool.c @@ -317,7 +317,7 @@ octnet_mdio45_access(struct lio *lio, int op, int loc, int *value) retval = octeon_send_soft_command(oct_dev, sc); - if (retval) { + if (retval == IQ_SEND_FAILED) { dev_err(&oct_dev->pci_dev->dev, "octnet_mdio45_access instruction failed status: %x\n", retval); @@ -722,7 +722,7 @@ static int octnet_set_intrmod_cfg(void *oct, struct oct_intrmod_cfg *intr_cfg) sc->wait_time = 1000; retval = octeon_send_soft_command(oct_dev, sc); - if (retval) { + if (retval == IQ_SEND_FAILED) { octeon_free_soft_command(oct_dev, sc); return -EINVAL; } |