[Commotion-admin] [serval-dna] fixed keyring-related memory leaks (#2)
dismantl
notifications at github.com
Fri Nov 1 01:57:43 UTC 2013
to test:
1. build `serval-crypto` against this version of `serval-dna`
2. make sure `servald` is running, and has only one key in its keyring
3. With Valgrind, run either the signing function (`valgrind --leak-check=yes --show-reachable=yes ./serval-crypto --sign -i <SID> -m <MSG>`) or verify function (`valgrind --leak-check=yes --show-reachable=yes ./ serval-crypto --verify -i <SID> -m <MSG> -s <SIG>`).
4. Valgrind should report 0 bytes {definitely,indirectly,possibly} lost (though there will be some blocks still reachable).
You can merge this Pull Request by running:
git pull https://github.com/opentechinstitute/serval-dna fix-memory-leaks
Or you can view, comment on it, or merge it online at:
https://github.com/opentechinstitute/serval-dna/pull/2
-- Commit Summary --
* fixed keyring-related memory leaks
-- File Changes --
M keyring.c (11)
-- Patch Links --
https://github.com/opentechinstitute/serval-dna/pull/2.patch
https://github.com/opentechinstitute/serval-dna/pull/2.diff
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.chambana.net/pipermail/commotion-admin/attachments/20131031/cc177b7f/attachment.html>
More information about the Commotion-admin
mailing list