geos_one/CASA
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix buffer overrun and double free problem.

Browse Source
This commit is contained in:
Jim Norman 2007-01-26 11:46:36 +00:00
parent 1ae3790b1b
commit de7dbd7b01
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
CASA/adlib/ad_ff/native/SignonManager.cpp
View File

@ -411,7 +411,7 @@ int count = 0;
// Add the name/value pair to the existing store.... // Add the name/value pair to the existing store....
if (count == 1) if (count == 1)
{ {
uname = (char *) malloc(strlen(clearData)*sizeof(char)); uname = (char *) malloc(strlen(clearData)*sizeof(char) + 1);
strcpy(uname, clearData); strcpy(uname, clearData);
} }
@ -427,7 +427,9 @@ int count = 0;
dataManager.AddHost(newHostName); dataManager.AddHost(newHostName);
} }
retValue = dataManager.AddHostElement(newHostName, name, clearData, isPassword); retValue = dataManager.AddHostElement(newHostName, name, clearData, isPassword);
free(newHostName);
if(count%2==0)
free(newHostName);
} }
else else
retValue = dataManager.AddHostElement(hostName, name, clearData, isPassword); retValue = dataManager.AddHostElement(hostName, name, clearData, isPassword);