Fix `make memcheck`#2785

Merged

jimklimov merged 22 commits intonetworkupstools:masterfrom jimklimov:fix-memcheck

Jan 26, 2025

+363-127

Commits on Jan 25, 2025

tests/Makefile.am: memcheck: report OVERALL FAILED cases (if any)
jimklimov
committed
scripts/valgrind/valgrind.sh: allow to customize the VALGRIND to be used
jimklimov
committed
tests/Makefile.am: memcheck: use the NUT-customized wrapper for valgrind
jimklimov
committed
configure.ac, scripts/valgrind/valgrind.sh.in, tests/Makefile.am: make the helper script a template
jimklimov
committed
scripts/valgrind/valgrind.sh.in: comment about --gen-suppressions=all option
jimklimov
committed
scripts/valgrind/.valgrind.supp: add suppressions for C++ tests (CPPUNIT leaky internals)
jimklimov
committed
scripts/valgrind/.valgrind.supp: add suppressions for dash interpreter
jimklimov
committed
common/nutstream.cpp: checkExistsWritableDir(): fix memory leak with opened DIRFD
jimklimov
committed
tests/nutconf_ut.cpp: NutConfigUnitTest::testUpsmonConfiguration(): fix memory leak with temporary BoolInt instances
jimklimov
committed
tests/generic_gpio_utest.c: refactor work with generic_gpio_close() to avoid use after free
jimklimov
committed
tests/generic_gpio_utest.c: comment away upsdrv_cleanup() in codepath which we did not upsdrv_init()
jimklimov
committed
tests/generic_gpio_utest.c: dstate_free() after the test run
jimklimov
committed
drivers/generic_gpio_common.c: fix code markup to adhere closer to NUT coding style
jimklimov
committed
drivers/generic_gpio_common.c: upsdrv_cleanup(): only free resources once
jimklimov
committed
tests/generic_gpio_utest.c: do upsdrv_cleanup() now that it is safe to occasionally repeat the attempt
jimklimov
committed
drivers/generic_gpio_common.c: generic_gpio_close(): free(gpioupsfdlocal->rules) itself too, not just entries in the list
jimklimov
committed
drivers/generic_gpio_common.{c,h}, tests/generic_gpio_utest.c: refactor generic_gpio_close() to take a pointer to pointer and nullify it
jimklimov
committed
tests/generic_gpio_utest.c: Be sure the internal gpioupsfd in driver code is freed before we (re?)init it in test
jimklimov
committed
tests/generic_gpio_utest.c: fix inverted strcmp() result treatment
jimklimov
committed
drivers/generic_gpio_common.c: generic_gpio_open(): reshuffle to optimize
jimklimov
committed
Revert "tests/generic_gpio_utest.c: Be sure the internal gpioupsfd in driver code is freed before we (re?)init it in test"
jimklimov
committed
scripts/valgrind/valgrind.sh.in: revise detection of VALGRIND and LIBTOOL values
jimklimov
committed