Clear the previous bitmap when calculating a new window shape Fixes https://github.com/libsdl-org/SDL/issues/6428
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39
diff --git a/src/video/SDL_shape.c b/src/video/SDL_shape.c
index ef4cc60..f393bdf 100644
--- a/src/video/SDL_shape.c
+++ b/src/video/SDL_shape.c
@@ -77,8 +77,12 @@ SDL_CalculateShapeBitmap(SDL_WindowShapeMode mode,SDL_Surface *shape,Uint8* bitm
int bytes_per_scanline = (shape->w + (ppb - 1)) / ppb;
Uint8 *bitmap_scanline;
SDL_Color key;
+
if(SDL_MUSTLOCK(shape))
SDL_LockSurface(shape);
+
+ SDL_memset(bitmap, 0, shape->h * bytes_per_scanline);
+
for(y = 0;y<shape->h;y++) {
bitmap_scanline = bitmap + y * bytes_per_scanline;
for(x=0;x<shape->w;x++) {
@@ -118,6 +122,7 @@ SDL_CalculateShapeBitmap(SDL_WindowShapeMode mode,SDL_Surface *shape,Uint8* bitm
bitmap_scanline[x / ppb] |= mask_value << (x % ppb);
}
}
+
if(SDL_MUSTLOCK(shape))
SDL_UnlockSurface(shape);
}
diff --git a/src/video/directfb/SDL_DirectFB_shape.c b/src/video/directfb/SDL_DirectFB_shape.c
index da2290f..ff334c8 100644
--- a/src/video/directfb/SDL_DirectFB_shape.c
+++ b/src/video/directfb/SDL_DirectFB_shape.c
@@ -96,8 +96,7 @@ DirectFB_SetWindowShape(SDL_WindowShaper *shaper,SDL_Surface *shape,SDL_WindowSh
SDL_DFB_CHECKERR(devdata->dfb->CreateSurface(devdata->dfb, &dsc, &data->surface));
/* Assume that shaper->alphacutoff already has a value, because SDL_SetWindowShape() should have given it one. */
- SDL_DFB_ALLOC_CLEAR(bitmap, shape->w * shape->h);
- SDL_CalculateShapeBitmap(shaper->mode,shape,bitmap,1);
+ SDL_CalculateShapeBitmap(shaper->mode, shape, bitmap, 1);
src = bitmap;