Commit Briefs
template: fix possible use-after-free and drop the `render' sugar
if some of the tp_* function fails, the inner code could free the template too (that's what galileo does), so `tp->tp_ret' would access free'd memory! Instead, add a local variable `tp_ret' to keep track of the return code. While here, drop the `render' shortand that adds the argument call, the template_reset function and adapt the regress.
always initialize the template
move the initialization to the proxy creation, and while here move the definition for the functions used by the template to fcgi.c too.
free vars before assigning them
prevent a leaks if the server sends some parameter twice
close fcgi socket when not marked as keepalive
if the server hasn't specified the FCGI_KEEP_CONN bit in the request, close the socket when we're done. This seems to be what httpd(8) (most of the time) wants.
drop useless proxy_accept wrapper
just call fcgi_accept (with args tweaked)