diff options
| author | Jack Morgenstein | 2015-10-08 16:14:02 +0200 |
|---|---|---|
| committer | David S. Miller | 2015-10-09 16:27:51 +0200 |
| commit | a5b3c56ef792009f6aff54ba93bd86ceca3e5bfc (patch) | |
| tree | 24c9fb39c47af96013e0f6f553cea9819f2f5125 /drivers/net/ethernet/mellanox/mlx4/qp.c | |
| parent | net/mlx4_en: Add steering rules after RSS creation (diff) | |
| download | kernel-qcow2-linux-a5b3c56ef792009f6aff54ba93bd86ceca3e5bfc.tar.gz kernel-qcow2-linux-a5b3c56ef792009f6aff54ba93bd86ceca3e5bfc.tar.xz kernel-qcow2-linux-a5b3c56ef792009f6aff54ba93bd86ceca3e5bfc.zip | |
net/mlx4_core: Fix mailbox leak in error flow when performing update qp
The procedure mlx4_update_qp leaks mailboxes in its error-flow, fix that.
Signed-off-by: Jack Morgenstein <jackm@dev.mellanox.co.il>
Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/mellanox/mlx4/qp.c')
| -rw-r--r-- | drivers/net/ethernet/mellanox/mlx4/qp.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx4/qp.c b/drivers/net/ethernet/mellanox/mlx4/qp.c index 20268634a9ab..3311f35d08e0 100644 --- a/drivers/net/ethernet/mellanox/mlx4/qp.c +++ b/drivers/net/ethernet/mellanox/mlx4/qp.c @@ -422,15 +422,15 @@ int mlx4_update_qp(struct mlx4_dev *dev, u32 qpn, u64 qp_mask = 0; int err = 0; + if (!attr || (attr & ~MLX4_UPDATE_QP_SUPPORTED_ATTRS)) + return -EINVAL; + mailbox = mlx4_alloc_cmd_mailbox(dev); if (IS_ERR(mailbox)) return PTR_ERR(mailbox); cmd = (struct mlx4_update_qp_context *)mailbox->buf; - if (!attr || (attr & ~MLX4_UPDATE_QP_SUPPORTED_ATTRS)) - return -EINVAL; - if (attr & MLX4_UPDATE_QP_SMAC) { pri_addr_path_mask |= 1ULL << MLX4_UPD_QP_PATH_MASK_MAC_INDEX; cmd->qp_context.pri_path.grh_mylmc = params->smac_index; |