Message ID | 1339760598-6459-1-git-send-email-kwolf@redhat.com |
---|---|
State | New |
Headers | show |
On Fri, 15 Jun 2012, Kevin Wolf wrote: > Some gcc versions seem not to be able to figure out that the switch > statement covers all possible values and that c is therefore always > initialised. Add a default branch for them. > Applied, thanks.
diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c index b663b9f..d7e0e19 100644 --- a/block/qcow2-cluster.c +++ b/block/qcow2-cluster.c @@ -471,6 +471,8 @@ int qcow2_get_cluster_offset(BlockDriverState *bs, uint64_t offset, QCOW_OFLAG_COMPRESSED | QCOW_OFLAG_ZERO); *cluster_offset &= L2E_OFFSET_MASK; break; + default: + abort(); } qcow2_cache_put(bs, s->l2_table_cache, (void**) &l2_table);
Some gcc versions seem not to be able to figure out that the switch statement covers all possible values and that c is therefore always initialised. Add a default branch for them. Reported-by: malc <av1474@comtv.ru> Signed-off-by: Kevin Wolf <kwolf@redhat.com> --- block/qcow2-cluster.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)