account for potentially false return value of dns_get_record

Author: transistive

src/Common/DNSAddressResolver.php

@@ -38,12 +38,16 @@ public function getAddresses(string $host): iterable
         yield $host;
 
         try {
-            /** @var list<array{ip?: string|null}> $records */
+            /** @var list<array{ip?: string|null}>|false $records */
             $records = dns_get_record($host, DNS_A | DNS_AAAA);
         } catch (Throwable) {
             $records = []; // Failed DNS queries should not halt execution
         }
 
+        if ($records === false) {
+            $records = [];
+        }
+
         if (count($records) === 0) {
             yield from $this->tryReverseLookup($host);
         } else {
@@ -59,12 +63,16 @@ public function getAddresses(string $host): iterable
     private function tryReverseLookup(string $host): iterable
     {
         try {
-            /** @var list<array{target?: string|null}> $records */
+            /** @var list<array{target?: string|null}>|false $records */
             $records = dns_get_record($host.'.in-addr.arpa');
         } catch (Throwable) {
             $records = []; // Failed DNS queries should not halt execution
         }
 
+        if ($records === false) {
+            $records = [];
+        }
+
         if (count($records) !== 0) {
             $records = array_map(static fn (array $x) => $x['target'] ?? '', $records);
             $records = array_filter($records, static fn (string $x) => $x !== '');