NO-JIRA fix lgtm.com warnings

Warnings enumerated at
https://lgtm.com/projects/g/apache/activemq-artemis/alerts/?mode=tree&severity=warning

artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/messages/ConsumerThread.java

@@ -201,7 +201,7 @@ public class ConsumerThread extends Thread {
                   System.out.println(threadName + " Committing transaction: " + transactions++);
                   session.commit();
                }
-            } else if (session.getAcknowledgeMode() == Session.CLIENT_ACKNOWLEDGE) {
+            } else if (session.getAcknowledgeMode() == Session.CLIENT_ACKNOWLEDGE && msg != null) {
                if (batchSize > 0 && received > 0 && received % batchSize == 0) {
                   System.out.println("Acknowledging last " + batchSize + " messages; messages so far = " + received);
                   msg.acknowledge();

artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/tools/PrintData.java

@@ -216,7 +216,6 @@ public class PrintData extends DBOption {
 
          if (pgStore != null) {
             folder = pgStore.getFolder();
-         }
             out.println("####################################################################################################");
             out.println("Exploring store " + store + " folder = " + folder);
             int pgid = (int) pgStore.getFirstPage();
@@ -276,6 +275,7 @@ public class PrintData extends DBOption {
             }
          }
       }
+   }
 
    /**
     * Calculate the acks on the page system

artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/tools/xml/XMLMessageImporter.java

@@ -71,9 +71,9 @@ public class XMLMessageImporter {
 
       Byte type = 0;
       Byte priority = 0;
-      Long expiration = 0L;
+      long expiration = 0L;
-      Long timestamp = 0L;
+      long timestamp = 0L;
-      Long id = 0L;
+      long id = 0L;
       org.apache.activemq.artemis.utils.UUID userId = null;
       ArrayList<String> queues = new ArrayList<>();
 
@@ -276,7 +276,7 @@ public class XMLMessageImporter {
     * CDATA has to be decoded in its entirety.
     *
     * @param processor used to deal with the decoded CDATA elements
-    * @param textMessage If this a text message we decode UTF8 and encode as a simple string
+    * @param decodeTextMessage If this a text message we decode UTF8 and encode as a simple string
     */
    private void getMessageBodyBytes(MessageBodyBytesProcessor processor, boolean decodeTextMessage) throws IOException, XMLStreamException {
       int currentEventType;

artemis-commons/src/main/java/org/apache/activemq/artemis/core/server/NetworkHealthCheck.java

@@ -86,7 +86,7 @@ public class NetworkHealthCheck extends ActiveMQScheduledComponent {
             netToUse = null;
          }
       } catch (Exception e) {
-         ActiveMQUtilLogger.LOGGER.failedToSetNIC(e, nicName == null ? " " : nicName);
+         ActiveMQUtilLogger.LOGGER.failedToSetNIC(e, nicName);
          netToUse = null;
       }
 
