Merge pull request #199 from mwierzbix/rstrcmp_nullTerminatorCheck

lgirdwood · web-flow · commit 7e5823d8b08c · 2018-08-07T09:39:40.000-07:00
lib: rstrcmp: null terminator check
diff --git a/src/lib/lib.c b/src/lib/lib.c
@@ -119,6 +119,12 @@ int rstrcmp(const char *s1, const char *s2)
 		s2++;
 	}
 
+	/* did both string end */
+	if (*s1 != 0)
+		return 1;
+	if (*s2 != 0)
+		return -1;
+
 	/* match */
 	return 0;
 }

Original file line number	Diff line number	Diff line change
`@@ -119,6 +119,12 @@ int rstrcmp(const char s1, const char s2)`
`119`	`119`	`s2++;`
`120`	`120`	`}`
`121`	`121`
	`122`	`+ /* did both string end */`
	`123`	`+ if (*s1 != 0)`
	`124`	`+ return 1;`
	`125`	`+ if (*s2 != 0)`
	`126`	`+ return -1;`
	`127`	`+`
`122`	`128`	`/* match */`
`123`	`129`	`return 0;`
`124`	`130`	`}`