ACK. Makes sense. The changes to crypto_mbedtls.c match the documentation for the mbedtls md functions, and the rest is just basic "if you make it, clean it up later on again" :-) - lightly tested ("t_client"), too lazy to come up with a test rig to trigger and truly test this.
Your patch has been applied to the master and release/2.4 branch. commit 07036fd3c456ed4ebf1809d8d9f34941d42865d0 (master) commit d04032c3f236cf7c726d6b162686bcb71fda2e15 (release/2.4) Author: Steffan Karger Date: Wed Jan 17 14:16:24 2018 +0100 Plug memory leak if push is interrupted Signed-off-by: Steffan Karger <steffan.kar...@fox-it.com> Acked-by: Gert Doering <g...@greenie.muc.de> Message-Id: <1516194984-1540-1-git-send-email-steffan.kar...@fox-it.com> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg16265.html Signed-off-by: Gert Doering <g...@greenie.muc.de> -- kind regards, Gert Doering ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel