Skip to content
Snippets Groups Projects
Commit dabfec32 authored by rswindell's avatar rswindell
Browse files

Bugfix: iniRemoveValue() wasn't checking if key_name() returned INI_NEW_SECTION 

(an invalid pointer).
parent 67534fe9
Branches
Tags
No related merge requests found
Show whitespace changes
Inline Side-by-side
Showing
with 3 additions and 1 deletion
src/xpdev/ini_file.c
+ 3
− 1
View file @ dabfec32
...@@ -262,11 +262,13 @@ BOOL iniRemoveValue(str_list_t* list, const char* section, const char* key) ...@@ -262,11 +262,13 @@ BOOL iniRemoveValue(str_list_t* list, const char* section, const char* key)
{ {
char val[INI_MAX_VALUE_LEN]; char val[INI_MAX_VALUE_LEN];
size_t i; size_t i;
char* p;
char* vp; char* vp;
i=find_value_index(*list, section, key, val); i=find_value_index(*list, section, key, val);
if(key_name((*list)[i], &vp)==NULL) p=key_name((*list)[i], &vp);
if(p===NULL || p==INI_NEW_SECTION)
return(FALSE); return(FALSE);
*vp=0; /* Terminate string at beginning of value */ *vp=0; /* Terminate string at beginning of value */
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment