From 29f9676126ea2c65fc50ba30cd04e36f4243f7f8 Mon Sep 17 00:00:00 2001 From: Reyk Floeter Date: Wed, 7 Jan 2015 12:10:37 +0100 Subject: revision 1.62 date: 2015/01/06 17:48:04; author: reyk; state: Exp; lines: +2 -3; commitid$ I missed one goto abort instead of free(line). Found by Fabian Raetz at gmail --- server_http.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/server_http.c b/server_http.c index 5568840..b0ae73d 100644 --- a/server_http.c +++ b/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 @@ -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; } /* -- cgit v1.2.3-54-g00ecf