Do not call substr_count() if ip is null to avoid deprecation warning in PHP 8.1

Author: W0rma

IpUtils.php

@@ -37,6 +37,10 @@ private function __construct()
      */
     public static function checkIp($requestIp, $ips)
     {
+        if (null === $requestIp) {
+            return false;
+        }
+
         if (!\is_array($ips)) {
             $ips = [$ips];
         }

Tests/IpUtilsTest.php

@@ -39,6 +39,8 @@ public function getIpv4Data()
             [true, '1.2.3.4', '192.168.1.0/0'],
             [false, '1.2.3.4', '256.256.256/0'], // invalid CIDR notation
             [false, 'an_invalid_ip', '192.168.1.0/24'],
+            [false, '', '1.2.3.4/1'],
+            [false, null, '1.2.3.4/1'],
         ];
     }
 
@@ -69,6 +71,8 @@ public function getIpv6Data()
             [false, '2a01:198:603:0:396e:4789:8e99:890f', ['::1', '1a01:198:603:0::/65']],
             [false, '}__test|O:21:"JDatabaseDriverMysqli":3:{s:2', '::1'],
             [false, '2a01:198:603:0:396e:4789:8e99:890f', 'unknown'],
+            [false, '', '::1'],
+            [false, null, '::1'],
         ];
     }