On Tue, Jul 14, 2015 at 09:57:45AM -0600, Todd C. Miller wrote:
> [...]
> Shouldn't this be:
> 
>                               p = (p - *store) + newstore;
> [...]

Of course, that makes way more sense. An amended patch is attached.

-- 
        Gregor

Index: misc.c
===================================================================
RCS file: /mnt/media/cvs/src/usr.bin/find/misc.c,v
retrieving revision 1.12
diff -u -p -u -r1.12 misc.c
--- misc.c      18 May 2014 08:10:00 -0000      1.12
+++ misc.c      14 Jul 2015 16:07:28 -0000
@@ -65,6 +65,7 @@ brace_subst(char *orig, char **store, ch
 
                                if (!(newstore = realloc(*store, newlen)))
                                        err(1, NULL);
+                               p = (p - *store) + newstore;
                                *store = newstore;
                                len = newlen;
                        }

Reply via email to