potential off by one in parse_arguments

Issue: #1799

Author: Felipe Zimmerle

CHANGES

@@ -1,6 +1,7 @@
 DD MMM YYYY - 2.9.3 - To be released
 ------------------------------------
-
+ * potential off by one in parse_arguments
+   [Issue #1799 - @tinselcity, @zimmerle]
  * Fix utf-8 character encoding conversion
    [Issue #1794 - @tinselcity, @zimmerle]
  * Fix ip tree lookup on netmask content

apache2/msc_parsers.c

@@ -266,7 +266,7 @@ int parse_arguments(modsec_rec *msr, const char *s, apr_size_t inputlength,
         if (status == 0) {
             /* parameter name */
             arg->name_origin_offset = i;
-            while ((s[i] != '=') && (s[i] != argument_separator) && (i < inputlength)) {
+            while ((i < inputlength) && (s[i] != '=') && (s[i] != argument_separator))  {
                 buf[j] = s[i];
                 j++;
                 i++;