Acked-by: Gert Doering <g...@greenie.muc.de> I do not really think this is *necessary* (if malloc() fails for a few bytes, how relevant are these free() calls, and how likely is that the next malloc() inside OpenVPN will just make everything stop?) - but it's good practice, and if it silences semi-irrelevant warnings, the check tools can show us the *real* issues...
I have only test compiled the plugins, not actually run anything. Your patch has been applied to the master and release/2.6 branch. commit 2d36678a2be15f7c00a44354ab71e4521ee3a4f3 (master) commit 13b8e155af702a25e86f20da694867c76c169673 (release/2.6) Author: Frank Lichtenheld Date: Tue May 16 11:35:34 2023 +0200 sample-plugins: Fix memleak in client-connect example plugin Signed-off-by: Frank Lichtenheld <fr...@lichtenheld.com> Acked-by: Gert Doering <g...@greenie.muc.de> Message-Id: <20230516093534.26384-1-fr...@lichtenheld.com> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg26668.html Signed-off-by: Gert Doering <g...@greenie.muc.de> -- kind regards, Gert Doering _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel