fixed bug involving freeing of memory that was never allocated, reviewed by Pinkerton

git-svn-id: svn://10.0.0.236/trunk@7674 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
morse%netscape.com
1998-08-10 20:36:41 +00:00
parent 82da783cbe
commit b28b86c55a

View File

@@ -1163,9 +1163,11 @@ SI_LoadSignonData(char * filename) {
if ((unmungedValue=SECNAV_UnMungeString(buffer+1)) == NULL) {
/* this is the free source and there is no obscuring of passwords */
unmungedValue = buffer+1;
StrAllocCopy(value_array[submit.value_cnt++], buffer+1);
} else {
StrAllocCopy(value_array[submit.value_cnt++], unmungedValue);
XP_FREE(unmungedValue);
}
StrAllocCopy(value_array[submit.value_cnt++], unmungedValue);
XP_FREE(unmungedValue);
} else {
StrAllocCopy(value_array[submit.value_cnt++], buffer+1);
}