From 02b02a76152183aecb895eba963f9fefaeda36c8 Mon Sep 17 00:00:00 2001 From: Emil Mikulic Date: Wed, 20 May 2015 00:06:59 +1000 Subject: [PATCH] Check the bound first. Found with cppcheck. --- darkhttpd.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/darkhttpd.c b/darkhttpd.c index eb47c54..b8d4180 100644 --- a/darkhttpd.c +++ b/darkhttpd.c @@ -1461,9 +1461,9 @@ static char *parse_field(const struct connection *conn, const char *field) { /* find end */ for (bound2 = bound1; - ((conn->request[bound2] != '\r') && - (conn->request[bound2] != '\n') && - (bound2 < conn->request_length)); + ((bound2 < conn->request_length) && + (conn->request[bound2] != '\r') && + (conn->request[bound2] != '\n')); bound2++) ;