diff options
| author | Jia Tan <jiat0218@gmail.com> | 2022-09-21 16:15:50 +0800 |
|---|---|---|
| committer | Lasse Collin <lasse.collin@tukaani.org> | 2022-09-28 15:48:03 +0300 |
| commit | 8cc9874a7974cd575aee44f218836f7acdbeb0ed (patch) | |
| tree | bc9ec8f2a8f225bd21496ef2f4fd2ce6e38ea661 /src/liblzma/common | |
| parent | Tests: Create a test for the lzma_index_cat bug. (diff) | |
| download | xz-8cc9874a7974cd575aee44f218836f7acdbeb0ed.tar.xz | |
liblzma: Add dest and src NULL checks to lzma_index_cat.
The documentation states LZMA_PROG_ERROR can be returned from
lzma_index_cat. Previously, lzma_index_cat could not return
LZMA_PROG_ERROR. Now, the validation is similar to
lzma_index_append, which does a NULL check on the index
parameter.
Diffstat (limited to '')
| -rw-r--r-- | src/liblzma/common/index.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/liblzma/common/index.c b/src/liblzma/common/index.c index e0b14a3d..24ec3c10 100644 --- a/src/liblzma/common/index.c +++ b/src/liblzma/common/index.c @@ -771,6 +771,9 @@ extern LZMA_API(lzma_ret) lzma_index_cat(lzma_index *restrict dest, lzma_index *restrict src, const lzma_allocator *allocator) { + if (dest == NULL || src == NULL) + return LZMA_PROG_ERROR; + const lzma_vli dest_file_size = lzma_index_file_size(dest); // Check that we don't exceed the file size limits. |