summary refs log tree commit diff stats
path: root/block/blkdebug.c
diff options
context:
space:
mode:
authorEduardo Habkost <ehabkost@redhat.com>2017-10-03 23:50:42 -0300
committerEduardo Habkost <ehabkost@redhat.com>2017-10-09 23:21:52 -0300
commite5766d6ec7524345f4c0fa284c065b68c5e93049 (patch)
tree4539468f42ed7811552714aeb8c82a5ee3fd9215 /block/blkdebug.c
parent3478eae990d0373a5a3614f9293e16ebb50e5cb6 (diff)
downloadfocaccia-qemu-e5766d6ec7524345f4c0fa284c065b68c5e93049.tar.gz
focaccia-qemu-e5766d6ec7524345f4c0fa284c065b68c5e93049.zip
config: qemu_config_parse() return number of config groups
Change qemu_config_parse() to return the number of config groups
in success and -EINVAL on error. This will allow callers of
qemu_config_parse() to check if something was really loaded from
the config file.

All existing callers of qemu_config_parse() and
qemu_read_config_file() only check if the return value was
negative, so the change shouldn't affect them.

Reviewed-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Message-Id: <20171004025043.3788-2-ehabkost@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Diffstat (limited to 'block/blkdebug.c')
-rw-r--r--block/blkdebug.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/block/blkdebug.c b/block/blkdebug.c
index 46e53f2f09..dfdf9b91aa 100644
--- a/block/blkdebug.c
+++ b/block/blkdebug.c
@@ -244,7 +244,6 @@ static int read_config(BDRVBlkdebugState *s, const char *filename,
         ret = qemu_config_parse(f, config_groups, filename);
         if (ret < 0) {
             error_setg(errp, "Could not parse blkdebug config file");
-            ret = -EINVAL;
             goto fail;
         }
     }