Skip to content

Conversation

@weltling
Copy link
Member

@weltling weltling commented Dec 8, 2025

The BufWriter must be flushed explicitly to handle errors properly. Without explicit flush, errors during the implicit drop flush are ignored.

This is the same issue fixed for write_pointer_table in commit 8555695.

@weltling weltling requested a review from a team as a code owner December 8, 2025 19:07
@russell-islam
Copy link
Contributor

@weltling Please fix the commit message issue.

@weltling weltling force-pushed the qcow-fixes-buf-sync branch from acc7286 to 7c5cde9 Compare December 9, 2025 13:20
The BufWriter must be flushed explicitly to handle errors
properly. Without explicit flush, errors during the implicit
drop flush are ignored.

This is the same issue fixed for write_pointer_table
in commit 8555695.

Signed-off-by: Anatol Belski <[email protected]>
@weltling weltling force-pushed the qcow-fixes-buf-sync branch from 7c5cde9 to a151b40 Compare December 9, 2025 13:25
@rbradford rbradford added this pull request to the merge queue Dec 9, 2025
Merged via the queue into cloud-hypervisor:main with commit 562af12 Dec 9, 2025
38 of 43 checks passed
@weltling weltling deleted the qcow-fixes-buf-sync branch December 9, 2025 15:56
@likebreath likebreath added the bug-fix Bug fix to include in release notes label Dec 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Bug fix to include in release notes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants