IB/mlx4: Fix check of opcode in mlx4_ib_post_send()
authorRoland Dreier <rolandd@cisco.com>
Sat, 19 May 2007 15:51:58 +0000 (08:51 -0700)
committerRoland Dreier <rolandd@cisco.com>
Sat, 19 May 2007 15:51:58 +0000 (08:51 -0700)
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>
drivers/infiniband/hw/mlx4/qp.c

index bd28af5..5706f98 100644 (file)
@@ -1238,7 +1238,7 @@ int mlx4_ib_post_send(struct ib_qp *ibqp, struct ib_send_wr *wr,
                 */
                wmb();
 
-               if (wr->opcode < 0 || wr->opcode > ARRAY_SIZE(mlx4_ib_opcode)) {
+               if (wr->opcode < 0 || wr->opcode >= ARRAY_SIZE(mlx4_ib_opcode)) {
                        err = -EINVAL;
                        goto out;
                }