Commit Diff

Commit:
dfa8b23f08ecd8b884c33986577983d380a4622a
From:
Omar Polo <op@omarpolo.com>
Date:
Message:
return after FCGI_END_REQUEST backport 17493a486c9e6325c3aae74920ddb10958ae2771 this fixes a possible crash if `client_write' closes the connection, because client_close can end up freeing the fastcgi bufferevent while we're looping. We don't support fastcgi multiplexing, so once we get an END_REQUEST there's nothing more to do. Prodded into looking here after a bug report from Allen Sobot, thanks!
Actions:
Patch | Tree
commit - c197ad99a36ea820b528cb9fd10ff80f96674bb8
commit + dfa8b23f08ecd8b884c33986577983d380a4622a
blob - 33df816bd4918ff6b57ec6007d2e19a365638ab2
blob + 4c71d8fa29d684f5d3e1e2e87ad6a972232e6875
--- fcgi.c
+++ fcgi.c
@@ -290,7 +290,7 @@ fcgi_read(struct bufferevent *bev, void *d)
 			/* TODO: do something with the status? */
 			c->type = REQUEST_DONE;
 			client_write(c->bev, c);
-			break;
+			return;
 
 		case FCGI_STDERR:
 			/* discard stderr (for now) */