fix: Added check for empty subfeature that could cause memory read-pa…

…st-end
LBNL-ETA · Apr 5, 2024 · 425d236 · 425d236
1 parent 1065f52
commit 425d236
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/rt/renderopts.c b/src/rt/renderopts.c
@@ -1,5 +1,5 @@
 #ifndef lint
-static const char	RCSid[] = "$Id: renderopts.c,v 2.25 2024/04/05 17:52:20 greg Exp $";
+static const char	RCSid[] = "$Id: renderopts.c,v 2.26 2024/04/05 17:55:25 greg Exp $";
 #endif
 /*
  *  renderopts.c - process common rendering options
@@ -75,7 +75,8 @@ match_subfeatures(	/* check if subfeatures are supported */
 		while (*reqs && (*cp = *reqs++) != ',')
 			cp++;
 		*cp = '\0';
-		n = cp - subfeat;
+		if (!(n = cp - subfeat))
+			continue;	/* empty subfeature */
 		for (cp = mysublist; (cp = strstr(cp, subfeat)) != NULL; cp++)
 			if ((cp[-1] == ',') | (cp[-1] == '=') &&
 					(cp[n] == ',') | (cp[n] == '\n'))