[PATCH] tst-fmon/tst-numeric: switch malloc to static stack space [BZ #19671]

Florian Weimer fw@deneb.enyo.de
Sat Apr 23 12:24:00 GMT 2016


* Mike Frysinger:

> 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).

This needs a ChangeLog entry.

> -  char *s = malloc (201);
> +  char s[201];

Please use a enum constant of 200, and also pass it to strfmon.
K think the current 200/201 choice is technically incorrect
(the maximum includes the terminating null byte).



More information about the Libc-alpha mailing list