Skip to content

Commit

Permalink
ImgIoUtilReadFile: check ftell() return
Browse files Browse the repository at this point in the history
This avoids attempting to allocate 0 bytes if the call fails. (An
additional byte is added to the result to allow a '\0' to be appended.)

Bug: chromium:334120888
Change-Id: Ifcb8ff7744c567cbd08051aa04cc66acf936078d
  • Loading branch information
jzern committed Apr 19, 2024
1 parent dc95058 commit 3cada4c
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions imageio/imageio_util.c
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,11 @@ int ImgIoUtilReadFile(const char* const file_name,
}
fseek(in, 0, SEEK_END);
file_size = ftell(in);
if (file_size == (size_t)-1) {
fclose(in);
WFPRINTF(stderr, "error getting size of '%s'\n", (const W_CHAR*)file_name);
return 0;
}
fseek(in, 0, SEEK_SET);
// we allocate one extra byte for the \0 terminator
file_data = (uint8_t*)WebPMalloc(file_size + 1);
Expand Down

0 comments on commit 3cada4c

Please sign in to comment.