error.c: don't return value for void function
authorAmos Kong <akong@redhat.com>
Mon, 16 Apr 2012 07:32:49 +0000 (15:32 +0800)
committerStefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Fri, 20 Apr 2012 12:14:53 +0000 (13:14 +0100)
It is invalid to return a value from a function
returning void.

[C99 6.8.6.4 says "A return statement with an expression shall not
appear in a function whose return type is void" but gcc 4.6.3 with QEMU
compile flags does not complain.  It's still worth fixing this.  Stefan]

Signed-off-by: Amos Kong <akong@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
error.c

diff --git a/error.c b/error.c
index d3455ab9e6dbb214d8e86ced200255540e5e2acc..a52b7710d265462648510ca5cf4babdd31a139b4 100644 (file)
--- a/error.c
+++ b/error.c
@@ -93,7 +93,7 @@ QDict *error_get_data(Error *err)
 void error_set_field(Error *err, const char *field, const char *value)
 {
     QDict *dict = qdict_get_qdict(err->obj, "data");
-    return qdict_put(dict, field, qstring_from_str(value));
+    qdict_put(dict, field, qstring_from_str(value));
 }
 
 void error_free(Error *err)