Applies the patch contributed by Dai Zhiwei which fixes the throwing of NullPointerException with details on the parameter being the cause in the Gibberish and Icq ProtocolProviderFactory implementations.

1 files changed, 8 insertions, 8 deletions

diff --git a/src/net/java/sip/communicator/impl/protocol/icq/ProtocolProviderFactoryIcqImpl.java b/src/net/java/sip/communicator/impl/protocol/icq/ProtocolProviderFactoryIcqImpl.java
index 6320497..8665b4c 100644
--- a/src/net/java/sip/communicator/impl/protocol/icq/ProtocolProviderFactoryIcqImpl.java
+++ b/src/net/java/sip/communicator/impl/protocol/icq/ProtocolProviderFactoryIcqImpl.java
@@ -313,6 +313,14 @@ public class ProtocolProviderFactoryIcqImpl
                                 Map accountProperties)
         throws NullPointerException
     {
+        // Make sure the specified arguments are valid.
+        if (protocolProvider == null)
+            throw new NullPointerException(
+                "The specified Protocol Provider was null");
+        if (accountProperties == null)
+            throw new NullPointerException(
+                "The specified property map was null");
+
         BundleContext context
             = IcqActivator.getBundleContext();
 
@@ -320,10 +328,6 @@ public class ProtocolProviderFactoryIcqImpl
             throw new NullPointerException(
                 "The specified BundleContext was null");
 
-        if (protocolProvider == null)
-            throw new NullPointerException(
-                "The specified Protocol Provider was null");
-
         IcqAccountID accountID = (IcqAccountID) protocolProvider.getAccountID();
 
         // If the given accountID doesn't correspond to an existing account
@@ -340,10 +344,6 @@ public class ProtocolProviderFactoryIcqImpl
 
         accountProperties.put(USER_ID, accountID.getUserID());
 
-        if (accountProperties == null)
-            throw new NullPointerException(
-                "The specified property map was null");
-
         if (!accountProperties.containsKey(PROTOCOL))
             accountProperties.put(PROTOCOL, ProtocolNames.ICQ);