diff options
author | Mike Frysinger <vapier@gentoo.org> | 2016-04-22 17:11:09 -0400 |
---|---|---|
committer | Mike Frysinger <vapier@gentoo.org> | 2016-04-23 16:14:01 -0400 |
commit | b2cae5d3f5f58e26fe3ceb9c9c2a4018e25785e6 (patch) | |
tree | c521f1356dd7d9f6577749d1e8c8492cfbbec068 /localedata/tst-numeric.c | |
parent | 4964bb4591b16b485eac3c5ecc8a383b528ea1a6 (diff) | |
download | glibc-b2cae5d3f5f58e26fe3ceb9c9c2a4018e25785e6.zip glibc-b2cae5d3f5f58e26fe3ceb9c9c2a4018e25785e6.tar.gz glibc-b2cae5d3f5f58e26fe3ceb9c9c2a4018e25785e6.tar.bz2 |
tst-fmon/tst-numeric: switch malloc to static stack space [BZ #19671]
The current test code doesn't check the return value of malloc.
This should rarely (if ever) cause a problem, but rather than add
some return value checks, just statically allocate the buffer on
the stack. This will never fail (or if it does, we've got much
bigger problems that don't matter to the test).
Diffstat (limited to 'localedata/tst-numeric.c')
-rw-r--r-- | localedata/tst-numeric.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/localedata/tst-numeric.c b/localedata/tst-numeric.c index 46a6b48..64abb0a 100644 --- a/localedata/tst-numeric.c +++ b/localedata/tst-numeric.c @@ -41,7 +41,7 @@ int main (int argc, char *argv[]) { - char *s = malloc (201); + char s[200]; double val; /* Make sure to read the value before setting of the locale, as @@ -54,7 +54,7 @@ main (int argc, char *argv[]) exit (EXIT_SETLOCALE); } - if (snprintf (s, 200, argv[2], val) == -1) + if (snprintf (s, sizeof (s), argv[2], val) == -1) { perror ("snprintf"); exit (EXIT_SNPRINTF); |