You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by ba...@apache.org on 2006/11/09 01:36:59 UTC

svn commit: r472716 - /james/server/trunk/src/java/org/apache/james/dnsserver/DNSServer.java

Author: bago
Date: Wed Nov  8 16:36:58 2006
New Revision: 472716

URL: http://svn.apache.org/viewvc?view=rev&rev=472716
Log:
Fixed an NPE in new code for DNSServer (JAMES-680)
Patch by Robert Burrell Donkin

Modified:
    james/server/trunk/src/java/org/apache/james/dnsserver/DNSServer.java

Modified: james/server/trunk/src/java/org/apache/james/dnsserver/DNSServer.java
URL: http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/dnsserver/DNSServer.java?view=diff&rev=472716&r1=472715&r2=472716
==============================================================================
--- james/server/trunk/src/java/org/apache/james/dnsserver/DNSServer.java (original)
+++ james/server/trunk/src/java/org/apache/james/dnsserver/DNSServer.java Wed Nov  8 16:36:58 2006
@@ -467,13 +467,16 @@
      * @see org.apache.james.services.DNSServer#getHostName(java.net.InetAddress)
      */
     public String getHostName(InetAddress addr){
+        String result = null;
         Name name = ReverseMap.fromAddress(addr);
         Record [] records = lookup(name.toString(), Type.PTR, "PTR");
         if (records == null) {
-            addr.getHostAddress();
+            result = addr.getHostAddress();
+        } else {
+            PTRRecord ptr = (PTRRecord) records[0];
+            result = ptr.getTarget().toString();
         }
-        PTRRecord ptr = (PTRRecord) records[0];
-        return ptr.getTarget().toString();
+        return result;
     }
 
     /**



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org