sysupdate: propagate transfer_instance_vacuum() errors
Both vacuum loops silently return 0 on failure instead of propagating the error code, hiding disk cleanup failures from the caller. Signed-off-by: dongshengyuan <dongshengyuan@uniontech.com>
This commit is contained in:
@@ -838,7 +838,7 @@ int transfer_vacuum(
|
||||
|
||||
r = transfer_instance_vacuum(t, instance);
|
||||
if (r < 0)
|
||||
return 0;
|
||||
return r;
|
||||
|
||||
instance_free(instance);
|
||||
memmove(t->target.instances + i, t->target.instances + i + 1, (t->target.n_instances - i - 1) * sizeof(Instance*));
|
||||
@@ -937,7 +937,7 @@ int transfer_vacuum(
|
||||
|
||||
r = transfer_instance_vacuum(t, oldest);
|
||||
if (r < 0)
|
||||
return 0;
|
||||
return r;
|
||||
|
||||
instance_free(oldest);
|
||||
memmove(t->target.instances + p, t->target.instances + p + 1, (t->target.n_instances - p - 1) * sizeof(Instance*));
|
||||
|
||||
Reference in New Issue
Block a user