diff options
author | Roland Dreier <rolandd@cisco.com> | 2007-05-19 11:51:58 -0400 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2007-05-19 11:51:58 -0400 |
commit | 59b0ed121297b57abb2352bdc8313959e7cb5635 (patch) | |
tree | 560ade0adc4a3ccf88336ee88fea0f079868862a /drivers/infiniband/hw/mlx4 | |
parent | 23c15c21d34a4b4b4d7b9a95ce498991c5339c77 (diff) |
IB/mlx4: Fix check of opcode in mlx4_ib_post_send()
wr->opcode is invalid if it's >= ARRAY_SIZE(mlx4_ib_opcode), not just
strictly >.
This was spotted by the Coverity checker (CID 1643).
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers/infiniband/hw/mlx4')
-rw-r--r-- | drivers/infiniband/hw/mlx4/qp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/mlx4/qp.c b/drivers/infiniband/hw/mlx4/qp.c index bd28af5753d1..5706f988e2ec 100644 --- a/drivers/infiniband/hw/mlx4/qp.c +++ b/drivers/infiniband/hw/mlx4/qp.c | |||
@@ -1238,7 +1238,7 @@ int mlx4_ib_post_send(struct ib_qp *ibqp, struct ib_send_wr *wr, | |||
1238 | */ | 1238 | */ |
1239 | wmb(); | 1239 | wmb(); |
1240 | 1240 | ||
1241 | if (wr->opcode < 0 || wr->opcode > ARRAY_SIZE(mlx4_ib_opcode)) { | 1241 | if (wr->opcode < 0 || wr->opcode >= ARRAY_SIZE(mlx4_ib_opcode)) { |
1242 | err = -EINVAL; | 1242 | err = -EINVAL; |
1243 | goto out; | 1243 | goto out; |
1244 | } | 1244 | } |