diff options
| author | Eric Blake <eblake@redhat.com> | 2017-04-27 16:58:18 -0500 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2017-05-09 09:14:39 +0200 |
| commit | ff6ed7141d87d26eafa2b8e4df969623e40fac49 (patch) | |
| tree | eedb7135d64638688c41dbd54ae577ff3727bbfe /util/qemu-option.c | |
| parent | 46f5ac205a9dc5e2c24274c7df371509a286281f (diff) | |
| download | qemu-ff6ed7141d87d26eafa2b8e4df969623e40fac49.zip qemu-ff6ed7141d87d26eafa2b8e4df969623e40fac49.tar.gz qemu-ff6ed7141d87d26eafa2b8e4df969623e40fac49.tar.bz2 | |
block: Simplify bdrv_append_temp_snapshot() logic
Noticed while checking Coccinelle results. Naming a label 'out:'
when it is only used on error paths is weird. Also, we had some
dead stores to 'ret'. Meanwhile we know that snapshot_options
is NULL on success and that QDECREF(NULL) is safe. So merge the
two exit paths into one by careful control over bs_snapshot.
Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20170427215821.19397-8-eblake@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'util/qemu-option.c')
0 files changed, 0 insertions, 0 deletions