Browse code
Fix possible access of uninitialized pipe handles
Compile time warning for openvpnserv.exe
interactive.c: In function ‘RunOpenvpn’:
interactive.c:160:27: warning: ‘svc_pipe’ may be used uninitialized in
this function [-Wmaybe-uninitialized]
When RunOpenvpn exits early due to errors, uninitialized svc_pipe and
ovpn_pipe vars could get passed to CloseHandleEx(). Fix by initializing
to NULL.
Signed-off-by: Selva Nair <selva.nair@gmail.com>
Acked-by: Lev Stipakov <lstipakov@gmail.com>
Message-Id: <1582163803-3342-1-git-send-email-selva.nair@gmail.com>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg19480.html
Signed-off-by: David Sommerseth <davids@openvpn.net>
Selva Nair authored on 2020/02/20 10:56:43Showing 1 changed files
| ... | ... |
@@ -1468,7 +1468,7 @@ static DWORD WINAPI |
| 1468 | 1468 |
RunOpenvpn(LPVOID p) |
| 1469 | 1469 |
{
|
| 1470 | 1470 |
HANDLE pipe = p; |
| 1471 |
- HANDLE ovpn_pipe, svc_pipe; |
|
| 1471 |
+ HANDLE ovpn_pipe = NULL, svc_pipe = NULL; |
|
| 1472 | 1472 |
PTOKEN_USER svc_user = NULL, ovpn_user = NULL; |
| 1473 | 1473 |
HANDLE svc_token = NULL, imp_token = NULL, pri_token = NULL; |
| 1474 | 1474 |
HANDLE stdin_read = NULL, stdin_write = NULL; |