[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v5 10/10] block/vvfat: fix setbuf stream parameter m
From: |
zhanghailiang |
Subject: |
[Qemu-devel] [PATCH v5 10/10] block/vvfat: fix setbuf stream parameter may be NULL |
Date: |
Mon, 11 Aug 2014 16:53:09 +0800 |
From: Li Liu <address@hidden>
fopen() may return NULL which will cause setbuf() segmentfault
Signed-off-by: zhanghailiang <address@hidden>
Signed-off-by: Li Liu <address@hidden>
---
block/vvfat.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/block/vvfat.c b/block/vvfat.c
index 70176b1..6889ea9 100644
--- a/block/vvfat.c
+++ b/block/vvfat.c
@@ -1084,7 +1084,10 @@ static int vvfat_open(BlockDriverState *bs, QDict
*options, int flags,
DLOG(if (stderr == NULL) {
stderr = fopen("vvfat.log", "a");
- setbuf(stderr, NULL);
+
+ if (stderr) {
+ setbuf(stderr, NULL);
+ }
})
opts = qemu_opts_create(&runtime_opts, NULL, 0, &error_abort);
--
1.7.12.4
- [Qemu-devel] [PATCH v5 04/10] ivshmem: check the value returned by fstat(), (continued)
- [Qemu-devel] [PATCH v5 04/10] ivshmem: check the value returned by fstat(), zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 03/10] virtio-blk: fix reference a pointer which might be freed, zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 06/10] slirp/misc: Use g_malloc() instead of malloc(), zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 05/10] util/path: Use the GLib memory allocation routines, zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 08/10] tests/bios-tables-test: check the value returned by fopen(), zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 02/10] monitor: fix access freed memory, zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 01/10] l2cap: fix access freed memory, zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 10/10] block/vvfat: fix setbuf stream parameter may be NULL,
zhanghailiang <=
- [Qemu-devel] [PATCH v5 09/10] tcg: check return value of fopen(), zhanghailiang, 2014/08/11
- [Qemu-devel] [PATCH v5 07/10] linux-user: check return value of malloc(), zhanghailiang, 2014/08/11