dany/libwebp
1
0
Fork 0
mirror of https://github.com/webmproject/libwebp.git synced 2025-04-08 09:56:49 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix next is invalid pointer when WebPSafeMalloc fails

Browse Source 
When WebPSafeMalloc fails on VP8LHuffmanTablesAllocate,
next is not initialized to NULL.
VP8LHuffmanTablesDeallocate uses next to know the following nodes.
A patch fixes this issue.

Change-Id: I144ae84cd97e5bca227018ef1afa95361267902c
This commit is contained in:
Masahiro Hanada 2023-09-14 19:37:24 +09:00 committed by Vincent Rabaud
parent c1ffd9ac75
commit dce8397fec
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/utils/huffman_utils.c
View File

@ -267,11 +267,11 @@ int VP8LHuffmanTablesAllocate(int size, HuffmanTables* huffman_tables) {
// Have 'segment' point to the first segment for now, 'root'. // Have 'segment' point to the first segment for now, 'root'.
HuffmanTablesSegment* const root = &huffman_tables->root; HuffmanTablesSegment* const root = &huffman_tables->root;
huffman_tables->curr_segment = root; huffman_tables->curr_segment = root;
root->next = NULL;
// Allocate root. // Allocate root.
root->start = (HuffmanCode*)WebPSafeMalloc(size, sizeof(*root->start)); root->start = (HuffmanCode*)WebPSafeMalloc(size, sizeof(*root->start));
if (root->start == NULL) return 0; if (root->start == NULL) return 0;
root->curr_table = root->start; root->curr_table = root->start;
root->next = NULL;
root->size = size; root->size = size;
return 1; return 1;
} }