diff options
| author |   Eyal Perry <eyalpe@mellanox.com> | 2014-05-14 12:15:17 +0300 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2014-05-14 15:40:34 -0400 |
| commit | 7572038409d0288ff5ba9c77a053ddec6791a43c (patch) | |
| tree | dddddda6e3f4e417065e91febffa04f16b9186cd /drivers/net/ethernet/mellanox/mlx4/mcg.c | |
| parent | net/mlx4_en: Using positive error value for unsigned (diff) | |
| download | linux-dev-7572038409d0288ff5ba9c77a053ddec6791a43c.tar.xz linux-dev-7572038409d0288ff5ba9c77a053ddec6791a43c.zip | |
net/mlx4_core: Fix inaccurate return value of mlx4_flow_attach()
Adopt the "info: why not propagate 'ret' from parse_trans_rule()..."
suggestion made by the smatch semantic checker on:
drivers/net/ethernet/mellanox/mlx4/mcg.c:867 mlx4_flow_attach()
Signed-off-by: Eyal Perry <eyalpe@mellanox.com>
Signed-off-by: Amir Vadai <amirv@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/mellanox/mlx4/mcg.c')
| -rw-r--r-- | drivers/net/ethernet/mellanox/mlx4/mcg.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx4/mcg.c b/drivers/net/ethernet/mellanox/mlx4/mcg.c index 7c6eba622186..4c36def8e10f 100644 --- a/drivers/net/ethernet/mellanox/mlx4/mcg.c +++ b/drivers/net/ethernet/mellanox/mlx4/mcg.c @@ -897,7 +897,7 @@ int mlx4_flow_attach(struct mlx4_dev *dev, ret = parse_trans_rule(dev, cur, mailbox->buf + size); if (ret < 0) { mlx4_free_cmd_mailbox(dev, mailbox); - return -EINVAL; + return ret; } size += ret; } |