From: Ilia Alshanetsky Date: Sun, 12 Dec 2010 19:54:21 +0000 (+0000) Subject: Fixed bug #47435 (FILTER_FLAG_NO_RES_RANGE don't work with ipv6). X-Git-Tag: php-5.4.0alpha1~191^2~516 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e90265ac978a5d92beb7c37ecf3107c2c5f1d7f5;p=php Fixed bug #47435 (FILTER_FLAG_NO_RES_RANGE don't work with ipv6). --- diff --git a/ext/filter/logical_filters.c b/ext/filter/logical_filters.c index f6a7c82fd2..52f7a851d2 100644 --- a/ext/filter/logical_filters.c +++ b/ext/filter/logical_filters.c @@ -735,8 +735,40 @@ void php_filter_validate_ip(PHP_INPUT_FILTER_PARAM_DECL) /* {{{ */ RETURN_VALIDATION_FAILED } } - if (flags & FILTER_FLAG_NO_RES_RANGE && Z_STRLEN_P(value) == 3 && !strcmp("::1", Z_STRVAL_P(value))) { - RETURN_VALIDATION_FAILED + if (flags & FILTER_FLAG_NO_RES_RANGE) { + switch (Z_STRLEN_P(value)) { + case 1: case 0: + break; + case 2: + if (!strcmp("::", Z_STRVAL_P(value))) { + RETURN_VALIDATION_FAILED + } + break; + case 3: + if (!strcmp("::1", Z_STRVAL_P(value)) || !strcmp("5f:", Z_STRVAL_P(value))) { + RETURN_VALIDATION_FAILED + } + break; + default: + if (Z_STRLEN_P(value) >= 5) { + if ( + !strncasecmp("fe8", Z_STRVAL_P(value), 3) || + !strncasecmp("fe9", Z_STRVAL_P(value), 3) || + !strncasecmp("fea", Z_STRVAL_P(value), 3) || + !strncasecmp("feb", Z_STRVAL_P(value), 3) + ) { + RETURN_VALIDATION_FAILED + } + } + if ( + (Z_STRLEN_P(value) >= 9 && !strncasecmp("2001:0db8", Z_STRVAL_P(value), 9)) || + (Z_STRLEN_P(value) >= 2 && !strncasecmp("5f", Z_STRVAL_P(value), 2)) || + (Z_STRLEN_P(value) >= 4 && !strncasecmp("3ff3", Z_STRVAL_P(value), 4)) || + (Z_STRLEN_P(value) >= 8 && !strncasecmp("2001:001", Z_STRVAL_P(value), 8)) + ) { + RETURN_VALIDATION_FAILED + } + } } } break; diff --git a/ext/filter/tests/018.phpt b/ext/filter/tests/018.phpt index 13122bdc55..af52b2e60d 100644 --- a/ext/filter/tests/018.phpt +++ b/ext/filter/tests/018.phpt @@ -24,6 +24,7 @@ var_dump(filter_var(-1, FILTER_VALIDATE_IP)); var_dump(filter_var("::1", FILTER_VALIDATE_IP, FILTER_FLAG_IPV4)); var_dump(filter_var("127.0.0.1", FILTER_VALIDATE_IP, FILTER_FLAG_IPV6)); var_dump(filter_var("::1", FILTER_VALIDATE_IP, FILTER_FLAG_IPV6)); +var_dump(filter_var("::1", FILTER_VALIDATE_IP, FILTER_FLAG_IPV6 | FILTER_FLAG_NO_RES_RANGE)); var_dump(filter_var("127.0.0.1", FILTER_VALIDATE_IP, FILTER_FLAG_IPV4)); echo "Done\n"; ?> @@ -48,5 +49,6 @@ bool(false) bool(false) bool(false) string(3) "::1" +bool(false) string(9) "127.0.0.1" Done diff --git a/ext/filter/tests/bug47435.phpt b/ext/filter/tests/bug47435.phpt new file mode 100644 index 0000000000..42705fdc6e --- /dev/null +++ b/ext/filter/tests/bug47435.phpt @@ -0,0 +1,34 @@ +--TEST-- +Bug #47435 (FILTER_FLAG_NO_PRIV_RANGE and FILTER_FLAG_NO_RES_RANGE don't work with ipv6) +--FILE-- + +--EXPECT-- +string(7) "FC00::1" +bool(false) +string(2) "::" +bool(false) +string(3) "::1" +bool(false) +string(10) "fe8:5:6::1" +bool(false) +string(12) "2001:0db8::1" +bool(false) +string(5) "5f::1" +bool(false) +string(7) "3ff3::1" +bool(false)