]> granicus.if.org Git - procps-ng/commitdiff
ps/sortformat.c: Avoid "sep_loc + 1" when sep_loc is NULL.
authorQualys Security Advisory <qsa@qualys.com>
Thu, 1 Jan 1970 00:00:00 +0000 (00:00 +0000)
committerCraig Small <csmall@enc.com.au>
Fri, 18 May 2018 21:32:22 +0000 (07:32 +1000)
ps/sortformat.c

index 08cc71fe7b372b61c87414727e33b8c6b235f9e5..2bac8a1c0889f5ee23af3c20eb3911a2bd70fd34 100644 (file)
@@ -260,6 +260,7 @@ static const char *format_parse(sf_node *sfn){
     format_node *endp;
     char *equal_loc;
     char *colon_loc;
+    if(!walk) catastrophic_failure(__FILE__, __LINE__, _("please report this bug"));
     sep_loc = strpbrk(walk," ,\t\n");
     /* if items left, then sep_loc is not in header override */
     if(items && sep_loc) *sep_loc = '\0';
@@ -302,7 +303,7 @@ static const char *format_parse(sf_node *sfn){
     endp = fnode; while(endp->next) endp = endp->next;  /* find end */
     endp->next = sfn->f_cooked;
     sfn->f_cooked = fnode;
-    walk = sep_loc + 1; /* point to next item, if any */
+    walk = sep_loc ? sep_loc + 1 : NULL; /* point to next item, if any */
   }
   free(buf);
   already_parsed_format = 1;