qemu-traditional: fix no-return-in-nonvoid-function

rpm post-build-checks found a few issues in qemu-xen-traditional and
marks them as errors.

I: Program returns random data in a function
E: xen no-return-in-nonvoid-function savevm.c:215
E: xen no-return-in-nonvoid-function /usr/src/packages/BUILD/xen-4.2.25602/non-dbg/stubdom/ioemu/xenfbfront.c:361

Signed-off-by: Olaf Hering <olaf@aepfle.de>

diff --git a/qemu-xen.h b/qemu-xen.h
index d50c89f1b8de9eef2c6ffd79469e9aea0f14f1b8..059866874a0a40b6d8a2a644e8d7b170c6b0e094 100644 (file)
--- a/qemu-xen.h
+++ b/qemu-xen.h
@@ -129,7 +129,7 @@ int xenstore_find_device(BlockDriverState *bs);
 
 /* xenfbfront.c */
 int xenfb_pv_display_init(DisplayState *ds);
-int xenfb_pv_display_vram(void *vram_start);
+void xenfb_pv_display_vram(void *vram_start);
 int xenfb_connect_vkbd(const char *path);
 int xenfb_connect_vfb(const char *path);
 

diff --git a/savevm.c b/savevm.c
index 71a57ebac5a58380ddca8c3986018ebbadf524ce..746a713b9445d29e7e4e072659bc8abadaf62c15 100644 (file)
--- a/savevm.c
+++ b/savevm.c
@@ -210,8 +210,8 @@ static int popen_close(void *opaque)
     QEMUFilePopen *s = opaque;
     pclose(s->popen_file);
     qemu_free(s);
-    return 0;
 #endif
+    return 0;
 }
 
 QEMUFile *qemu_popen(FILE *popen_file, const char *mode)

diff --git a/xenfbfront.c b/xenfbfront.c
index ed79cde7359673a0914423e31e947ab0b171dbaf..d83693fca208828d080e6b31c744ffc709cbf349 100644 (file)
--- a/xenfbfront.c
+++ b/xenfbfront.c
@@ -355,7 +355,7 @@ int xenfb_pv_display_init(DisplayState *ds)
     return 0;
 }
 
-int xenfb_pv_display_vram(void *data)
+void xenfb_pv_display_vram(void *data)
 {
     vga_vram = data;
 }