Reduce valgrind memory leak warnings in depmod and modprobe
authorAlan Jenkins <alan-jenkins@tuffmail.co.uk>
Mon, 13 Oct 2008 05:31:49 +0000 (01:31 -0400)
committerJon Masters <jcm@jonmasters.org>
Mon, 13 Oct 2008 05:31:49 +0000 (01:31 -0400)
commit04170aaff08cedcf3a446c2a5d7d350474592bf4
tree0c22dbfd03fc0741a2d6d69954f9402a8be02d05
parentf5b39425681002a2407bc6f2e95ba8d5113a87ab
Reduce valgrind memory leak warnings in depmod and modprobe

Reducing valgrind leak warnings makes it easier to see problems in new code.
Specifically, the new indexing code will introduce in-memory index data
structures with hairy algorithms.  Memory leaks there could be a sign of
serious problems.

These are slightly less clean fixes to some remaining issues found by valgrind.

* depmod should now be clean of valgrind leaks.

* vg-suppressions is extended to ignore errors from aggressive GCC optimization.
  The errors affected depmod when run on my system, but not the test cases.

* Two sources of leaks remain in modprobe
   1) Lists of modules, blacklists, options etc. leak after loading a module.
   2) gzFile leak justified by POSIX locking semantics being "horribly broken".
  These require rather more complex fixes, which would probably be pointless.
  modprobe is short lived and won't miss the leaked memory.

Signed-off-by: Jon Masters <jcm@jonmasters.org>
depmod.c
tests/vg-suppressions