[PATCH v4 17/19] vbox: change type to void vboxDumpDisplay()

[Dmitry Frolov <frolov@xxxxxxxxx>]

vboxDumpDisplay() returns nothing except 0. Changing
return type to void.

Found by Linux Verification Center (linuxtesting.org) with SVACE.

Signed-off-by: Dmitry Frolov <frolov@xxxxxxxxx>
---
 src/vbox/vbox_common.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/vbox/vbox_common.c b/src/vbox/vbox_common.c
index 59e7a91081..305c85f4ec 100644
--- a/src/vbox/vbox_common.c
+++ b/src/vbox/vbox_common.c
@@ -3538,7 +3538,7 @@ vboxDumpVideo(virDomainDef *def, struct _vboxDriver *data G_GNUC_UNUSED,
     return 0;
 }
 
-static int
+static void
 vboxDumpDisplay(virDomainDef *def, struct _vboxDriver *data, IMachine *machine)
 {
     /* dump display options vrdp/gui/sdl */
@@ -3641,7 +3641,6 @@ vboxDumpDisplay(virDomainDef *def, struct _vboxDriver *data, IMachine *machine)
     VBOX_UTF8_FREE(valueTypeUtf8);
     VBOX_UTF8_FREE(netAddressUtf8);
     virDomainGraphicsDefFree(graphics);
-    return 0;
 }
 
 static int
@@ -4157,8 +4156,7 @@ static char *vboxDomainGetXMLDesc(virDomainPtr dom, unsigned int flags)
 
     if (vboxDumpVideo(def, data, machine) < 0)
         goto cleanup;
-    if (vboxDumpDisplay(def, data, machine) < 0)
-        goto cleanup;
+    vboxDumpDisplay(def, data, machine);
     if (vboxDumpStorageControllers(def, machine) < 0)
         goto cleanup;
     if (vboxDumpDisks(def, data, machine) < 0)
-- 
2.34.1
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx