On 2014-11-11 at 19:12, Eric Blake wrote:
On 11/10/2014 06:45 AM, Max Reitz wrote:
Signed-off-by: Max Reitz <mre...@redhat.com>
---
block/qcow2.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Eric Blake <ebl...@redhat.com>
Is it worth hoisting this one into 2.2 via the -trivial tree?
No, as explained this point can only be reached if there is some
creation option for qcow2 images which is not handled by any of the
branches in this function. Since there is no such thing currently in
master and there most certainly won't be in 2.2 (thanks to hard freeze),
it's fine to keep it out of 2.2.
Max
diff --git a/block/qcow2.c b/block/qcow2.c
index beb7187..ebf843f 100644
--- a/block/qcow2.c
+++ b/block/qcow2.c
@@ -2718,9 +2718,9 @@ static int qcow2_amend_options(BlockDriverState *bs,
QemuOpts *opts,
error_report("Cannot change refcount entry width");
return -ENOTSUP;
} else {
- /* if this assertion fails, this probably means a new option was
+ /* if this point is reached, this probably means a new option was
* added without having it covered here */
- assert(false);
+ abort();
}
desc++;
