Browse Source

Spec conformance on protover: always reject ranges where lo>hi

Nick Mathewson 6 years ago
parent
commit
c5295cc1be
1 changed files with 3 additions and 4 deletions
  1. 3 4
      src/or/protover.c

+ 3 - 4
src/or/protover.c

@@ -158,6 +158,9 @@ parse_version_range(const char *s, const char *end_of_range,
   if (next != end_of_range)
     goto error;
 
+  if (low > high)
+    goto error;
+
  done:
   *high_out = high;
   *low_out = low;
@@ -208,10 +211,6 @@ parse_single_entry(const char *s, const char *end_of_entry)
       goto error;
     }
 
-    if (range->low > range->high) {
-      goto error;
-    }
-
     s = comma;
     while (*s == ',' && s < end_of_entry)
       ++s;