On Fri, 06 May 2016 17:56:16 +0200, Hiltjo Posthuma wrote: > Any OK's for this? > > Please also see my mail with subject "httpd: fix/style: unbalanced > va_start and va_end macros" (don't want to spam the mailinglist :)).
I didn't change those before because body and hstsheader are initialized to NULL. All known asprintf() implementations either leave the ret pointer unchanged on error or set it to NULL. So this would only be a problem on systems where asprintf() modifies ret on error and sets it to a non-NULL value. Is that really an issue? - todd
