Hi, Ping...
Nine patches of this patch series has been reviewd, and the last patch is not reviewed. Are these patches accepted? Though this series contains ten patches, but most of them only touch a few lines of code, and it involves several modules. So, should these patched be applied to qemu trivial branch? Any help will be greatly appreciated. Thanks, zhanghailiang This patch set fix three bugs about accessing freed memory and several api abuse. In qemu, there are serveral places that do not check the return value of fstat()/fopen()/malloc(),etc. Though it is a small probability for the these functions to fail, but it is better to fix them, Or there may be a serious segmentfault. v5 -> v6: * Add reviewd-by info v4 -> v5: util/path: * Use the GLib memory APIs g_malloc/g_strdup/g_realloc which would abort on failure (Thanks for the suggestion of Alex Bennée) slirp: * Again use of g_malloc to replace malloc(based on the review of Alex Bennée) bios-tables-test: * Correct the wrong use of g_assert v3 -> v4: slirp: * Check return value of '*ex_ptr', not 'ex_ptr',also add error message (basedon the review of GongLei) linux-user: * It should call unlock_user_struct() before return (based on the review of Richard Henderson) tests/bios-tables-test: * Remove unnecessary check then return value of fopen() in qtest_init() v2 -> v3: ivshmem: * Change the error message which advised by Levente Kurusa others: * Add six new patches which check the return value of malloc() and fopen(), which may be failed. v1 -> v2: ivshmem: * Modified the log message according to reviewing suggestion of Michael Li Liu (3): tcg: check return value of fopen() block/vvfat: fix setbuf stream parameter may be NULL qtest: check the value returned by fopen() zhanghailiang (7): l2cap: fix access freed memory monitor: fix access freed memory virtio-blk: fix reference a pointer which might be freed ivshmem: check the value returned by fstat() util/path: check return value of malloc() slirp: check return value of malloc() linux-user: check return value of malloc() block/vvfat.c | 5 ++++- hw/block/virtio-blk.c | 5 +++-- hw/bt/l2cap.c | 2 +- hw/misc/ivshmem.c | 6 +++++- linux-user/syscall.c | 4 ++++ monitor.c | 4 +++- slirp/misc.c | 4 ++-- tcg/tcg.c | 4 ++++ tests/bios-tables-test.c | 5 +++++ util/path.c | 6 +++--- 10 files changed, 34 insertions(+), 11 deletions(-) -- 1.7.12.4
