mirrors/FreeRDP
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed #4993: Allow uninitialized bio socket in transport_bio_simple_uninit

Browse Source 
When the TLS connection is aborted early (certificate issues, ...)
the bio simple socket might not have been initialized.
Do not access the NULL pointer on cleanup.
This commit is contained in:
Armin Novak 2018-11-07 17:01:28 +01:00
parent cf43406dc3
commit 5eca5ebde8
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
libfreerdp/core/tcp.c
View File

@ -380,7 +380,7 @@ static int transport_bio_simple_uninit(BIO* bio)
if (BIO_get_shutdown(bio)) if (BIO_get_shutdown(bio))
{ {
if (BIO_get_init(bio)) if (BIO_get_init(bio) && ptr)
{ {
_shutdown(ptr->socket, SD_BOTH); _shutdown(ptr->socket, SD_BOTH);
closesocket(ptr->socket); closesocket(ptr->socket);
@ -388,7 +388,7 @@ static int transport_bio_simple_uninit(BIO* bio)
} }
} }
if (ptr->hEvent) if (ptr && ptr->hEvent)
{ {
CloseHandle(ptr->hEvent); CloseHandle(ptr->hEvent);
ptr->hEvent = NULL; ptr->hEvent = NULL;