From 38b9c0274974a1b9941adb2e59238b153704151f Mon Sep 17 00:00:00 2001 From: Rich Felker Date: Sat, 19 Oct 2013 22:36:30 -0400 Subject: [PATCH] fix return value for inet_pton in ipv6 failure cases in the case of input that does not match the expected form, the correct return value is 0, not -1. --- src/network/inet_pton.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/network/inet_pton.c b/src/network/inet_pton.c index 5c4850a6..8f7c17b5 100644 --- a/src/network/inet_pton.c +++ b/src/network/inet_pton.c @@ -46,24 +46,24 @@ int inet_pton(int af, const char *restrict s, void *restrict a0) if (!s[1]) break; continue; } - if (hexval(s[0])<0) return -1; + if (hexval(s[0])<0) return 0; while (s[0]=='0' && s[1]=='0') s++; for (v=j=0; j<5 && (d=hexval(s[j]))>=0; j++) v=16*v+d; - if (v > 65535) return -1; + if (v > 65535) return 0; ip[i] = v; if (!s[j]) { - if (brk<0 && i!=7) return -1; + if (brk<0 && i!=7) return 0; break; } if (i<7) { if (s[j]==':') continue; - if (s[j]!='.') return -1; + if (s[j]!='.') return 0; need_v4=1; i++; break; } - return -1; + return 0; } if (brk>=0) { memmove(ip+brk+7-i, ip+brk, 2*(i+1-brk)); @@ -73,6 +73,6 @@ int inet_pton(int af, const char *restrict s, void *restrict a0) *a++ = ip[j]>>8; *a++ = ip[j]; } - if (need_v4 &&inet_pton(AF_INET, (void *)s, a-4) <= 0) return -1; + if (need_v4 && inet_pton(AF_INET, (void *)s, a-4) <= 0) return 0; return 1; } -- 2.25.1