Commit 32c0e902b2 for qemu.org
commit 32c0e902b2fbe30592cbe1afe130095d8fec5c93
Author: Marc-André Lureau <marcandre.lureau@redhat.com>
Date: Mon Sep 7 14:50:13 2026 +0400
qga/vss-win32: initialize *num_vols at start of requester_freeze
Initialize *num_vols to 0 at the top of requester_freeze() so all error
paths and zero-volume early exits return a defined value to the caller.
Previously, only the already-frozen path and the success path set
*num_vols; every goto-out error path left it indeterminate.
The sole caller is qga_vss_freeze(), which already sets *nr_volume = 0.
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Kostiantyn Kostiuk <kkostiuk@redhat.com>
Link: https://lore.kernel.org/qemu-devel/20260907-qga-v1-15-86456ad356e2@redhat.com
Signed-off-by: Kostiantyn Kostiuk <kkostiuk@redhat.com>
diff --git a/qga/vss-win32/requester.cpp b/qga/vss-win32/requester.cpp
index f14c47f9f7..d36a90c17d 100644
--- a/qga/vss-win32/requester.cpp
+++ b/qga/vss-win32/requester.cpp
@@ -306,8 +306,9 @@ void requester_freeze(int *num_vols, void *mountpoints, ErrorSet *errset)
int num_mount_points = 0;
VSS_BACKUP_TYPE vss_bt = get_vss_backup_type();
+ *num_vols = 0;
+
if (vss_ctx.pVssbc) { /* already frozen */
- *num_vols = 0;
qga_debug("finished, already frozen");
return;
}