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

Reply via email to