Changeset 448 for trunk/src

Timestamp:

Mar 29, 2014, 7:59:38 AM (11 years ago)

Author:

katerina

Message:

Fic for ticket #350 (numeric hostnames).

Location:

trunk/src

Files:

• cutest_sh_tools.c (modified) (1 diff)
• sh_ipvx.c (modified) (2 diffs)

trunk/src/cutest_sh_tools.c

@@ -114 +114 @@
   CuAssertTrue(tc, !sh_ipvx_is_numeric(input));
   input  = strdup("127");
-  CuAssertTrue(tc, sh_ipvx_is_numeric(input));
+  CuAssertTrue(tc, !sh_ipvx_is_numeric(input));
 }
 

trunk/src/sh_ipvx.c

@@ -41 +41 @@
 {
   int j;
+  int count = 0;
   int len = sl_strlen(addr);
+  unsigned int a, b, c, d;
   
-  for (j = 0; j < len; ++j)
+  if (len > 15) /* 3*4 + 3 dots */
+    return (1 == 0);
+
+  for (j = 0; j < len; ++j) {
     if ( (addr[j] < '0' || addr[j] > '9') && addr[j] != '.')
       return (1 == 0);
+    if (addr[j] == '.') ++count;
+  }
+
+  if (count != 3)
+    return (1 == 0);
+
+  if (sscanf(addr, "%3u.%3u.%3u.%3u", &a, &b, &c, &d) != 4)
+    return( 1 == 0 );
+
+  if ((a|b|c|d) > 255) 
+    return( 1 == 0 );
+
   return (1 == 1);
 }
@@ -538 +555 @@
   return 0;
 }
+
+#ifdef SH_CUTEST
+#include <stdlib.h>
+#include "CuTest.h"
+
+void Test_ipvx (CuTest *tc) {
+
+  int result;
+
+  result = sh_ipvx_is_ipv4("123456789");
+  CuAssertTrue(tc, result == FALSE);
+
+  result = sh_ipvx_is_ipv4("123456789123...");
+  CuAssertTrue(tc, result == FALSE);
+
+  result = sh_ipvx_is_ipv4("123.456.789.123");
+  CuAssertTrue(tc, result == FALSE);
+
+  result = sh_ipvx_is_ipv4("123.156.189.254");
+  CuAssertTrue(tc, result == TRUE);
+
+  result = sh_ipvx_is_ipv4("255.255.255.255");
+  CuAssertTrue(tc, result == TRUE);
+
+  result = sh_ipvx_is_ipv4("0.0.0.0");
+  CuAssertTrue(tc, result == TRUE);
+
+  result = sh_ipvx_is_ipv4("0022.156.189.254");
+  CuAssertTrue(tc, result == FALSE);
+
+  result = sh_ipvx_is_ipv4("999999999.1.1.2");
+  CuAssertTrue(tc, result == FALSE);
+
+}
+#endif
+