|
 |
383d26 |
From 27956f992c0e7622feed76bce5c5ff5cc9139e4d Mon Sep 17 00:00:00 2001
|
|
|
383d26 |
From: Kevin Wolf <kwolf@redhat.com>
|
|
|
383d26 |
Date: Tue, 26 Jun 2018 09:47:48 +0200
|
|
|
383d26 |
Subject: [PATCH 19/89] nfs: Remove processed options from QDict
|
|
|
383d26 |
|
|
|
383d26 |
RH-Author: Kevin Wolf <kwolf@redhat.com>
|
|
|
383d26 |
Message-id: <20180626094856.6924-6-kwolf@redhat.com>
|
|
|
383d26 |
Patchwork-id: 81058
|
|
|
383d26 |
O-Subject: [RHV-7.6 qemu-kvm-rhev PATCH v2 05/73] nfs: Remove processed options from QDict
|
|
|
383d26 |
Bugzilla: 1513543
|
|
|
383d26 |
RH-Acked-by: Jeffrey Cody <jcody@redhat.com>
|
|
|
383d26 |
RH-Acked-by: Max Reitz <mreitz@redhat.com>
|
|
|
383d26 |
RH-Acked-by: Fam Zheng <famz@redhat.com>
|
|
|
383d26 |
|
|
|
383d26 |
Commit c22a03454 QAPIfied option parsing in the NFS block driver, but
|
|
|
383d26 |
forgot to remove all the options we processed. Therefore, we get an
|
|
|
383d26 |
error in bdrv_open_inherit(), which thinks the remaining options are
|
|
|
383d26 |
invalid. Trying to open an NFS image will result in an error like this:
|
|
|
383d26 |
|
|
|
383d26 |
Block protocol 'nfs' doesn't support the option 'server.host'
|
|
|
383d26 |
|
|
|
383d26 |
Remove all options from the QDict to make the NFS driver work again.
|
|
|
383d26 |
|
|
|
383d26 |
Cc: qemu-stable@nongnu.org
|
|
|
383d26 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
383d26 |
Message-id: 20180516160816.26259-1-kwolf@redhat.com
|
|
|
383d26 |
Reviewed-by: Eric Blake <eblake@redhat.com>
|
|
|
383d26 |
Reviewed-by: Jeff Cody <jcody@redhat.com>
|
|
|
383d26 |
Signed-off-by: Jeff Cody <jcody@redhat.com>
|
|
|
383d26 |
(cherry picked from commit c82be42cc803b36fd7aed5dceec68312c7056fd5)
|
|
|
383d26 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
383d26 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
383d26 |
---
|
|
|
383d26 |
block/nfs.c | 7 +++++++
|
|
|
383d26 |
1 file changed, 7 insertions(+)
|
|
|
383d26 |
|
|
|
383d26 |
diff --git a/block/nfs.c b/block/nfs.c
|
|
|
383d26 |
index 3059ef2..743ca04 100644
|
|
|
383d26 |
--- a/block/nfs.c
|
|
|
383d26 |
+++ b/block/nfs.c
|
|
|
383d26 |
@@ -557,6 +557,7 @@ static BlockdevOptionsNfs *nfs_options_qdict_to_qapi(QDict *options,
|
|
|
383d26 |
{
|
|
|
383d26 |
BlockdevOptionsNfs *opts = NULL;
|
|
|
383d26 |
Visitor *v;
|
|
|
383d26 |
+ const QDictEntry *e;
|
|
|
383d26 |
Error *local_err = NULL;
|
|
|
383d26 |
|
|
|
383d26 |
v = qobject_input_visitor_new_flat_confused(options, errp);
|
|
|
383d26 |
@@ -572,6 +573,12 @@ static BlockdevOptionsNfs *nfs_options_qdict_to_qapi(QDict *options,
|
|
|
383d26 |
return NULL;
|
|
|
383d26 |
}
|
|
|
383d26 |
|
|
|
383d26 |
+ /* Remove the processed options from the QDict (the visitor processes
|
|
|
383d26 |
+ * _all_ options in the QDict) */
|
|
|
383d26 |
+ while ((e = qdict_first(options))) {
|
|
|
383d26 |
+ qdict_del(options, e->key);
|
|
|
383d26 |
+ }
|
|
|
383d26 |
+
|
|
|
383d26 |
return opts;
|
|
|
383d26 |
}
|
|
|
383d26 |
|
|
|
383d26 |
--
|
|
|
383d26 |
1.8.3.1
|
|
|
383d26 |
|