RDMA/rxe: Set wqe->status correctly if an unexpected response is received
authorBart Van Assche <bart.vanassche@wdc.com>
Tue, 26 Jun 2018 15:39:36 +0000 (08:39 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 9 Sep 2018 18:01:20 +0000 (20:01 +0200)
commit 61b717d041b1976530f68f8b539b2e3a7dd8e39c upstream.

Every function that returns COMPST_ERROR must set wqe->status to another
value than IB_WC_SUCCESS before returning COMPST_ERROR. Fix the only code
path for which this is not yet the case.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
Cc: <stable@vger.kernel.org>
Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/infiniband/sw/rxe/rxe_comp.c

index 6c5e29db88e360b0188d4c3bf3ee74214afc3c79..df15b6d7b645e2afaa730d100c9e32873a1f9232 100644 (file)
@@ -273,6 +273,7 @@ static inline enum comp_state check_ack(struct rxe_qp *qp,
        case IB_OPCODE_RC_RDMA_READ_RESPONSE_MIDDLE:
                if (wqe->wr.opcode != IB_WR_RDMA_READ &&
                    wqe->wr.opcode != IB_WR_RDMA_READ_WITH_INV) {
+                       wqe->status = IB_WC_FATAL_ERR;
                        return COMPST_ERROR;
                }
                reset_retry_counters(qp);