storm/w3m
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Check for NULL before dereferencing the pointer

Browse Source 
In case of an error the whole frame is freed, break out of the loop to
return early.
This commit is contained in:
Rene Kita
2022-02-13 10:49:23 +01:00
parent 9f5c311e45
commit 1308f5efe5
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
w3mimg/fb/fb.c
View File

@@ -426,11 +426,13 @@ fb_frame_new(int w, int h, int n)
for (i = 0; i < n; i++) { for (i = 0; i < n; i++) {
frame[i] = fb_image_new(w, h); frame[i] = fb_image_new(w, h);
if (frame[i] == NULL) {
error = 1;
break;
}
frame[i]->num = n; frame[i]->num = n;
frame[i]->id = i; frame[i]->id = i;
frame[i]->delay = 1000; frame[i]->delay = 1000;
if (frame[i] == NULL)
error = 1;
} }
if (error) { if (error) {