"Brent W. Baccala" <cos...@freesoft.org> writes:

> +  va_start(argptr, fmt);
> +  retval = vasprintf(strp, fmt, argptr);
> +  va_end(argptr);
> +
> +  assert_perror(retval == -1);

The argument of assert_perror should be an errno_t value, not
just a Boolean.

It isn't clear from the commit message that the warning message
occurs at compile time.

Could add an __attribute__ so that GCC can validate that the
arguments of easprintf match the format string, like it already
validates the arguments of asprintf.
https://gcc.gnu.org/onlinedocs/gcc/Common-Function-Attributes.html

Reply via email to