Changeset 910 for trunk/src/gmakenew/file.c

Timestamp:

May 24, 2007, 5:10:51 AM (18 years ago)

Author:

bird

Message:

Fixed a few potential variable_buffer realloc problems.

File:

• trunk/src/gmakenew/file.c (modified) (5 diffs)

trunk/src/gmakenew/file.c

@@ -474 +474 @@
   for (d = old; d != 0; d = d->next)
     {
+      size_t buffer_offset; /* bird */
       struct dep *new, *d1;
       char *p;
@@ -485 +486 @@
       if (! d->need_2nd_expansion)
         {
-          size_t l;                         /* bird */
           p = variable_expand ("");
-          l = p - variable_buffer;          /* bird */
+          buffer_offset = p - variable_buffer;
           variable_buffer_output (p, d->name, strlen (d->name) + 1);
-          p = variable_buffer + l;          /* bird - this may have been reallocated! */
+          p = variable_buffer + buffer_offset; /* bird - variable_buffer may have been reallocated. (observed it) */
         }
       else
@@ -499 +499 @@
               char *o;
               char *buffer = variable_expand ("");
+              buffer_offset = buffer - variable_buffer; /* bird */
 
               o = subst_expand (buffer, d->name, "%", "$*", 1, 2, 0);
+              buffer = variable_buffer + buffer_offset; /* bird - variable_buffer may have been reallocated. */
 
               d->name = strcache_add_len (buffer, o - buffer);
@@ -538 +540 @@
           const char *pattern = "%";
           char *buffer = variable_expand ("");
+          const size_t buffer_offset = buffer - variable_buffer; /* bird */
           struct dep *dp = new, *dl = 0;
 
@@ -562 +565 @@
                     o = patsubst_expand_pat (buffer, d->stem, pattern, nm,
                                              pattern+1, percent+1);
+                  buffer = variable_buffer + buffer_offset; /* bird - variable_buffer may have been reallocated. */
+
 
                   /* If the name expanded to the empty string, ignore it.  */