From: Kamal Heib <kamalhe...@gmail.com>
Make sure to zero-initialize only the pvrdma_cmd_query_qp_resp and not
the whole pvrdma_cmd_resp for query_qp, in modify_qp the resp isn't used
so remove it.
Reviewed-by: Yuval Shaia <yuval.sh...@oracle.com>
Signed-off-by: Kamal Heib <kamalhe...@gmail.com>
Message-Id: <20190314153031.7197-5-kamalhe...@gmail.com>
Signed-off-by: Marcel Apfelbaum <marcel.apfelb...@gmail.com>
---
hw/rdma/vmw/pvrdma_cmd.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/hw/rdma/vmw/pvrdma_cmd.c b/hw/rdma/vmw/pvrdma_cmd.c
index b682e919d5..be8c2b61c9 100644
--- a/hw/rdma/vmw/pvrdma_cmd.c
+++ b/hw/rdma/vmw/pvrdma_cmd.c
@@ -488,8 +488,6 @@ static int modify_qp(PVRDMADev *dev, union pvrdma_cmd_req
*req,
struct pvrdma_cmd_modify_qp *cmd = &req->modify_qp;
int rc;
- memset(rsp, 0, sizeof(*rsp));
-
/* No need to verify sgid_index since it is u8 */
rc = rdma_rm_modify_qp(&dev->rdma_dev_res, &dev->backend_dev,
@@ -512,7 +510,7 @@ static int query_qp(PVRDMADev *dev, union pvrdma_cmd_req
*req,
struct ibv_qp_init_attr init_attr;
int rc;
- memset(rsp, 0, sizeof(*rsp));
+ memset(resp, 0, sizeof(*resp));
rc = rdma_rm_query_qp(&dev->rdma_dev_res, &dev->backend_dev,
cmd->qp_handle,
(struct ibv_qp_attr *)&resp->attrs, cmd->attr_mask,
--
2.17.1
