http-backend: Don't infinite loop during die()
authorShawn O. Pearce <spearce@spearce.org>
Mon, 22 Mar 2010 14:22:04 +0000 (07:22 -0700)
committerJunio C Hamano <gitster@pobox.com>
Wed, 24 Mar 2010 21:40:56 +0000 (14:40 -0700)
commit5856b5f568ba957a0ea094fa676641ab246021fc
treeef078b3d6bbf301f26f2f3517180eb8dcb145b51
parent0b3dcfe721dc8734e2688f936afad055d8541d97
http-backend: Don't infinite loop during die()

If stdout has already been closed by the CGI and die() gets called,
the CGI will fail to write the "Status: 500 Internal Server Error" to
the pipe, which results in die() being called again (via safe_write).
This goes on in an infinite loop until the stack overflows and the
process is killed by SIGSEGV.

Instead set a flag on the first die() invocation and if we came back to
the handler, just die silently, as it only means we failed to report the
failure---we cannot report anything anyway in such a case.  This way
failures to write the error messages to the stdout pipe do not result in
an infinite loop.

We also now report on the death to stderr before we report to stdout,
to increase the chances that the cause of the die() invocation will
appear in the server's error log.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
fixup! http-backend.c: Don't infinite loop

Now die_webcgi() actually can return during a recursive call into it,
causing

    http-backend.c:554: error: 'noreturn' function does return

The only reason we would come back to the die handler is because we
failed during it, so we cannot report anything anyway.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
http-backend.c