qemu-char: Avoid qerror_report_err() outside QMP command handlers
authorMarkus Armbruster <armbru@redhat.com>
Tue, 10 Feb 2015 14:21:26 +0000 (15:21 +0100)
committerMarkus Armbruster <armbru@redhat.com>
Wed, 18 Feb 2015 09:51:40 +0000 (10:51 +0100)
qerror_report_err() is a transitional interface to help with
converting existing monitor commands to QMP.  It should not be used
elsewhere.  Replace by error_report_err() in legacy chardev parser
qemu_chr_parse_compat().  Legacy chardev syntax is not to be used in
QMP.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
qemu-char.c

index f901ed1583642960dc4bdc8f49fcd26ec7a7af01..8159462fc0bbd973481b8dcefa7b980c2ff6d103 100644 (file)
@@ -3306,8 +3306,7 @@ QemuOpts *qemu_chr_parse_compat(const char *label, const char *filename)
 
     opts = qemu_opts_create(qemu_find_opts("chardev"), label, 1, &local_err);
     if (local_err) {
-        qerror_report_err(local_err);
-        error_free(local_err);
+        error_report_err(local_err);
         return NULL;
     }