Skip to content

Commit 32eba9b

Browse files
committed
Fix CodeQL error
Multiplication result converted to larger type
1 parent 6bb4dca commit 32eba9b

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

i3lock.c

+5-4
Original file line numberDiff line numberDiff line change
@@ -1230,19 +1230,20 @@ static cairo_surface_t *read_gif_image(const char *image_path) {
12301230
int img_bottom = img_top + pimg->ImageDesc.Height;
12311231

12321232
/* Handle disposal mode */
1233+
int data_size = width*height*((int)sizeof(width));
12331234
switch (gc.DisposalMode) {
12341235
case DISPOSE_DO_NOT:
12351236
if (data_prev) {
1236-
memcpy(data, data_prev, width*height*sizeof(uint32_t));
1237+
memcpy(data, data_prev, data_size);
12371238
} else {
1238-
memset(data, 0, width*height*sizeof(uint32_t));
1239+
memset(data, 0, data_size);
12391240
}
12401241
break;
12411242
case DISPOSE_BACKGROUND:
1242-
memset(data, bg_color, width*height*sizeof(uint32_t));
1243+
memset(data, bg_color, data_size);
12431244
break;
12441245
default:
1245-
memset(data, 0, width*height*sizeof(uint32_t));
1246+
memset(data, 0, data_size);
12461247
}
12471248

12481249
/* Read RGB */

0 commit comments

Comments
 (0)