]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
curl: glob_range: no need to check unsigned variable for negative
authorDaniel Stenberg <daniel@haxx.se>
Mon, 14 Mar 2016 09:50:53 +0000 (10:50 +0100)
committerDaniel Stenberg <daniel@haxx.se>
Mon, 14 Mar 2016 09:51:25 +0000 (10:51 +0100)
cppcheck warned:

[src/tool_urlglob.c:283]: (style) Checking if unsigned variable 'step_n'
is less than zero.

src/tool_urlglob.c

index f1dd27fad064267fb32b0de10cbc86620d732b03..70d17fed1145236a5c007817cd5fe8b3c078fe9f 100644 (file)
@@ -5,7 +5,7 @@
  *                            | (__| |_| |  _ <| |___
  *                             \___|\___/|_| \_\_____|
  *
- * Copyright (C) 1998 - 2015, Daniel Stenberg, <daniel@haxx.se>, et al.
+ * Copyright (C) 1998 - 2016, Daniel Stenberg, <daniel@haxx.se>, et al.
  *
  * This software is licensed as described in the file COPYING, which
  * you should have received as part of this distribution. The terms
@@ -279,8 +279,7 @@ static CURLcode glob_range(URLGlob *glob, char **patternp,
 
     *posp += (pattern - *patternp);
 
-    if(!endp || (min_n > max_n) || (step_n > (max_n - min_n)) ||
-       (step_n <= 0) )
+    if(!endp || (min_n > max_n) || (step_n > (max_n - min_n)) || !step_n)
       /* the pattern is not well-formed */
       return GLOBERROR("bad range", *posp, CURLE_URL_MALFORMAT);