diff options
author | Kirill A. Shutemov <kirill@shutemov.name> | 2010-01-20 00:56:19 +0100 |
---|---|---|
committer | Anthony Liguori <aliguori@us.ibm.com> | 2010-01-26 14:59:20 -0600 |
commit | 27e3ddd3e0e9a8454e4cd4eebff9db8cd16c1949 (patch) | |
tree | eecedafe35ca59bbb79964ac965a1ad7241ba34b | |
parent | dc330e282a9aa2daf1a8703140917a4a2f85397a (diff) |
monitor.c: fix warnings with _FORTIFY_SOURCE
CC i386-softmmu/monitor.o
cc1: warnings being treated as errors
/usr/src/RPM/BUILD/qemu-0.11.92/monitor.c: In function 'do_memory_save':
/usr/src/RPM/BUILD/qemu-0.11.92/monitor.c:1318: error: ignoring return value of 'fwrite', declared with attribute warn_unused_result
/usr/src/RPM/BUILD/qemu-0.11.92/monitor.c: In function 'do_physical_memory_save':
/usr/src/RPM/BUILD/qemu-0.11.92/monitor.c:1345: error: ignoring return value of 'fwrite', declared with attribute warn_unused_result
make[1]: *** [monitor.o] Error 1
Signed-off-by: Kirill A. Shutemov <kirill@shutemov.name>
Signed-off-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
-rw-r--r-- | monitor.c | 12 |
1 files changed, 10 insertions, 2 deletions
@@ -1324,10 +1324,14 @@ static void do_memory_save(Monitor *mon, const QDict *qdict, QObject **ret_data) if (l > size) l = size; cpu_memory_rw_debug(env, addr, buf, l, 0); - fwrite(buf, 1, l, f); + if (fwrite(buf, 1, l, f) != l) { + monitor_printf(mon, "fwrite() error in do_memory_save\n"); + goto exit; + } addr += l; size -= l; } +exit: fclose(f); } @@ -1351,11 +1355,15 @@ static void do_physical_memory_save(Monitor *mon, const QDict *qdict, if (l > size) l = size; cpu_physical_memory_rw(addr, buf, l, 0); - fwrite(buf, 1, l, f); + if (fwrite(buf, 1, l, f) != l) { + monitor_printf(mon, "fwrite() error in do_physical_memory_save\n"); + goto exit; + } fflush(f); addr += l; size -= l; } +exit: fclose(f); } |