]> git.ipfire.org Git - thirdparty/zstd.git/commitdiff
fix scope of scan-build tests
authorYann Collet <cyan@fb.com>
Thu, 16 Aug 2018 00:41:44 +0000 (17:41 -0700)
committerYann Collet <cyan@fb.com>
Thu, 16 Aug 2018 00:41:44 +0000 (17:41 -0700)
exclude zlib code

Makefile
lib/decompress/zstd_decompress.c

index d3375760329775eb1867e3a5a85ec826255f510b..e338c5ca199b5cc59e9892ffeb8b70ead9efa231 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -32,7 +32,7 @@ all: allmost examples manual contrib
 .PHONY: allmost
 allmost: allzstd zlibwrapper
 
-#skip zwrapper, can't build that on alternate architectures without the proper zlib installed
+# skip zwrapper, can't build that on alternate architectures without the proper zlib installed
 .PHONY: allzstd
 allzstd: lib
        $(MAKE) -C $(PRGDIR) all
@@ -44,8 +44,7 @@ all32:
        $(MAKE) -C $(TESTDIR) all32
 
 .PHONY: lib lib-release libzstd.a
-lib : libzstd.a
-lib lib-release libzstd.a:
+lib lib-release :
        @$(MAKE) -C $(ZSTDDIR) $@
 
 .PHONY: zstd zstd-release
@@ -59,7 +58,7 @@ zstdmt:
        cp $(PRGDIR)/zstd$(EXT) ./zstdmt$(EXT)
 
 .PHONY: zlibwrapper
-zlibwrapper: libzstd.a
+zlibwrapper: lib
        $(MAKE) -C $(ZWRAPDIR) all
 
 .PHONY: test
@@ -351,7 +350,9 @@ bmi32build: clean
        $(CC) -v
        CFLAGS="-O3 -mbmi -m32 -Werror" $(MAKE) -C $(TESTDIR) test
 
+# static analyzer test uses clang's scan-build
+# does not analyze zlibWrapper, due to detected issues in zlib source code
 staticAnalyze:
        $(CC) -v
-       CC=$(CC) CPPFLAGS=-g scan-build --status-bugs -v $(MAKE) all
+       CC=$(CC) CPPFLAGS=-g scan-build --status-bugs -v $(MAKE) allzstd examples contrib
 endif
index 5d9f0ba0b02ac885f8e50ed8c9b1582410ef8a9a..8fef7e51b4ab4bf0f6b4864797a99f6c00ed7184 100644 (file)
@@ -2399,6 +2399,7 @@ static size_t ZSTD_initDDict_internal(ZSTD_DDict* ddict,
     if ((dictLoadMethod == ZSTD_dlm_byRef) || (!dict) || (!dictSize)) {
         ddict->dictBuffer = NULL;
         ddict->dictContent = dict;
+        if (!dict) dictSize = 0;
     } else {
         void* const internalBuffer = ZSTD_malloc(dictSize, ddict->cMem);
         ddict->dictBuffer = internalBuffer;