-
Michał Kępień authored
cppcheck 2.0 reports false positives about uninitialized variables in a lot of places throughout BIND source code, e.g.: bin/dnssec/dnssec-cds.c:283:6: error: Uninitialized variable: length [uninitvar] if (isc_buffer_availablelength(&buf) <= len) { ^ Apparently cppcheck 2.0 has issues with processing (&var)->field syntax, which is what the macros from lib/isc/include/isc/buffer.h are evaluated to. This issue was reported upstream [1] and will hopefully be addressed in a future cppcheck release. In the meantime, to avoid modifying BIND source code in multiple places just because of a static checker false positive, work around the issue by adding intermediate variables to buffer macro definitions using a sed invocation in the cppcheck job script. [1] https://sourceforge.net/p/cppcheck/discussion/general/thread/122153e3c1/
481fa34e
Validating GitLab CI configuration…
Learn more