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

core/info: Fix leak found by covscan

Browse Source 
leaked_storage: Variable "wString" going out of scope leaks the storage it points to.
This commit is contained in:
Ondrej Holy 2018-08-17 16:10:26 +02:00
parent 83e966d9e2
commit 4a7bb18428
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
libfreerdp/core/info.c
View File

@ -1218,7 +1218,10 @@ static BOOL rdp_write_logon_info_v1(wStream* s, logon_info* info)
len *= 2;
if (len > 52)
{
free(wString);
return FALSE;
}
Stream_Write_UINT32(s, len);
Stream_Write(s, wString, len);
@ -1233,7 +1236,10 @@ static BOOL rdp_write_logon_info_v1(wStream* s, logon_info* info)
len *= 2;
if (len > 512)
{
free(wString);
return FALSE;
}
Stream_Write_UINT32(s, len);
Stream_Write(s, wString, len);