From: reyk <reyk@openbsd.org>
Date: Tue, 6 Jan 2015 17:48:04 +0000 (+0000)
Subject: I missed one goto abort instead of free(line).
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=0859b34fddfdff19f1f36b3bda6e110afef66a1f;p=openbsd

I missed one goto abort instead of free(line).

Found by Fabian Raetz at gmail
---

diff --git a/usr.sbin/httpd/server_http.c b/usr.sbin/httpd/server_http.c
index 5568840e3a5..b0ae73d2e05 100644
--- a/usr.sbin/httpd/server_http.c
+++ b/usr.sbin/httpd/server_http.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: server_http.c,v 1.61 2015/01/06 13:48:15 reyk Exp $	*/
+/*	$OpenBSD: server_http.c,v 1.62 2015/01/06 17:48:04 reyk Exp $	*/
 
 /*
  * Copyright (c) 2006 - 2014 Reyk Floeter <reyk@openbsd.org>
@@ -166,9 +166,8 @@ server_read_http(struct bufferevent *bev, void *arg)
 		/* Limit the total header length minus \r\n */
 		clt->clt_headerlen += linelen;
 		if (clt->clt_headerlen > SERVER_MAXHEADERLENGTH) {
-			free(line);
 			server_abort_http(clt, 413, "request too large");
-			return;
+			goto abort;
 		}
 
 		/*