User "Brion VIBBER" changed the status of MediaWiki.r76560. Old Status: ok New Status: fixme
User "Brion VIBBER" also posted a comment on MediaWiki.r76560. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/76560#c21992 Commit summary: * Changes in IP.php: ** Fixed hexToOctet()/toOctet() padding (pad left not right) ** Made hexToQuad() left-pad input (e.g. C -> 0000000C) ** Added isPublic6() function (checked as needed by isPublic()) ** Rewrote isValidBlock() to not do flaky and roundabout isArray() check. Works for v6 now. ** Removed toArray(), unused outside IP.php and broken for v6 ** Removed toOctet() duplication ** Added new private IPv6ToRawHex() function. Used to make toHex() faster. ** Made some functions private ** Reverted r20435, pointless ** Updated credits * Changes in IPTests: ** Added a bunch of tests (mostly v6) ** Removed weird array test (especially with toArray() gone) after r76514 ** Padding *no* longer needed for hexToX functions - assertion flipped * CheckUser: removed parseRange6() reference (now private) Comment: The private ranges in IP::isPublic6() appear to be incorrect; per notes on bug 30739 it looks like it should look for private addresses in fc00::/7 rather than 00fc::/7. Currently, publicly routable addresses get misinterpreted as private due to having a prefix anywhere between 00fc and fbff... _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
