Bug #52260 [Asn]: Windows implementation of dns_get_record fails with non-existing domain
Edit report at http://bugs.php.net/bug.php?id=52260&edit=1
ID: 52260
Updated by: pajoye@php.net
Reported by: a_jelly_doughnut at phpbb dot com
Summary: Windows implementation of dns_get_record fails with
non-existing domain
Status: Assigned
Type: Bug
Package: Network related
Operating System: Windows (All)
PHP Version: 5.3.2
Assigned To: pajoye
Block user comment: N
New Comment:
The not found error has been fixed in svn (5.3/trunk).
However I keep this issue open and assigned to me as I think the DNS
function should not be as noisy as they are now (what's the point of
raising a warning if all one wants is to actually check a domain...).
Previous Comments:
------------------------------------------------------------------------
[2010-09-06 12:57:43] pajoye@php.net
Automatic comment from SVN on behalf of pajoye
Revision: http://svn.php.net/viewvc/?view=revision&revision=303069
Log: - fix bug #52260, catch all DNS not found error
------------------------------------------------------------------------
[2010-07-06 02:08:26] a_jelly_doughnut at phpbb dot com
Description:
------------
Windows implementation of dns_get_record fails with non-existing domain
---
var_dump(dns_get_record('dalfkjdaslfj.net')); // return false, throw
E_WARNING
---
This is inconsistent with the *NIX implementation, which never (in our
testing, see * note at bottom) throws E_WARNING on a non-existing
domain, and returns an empty array, as demonstrated below:
---
afischer@miraculix:~$ php -v
PHP 5.3.2-1ubuntu4.2 with Suhosin-Patch (cli) (built: May 13 2010
20:01:00)
Copyright (c) 1997-2009 The PHP Group
Zend Engine v2.3.0, Copyright (c) 1998-2010 Zend Technologies
with Xdebug v2.0.5, Copyright (c) 2002-2008, by Derick Rethans
afischer@miraculix:~$ php -r "error_reporting(E_ALL);
var_dump(dns_get_record('dalfkjdaslfj.net'));"
array(0) {
}
---
There is code in dns_win32.c to attempt to catch some cases of
non-existing domains, but it does not seem to be complete.
Changing
---
if (status) {
if (status == DNS_INFO_NO_RECORDS) {
continue;
---
to
---
if (status) {
if (status == DNS_INFO_NO_RECORDS || status ==
DNS_ERROR_RCODE_NAME_ERROR) {
continue;
---
Seems to fix the problems for me. For reference, all error codes which
can be returned by DNSQuery_A are listed by Microsoft:
http://msdn.microsoft.com/en-us/library/ms681391
* I cannot say for sure if reports like
http://bugs.php.net/bug.php?id=50903 are valid.
I do know from reading
the PHP source that there are three separate *NIX implementations of
php_dns_search(), but Andreas has not investigated which one his Ubuntu
server is using. It seems likely that this inconsistency affects NIX
implementations as well.
Test script:
---------------
<?php
var_dump(dns_get_record('dalfkjdaslfj.net')); // return false, throw
E_WARNING
?>
Expected result:
----------------
array(0) {
}
Actual result:
--------------
bool(false)
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/bug.php?id=52260&edit=1
Thread (5 messages)