]> git.proxmox.com Git - mirror_qemu.git/commit
qemu-img/convert: Use @opts for one thing only
authorMax Reitz <mreitz@redhat.com>
Thu, 13 Apr 2017 20:34:00 +0000 (22:34 +0200)
committerMichael Roth <mdroth@linux.vnet.ibm.com>
Mon, 31 Jul 2017 21:20:06 +0000 (16:20 -0500)
commita1c850fd9d491594b39b23dace01b1fb3ca83236
tree80c0159c47166cf0d3857ae8466ae599c6b9c4f5
parentc37a62b751647927ec8d09fd885cf4f36f81255c
qemu-img/convert: Use @opts for one thing only

After storing the creation options for the new image into @opts, we
fetch some things for our own information, like the backing file name,
or whether to use encryption or preallocation.

With the -n parameter, there will not be any creation options; this is
not too bad because this just means that querying a NULL @opts will
always return the default value.

However, we also use @opts for the --object options. Therefore, @opts is
not necessarily NULL if -n was specified; instead, it may contain those
options. In practice, this probably does not cause any problems because
there most likely is no object that supports any of the parameters we
query here, but this is neither something we should rely on nor does
this variable reuse make the code very nice to read.

Therefore, just use an own variable for the --object options.

Cc: qemu-stable <qemu-stable@nongnu.org>
Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
* applied patch from v1 of series as suggested by author
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
qemu-img.c