diff options
| author | Kevin Wolf <kwolf@redhat.com> | 2014-09-25 09:54:02 +0200 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2014-09-25 15:24:32 +0200 |
| commit | 9aebf3b89281a173d2dfeee379b800be5e3f363e (patch) | |
| tree | 0e1ae715f5268dc2b4592c10ad521285a0cead26 /include | |
| parent | a4127c428e61317b2966ec7fe480c3c6fb2d08df (diff) | |
| download | qemu-9aebf3b89281a173d2dfeee379b800be5e3f363e.zip | |
block: Validate node-name
The device_name of a BlockDriverState is currently checked because it is
always used as a QemuOpts ID and qemu_opts_create() checks whether such
IDs are wellformed.
node-name is supposed to share the same namespace, but it isn't checked
currently. This patch adds explicit checks both for device_name and
node-name so that the same rules will still apply even if QemuOpts won't
be used any more at some point.
qemu-img used to use names with spaces in them, which isn't allowed any
more. Replace them with underscores.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'include')
| -rw-r--r-- | include/qemu/option.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/qemu/option.h b/include/qemu/option.h index 59bea759a2..945347cc8f 100644 --- a/include/qemu/option.h +++ b/include/qemu/option.h @@ -103,6 +103,7 @@ typedef int (*qemu_opt_loopfunc)(const char *name, const char *value, void *opaq int qemu_opt_foreach(QemuOpts *opts, qemu_opt_loopfunc func, void *opaque, int abort_on_failure); +int qemu_opts_id_wellformed(const char *id); QemuOpts *qemu_opts_find(QemuOptsList *list, const char *id); QemuOpts *qemu_opts_create(QemuOptsList *list, const char *id, int fail_if_exists, Error **errp); |