From 94c8503486ad394245f455924fa0bbb8539d8282 Mon Sep 17 00:00:00 2001 From: Jia Tan Date: Sat, 16 Dec 2023 20:18:47 +0800 Subject: [PATCH] liblzma: Improve lzma encoder init function consistency. lzma_encoder_init() did not check for NULL options, but lzma2_encoder_init() did. This is more of a code style improvement than anything else to help make lzma_encoder_init() and lzma2_encoder_init() more similar. (cherry picked from commit 019afd72e02339a6bf00c32bfb56f649c637dd6b) --- src/liblzma/lzma/lzma_encoder.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/liblzma/lzma/lzma_encoder.c b/src/liblzma/lzma/lzma_encoder.c index bbbb5d2d..01ca1b4f 100644 --- a/src/liblzma/lzma/lzma_encoder.c +++ b/src/liblzma/lzma/lzma_encoder.c @@ -620,6 +620,9 @@ static lzma_ret lzma_encoder_init(lzma_lz_encoder *lz, const lzma_allocator *allocator, const void *options, lzma_lz_options *lz_options) { + if (options == NULL) + return LZMA_PROG_ERROR; + lz->code = &lzma_encode; return lzma_lzma_encoder_create( &lz->coder, allocator, options, lz_options);