Changes in uspace/lib/c/generic/inet/addr.c [a62ceaf:683e584] in mainline

File:

• uspace/lib/c/generic/inet/addr.c (modified) (13 diffs)

uspace/lib/c/generic/inet/addr.c

@@ -290 +290 @@
 
 static int inet_addr_parse_v4(const char *str, inet_addr_t *raddr,
-    int *prefix, char **endptr)
+    int *prefix)
 {
         uint32_t a = 0;
@@ -306 +306 @@
                 i++;
 
+                if (*cur == '\0')
+                        break;
+
                 if (*cur != '.')
-                        break;
+                        return EINVAL;
 
                 if (i < 4)
@@ -314 +317 @@
 
         if (prefix != NULL) {
-                if (*cur != '/')
-                        return EINVAL;
-                cur++;
-
                 *prefix = strtoul(cur, &cur, 10);
                 if (*prefix > 32)
@@ -323 +322 @@
         }
 
-        if (i != 4)
-                return EINVAL;
-
-        if (endptr == NULL && *cur != '\0')
+        if (i != 4 || (*cur != '\0'))
                 return EINVAL;
 
@@ -332 +328 @@
         raddr->addr = a;
 
-        if (endptr != NULL)
-                *endptr = cur;
-
         return EOK;
 }
 
-static int inet_addr_parse_v6(const char *str, inet_addr_t *raddr, int *prefix,
-    char **endptr)
+static int inet_addr_parse_v6(const char *str, inet_addr_t *raddr, int *prefix)
 {
         uint8_t data[16];
-        int explicit_groups;
 
         memset(data, 0, 16);
@@ -356 +347 @@
                 wildcard_pos = 0;
                 wildcard_size = 16;
+
+                /* Handle the unspecified address */
+                if (*cur == '\0')
+                        goto success;
         }
 
         while (i < 16) {
                 uint16_t bioctet;
-                const char *gend;
-                int rc = str_uint16_t(cur, &gend, 16, false, &bioctet);
+                int rc = str_uint16_t(cur, &cur, 16, false, &bioctet);
                 if (rc != EOK)
-                        break;
+                        return rc;
 
                 data[i] = (bioctet >> 8) & 0xff;
@@ -377 +371 @@
                 i += 2;
 
-                if (*gend != ':') {
-                        cur = gend;
+                if (*cur != ':')
                         break;
-                }
 
                 if (i < 16) {
+                        cur++;
+
                         /* Handle wildcard */
-                        if (gend[1] == ':') {
+                        if (*cur == ':') {
                                 if (wildcard_pos != (size_t) -1)
                                         return EINVAL;
@@ -390 +384 @@
                                 wildcard_pos = i;
                                 wildcard_size = 16 - i;
-                                cur = gend + 2;
+                                cur++;
+
+                                if (*cur == '\0' || *cur == '/')
+                                        break;
                         }
                 }
         }
-
-        /* Number of explicitly specified groups */
-        explicit_groups = i;
 
         if (prefix != NULL) {
@@ -408 +402 @@
         }
 
-        if (endptr == NULL && *cur != '\0')
+        if (*cur != '\0')
                 return EINVAL;
 
@@ -420 +414 @@
                         data[j] = 0;
                 }
-        } else {
-                /* Verify that all groups have been specified */
-                if (explicit_groups != 16)
-                        return EINVAL;
-        }
-
+        }
+
+success:
         raddr->version = ip_v6;
         memcpy(raddr->addr6, data, 16);
-        if (endptr != NULL)
-                *endptr = (char *)cur;
         return EOK;
 }
 
 /** Parse node address.
- *
- * Will fail if @a text contains extra characters at the and and @a endptr
- * is @c NULL.
  *
  * @param text Network address in common notation.
  * @param addr Place to store node address.
- * @param endptr Place to store pointer to next character oc @c NULL
  *
  * @return EOK on success, EINVAL if input is not in valid format.
  *
  */
-int inet_addr_parse(const char *text, inet_addr_t *addr, char **endptr)
+int inet_addr_parse(const char *text, inet_addr_t *addr)
 {
         int rc;
 
-        rc = inet_addr_parse_v4(text, addr, NULL, endptr);
+        rc = inet_addr_parse_v4(text, addr, NULL);
         if (rc == EOK)
                 return EOK;
 
-        rc = inet_addr_parse_v6(text, addr, NULL, endptr);
+        rc = inet_addr_parse_v6(text, addr, NULL);
         if (rc == EOK)
                 return EOK;
@@ -462 +447 @@
 /** Parse network address.
  *
- * Will fail if @a text contains extra characters at the and and @a endptr
- * is @c NULL.
- *
  * @param text  Network address in common notation.
  * @param naddr Place to store network address.
- * @param endptr Place to store pointer to next character oc @c NULL
  *
  * @return EOK on success, EINVAL if input is not in valid format.
  *
  */
-int inet_naddr_parse(const char *text, inet_naddr_t *naddr, char **endptr)
+int inet_naddr_parse(const char *text, inet_naddr_t *naddr)
 {
         int rc;
@@ -478 +459 @@
         int prefix;
 
-        rc = inet_addr_parse_v4(text, &addr, &prefix, endptr);
+        rc = inet_addr_parse_v4(text, &addr, &prefix);
         if (rc == EOK) {
                 inet_addr_naddr(&addr, prefix, naddr);
@@ -484 +465 @@
         }
 
-        rc = inet_addr_parse_v6(text, &addr, &prefix, endptr);
+        rc = inet_addr_parse_v6(text, &addr, &prefix);
         if (rc == EOK) {
                 inet_addr_naddr(&addr, prefix, naddr);