| Index: trunk/phase3/includes/IP.php |
| — | — | @@ -426,12 +426,16 @@ |
| 427 | 427 | } elseif ( strpos( $range, '-' ) !== false ) { |
| 428 | 428 | # Explicit range |
| 429 | 429 | list( $start, $end ) = array_map( 'trim', explode( '-', $range, 2 ) ); |
| 430 | | - $start = self::toUnsigned( $start ); $end = self::toUnsigned( $end ); |
| 431 | | - if ( $start > $end ) { |
| | 430 | + if( self::isIPAddress( $start ) && self::isIPAddress( $end ) ) { |
| | 431 | + $start = self::toUnsigned( $start ); $end = self::toUnsigned( $end ); |
| | 432 | + if ( $start > $end ) { |
| | 433 | + $start = $end = false; |
| | 434 | + } else { |
| | 435 | + $start = sprintf( '%08X', $start ); |
| | 436 | + $end = sprintf( '%08X', $end ); |
| | 437 | + } |
| | 438 | + } else { |
| 432 | 439 | $start = $end = false; |
| 433 | | - } else { |
| 434 | | - $start = sprintf( '%08X', $start ); |
| 435 | | - $end = sprintf( '%08X', $end ); |
| 436 | 440 | } |
| 437 | 441 | } else { |
| 438 | 442 | # Single IP |
| Index: trunk/phase3/RELEASE-NOTES |
| — | — | @@ -63,7 +63,11 @@ |
| 64 | 64 | * Special pages are now not subject to special handling for "self-links" |
| 65 | 65 | * (bug 15053) Syntactically incorrect redirects with another link in them |
| 66 | 66 | no longer redirect to the second link |
| | 67 | +* (bug 15049) Fix for CheckUser extension's log search: usernames containing |
| | 68 | + a "-" were incorrectly turned into bogus IP range searches. |
| | 69 | + Patch by Max Semenik. |
| 67 | 70 | |
| | 71 | + |
| 68 | 72 | === API changes in 1.14 === |
| 69 | 73 | |
| 70 | 74 | * Registration time of users registered before the DB field was created is now |