summaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorYann Collet <yann.collet.73@gmail.com>2015-04-12 07:21:35 (GMT)
committerYann Collet <yann.collet.73@gmail.com>2015-04-12 07:21:35 (GMT)
commit62ed15319570d80690915f0d0dba0dc2e9478631 (patch)
treef3728e1d237aed86798c5c9f0366bcdb3f500379 /lib
parent9443f3d9244faedb08527bf684994a0495117833 (diff)
downloadlz4-62ed15319570d80690915f0d0dba0dc2e9478631.zip
lz4-62ed15319570d80690915f0d0dba0dc2e9478631.tar.gz
lz4-62ed15319570d80690915f0d0dba0dc2e9478631.tar.bz2
Fixed : a few minor coverity warnings
Diffstat (limited to 'lib')
-rw-r--r--lib/lz4frame.c18
1 files changed, 12 insertions, 6 deletions
diff --git a/lib/lz4frame.c b/lib/lz4frame.c
index b6dbd20..7094364 100644
--- a/lib/lz4frame.c
+++ b/lib/lz4frame.c
@@ -372,9 +372,12 @@ LZ4F_errorCode_t LZ4F_freeCompressionContext(LZ4F_compressionContext_t LZ4F_comp
{
LZ4F_cctx_internal_t* cctxPtr = (LZ4F_cctx_internal_t*)LZ4F_compressionContext;
- FREEMEM(cctxPtr->lz4CtxPtr);
- FREEMEM(cctxPtr->tmpBuff);
- FREEMEM(LZ4F_compressionContext);
+ if (cctxPtr != NULL) /* null pointers can be safely provided to this function, like free() */
+ {
+ FREEMEM(cctxPtr->lz4CtxPtr);
+ FREEMEM(cctxPtr->tmpBuff);
+ FREEMEM(LZ4F_compressionContext);
+ }
return OK_NoError;
}
@@ -768,9 +771,12 @@ LZ4F_errorCode_t LZ4F_createDecompressionContext(LZ4F_decompressionContext_t* LZ
LZ4F_errorCode_t LZ4F_freeDecompressionContext(LZ4F_decompressionContext_t LZ4F_decompressionContext)
{
LZ4F_dctx_internal_t* dctxPtr = (LZ4F_dctx_internal_t*)LZ4F_decompressionContext;
- FREEMEM(dctxPtr->tmpIn);
- FREEMEM(dctxPtr->tmpOutBuffer);
- FREEMEM(dctxPtr);
+ if (dctxPtr != NULL) /* can accept NULL input, like free() */
+ {
+ FREEMEM(dctxPtr->tmpIn);
+ FREEMEM(dctxPtr->tmpOutBuffer);
+ FREEMEM(dctxPtr);
+ }
return OK_NoError;
}