On Tue, Feb 23, 2016 at 09:49:07AM +0100, Michal Privoznik wrote: > After 457ff97fa there are two defects in our code. In both of > them we use a signed variable to hold up a number of snapshots > that domain has. We use a helper function to count the number. > However, the helper function may fail in which case it returns > a negative one and control jumps to cleanup label where an > unsigned variable is used to iterate over array of snapshots. The > loop condition thus compare signed and unsigned variables which > in this specific case ends up badly for us. > > Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> > --- > src/vbox/vbox_common.c | 8 +++----- > 1 file changed, 3 insertions(+), 5 deletions(-) > ACK. Does virStorageBackendRBDCleanupSnapshots need the same treatment? I'm having trouble deciphering the code flow. Jan -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list