From: Frank Lichtenheld <[email protected]> Probably not a real issue since it should always use the W version in our case, but still unclean.
Fixes a cppcheck complaint. Change-Id: I3d391016dcd3c66e58f3f05b356bcd419224acb1 Signed-off-by: Frank Lichtenheld <[email protected]> Acked-by: Heiko Hund <[email protected]> Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1671 --- This change was reviewed on Gerrit and approved by at least one developer. I request to merge it to master. Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1671 This mail reflects revision 2 of this Change. Acked-by according to Gerrit (reflected above): Heiko Hund <[email protected]> diff --git a/src/openvpnserv/interactive.c b/src/openvpnserv/interactive.c index e25c511..bd310e4 100644 --- a/src/openvpnserv/interactive.c +++ b/src/openvpnserv/interactive.c @@ -3484,7 +3484,7 @@ swprintf(ovpn_pipe_name, _countof(ovpn_pipe_name), L"\\\\.\\pipe\\" _L(PACKAGE) L"%ls\\service_%lu_%ls", service_instance, GetCurrentThreadId(), pipe_uuid_str); - RpcStringFree(&pipe_uuid_str); + RpcStringFreeW(&pipe_uuid_str); /* make a security descriptor for the named pipe with access * restricted to the user and SYSTEM _______________________________________________ Openvpn-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
