Commit 87b5102f authored by Per Cederqvist's avatar Per Cederqvist
Browse files

Valgrind improvements. Plug a memory leak.

parent cfa29fef
2002-08-02 Per Cederqvist <>
Valgrind refinements.
* src/server/testsuite/config/unix.exp (spawn_lyskomd): Use
* src/server/testsuite/ (valgrind.wrap): New target.
Since valgrind can only send the log to a specified file
descriptor, and expect cannot open a specified file descriptor, we
have to use a wrapper for valgrind.
(noinst_DATA): Added
(MOSTLYCLEANFILES): Added valgrind-*.log, valgrind.log and
(check-l2g): Depend on valgrind.wrap.
(check-lyskomd): Ditto.
(check-leaks): Ditto.
Plug a memory leak.
* src/server/aux-items.c (aux_item_definition_cache_regexp): Call
regfree() when a regex compilation fail, since our
re_compile_pattern may leave allocated stuff in the buffer.
Improved valgrind support.
* Remove all remnants of the purify support, since
it was broken when we switched to automake. Added valgrind
* src/server/testsuite/.cvsignore: Ignore valgrind-*.log.
* src/server/testsuite/ (site.exp): Set valgrind.
* src/server/testsuite/config/unix.exp: Use the "valgrind"
variable from site.exp. It contains a the path name of the
valgrind binary to use.
......@@ -18,6 +39,8 @@
* src/server/testsuite/lyskomd.0/regexp-match-cov.exp: Bug 689 is
a hard-to-fix memory leak. Mark it as an expected failure.
(shutdown): Pass info about expected leaks to lyskomd_death.
* src/server/testsuite/lyskomd.0/aux-items-cov.exp: This triggers
a hard-to-fix memory leak (Bug 689).
Shut up valgrind.
* src/server/memory.c (init_person): Clear the pwd field. This
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment