]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
projects / thirdparty / suricata.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: ca6e738)
parse/size: fix unit test checks
authorVictor Julien <vjulien@oisf.net>
Wed, 10 Jul 2024 17:46:34 +0000 (19:46 +0200)
committerVictor Julien <victor@inliniac.net>
Fri, 12 Jul 2024 08:47:50 +0000 (10:47 +0200)
src/util-misc.c patch | blob | blame | history

diff --git a/src/util-misc.c b/src/util-misc.c
index 4380e694a8a02e3a74ebeffd15db8507fc2d47b7..4ed00575e61003be20dffedd95726c6f039dad60 100644 (file)
--- a/src/util-misc.c
+++ b/src/util-misc.c
@@ -252,128 +252,128 @@ static int UtilMiscParseSizeStringTest01(void)
 
     str = "10";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10);
 
     str = "10kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10Kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10KB";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10mb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024 * 1024);
 
     str = "10gb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10737418240UL);
 
     /* space start */
 
     str = " 10";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10);
 
     str = " 10kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10Kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10KB";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10mb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024 * 1024);
 
     str = " 10gb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10737418240);
 
     /* space end */
 
     str = "10 ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10);
 
     str = "10kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10Kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10KB ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10mb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024 * 1024);
 
     str = "10gb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10737418240);
 
     /* space start - space end */
 
     str = " 10 ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10);
 
     str = " 10kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10Kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10KB ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10mb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024 * 1024);
 
     str = " 10gb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10737418240);
 
     /* space between number and scale */
@@ -382,256 +382,256 @@ static int UtilMiscParseSizeStringTest01(void)
 
     str = "10";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10);
 
     str = "10 kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10 Kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10 KB";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10 mb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024 * 1024);
 
     str = "10 gb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10737418240);
 
     /* space start */
 
     str = " 10";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10);
 
     str = " 10 kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10 Kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10 KB";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10 mb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024 * 1024);
 
     str = " 10 gb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10737418240);
 
     /* space end */
 
     str = "10 ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10);
 
     str = "10 kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10 Kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10 KB ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = "10 mb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024 * 1024);
 
     str = "10 gb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10737418240);
 
     /* space start - space end */
 
     str = " 10 ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10);
 
     str = " 10 kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10 Kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10 KB ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024);
 
     str = " 10 mb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10 * 1024 * 1024);
 
     str = " 10 gb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10737418240);
 
     /* no space */
 
     str = "10.5";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5);
 
     str = "10.5kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5Kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5KB";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5mb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024);
 
     str = "10.5gb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024 * 1024);
 
     /* space start */
 
     str = " 10.5";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5);
 
     str = " 10.5kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5Kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5KB";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5mb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024);
 
     str = " 10.5gb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024 * 1024);
 
     /* space end */
 
     str = "10.5 ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5);
 
     str = "10.5kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5Kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5KB ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5mb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024);
 
     str = "10.5gb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024 * 1024);
 
     /* space start - space end */
 
     str = " 10.5 ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5);
 
     str = " 10.5kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5Kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5KB ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5mb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024);
 
     str = " 10.5gb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024 * 1024);
 
     /* space between number and scale */
@@ -640,132 +640,132 @@ static int UtilMiscParseSizeStringTest01(void)
 
     str = "10.5";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5);
 
     str = "10.5 kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5 Kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5 KB";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5 mb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024);
 
     str = "10.5 gb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024 * 1024);
 
     /* space start */
 
     str = " 10.5";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5);
 
     str = " 10.5 kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5 Kb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5 KB";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5 mb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024);
 
     str = " 10.5 gb";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024 * 1024);
 
     /* space end */
 
     str = "10.5 ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5);
 
     str = "10.5 kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5 Kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5 KB ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = "10.5 mb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024);
 
     str = "10.5 gb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024 * 1024);
 
     /* space start - space end */
 
     str = " 10.5 ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5);
 
     str = " 10.5 kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5 Kb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5 KB ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024);
 
     str = " 10.5 mb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024);
 
     str = " 10.5 gb ";
     result = 0;
-    FAIL_IF(ParseSizeString(str, &result) > 0);
+    FAIL_IF(ParseSizeString(str, &result) != 0);
     FAIL_IF(result != 10.5 * 1024 * 1024 * 1024);
 
     /* Should fail on unknown units. */
-    FAIL_IF(ParseSizeString("32eb", &result) > 0);
+    FAIL_IF(ParseSizeString("32eb", &result) == 0);
 
     PASS;
 }
Mirror of https://github.com/OISF/suricata.git