| Message ID | 20220825143109.176582-3-alexander.ivanov@virtuozzo.com |
|---|---|
| State | New |
| Headers | show |
| Series | parallels: Refactor the code of images checks and fix a bug | expand |
On 25.08.2022 16:31, Alexander Ivanov wrote: > Don't let high_off be more than the file size even if we don't fix the image. > > Signed-off-by: Alexander Ivanov <alexander.ivanov@virtuozzo.com> > --- > block/parallels.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/block/parallels.c b/block/parallels.c > index 93bc2750ef..7e8cdbbc3a 100644 > --- a/block/parallels.c > +++ b/block/parallels.c > @@ -460,12 +460,12 @@ static int coroutine_fn parallels_co_check(BlockDriverState *bs, > fix & BDRV_FIX_ERRORS ? "Repairing" : "ERROR", i); > res->corruptions++; > if (fix & BDRV_FIX_ERRORS) { > - prev_off = 0; > s->bat_bitmap[i] = 0; > res->corruptions_fixed++; > flush_bat = true; > - continue; > } > + prev_off = 0; > + continue; > } > > res->bfi.allocated_clusters++; with string length fixes in the commit message (more that 74 chars) Reviewed-by: Denis V. Lunev <den@openvz.org>
diff --git a/block/parallels.c b/block/parallels.c index 93bc2750ef..7e8cdbbc3a 100644 --- a/block/parallels.c +++ b/block/parallels.c @@ -460,12 +460,12 @@ static int coroutine_fn parallels_co_check(BlockDriverState *bs, fix & BDRV_FIX_ERRORS ? "Repairing" : "ERROR", i); res->corruptions++; if (fix & BDRV_FIX_ERRORS) { - prev_off = 0; s->bat_bitmap[i] = 0; res->corruptions_fixed++; flush_bat = true; - continue; } + prev_off = 0; + continue; } res->bfi.allocated_clusters++;
Don't let high_off be more than the file size even if we don't fix the image. Signed-off-by: Alexander Ivanov <alexander.ivanov@virtuozzo.com> --- block/parallels.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)