m_getfld2: Whitespace in component name is forbidden
authormarkus schnalke <meillo@marmaro.de>
Thu, 12 Nov 2015 07:14:59 +0000 (08:14 +0100)
committermarkus schnalke <meillo@marmaro.de>
Thu, 12 Nov 2015 21:41:46 +0000 (22:41 +0100)
Also on style: No function calls should be used to initialize
variables in definition statements. Use separate assignments in
this case.

sbr/m_getfld2.c

index 91164f3..a17eb03 100644 (file)
@@ -175,11 +175,17 @@ static size_t
 copyname(char *dst, char *src)
 {
        size_t len;
-       char *sep = strchr(src, ':');
+       char *cp, *sep;
 
-       if (!sep) {
+       if (!(sep = strchr(src, ':'))) {
                return 0;
        }
+       /* whitespace is forbidden in name */
+       for (cp=src; cp<sep; cp++) {
+               if (isspace(*cp)) {
+                       return 0;
+               }
+       }
 
        len = sep - src;
        if (len >= NAMESZ) {