@@ -326,6 +326,10 @@ public class NetworkHealthCheck extends ActiveMQScheduledComponent {
    }
 
    public boolean check(InetAddress address) {
+      if (address == null) {
+         return false;
+      }
+
       try {
          if (address.isReachable(networkInterface, 0, networkTimeout)) {
             if (logger.isTraceEnabled()) {
@@ -336,7 +340,7 @@ public class NetworkHealthCheck extends ActiveMQScheduledComponent {
             return purePing(address);
          }
       } catch (Exception e) {
-         ActiveMQUtilLogger.LOGGER.failedToCheckAddress(e, address == null ? " " : address.toString());
+         ActiveMQUtilLogger.LOGGER.failedToCheckAddress(e, address.toString());
          return false;
       }
    }
@@ -392,6 +396,10 @@ public class NetworkHealthCheck extends ActiveMQScheduledComponent {
    }
 
    public boolean check(URL url) {
+      if (url == null) {
+         return false;
+      }
+
       try {
          URLConnection connection = url.openConnection();
          connection.setReadTimeout(networkTimeout);
@@ -399,7 +407,7 @@ public class NetworkHealthCheck extends ActiveMQScheduledComponent {
          is.close();
          return true;
       } catch (Exception e) {
-         ActiveMQUtilLogger.LOGGER.failedToCheckURL(e, url == null ? " " : url.toString());
+         ActiveMQUtilLogger.LOGGER.failedToCheckURL(e, url.toString());
          return false;
       }
    }

artemis-commons/src/main/java/org/apache/activemq/artemis/utils/FactoryFinder.java

@@ -101,17 +101,10 @@ public class FactoryFinder {
          }
 
          // lets load the file
-         BufferedInputStream reader = null;
+         try (BufferedInputStream reader = new BufferedInputStream(in)) {
-         try {
-            reader = new BufferedInputStream(in);
             Properties properties = new Properties();
             properties.load(reader);
             return properties;
-         } finally {
-            try {
-               reader.close();
-            } catch (Exception e) {
-            }
          }
       }
    }

artemis-commons/src/main/java/org/apache/activemq/artemis/utils/collections/ConcurrentLongHashMap.java

@@ -295,12 +295,12 @@ public class ConcurrentLongHashMap<V> {
 
                if (storedKey == key) {
                   if (storedValue == EmptyValue) {
-                     values[bucket] = value != null ? value : valueProvider.apply(key);
+                     values[bucket] = value != null ? value : (valueProvider != null ? valueProvider.apply(key) : null);
                      ++size;
                      ++usedBuckets;
                      return valueProvider != null ? values[bucket] : null;
                   } else if (storedValue == DeletedValue) {
-                     values[bucket] = value != null ? value : valueProvider.apply(key);
+                     values[bucket] = value != null ? value : (valueProvider != null ? valueProvider.apply(key) : null);
                      ++size;
                      return valueProvider != null ? values[bucket] : null;
                   } else if (!onlyIfAbsent) {
@@ -320,7 +320,7 @@ public class ConcurrentLongHashMap<V> {
                   }
 
                   keys[bucket] = key;
-                  values[bucket] = value != null ? value : valueProvider.apply(key);
+                  values[bucket] = value != null ? value : (valueProvider != null ? valueProvider.apply(key) : null);
                   ++size;
                   return valueProvider != null ? values[bucket] : null;
                } else if (storedValue == DeletedValue) {

artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientProducerImpl.java

@@ -145,7 +145,7 @@ public class ClientProducerImpl implements ClientProducerInternal {
          doSend(address1, message, null);
          if (handler != null) {
             if (logger.isDebugEnabled()) {
-               logger.debug("Handler was used on producing messages towards address " + address1.toString() + " however there is no confirmationWindowEnabled");
+               logger.debug("Handler was used on producing messages towards address " + (address1 == null ? null : address1.toString()) + " however there is no confirmationWindowEnabled");
             }
 
             if (!confirmationNotSetLogged) {

artemis-core-client/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/ActiveMQClientProtocolManager.java

@@ -469,8 +469,8 @@ public class ActiveMQClientProtocolManager implements ClientProtocolManager {
          } else if (type == PacketImpl.CLUSTER_TOPOLOGY_V2) {
             ClusterTopologyChangeMessage_V2 topMessage = (ClusterTopologyChangeMessage_V2) packet;
             notifyTopologyChange(updateTransportConfiguration(topMessage));
-         } else if (type == PacketImpl.CLUSTER_TOPOLOGY || type == PacketImpl.CLUSTER_TOPOLOGY_V2 || type == PacketImpl.CLUSTER_TOPOLOGY_V3) {
+         } else if (type == PacketImpl.CLUSTER_TOPOLOGY_V3) {
-            ClusterTopologyChangeMessage topMessage = (ClusterTopologyChangeMessage) packet;
+            ClusterTopologyChangeMessage_V3 topMessage = (ClusterTopologyChangeMessage_V3) packet;
             notifyTopologyChange(updateTransportConfiguration(topMessage));
          } else if (type == PacketImpl.CHECK_FOR_FAILOVER_REPLY) {
             System.out.println("Channel0Handler.handlePacket");

artemis-core-client/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/ChannelImpl.java

@@ -695,7 +695,7 @@ public final class ChannelImpl implements Channel {
             clearUpTo(msg.getCommandID());
          }
 
-         if (!connection.isClient()) {
+         if (!connection.isClient() && handler != null) {
             handler.handlePacket(packet);
          }
 

artemis-core-client/src/main/java/org/apache/activemq/artemis/core/remoting/impl/netty/NettyConnector.java

@@ -1181,7 +1181,7 @@ public class NettyConnector extends AbstractConnector {
 
    @Override
    public boolean isEquivalent(Map<String, Object> configuration) {
-      Boolean httpUpgradeEnabled = ConfigurationHelper.getBooleanProperty(TransportConstants.HTTP_UPGRADE_ENABLED_PROP_NAME, TransportConstants.DEFAULT_HTTP_UPGRADE_ENABLED, configuration);
+      boolean httpUpgradeEnabled = ConfigurationHelper.getBooleanProperty(TransportConstants.HTTP_UPGRADE_ENABLED_PROP_NAME, TransportConstants.DEFAULT_HTTP_UPGRADE_ENABLED, configuration);
       if (httpUpgradeEnabled) {
          // we need to look at the activemqServerName to distinguish between ActiveMQ servers that could be proxied behind the same
          // HTTP upgrade handler in the Web server
@@ -1198,9 +1198,9 @@ public class NettyConnector extends AbstractConnector {
       //here we only check host and port because these two parameters
       //is sufficient to determine the target host
       String host = ConfigurationHelper.getStringProperty(TransportConstants.HOST_PROP_NAME, TransportConstants.DEFAULT_HOST, configuration);
-      Integer port = ConfigurationHelper.getIntProperty(TransportConstants.PORT_PROP_NAME, TransportConstants.DEFAULT_PORT, configuration);
+      int port = ConfigurationHelper.getIntProperty(TransportConstants.PORT_PROP_NAME, TransportConstants.DEFAULT_PORT, configuration);
 
-      if (!port.equals(this.port))
+      if (port != this.port)
          return false;
 
       if (host.equals(this.host))

artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/drivers/AbstractJDBCDriver.java

@@ -199,7 +199,7 @@ public abstract class AbstractJDBCDriver {
          connection.setAutoCommit(false);
          final boolean tableExists;
          try (ResultSet rs = connection.getMetaData().getTables(null, null, tableName, null)) {
-            if ((rs == null) || (rs != null && !rs.next())) {
+            if (rs == null || !rs.next()) {
                tableExists = false;
                if (logger.isTraceEnabled()) {
                   logger.tracef("Table %s did not exist, creating it with SQL=%s", tableName, Arrays.toString(sqls));

artemis-jms-client/src/main/java/org/apache/activemq/artemis/jms/client/ActiveMQMessage.java

@@ -433,7 +433,7 @@ public class ActiveMQMessage implements javax.jms.Message {
             dest = (ActiveMQDestination) ActiveMQDestination.fromPrefixedName(address.toString());
          }
 
-         if (changedAddress != null) {
+         if (changedAddress != null && dest != null) {
             ((ActiveMQDestination) dest).setName(changedAddress.toString());
          }
       }
@@ -902,7 +902,7 @@ public class ActiveMQMessage implements javax.jms.Message {
 
    private void checkProperty(final String name) throws JMSException {
       if (propertiesReadOnly) {
-         if (name.equals(ActiveMQJMSConstants.JMS_ACTIVEMQ_INPUT_STREAM)) {
+         if (name != null && name.equals(ActiveMQJMSConstants.JMS_ACTIVEMQ_INPUT_STREAM)) {
             throw new MessageNotWriteableException("You cannot set the Input Stream on received messages. Did you mean " + ActiveMQJMSConstants.JMS_ACTIVEMQ_OUTPUT_STREAM +
                                                       " or " +
                                                       ActiveMQJMSConstants.JMS_ACTIVEMQ_SAVE_STREAM +

artemis-journal/src/main/java/org/apache/activemq/artemis/core/io/aio/AIOSequentialFileFactory.java

@@ -370,7 +370,9 @@ public final class AIOSequentialFileFactory extends AbstractSequentialFileFactor
       public void sequentialDone() {
 
          if (error) {
+            if (callback != null) {
                callback.onError(errorCode, errorMessage);
+            }
             onIOError(new ActiveMQException(errorCode, errorMessage), errorMessage, null);
             errorMessage = null;
          } else {

artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/converter/CoreAmqpConverter.java

@@ -323,12 +323,8 @@ public class CoreAmqpConverter {
          if (daMap != null) {
             encoder.writeObject(new DeliveryAnnotations(daMap));
          }
-         if (maMap != null) {
          encoder.writeObject(new MessageAnnotations(maMap));
-         }
-         if (properties != null) {
          encoder.writeObject(properties);
-         }
          if (apMap != null) {
             encoder.writeObject(new ApplicationProperties(apMap));
          }

artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireConnection.java

@@ -1261,8 +1261,10 @@ public class OpenWireConnection extends AbstractRemotingConnection implements Se
                }
             }
          } else {
+            if (tx != null) {
                tx.rollback();
             }
+         }
 
          return null;
       }
@@ -1413,8 +1415,10 @@ public class OpenWireConnection extends AbstractRemotingConnection implements Se
                }
             }
          } else {
+            if (tx != null) {
                tx.commit(onePhase);
             }
+         }
 
          return null;
       }

artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireMessageConverter.java

@@ -782,7 +782,7 @@ public final class OpenWireMessageConverter {
                   MarshallingSupport.marshalDouble(dataOut, doubleVal);
                   break;
                case DataConstants.FLOAT:
-                  Float floatVal = Float.intBitsToFloat(buffer.readInt());
+                  float floatVal = Float.intBitsToFloat(buffer.readInt());
                   MarshallingSupport.marshalFloat(dataOut, floatVal);
                   break;
                case DataConstants.INT:

artemis-ra/src/main/java/org/apache/activemq/artemis/ra/ActiveMQRAManagedConnection.java

@@ -250,9 +250,8 @@ public final class ActiveMQRAManagedConnection implements ManagedConnection, Exc
       } catch (JMSException e) {
          logger.debug("Error unsetting the exception listener " + this, e);
       }
-      if (connection != null) {
+
       connection.signalStopToAllSessions();
-      }
 
       try {
          // we must close the ActiveMQConnectionFactory because it contains a ServerLocator
@@ -277,9 +276,7 @@ public final class ActiveMQRAManagedConnection implements ManagedConnection, Exc
              * <p>
              * connection close will close the ClientSessionFactory which will close all sessions.
              */
-            if (connection != null) {
             connection.close();
-            }
 
             if (nonXAsession != null) {
                nonXAsession.close();

artemis-ra/src/main/java/org/apache/activemq/artemis/ra/ConnectionFactoryProperties.java

@@ -934,12 +934,11 @@ public class ConnectionFactoryProperties implements ConnectionFactoryOptions {
             return false;
       } else if (!this.producerWindowSize.equals(other.producerWindowSize))
          return false;
-      else if (!protocolManagerFactoryStr.equals(other.protocolManagerFactoryStr))
-         return false;
       if (this.protocolManagerFactoryStr == null) {
          if (other.protocolManagerFactoryStr != null)
             return false;
-      }
+      } else if (!protocolManagerFactoryStr.equals(other.protocolManagerFactoryStr))
+         return false;
       if (this.reconnectAttempts == null) {
          if (other.reconnectAttempts != null)
             return false;
@@ -1007,7 +1006,7 @@ public class ConnectionFactoryProperties implements ConnectionFactoryOptions {
       if (enableSharedClientID == null) {
          if (other.enableSharedClientID != null)
             return false;
-      } else if (!enableSharedClientID == other.enableSharedClientID)
+      } else if (!this.enableSharedClientID.equals(other.enableSharedClientID))
          return false;
 
       return true;

artemis-rest/src/main/java/org/apache/activemq/artemis/rest/topic/SubscriptionsResource.java

@@ -148,7 +148,7 @@ public class SubscriptionsResource implements TimeoutTask.Callback {
       ActiveMQRestLogger.LOGGER.debug("Handling POST request for \"" + uriInfo.getPath() + "\"");
 
       if (timeout == null)
-         timeout = Long.valueOf(consumerTimeoutSeconds * 1000);
+         timeout = Long.valueOf((long) consumerTimeoutSeconds * 1000);
       boolean deleteWhenIdle = !durable; // default is true if non-durable
       if (destroyWhenIdle != null)
          deleteWhenIdle = destroyWhenIdle.booleanValue();

artemis-rest/src/main/java/org/apache/activemq/artemis/rest/util/TimeoutTask.java

@@ -96,7 +96,7 @@ public class TimeoutTask implements Runnable {
    public void run() {
       while (running) {
          try {
-            Thread.sleep(interval * 1000);
+            Thread.sleep((long) interval * 1000);
          } catch (InterruptedException e) {
             running = false;
             break;

artemis-server/src/main/java/org/apache/activemq/artemis/core/config/impl/ConfigurationImpl.java

@@ -2044,9 +2044,6 @@ public class ConfigurationImpl implements Configuration, Serializable {
       if (diskScanPeriod != other.diskScanPeriod) {
          return false;
       }
-      if (connectionTtlCheckInterval != other.connectionTtlCheckInterval) {
-         return false;
-      }
 
       return true;
    }

artemis-server/src/main/java/org/apache/activemq/artemis/core/management/impl/QueueControlImpl.java

@@ -1131,7 +1131,7 @@ public class QueueControlImpl extends AbstractControl implements QueueControl {
       clearIO();
       try {
          long index = 0;
-         long start = (page - 1) * pageSize;
+         long start = (long) (page - 1) * pageSize;
          long end = Math.min(page * pageSize, queue.getMessageCount());
 
          ArrayList<CompositeData> c = new ArrayList<>();

artemis-server/src/main/java/org/apache/activemq/artemis/core/postoffice/impl/PostOfficeImpl.java

@@ -1325,7 +1325,7 @@ public class PostOfficeImpl implements PostOffice, NotificationListener, Binding
                   storageManager.storeReference(queue.getID(), message.getMessageID(), !iter.hasNext());
                }
 
-               if (deliveryTime > 0) {
+               if (deliveryTime != null && deliveryTime > 0) {
                   if (tx != null) {
                      storageManager.updateScheduledDeliveryTimeTransactional(tx.getID(), reference);
                   } else {

artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/ha/ColocatedHAManager.java

@@ -291,8 +291,8 @@ public class ColocatedHAManager implements HAManager {
          }
          Object serverId = params.get(org.apache.activemq.artemis.core.remoting.impl.invm.TransportConstants.SERVER_ID_PROP_NAME);
          if (serverId != null) {
-            Integer newid = Integer.parseInt(serverId.toString()) + portOffset;
+            int newid = Integer.parseInt(serverId.toString()) + portOffset;
-            params.put(org.apache.activemq.artemis.core.remoting.impl.invm.TransportConstants.SERVER_ID_PROP_NAME, newid.toString());
+            params.put(org.apache.activemq.artemis.core.remoting.impl.invm.TransportConstants.SERVER_ID_PROP_NAME, newid);
          }
          params.put(TransportConstants.ACTIVEMQ_SERVER_NAME, name);
       }

artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/impl/ClusterConnectionBridge.java

@@ -196,6 +196,7 @@ public class ClusterConnectionBridge extends BridgeImpl {
    }
 
    private void setupNotificationConsumer() throws Exception {
+      if (flowRecord != null) {
          if (logger.isDebugEnabled()) {
             logger.debug("Setting up notificationConsumer between " + this.clusterConnection.getConnector() +
                             " and " +
@@ -205,7 +206,6 @@ public class ClusterConnectionBridge extends BridgeImpl {
                             " on server " +
                             clusterConnection.getServer());
          }
-      if (flowRecord != null) {
          flowRecord.reset();
 
          if (notifConsumer != null) {

artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/LiveOnlyActivation.java

@@ -146,10 +146,10 @@ public class LiveOnlyActivation extends Activation {
          ClientSessionFactoryInternal clientSessionFactory = null;
          while (clientSessionFactory == null) {
             Pair<TransportConfiguration, TransportConfiguration> possibleLive = null;
-            try {
             possibleLive = nodeLocator.getLiveConfiguration();
             if (possibleLive == null)  // we've tried every connector
                break;
+            try {
                clientSessionFactory = (ClientSessionFactoryInternal) scaleDownServerLocator.createSessionFactory(possibleLive.getA(), 0, false);
             } catch (Exception e) {
                logger.trace("Failed to connect to " + possibleLive.getA());

artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java

@@ -3685,7 +3685,10 @@ public class QueueImpl extends CriticalComponentImpl implements Queue {
          if (slowConsumerReaperRunnable == null) {
             scheduleSlowConsumerReaper(settings);
          } else if (slowConsumerReaperRunnable.checkPeriod != settings.getSlowConsumerCheckPeriod() || slowConsumerReaperRunnable.threshold != settings.getSlowConsumerThreshold() || !slowConsumerReaperRunnable.policy.equals(settings.getSlowConsumerPolicy())) {
+            if (slowConsumerReaperFuture != null) {
                slowConsumerReaperFuture.cancel(false);
+               slowConsumerReaperFuture = null;
+            }
             scheduleSlowConsumerReaper(settings);
          }
       }

artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ServerConsumerImpl.java

@@ -389,7 +389,7 @@ public class ServerConsumerImpl implements ServerConsumer, ReadyListener {
          // should go back into the
          // queue for delivery later.
          // TCP-flow control has to be done first than everything else otherwise we may lose notifications
-         if (!callback.isWritable(this, protocolContext) || !started || transferring) {
+         if ((callback != null && !callback.isWritable(this, protocolContext)) || !started || transferring) {
             return HandleStatus.BUSY;
          }
 

artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/SharedNothingBackupActivation.java

@@ -215,7 +215,7 @@ public final class SharedNothingBackupActivation extends Activation {
 
             try {
                if (logger.isTraceEnabled()) {
-                  logger.trace("Calling clusterController.connectToNodeInReplicatedCluster(" + possibleLive.getA() + ")");
+                  logger.trace("Calling clusterController.connectToNodeInReplicatedCluster(" + possibleLive != null ? possibleLive.getA() : null + ")");
                }
                clusterControl = clusterController.connectToNodeInReplicatedCluster(possibleLive.getA());
             } catch (Exception e) {

artemis-server/src/main/java/org/apache/activemq/artemis/core/server/plugin/ActiveMQServerConsumerPlugin.java

@@ -51,7 +51,7 @@ public interface ActiveMQServerConsumerPlugin extends ActiveMQServerBasePlugin {
     * Before a consumer is created
     *
     * @param consumerID
-    * @param QueueBinding
+    * @param queueBinding
     * @param filterString
     * @param browseOnly
     * @param supportLargeMessage

artemis-server/src/main/java/org/apache/activemq/artemis/core/transaction/impl/TransactionImpl.java

@@ -181,9 +181,9 @@ public class TransactionImpl implements Transaction {
       synchronized (timeoutLock) {
          boolean timedout;
          if (timeoutSeconds == -1) {
-            timedout = getState() != Transaction.State.PREPARED && currentTime > createTime + defaultTimeout * 1000;
+            timedout = getState() != Transaction.State.PREPARED && currentTime > createTime + (long) defaultTimeout * 1000;
          } else {
-            timedout = getState() != Transaction.State.PREPARED && currentTime > createTime + timeoutSeconds * 1000;
+            timedout = getState() != Transaction.State.PREPARED && currentTime > createTime + (long) timeoutSeconds * 1000;
          }
 
          if (timedout) {

artemis-server/src/main/java/org/apache/activemq/artemis/spi/core/security/jaas/LDAPLoginModule.java

@@ -184,7 +184,7 @@ public class LDAPLoginModule implements LoginModule {
        * requests (by verifying that the supplied password is not empty) and
        * react appropriately.
        */
-      if (password == null || (password != null && password.length() == 0))
+      if (password == null || password.length() == 0)
          throw new FailedLoginException("Password cannot be null or empty");
 
       // authenticate will throw LoginException