From dabfec32dff1740815bcaefbc005aecbcf26373c Mon Sep 17 00:00:00 2001
From: rswindell <>
Date: Sat, 3 Jul 2004 00:09:05 +0000
Subject: [PATCH] Bugfix: iniRemoveValue() wasn't checking if key_name()
 returned INI_NEW_SECTION (an invalid pointer).

---
 src/xpdev/ini_file.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/xpdev/ini_file.c b/src/xpdev/ini_file.c
index 58ae3bb92b..e2e480d18a 100644
--- a/src/xpdev/ini_file.c
+++ b/src/xpdev/ini_file.c
@@ -262,11 +262,13 @@ BOOL iniRemoveValue(str_list_t* list, const char* section, const char* key)
 {
 	char	val[INI_MAX_VALUE_LEN];
 	size_t	i;
+    char*   p;
 	char*	vp;
 
 	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);
 
 	*vp=0;	/* Terminate string at beginning of value */
-- 
GitLab