diff options
| author | Kevin Wolf <kwolf@redhat.com> | 2018-05-16 18:08:16 +0200 |
|---|---|---|
| committer | Jeff Cody <jcody@redhat.com> | 2018-05-16 13:37:47 -0400 |
| commit | c82be42cc803b36fd7aed5dceec68312c7056fd5 (patch) | |
| tree | 11c50b970d2f91c9d95e4e476d1f6f386abe4e1c | |
| parent | 54b7af4369a37afbd82573d0dcfb27febdb6dd24 (diff) | |
| download | qemu-c82be42cc803b36fd7aed5dceec68312c7056fd5.zip | |
nfs: Remove processed options from QDict
Commit c22a03454 QAPIfied option parsing in the NFS block driver, but
forgot to remove all the options we processed. Therefore, we get an
error in bdrv_open_inherit(), which thinks the remaining options are
invalid. Trying to open an NFS image will result in an error like this:
Block protocol 'nfs' doesn't support the option 'server.host'
Remove all options from the QDict to make the NFS driver work again.
Cc: qemu-stable@nongnu.org
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Message-id: 20180516160816.26259-1-kwolf@redhat.com
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Jeff Cody <jcody@redhat.com>
Signed-off-by: Jeff Cody <jcody@redhat.com>
| -rw-r--r-- | block/nfs.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/block/nfs.c b/block/nfs.c index 4ee2ad59d9..3349b67a76 100644 --- a/block/nfs.c +++ b/block/nfs.c @@ -557,6 +557,7 @@ static BlockdevOptionsNfs *nfs_options_qdict_to_qapi(QDict *options, BlockdevOptionsNfs *opts = NULL; QObject *crumpled = NULL; Visitor *v; + const QDictEntry *e; Error *local_err = NULL; crumpled = qdict_crumple(options, errp); @@ -574,6 +575,12 @@ static BlockdevOptionsNfs *nfs_options_qdict_to_qapi(QDict *options, return NULL; } + /* Remove the processed options from the QDict (the visitor processes + * _all_ options in the QDict) */ + while ((e = qdict_first(options))) { + qdict_del(options, e->key); + } + return opts; } |