Changeset 1477 for trunk/src

Timestamp:

Apr 1, 2008, 2:05:01 AM (17 years ago)

Author:

bird

Message:

Check for '\0' in defines, just to keep the value_length sane.

File:

• trunk/src/kmk/read.c (modified) (1 diff)

trunk/src/kmk/read.c

@@ -629 +629 @@
           if (!found_endef)
             {
-              error (f, "%s(%d): bogus define statement.", name, line_no);
+              error (f, "%s(%d): missing endef, dropping the rest of the file.", name, line_no);
               break;
             }
+          value_len = value_end - value_start;
+          if (memchr (value_start, '\0', value_len))
+            {
+              error (f, "%s(%d): '\\0' in define, dropping the rest of the file.", name, line_no);
+              break;
+            }
 
           /* make a copy of the value, converting \r\n to \n, and define it. */
-          value_len = value_end - value_start;
           value = xmalloc (value_len + 1);
           endp = memchr (value_start, '\r', value_len);