wivet icon indicating copy to clipboard operation
wivet copied to clipboard

Published 20 hours ago • verified publisher icon bedirhan

long2ip is unnecessary in statistics.php

Open JackNoire opened this issue 3 years ago • 1 comments

https://github.com/bedirhan/wivet/blob/1b3f580b4f9cabb9451f944a3b25fdca5432b09c/offscanpages/statistics.php#L34 https://github.com/bedirhan/wivet/blob/1b3f580b4f9cabb9451f944a3b25fdca5432b09c/offscanpages/statistics.php#L170 $scan['ipaddress'] seems to be already an IP string, so long2ip may not be needed. In PHP 8.0, this may result in an error: Uncaught TypeError: long2ip(): Argument #1 ($ip) must be of type int

JackNoire avatar Jan 05 '22 08:01 JackNoire

Hi Jack, Thanks for the issue. I don't maintain this repo anymore, though. Thanks anyway, appreciated.

bedirhan avatar Jan 06 '22 13:01 bedirhan