michaelvogt / rpms / qemu

Forked from rpms/qemu 3 months ago
Clone
4d7edd7
From: Li Qiang <liqiang6-s@360.cn>
4d7edd7
Date: Mon, 17 Oct 2016 14:13:58 +0200
4d7edd7
Subject: [PATCH] 9pfs: fix information leak in xattr read
4d7edd7
4d7edd7
9pfs uses g_malloc() to allocate the xattr memory space, if the guest
4d7edd7
reads this memory before writing to it, this will leak host heap memory
4d7edd7
to the guest. This patch avoid this.
4d7edd7
4d7edd7
Signed-off-by: Li Qiang <liqiang6-s@360.cn>
4d7edd7
Reviewed-by: Greg Kurz <groug@kaod.org>
4d7edd7
Signed-off-by: Greg Kurz <groug@kaod.org>
4d7edd7
(cherry picked from commit eb687602853b4ae656e9236ee4222609f3a6887d)
4d7edd7
---
4d7edd7
 hw/9pfs/9p.c | 2 +-
4d7edd7
 1 file changed, 1 insertion(+), 1 deletion(-)
4d7edd7
4d7edd7
diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c
4d7edd7
index 856544d..0735246 100644
4d7edd7
--- a/hw/9pfs/9p.c
4d7edd7
+++ b/hw/9pfs/9p.c
4d7edd7
@@ -3259,7 +3259,7 @@ static void v9fs_xattrcreate(void *opaque)
4d7edd7
     xattr_fidp->fs.xattr.flags = flags;
4d7edd7
     v9fs_string_init(&xattr_fidp->fs.xattr.name);
4d7edd7
     v9fs_string_copy(&xattr_fidp->fs.xattr.name, &name);
4d7edd7
-    xattr_fidp->fs.xattr.value = g_malloc(size);
4d7edd7
+    xattr_fidp->fs.xattr.value = g_malloc0(size);
4d7edd7
     err = offset;
4d7edd7
     put_fid(pdu, file_fidp);
4d7edd7
 out_nofid: