diff options
| author | Greg Kurz <groug@kaod.org> | 2017-09-04 09:59:01 +0200 |
|---|---|---|
| committer | Michael Roth <mdroth@linux.vnet.ibm.com> | 2017-09-05 15:42:17 -0500 |
| commit | c24c5910b78db714efe0698d9d7a8868656d22d8 (patch) | |
| tree | f46f6927b23781f018baec56a80bd0cfc09282a8 /util/qemu-thread-win32.c | |
| parent | 2d1bbf51c2cb948da4b6fd5f91cf3ecc80b28156 (diff) | |
| download | qemu-c24c5910b78db714efe0698d9d7a8868656d22d8.zip qemu-c24c5910b78db714efe0698d9d7a8868656d22d8.tar.gz qemu-c24c5910b78db714efe0698d9d7a8868656d22d8.tar.bz2 | |
virtfs: error out gracefully when mandatory suboptions are missing
We internally convert -virtfs to -fsdev/-device. If the user doesn't
provide the path or security_model suboptions, and the fsdev backend
requires them, we hit an assertion when populating the internal -fsdev
option:
util/qemu-option.c:547: opt_set: Assertion `opt->str' failed.
Aborted (core dumped)
Let's test the suboption presence on the command line before trying
to set it in the internal -fsdev option, and let the backend code
error out gracefully (ie, like it already does when the user passes
-fsdev on the command line).
Reported-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Greg Kurz <groug@kaod.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
(cherry picked from commit 32b6943699948f7adc35ada233fbd25daffad5e9)
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
Diffstat (limited to 'util/qemu-thread-win32.c')
0 files changed, 0 insertions, 0 deletions