From 1d498daa95384e5c9ad5bcb35e7a996e5869ac39 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Fri, 10 Apr 2009 03:56:06 +0000 Subject: Simplified code and possible copy problem fixed. --- ChangeLog | 1 + stdio-common/vfprintf.c | 20 +++----------------- 2 files changed, 4 insertions(+), 17 deletions(-) diff --git a/ChangeLog b/ChangeLog index fe0a899819..c0d26fccce 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,6 +1,7 @@ 2009-04-09 Ulrich Drepper * stdio-common/vfprintf.c (vfprintf): Slightly more compact code. + Simplified code and possible copy problem fixed. * sysdeps/unix/sysv/linux/preadv.c: Avoid prototype for static function if it is not defined. Add some necessary casts. diff --git a/stdio-common/vfprintf.c b/stdio-common/vfprintf.c index 819865f61e..5a24e725d5 100644 --- a/stdio-common/vfprintf.c +++ b/stdio-common/vfprintf.c @@ -1662,24 +1662,10 @@ do_positional: { /* Extend the array of format specifiers. */ struct printf_spec *old = specs; + specs = extend_alloca (specs, nspecs_max, 2 * nspecs_max); - nspecs_max *= 2; - specs = alloca (nspecs_max * sizeof (struct printf_spec)); - - if (specs == &old[nspecs]) - /* Stack grows up, OLD was the last thing allocated; - extend it. */ - nspecs_max += nspecs_max / 2; - else - { - /* Copy the old array's elements to the new space. */ - memcpy (specs, old, nspecs * sizeof (struct printf_spec)); - if (old == &specs[nspecs]) - /* Stack grows down, OLD was just below the new - SPECS. We can use that space when the new space - runs out. */ - nspecs_max += nspecs_max / 2; - } + /* Copy the old array's elements to the new space. */ + memmove (specs, old, nspecs * sizeof (struct printf_spec)); } /* Parse the format specifier. */ -- cgit 1.4.1