summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChen Gang2014-03-01 18:34:10 +0100
committerAneesh Kumar K.V2014-03-03 17:25:01 +0100
commit75b7931ec63577acb2dd472eb8d772012f1a4807 (patch)
tree202144ace4499693540e6ad0e183466a9692e904
parentfsdev: Fix overrun after readlink() fills buffer completely (diff)
downloadqemu-75b7931ec63577acb2dd472eb8d772012f1a4807.tar.gz
qemu-75b7931ec63577acb2dd472eb8d772012f1a4807.tar.xz
qemu-75b7931ec63577acb2dd472eb8d772012f1a4807.zip
hw/9pfs/virtio-9p-local.c: move v9fs_string_free() to below "err_out:"
When "goto err_out", 'v9fs_string' already was allocated, so still need free 'v9fs_string' before return. Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
-rw-r--r--hw/9pfs/virtio-9p-local.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/9pfs/virtio-9p-local.c b/hw/9pfs/virtio-9p-local.c
index df0dbffa7a..62e694370f 100644
--- a/hw/9pfs/virtio-9p-local.c
+++ b/hw/9pfs/virtio-9p-local.c
@@ -1059,9 +1059,9 @@ static int local_unlinkat(FsContext *ctx, V9fsPath *dir,
}
/* Remove the name finally */
ret = remove(rpath(ctx, fullname.data, buffer));
- v9fs_string_free(&fullname);
err_out:
+ v9fs_string_free(&fullname);
return ret;
}