[389-commits] ldap/admin

Noriko Hosoi nhosoi at fedoraproject.org
Fri Mar 30 17:44:48 UTC 2012


 ldap/admin/src/scripts/DSUtil.pm.in |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 796dcfedee3d8e884dafb02ed0fce86cd95199b2
Author: Noriko Hosoi <nhosoi at totoro.usersys.redhat.com>
Date:   Thu Mar 29 15:38:16 2012 -0700

    Trac Ticket #46 - (additional) setup-ds-admin.pl does not
        like ipv6 only hostnames
    
    commit 850005499bd92c5b9b0027f944fcd33633c8db46 included a logic bug.
    The host name returned from getnameinfo needs to be checked with the
    original when the host name is not NULL.

diff --git a/ldap/admin/src/scripts/DSUtil.pm.in b/ldap/admin/src/scripts/DSUtil.pm.in
index a3a8abf..e84f482 100644
--- a/ldap/admin/src/scripts/DSUtil.pm.in
+++ b/ldap/admin/src/scripts/DSUtil.pm.in
@@ -269,7 +269,7 @@ sub checkHostname {
             debug(1, "ipaddr=", $ip, "\n");
             # see if reverse resolution works
             my ($hn2, $service) = getnameinfo($saddr);
-            if (!$hn2) {
+            if ($hn2) {
                 push @hostip, [$hn2, $ip];
                 if (lc($hn) eq lc($hn2)) {
                     $found = 1;




More information about the 389-commits mailing list