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 (file)
--- 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';
                }