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 17:55:54 +0000 (19:55 +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 9eb12c2e3c74cf69499124e21fcb1b4a8be4a40f..83cfe44f070ec47d85f43a02e7ce4a44158bab35 100644 (file)
@@ -276,6 +276,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);