Willy,
these patches are the result of running
scan-build make -j4 all TARGET=linux-glibc DEBUG=-DDEBUG_STRICT=1
While it spits out a bunch of false negatives that are quite convoluted these
patches either:
1. Make it easier for clang analyzer to understand the code by making a few
*small*
adjustments. The one in `proxy_parse_declare` might be questionable, though.
2. Actually fix an issue I could reproduce with a carefully crafted example
configuration.
Cc'd William on patch 4 (worker) and Christopher on 5 (spoe).
Tim Duesterhus (9):
BUG/MINOR: cfgparse: Pass correct type to `calloc`
BUG/MINOR: log: Detect missing sampling ranges in config
BUG/MINOR: cfgparse: Stop passing NULL to memcpy
BUG/MINOR: mworker: Fix segmentation fault during cfgparse
BUG/MINOR: spoe: Fix memory leak if failing to allocate memory
BUG/MINOR: debug: Silence warning about ignored return value
MINOR: debug: Make ABORT_NOW() emit abort() for clang analyzer
MINOR: proxy: Restructure code to assert that `proxy_parse_declare`
does not leak
MINOR: h2: Assert that a status line exists in
h2_make_htx_(request|response)
include/common/debug.h | 6 +++++-
src/cfgparse.c | 5 +++--
src/flt_spoe.c | 1 +
src/h2.c | 8 ++++++++
src/log.c | 5 +++++
src/mworker-prog.c | 30 ++++++++++++++++--------------
src/proxy.c | 6 +++++-
7 files changed, 43 insertions(+), 18 deletions(-)
--
2.21.0
