diff options
| author | Reyk Floeter <reyk@esdenera.com> | 2015-01-07 12:10:37 +0100 |
|---|---|---|
| committer | Reyk Floeter <reyk@esdenera.com> | 2015-01-07 12:10:37 +0100 |
| commit | 29f9676126ea2c65fc50ba30cd04e36f4243f7f8 (patch) | |
| tree | f271466fac05cc7aeed8a7b735d396b56acb9b19 | |
| parent | a93d310ccd3c02ba4f336c7ca1d8911874fea847 (diff) | |
| download | httpd-29f9676126ea2c65fc50ba30cd04e36f4243f7f8.tar.gz httpd-29f9676126ea2c65fc50ba30cd04e36f4243f7f8.zip | |
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
| -rw-r--r-- | server_http.c | 5 |
1 files 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 <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; } /* |