diff options
| author | Petr Baudis <pasky@suse.cz> | 2011-02-20 07:59:49 -0500 |
|---|---|---|
| committer | Ulrich Drepper <drepper@gmail.com> | 2011-02-20 07:59:49 -0500 |
| commit | 84a4211850e3d23a9d3a4f3b294752a3b30bc0ff (patch) | |
| tree | 04faa4e3387e0e86f2c0fc9ef47af138f1258006 /stdio-common | |
| parent | e23fe25b33324a9ea992276c1a4f04127bf9ba4b (diff) | |
| download | glibc-84a4211850e3d23a9d3a4f3b294752a3b30bc0ff.tar.gz | |
Fix allocation when handling positional parameters in printf.
Diffstat (limited to 'stdio-common')
| -rw-r--r-- | stdio-common/Makefile | 2 | ||||
| -rw-r--r-- | stdio-common/bug23.c | 21 | ||||
| -rw-r--r-- | stdio-common/vfprintf.c | 5 |
3 files changed, 25 insertions, 3 deletions
diff --git a/stdio-common/Makefile b/stdio-common/Makefile index 6aabfb6b54..6c71f1c37f 100644 --- a/stdio-common/Makefile +++ b/stdio-common/Makefile @@ -60,7 +60,7 @@ tests := tstscanf test_rdwr test-popen tstgetln test-fseek \ tst-popen tst-unlockedio tst-fmemopen2 tst-put-error tst-fgets \ tst-fwrite bug16 bug17 tst-swscanf tst-sprintf2 bug18 bug18a \ bug19 bug19a tst-popen2 scanf13 scanf14 scanf15 bug20 bug21 bug22 \ - scanf16 scanf17 tst-setvbuf1 tst-grouping + scanf16 scanf17 tst-setvbuf1 tst-grouping bug23 test-srcs = tst-unbputc tst-printf diff --git a/stdio-common/bug23.c b/stdio-common/bug23.c new file mode 100644 index 0000000000..dcc5428078 --- /dev/null +++ b/stdio-common/bug23.c @@ -0,0 +1,21 @@ +#include <stdio.h> +#include <string.h> + +static char buf[32768]; +static const char expected[] = "\ +\n\ +a\n\ +abbcd55%%%%%%%%%%%%%%%%%%%%%%%%%%\n"; + +static int +do_test (void) +{ + snprintf (buf, sizeof (buf), + "\n%1$s\n" "%1$s" "%2$s" "%2$s" "%3$s" "%4$s" "%5$d" "%5$d" + "%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%\n", + "a", "b", "c", "d", 5); + return strcmp (buf, expected) != 0; +} + +#define TEST_FUNCTION do_test () +#include "../test-skeleton.c" diff --git a/stdio-common/vfprintf.c b/stdio-common/vfprintf.c index fc370e8cbc..cfa4c30b78 100644 --- a/stdio-common/vfprintf.c +++ b/stdio-common/vfprintf.c @@ -1,4 +1,4 @@ -/* Copyright (C) 1991-2008, 2009, 2010 Free Software Foundation, Inc. +/* Copyright (C) 1991-2008, 2009, 2010, 2011 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -1682,7 +1682,8 @@ do_positional: { /* Extend the array of format specifiers. */ struct printf_spec *old = specs; - specs = extend_alloca (specs, nspecs_max, 2 * nspecs_max); + specs = extend_alloca (specs, nspecs_max, + 2 * nspecs_max * sizeof (*specs)); /* Copy the old array's elements to the new space. */ memmove (specs, old, nspecs * sizeof (struct printf_spec)); |