summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPedro Magalhães <mail@pmmaga.net>2017-07-09 16:02:27 +0200
committerJoe Watkins <krakjoe@php.net>2017-07-10 06:55:31 +0100
commit2689917b17fd6018950b2af0711cb7fff702135d (patch)
tree177c82f55ddc9e4c796023761cc8984ffd9abc14
parent5cf54f6073f9e3cfc99eca9e8a5706a08dd404a1 (diff)
downloadphp-git-2689917b17fd6018950b2af0711cb7fff702135d.tar.gz
#73594 tests only check the extra params if dns_get_record is successful
-rw-r--r--ext/standard/tests/network/bug73594.phpt5
-rw-r--r--ext/standard/tests/network/bug73594a.phpt5
2 files changed, 6 insertions, 4 deletions
diff --git a/ext/standard/tests/network/bug73594.phpt b/ext/standard/tests/network/bug73594.phpt
index a3068360ce..370b6162fd 100644
--- a/ext/standard/tests/network/bug73594.phpt
+++ b/ext/standard/tests/network/bug73594.phpt
@@ -19,8 +19,9 @@ if (empty($out)) die("skip local resolver does not return additional records");
<?php
$auth = array();
$additional = array();
-dns_get_record('php.net', DNS_MX, $auth, $additional);
-var_dump(empty($additional));
+$res = dns_get_record('php.net', DNS_MX, $auth, $additional);
+// only check $additional if dns_get_record is successful
+var_dump(!empty($res) && empty($additional));
?>
--EXPECT--
bool(false)
diff --git a/ext/standard/tests/network/bug73594a.phpt b/ext/standard/tests/network/bug73594a.phpt
index a0a08e4a9e..9a07fb71ed 100644
--- a/ext/standard/tests/network/bug73594a.phpt
+++ b/ext/standard/tests/network/bug73594a.phpt
@@ -18,8 +18,9 @@ if (empty($out)) die("skip local resolver does not return authority records");
--FILE--
<?php
$auth = array();
-dns_get_record('php.net', DNS_MX, $auth);
-var_dump(empty($auth));
+$res = dns_get_record('php.net', DNS_MX, $auth);
+// only check $auth if dns_get_record is successful
+var_dump(!empty($res) && empty($auth));
?>
--EXPECT--
bool(false)