From: markus schnalke <meillo@marmaro.de>
Date: Wed, 4 Jan 2012 10:16:12 +0000 (+0100)
Subject: Fixed bug in brkstring(), newly introduced by the switch to strchr().
X-Git-Tag: mmh-thesis-end~433
X-Git-Url: http://git.marmaro.de/?a=commitdiff_plain;h=909cdf33782fb9803b8e18f7f378bfcec21e4b15;p=mmh

Fixed bug in brkstring(), newly introduced by the switch to strchr().
strchr() treats the trailing null byte surprisingly:
``The terminating null byte is considered to be part of the string.''
---

diff --git a/sbr/brkstring.c b/sbr/brkstring.c
index 68c7b70..da42d61 100644
--- a/sbr/brkstring.c
+++ b/sbr/brkstring.c
@@ -46,7 +46,7 @@ brkstring(char *str, char *brksep, char *brkterm)
 		}
 
 		/* handle separators */
-		while (strchr(brksep, c = *s)) {
+		while ((c=*s) && strchr(brksep, c)) {
 			*s++ = '\0';
 		}