Bug #10681 Updated: Fix for #9698 breaks more than it fixes
| From: | derick@php.net | Date: | Sun, 06 May 2001 16:00:05 +0000 |
| Subject: | Bug #10681 Updated: Fix for #9698 breaks more than it fixes | ||
| Groups: | php.dev | ||
| Request: | Send a blank email to php-dev+get-54121@lists.php.net to get a copy of this message | ||
ID: 10681
Updated by: derick
Reported By: kettler@gmx.net
Old-Status: Open
Status: Feedback
Bug Type: Network related
Operating system:
PHP Version: 4.0 Latest CVS (05/05/2001)
Assigned To:
Comments:
Sounds ok to me, are you willing to make a patch for this?
Previous Comments:
---------------------------------------------------------------------------
[2001-05-05 04:40:03] kettler@gmx.net
<?php
$ipaslong = ip2long("208.247.106.187");
print long2ip($ipaslong)."n";
print bin2hex(pack("N", $ipaslong))."n";
?>
That script run with PHP 4.0.5 prints:
208.247.106.187
d0f76abb
whereas with the latest CVS version it prints:
127.255.255.255
7fffffff
That makes that function completely unusable for me (and for others probably too). I have another
proposal for fixing bug #9698, and that is implementing %u as a sprintf format specifier to output
unsigned longs instead of signed.
This way he could use the following script for his problem (should go onto the ip2long page then):
<?php
$ip = gethostbyname("www.php.net");
$out = sprintf("http://%u/<br>n", ip2long($ip));
echo $out;
?>
If that fix would be accepted (and the other fix rolled back), I could add %u to sprintf myself and
post it here.
---------------------------------------------------------------------------
ATTENTION! Do NOT reply to this email!
To reply, use the web interface found at http://bugs.php.net/?id=10681&edit=2