[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