From 0d4b927f3d94ede568b18e02c6e7aa301d3bebc9 Mon Sep 17 00:00:00 2001
From: rswindell <>
Date: Sun, 2 Jan 2005 00:52:06 +0000
Subject: [PATCH] Use strdup() (instead of malloc/strcpy) in strListCopy().
strListFree() now zero's the list pointer.
---
src/xpdev/str_list.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/xpdev/str_list.c b/src/xpdev/str_list.c
index d199e915bb..ceb832ca00 100644
--- a/src/xpdev/str_list.c
+++ b/src/xpdev/str_list.c
@@ -272,11 +272,9 @@ str_list_t strListSplitCopy(str_list_t* list, const char* str, const char* delim
if(str==NULL)
return(NULL);
- if((buf=(char*)malloc(strlen(str)+1))==NULL)
+ if((buf=strdup(str))==NULL)
return(NULL);
- strcpy(buf,str);
-
*list=strListSplit(list,buf,delimit);
free(buf);
@@ -347,7 +345,7 @@ void strListFree(str_list_t* list)
{
if(*list!=NULL) {
strListFreeStrings(*list);
- free(*list);
+ FREE_AND_NULL(*list);
}
}
--
GitLab