Changeset 1699 for trunk/src

Timestamp:

Sep 2, 2008, 3:45:50 AM (17 years ago)

Author:

bird

Message:

kmk / $(commands* ): Check for empty name to prevent lookup_file from asserting, print error and return. Print error message when detecting recursion.

File:

• trunk/src/kmk/function.c (modified) (2 diffs)

trunk/src/kmk/function.c

@@ -3652 +3652 @@
 
   if (recursive)
-    return variable_buffer_output (o, "recursive", sizeof ("recursive") - 1);
+    {
+      error (reading_file, _("$(%s ) was invoked recursivly"), funcname);
+      return variable_buffer_output (o, "recursive", sizeof ("recursive") - 1);
+    }
+  if (*argv[0] == '\0')
+    {
+      error (reading_file, _("$(%s ) was invoked with an empty target name"), funcname);
+      return o;
+    }
   recursive = 1;
-
+  
   file = lookup_file (argv[0]);
   if (file && file->cmds)
@@ -3814 +3822 @@
           else
             o = p;
-        }
+        } /* for each command line */
     }
   /* else FIXME: bitch about it? */