On 04/30/2018 05:02 PM, Marcel Apfelbaum wrote:
> Coverity CID 1390586; The cq handle is provided by the guest
> and cannot be trusted to be previuosly allocated.
> Fix it by exiting the completion flow.
>
> Reported-by: Peter Maydell <peter.mayd...@linaro.org>
> Signed-off-by: Marcel Apfelbaum <marcel.apfelb...@gmail.com>
> Reviewed-by: Yuval Shaia <yuval.sh...@oracle.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
> ---
> hw/rdma/vmw/pvrdma_qp_ops.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/hw/rdma/vmw/pvrdma_qp_ops.c b/hw/rdma/vmw/pvrdma_qp_ops.c
> index 750ade6c31..99bb51111e 100644
> --- a/hw/rdma/vmw/pvrdma_qp_ops.c
> +++ b/hw/rdma/vmw/pvrdma_qp_ops.c
> @@ -216,6 +216,7 @@ void pvrdma_cq_poll(RdmaDeviceResources *dev_res,
> uint32_t cq_handle)
> cq = rdma_rm_get_cq(dev_res, cq_handle);
> if (!cq) {
> pr_dbg("Invalid CQ# %d\n", cq_handle);
> + return;
> }
>
> rdma_backend_poll_cq(dev_res, &cq->backend_cq);
>
