Skip to content

Commit

Permalink
qcow2: Fix unaligned preallocated truncation
Browse files Browse the repository at this point in the history
A qcow2 image file's length is not required to have a length that is a
multiple of the cluster size.  However, qcow2_refcount_area() expects an
aligned value for its @start_offset parameter, so we need to round
@old_file_size up to the next cluster boundary.

Reported-by: Ping Li <[email protected]>
Bug: https://bugzilla.redhat.com/show_bug.cgi?id=1414049
Signed-off-by: Max Reitz <[email protected]>
Message-id: [email protected]
Cc: [email protected]
Reviewed-by: Eric Blake <[email protected]>
Reviewed-by: Jeff Cody <[email protected]>
Reviewed-by: Stefan Hajnoczi <[email protected]>
Signed-off-by: Max Reitz <[email protected]>
  • Loading branch information
XanClic committed Oct 26, 2017
1 parent 233521b commit e400ad1
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions block/qcow2.c
Original file line number Diff line number Diff line change
Expand Up @@ -3188,6 +3188,7 @@ static int qcow2_truncate(BlockDriverState *bs, int64_t offset,
"Failed to inquire current file length");
return old_file_size;
}
old_file_size = ROUND_UP(old_file_size, s->cluster_size);

nb_new_data_clusters = DIV_ROUND_UP(offset - old_length,
s->cluster_size);
Expand Down

0 comments on commit e400ad1

Please sign in to comment.