From: claudio <claudio@openbsd.org>
Date: Fri, 12 Aug 2022 08:40:25 +0000 (+0000)
Subject: Use break instead of return so that a HEAD request still consumes all data.
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=5c9c220ee92279987a628c9b165f9b29c6c83c50;p=openbsd

Use break instead of return so that a HEAD request still consumes all data.
OK op@
---

diff --git a/usr.sbin/httpd/server_fcgi.c b/usr.sbin/httpd/server_fcgi.c
index 4b2c2690f8e..e29ad9936d3 100644
--- a/usr.sbin/httpd/server_fcgi.c
+++ b/usr.sbin/httpd/server_fcgi.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: server_fcgi.c,v 1.92 2022/08/12 06:41:41 op Exp $	*/
+/*	$OpenBSD: server_fcgi.c,v 1.93 2022/08/12 08:40:25 claudio Exp $	*/
 
 /*
  * Copyright (c) 2014 Florian Obser <florian@openbsd.org>
@@ -584,7 +584,7 @@ server_fcgi_read(struct bufferevent *bev, void *arg)
 				    ((struct http_descriptor *)
 				    clt->clt_descreq)->http_method
 				    == HTTP_METHOD_HEAD)
-					return;
+					break;
 				if (server_fcgi_writechunk(clt) == -1) {
 					server_abort_http(clt, 500,
 					    "encoding error");