| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.3.0b4 |
| Bug #21046 | checkIPv6 wrongly validates IPv6 IPs with leading and/or trailing : | ||
|---|---|---|---|
| Submitted: | 2016-03-24 23:39 UTC | ||
| From: | cbuechler | Assigned: | alexmerz |
| Status: | Closed | Package: | Net_IPv6 (version 1.2.1) |
| PHP Version: | 5.6.19 | OS: | FreeBSD |
| Roadmaps: | (Not assigned) | ||
[2016-03-24 23:39 UTC] cbuechler
(Chris Buechler)
Description:
------------
Net_IPv6::checkIPv6 wrongly validates full length IPv6 IPs that have a
leading and/or trailing colon.
Test script:
---------------
these three examples should return false, but return true:
var_dump(Net_IPv6::checkIPv6("1234:5678:90AB:CDEF:1234:5678:90AB:CDEF:"));
var_dump(Net_IPv6::checkIPv6(":1234:5678:90AB:CDEF:1234:5678:90AB:CDEF"));
var_dump(Net_IPv6::checkIPv6(":1234:5678:90AB:CDEF:1234:5678:90AB:CDEF:"));
Expected result:
----------------
The three examples provided aren't valid IPv6 IPs and should return false,
not true.
Comments
[2016-03-25 05:10 UTC] phildavis
(Phil Davis)
Proposed fix at https://github.com/pear/Net_IPv6/pull/14
[2016-03-25 12:16 UTC] alexmerz
(Alexander Merz)
-Status: Open
+Status: Closed
-Assigned To:
+Assigned To: alexmerz
This bug has been fixed in SVN.
If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET).
If this was a problem with the pear.php.net website, the change should be live shortly.
Otherwise, the fix will appear in the package's next release.
Thank you for the report and for helping us make PEAR better.