|
 |
00db10 |
Revert this upstream commit:
|
|
|
00db10 |
|
|
|
00db10 |
commit 9a0cc8c1bd7645bf3c988890ffb59639c07a5812
|
|
|
00db10 |
Author: Florian Weimer <fweimer@redhat.com>
|
|
|
00db10 |
Date: Fri Jun 23 22:51:00 2017 +0200
|
|
|
00db10 |
|
|
|
00db10 |
inet_pton: Reject IPv6 addresses with many leading zeros [BZ #16637]
|
|
|
00db10 |
|
|
|
00db10 |
2001:db8:00001::f is not a valid IPv6 address according to RFC 2373.
|
|
|
00db10 |
|
|
|
00db10 |
We do not want this behavioral change in Red Hat Enterprise Linux 7.
|
|
|
00db10 |
See rhbz#1484034 for some discussion.
|
|
|
00db10 |
|
|
|
00db10 |
diff --git a/resolv/inet_pton.c b/resolv/inet_pton.c
|
|
|
00db10 |
index 16ee33e0c0dfb015..b95da47c17ef8afc 100644
|
|
|
00db10 |
--- a/resolv/inet_pton.c
|
|
|
00db10 |
+++ b/resolv/inet_pton.c
|
|
|
00db10 |
@@ -144,8 +144,7 @@ inet_pton6 (const char *src, const char *src_endp, unsigned char *dst)
|
|
|
00db10 |
{
|
|
|
00db10 |
unsigned char tmp[NS_IN6ADDRSZ], *tp, *endp, *colonp;
|
|
|
00db10 |
const char *curtok;
|
|
|
00db10 |
- int ch;
|
|
|
00db10 |
- size_t xdigits_seen; /* Number of hex digits since colon. */
|
|
|
00db10 |
+ int ch, saw_xdigit;
|
|
|
00db10 |
unsigned int val;
|
|
|
00db10 |
|
|
|
00db10 |
tp = memset (tmp, '\0', NS_IN6ADDRSZ);
|
|
|
00db10 |
@@ -163,7 +162,7 @@ inet_pton6 (const char *src, const char *src_endp, unsigned char *dst)
|
|
|
00db10 |
}
|
|
|
00db10 |
|
|
|
00db10 |
curtok = src;
|
|
|
00db10 |
- xdigits_seen = 0;
|
|
|
00db10 |
+ saw_xdigit = 0;
|
|
|
00db10 |
val = 0;
|
|
|
00db10 |
while (src < src_endp)
|
|
|
00db10 |
{
|
|
|
00db10 |
@@ -171,19 +170,17 @@ inet_pton6 (const char *src, const char *src_endp, unsigned char *dst)
|
|
|
00db10 |
int digit = hex_digit_value (ch);
|
|
|
00db10 |
if (digit >= 0)
|
|
|
00db10 |
{
|
|
|
00db10 |
- if (xdigits_seen == 4)
|
|
|
00db10 |
- return 0;
|
|
|
00db10 |
val <<= 4;
|
|
|
00db10 |
val |= digit;
|
|
|
00db10 |
if (val > 0xffff)
|
|
|
00db10 |
return 0;
|
|
|
00db10 |
- ++xdigits_seen;
|
|
|
00db10 |
+ saw_xdigit = 1;
|
|
|
00db10 |
continue;
|
|
|
00db10 |
}
|
|
|
00db10 |
if (ch == ':')
|
|
|
00db10 |
{
|
|
|
00db10 |
curtok = src;
|
|
|
00db10 |
- if (xdigits_seen == 0)
|
|
|
00db10 |
+ if (!saw_xdigit)
|
|
|
00db10 |
{
|
|
|
00db10 |
if (colonp)
|
|
|
00db10 |
return 0;
|
|
|
00db10 |
@@ -196,7 +193,7 @@ inet_pton6 (const char *src, const char *src_endp, unsigned char *dst)
|
|
|
00db10 |
return 0;
|
|
|
00db10 |
*tp++ = (unsigned char) (val >> 8) & 0xff;
|
|
|
00db10 |
*tp++ = (unsigned char) val & 0xff;
|
|
|
00db10 |
- xdigits_seen = 0;
|
|
|
00db10 |
+ saw_xdigit = 0;
|
|
|
00db10 |
val = 0;
|
|
|
00db10 |
continue;
|
|
|
00db10 |
}
|
|
|
00db10 |
@@ -204,12 +201,12 @@ inet_pton6 (const char *src, const char *src_endp, unsigned char *dst)
|
|
|
00db10 |
&& inet_pton4 (curtok, src_endp, tp) > 0)
|
|
|
00db10 |
{
|
|
|
00db10 |
tp += NS_INADDRSZ;
|
|
|
00db10 |
- xdigits_seen = 0;
|
|
|
00db10 |
+ saw_xdigit = 0;
|
|
|
00db10 |
break; /* '\0' was seen by inet_pton4. */
|
|
|
00db10 |
}
|
|
|
00db10 |
return 0;
|
|
|
00db10 |
}
|
|
|
00db10 |
- if (xdigits_seen > 0)
|
|
|
00db10 |
+ if (saw_xdigit)
|
|
|
00db10 |
{
|
|
|
00db10 |
if (tp + NS_INT16SZ > endp)
|
|
|
00db10 |
return 0;
|
|
|
00db10 |
diff --git a/resolv/tst-inet_pton.c b/resolv/tst-inet_pton.c
|
|
|
00db10 |
index 4bb9f8119378b467..7fffb24cdf9eb1f4 100644
|
|
|
00db10 |
--- a/resolv/tst-inet_pton.c
|
|
|
00db10 |
+++ b/resolv/tst-inet_pton.c
|
|
|
00db10 |
@@ -226,7 +226,13 @@ const struct test_case test_cases[] =
|
|
|
00db10 |
},
|
|
|
00db10 |
{.input = "2", },
|
|
|
00db10 |
{.input = "2.", },
|
|
|
00db10 |
- {.input = "2001:db8:00001::f", },
|
|
|
00db10 |
+ {.input = "2001:db8:00001::f",
|
|
|
00db10 |
+ .ipv6_ok = true,
|
|
|
00db10 |
+ .ipv6_expected = {
|
|
|
00db10 |
+ 0x20, 0x1, 0xd, 0xb8, 0x0, 0x1, 0x0, 0x0,
|
|
|
00db10 |
+ 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0xf
|
|
|
00db10 |
+ },
|
|
|
00db10 |
+ },
|
|
|
00db10 |
{.input = "2001:db8:10000::f", },
|
|
|
00db10 |
{.input = "2001:db8:1234:5678:abcd:ef01:2345:67",
|
|
|
00db10 |
.ipv6_ok = true,
|
|
|
00db10 |
@@ -448,7 +454,13 @@ const struct test_case test_cases[] =
|
|
|
00db10 |
0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0
|
|
|
00db10 |
},
|
|
|
00db10 |
},
|
|
|
00db10 |
- {.input = "::00001", },
|
|
|
00db10 |
+ {.input = "::00001",
|
|
|
00db10 |
+ .ipv6_ok = true,
|
|
|
00db10 |
+ .ipv6_expected = {
|
|
|
00db10 |
+ 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0,
|
|
|
00db10 |
+ 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x1
|
|
|
00db10 |
+ },
|
|
|
00db10 |
+ },
|
|
|
00db10 |
{.input = "::1",
|
|
|
00db10 |
.ipv6_ok = true,
|
|
|
00db10 |
.ipv6_expected = {
|