MINOR: tools: make url2ipv4 return the exact number of bytes parsed
The function's return value is currently used as a boolean but we'll
need it to return the number of bytes parsed. Right now it returns
it minus one, unless the last char doesn't match what is permitted.
Let's update this to make it more usable.
diff --git a/src/tools.c b/src/tools.c
index 98adb32..546ad44 100644
--- a/src/tools.c
+++ b/src/tools.c
@@ -1447,7 +1447,9 @@
/*
- * Parse IPv4 address found in url.
+ * Parse IPv4 address found in url. Return the number of bytes parsed. It
+ * expects exactly 4 numbers between 0 and 255 delimited by dots, and returns
+ * zero in case of mismatch.
*/
int url2ipv4(const char *addr, struct in_addr *dst)
{
@@ -1460,9 +1462,10 @@
*(tp = tmp) = 0;
while (*addr) {
- unsigned char digit = (ch = *addr++) - '0';
+ unsigned char digit = (ch = *addr) - '0';
if (digit > 9 && ch != '.')
break;
+ addr++;
if (digit <= 9) {
u_int new = *tp * 10 + digit;
if (new > 255)
@@ -1486,7 +1489,7 @@
return 0;
memcpy(&dst->s_addr, tmp, 4);
- return addr-cp-1;
+ return addr - cp;
}
/*