Mercury-IM/Smack
Mercury-IM/Smack
1
0
Fork 0
mirror of https://codeberg.org/Mercury-IM/Smack synced 2024-09-28 10:49:34 +02:00
Code Issues Releases Wiki Activity

Return empty list instead of null in DNSUtil

Browse source
This commit is contained in:
Florian Schmaus 2014-03-26 22:48:32 +01:00
parent 42d0557682
commit c59a6010e1
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
core/src/main/java/org/jivesoftware/smack/util/DNSUtil.java
View file

@ -122,8 +122,7 @@ public class DNSUtil {
} }
List<SRVRecord> srvRecords = dnsResolver.lookupSRVRecords(srvDomain); List<SRVRecord> srvRecords = dnsResolver.lookupSRVRecords(srvDomain);
List<HostAddress> sortedRecords = sortSRVRecords(srvRecords); List<HostAddress> sortedRecords = sortSRVRecords(srvRecords);
if (sortedRecords != null) addresses.addAll(sortedRecords);
addresses.addAll(sortedRecords);
// Step two: Add the hostname to the end of the list // Step two: Add the hostname to the end of the list
addresses.add(new HostAddress(domain)); addresses.add(new HostAddress(domain));
@ -139,11 +138,11 @@ public class DNSUtil {
* @param records * @param records
* @return the list of resolved HostAddresses * @return the list of resolved HostAddresses
*/ */
protected static List<HostAddress> sortSRVRecords(List<SRVRecord> records) { private static List<HostAddress> sortSRVRecords(List<SRVRecord> records) {
// RFC 2782, Usage rules: "If there is precisely one SRV RR, and its Target is "." // RFC 2782, Usage rules: "If there is precisely one SRV RR, and its Target is "."
// (the root domain), abort." // (the root domain), abort."
if (records.size() == 1 && records.get(0).getFQDN().equals(".")) if (records.size() == 1 && records.get(0).getFQDN().equals("."))
return null; return Collections.emptyList();
// sorting the records improves the performance of the bisection later // sorting the records improves the performance of the bisection later
Collections.sort(records); Collections.sort(records);