Skip to content

Commit 2b8afc6

Browse files
flichtenheldcron2
authored andcommitted
openvpnserv: always use W variant of RpcStringFree()
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 <frank@lichtenheld.com> Acked-by: Heiko Hund <heiko@openvpn.net> Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1671 Message-Id: <20260514091918.18197-1-gert@greenie.muc.de> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg36915.html Signed-off-by: Gert Doering <gert@greenie.muc.de>
1 parent ae63905 commit 2b8afc6

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/openvpnserv/interactive.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3487,7 +3487,7 @@ RunOpenvpn(LPVOID p)
34873487
swprintf(ovpn_pipe_name, _countof(ovpn_pipe_name),
34883488
L"\\\\.\\pipe\\" _L(PACKAGE) L"%ls\\service_%lu_%ls", service_instance,
34893489
GetCurrentThreadId(), pipe_uuid_str);
3490-
RpcStringFree(&pipe_uuid_str);
3490+
RpcStringFreeW(&pipe_uuid_str);
34913491

34923492
/* make a security descriptor for the named pipe with access
34933493
* restricted to the user and SYSTEM

0 commit comments

Comments
 (0)