Check the bound first.

Found with cppcheck.
2015-05-20 00:06:59 +10:00 · 2015-05-20 00:06:59 +10:00 · 02b02a7615
parent 1c5fdb5607
commit 02b02a7615
1 changed files with 3 additions and 3 deletions
--- 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++)
            ;