diff options
| author | Markus Armbruster | 2020-07-23 16:27:38 +0200 |
|---|---|---|
| committer | Markus Armbruster | 2020-09-03 09:44:33 +0200 |
| commit | 3a14019e8216eb5f48074d781343317274b8292a (patch) | |
| tree | 98a8425c90c246b3c09bdc8153b7191a43b6f474 | |
| parent | scripts/qmp/qom-fuse: Port to current Python module fuse (diff) | |
| download | qemu-3a14019e8216eb5f48074d781343317274b8292a.tar.gz qemu-3a14019e8216eb5f48074d781343317274b8292a.tar.xz qemu-3a14019e8216eb5f48074d781343317274b8292a.zip | |
scripts/qmp/qom-fuse: Fix getattr(), read() for files in /
path, prop = "type".rsplit('/', 1) sets path to "", which doesn't
work. Correct to "/".
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20200723142738.1868568-4-armbru@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: John Snow <jsnow@redhat.com>
| -rwxr-xr-x | scripts/qmp/qom-fuse | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/scripts/qmp/qom-fuse b/scripts/qmp/qom-fuse index 405e6ebd67..7c7cff8edf 100755 --- a/scripts/qmp/qom-fuse +++ b/scripts/qmp/qom-fuse @@ -45,8 +45,10 @@ class QOMFS(Operations): return False def is_property(self, path): + path, prop = path.rsplit('/', 1) + if path == '': + path = '/' try: - path, prop = path.rsplit('/', 1) for item in self.qmp.command('qom-list', path=path): if item['name'] == prop: return True @@ -55,8 +57,10 @@ class QOMFS(Operations): return False def is_link(self, path): + path, prop = path.rsplit('/', 1) + if path == '': + path = '/' try: - path, prop = path.rsplit('/', 1) for item in self.qmp.command('qom-list', path=path): if item['name'] == prop: if item['type'].startswith('link<'): @@ -71,6 +75,8 @@ class QOMFS(Operations): return -ENOENT path, prop = path.rsplit('/', 1) + if path == '': + path = '/' try: data = self.qmp.command('qom-get', path=path, property=prop) data += '\n' # make values shell friendly |