diff options
| author | Max Reitz <mreitz@redhat.com> | 2015-09-11 18:47:51 +0200 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2015-09-14 16:51:37 +0200 |
| commit | 2ac01520be8717f3492b10a083c3e0e22cb52cda (patch) | |
| tree | cf39358b64bb3eb0d227db2b753d163f8012797a /include | |
| parent | 3efffc3292d94271a15b1606b4a56adf6c6f04ed (diff) | |
| download | qemu-2ac01520be8717f3492b10a083c3e0e22cb52cda.tar.gz qemu-2ac01520be8717f3492b10a083c3e0e22cb52cda.tar.bz2 qemu-2ac01520be8717f3492b10a083c3e0e22cb52cda.zip | |
qcow2: Make qcow2_alloc_bytes() more explicit
In case of -EAGAIN returned by update_refcount(), we should discard the
cluster offset we were trying to allocate and request a new one, because
in theory that old offset might now be taken by a refcount block.
In practice, this was not the case due to update_refcount() generally
returning strictly monotonic increasing cluster offsets. However, this
behavior is not set in stone, and it is also not obvious when looking at
qcow2_alloc_bytes() alone, so we should not rely on it.
Reported-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions