diff options
| author | Yuval Shaia | 2018-10-25 08:17:00 +0200 |
|---|---|---|
| committer | Marcel Apfelbaum | 2018-12-22 10:09:56 +0100 |
| commit | dee2e53c86de37dea4f36936eab5a6aef340b8f8 (patch) | |
| tree | 3012d9228baf7db77de63587fbd7bba9fdd34d3e | |
| parent | Merge remote-tracking branch 'remotes/dgibson/tags/ppc-for-4.0-20181221' into... (diff) | |
| download | qemu-dee2e53c86de37dea4f36936eab5a6aef340b8f8.tar.gz qemu-dee2e53c86de37dea4f36936eab5a6aef340b8f8.tar.xz qemu-dee2e53c86de37dea4f36936eab5a6aef340b8f8.zip | |
hw/pvrdma: Check the correct return value
Return value of 0 means ok, we want to free the memory only in case of
error.
Signed-off-by: Yuval Shaia <yuval.shaia@oracle.com>
Message-Id: <20181025061700.17050-1-yuval.shaia@oracle.com>
Reviewed-by: Marcel Apfelbaum<marcel.apfelbaum@gmail.com>
Signed-off-by: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
| -rw-r--r-- | hw/rdma/vmw/pvrdma_cmd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/rdma/vmw/pvrdma_cmd.c b/hw/rdma/vmw/pvrdma_cmd.c index 4faeb21631..57d6f41ae6 100644 --- a/hw/rdma/vmw/pvrdma_cmd.c +++ b/hw/rdma/vmw/pvrdma_cmd.c @@ -232,7 +232,7 @@ static int create_mr(PVRDMADev *dev, union pvrdma_cmd_req *req, cmd->start, cmd->length, host_virt, cmd->access_flags, &resp->mr_handle, &resp->lkey, &resp->rkey); - if (host_virt && !resp->hdr.err) { + if (resp->hdr.err && host_virt) { munmap(host_virt, cmd->length); } |