Skip to content
Snippets Groups Projects
Commit 30f63a5c authored by Rob Swindell's avatar Rob Swindell :speech_balloon:
Browse files

NULL pointer checks in strListSort*() and strListDup()

parent f615b604
No related branches found
No related tags found
1 merge request!463MRC mods by Codefenix (2024-10-20)
...@@ -483,21 +483,25 @@ static int QSORT_CALLBACK_TYPE strListCompareAlphaCaseReverse(const void *arg1, ...@@ -483,21 +483,25 @@ static int QSORT_CALLBACK_TYPE strListCompareAlphaCaseReverse(const void *arg1,
void strListSortAlpha(str_list_t list) void strListSortAlpha(str_list_t list)
{ {
if(list != NULL)
qsort(list,strListCount(list),sizeof(char*),strListCompareAlpha); qsort(list,strListCount(list),sizeof(char*),strListCompareAlpha);
} }
void strListSortAlphaReverse(str_list_t list) void strListSortAlphaReverse(str_list_t list)
{ {
if(list != NULL)
qsort(list,strListCount(list),sizeof(char*),strListCompareAlphaReverse); qsort(list,strListCount(list),sizeof(char*),strListCompareAlphaReverse);
} }
void strListSortAlphaCase(str_list_t list) void strListSortAlphaCase(str_list_t list)
{ {
if(list != NULL)
qsort(list,strListCount(list),sizeof(char*),strListCompareAlphaCase); qsort(list,strListCount(list),sizeof(char*),strListCompareAlphaCase);
} }
void strListSortAlphaCaseReverse(str_list_t list) void strListSortAlphaCaseReverse(str_list_t list)
{ {
if(list != NULL)
qsort(list,strListCount(list),sizeof(char*),strListCompareAlphaCaseReverse); qsort(list,strListCount(list),sizeof(char*),strListCompareAlphaCaseReverse);
} }
...@@ -506,6 +510,8 @@ str_list_t strListDup(str_list_t list) ...@@ -506,6 +510,8 @@ str_list_t strListDup(str_list_t list)
str_list_t ret; str_list_t ret;
size_t count=0; size_t count=0;
if(list == NULL)
return NULL;
ret = strListInit(); ret = strListInit();
for(; *list; list++) for(; *list; list++)
strListAppend(&ret, *list, count++); strListAppend(&ret, *list, count++);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment