PMD Results

The following document contains the results of PMD 3.7.

Files

org/apache/geronimo/security/deploy/Security.java

ViolationLine
Avoid unused imports such as 'java.util.HashSet'21
Avoid unused imports such as 'java.util.Iterator'22
Avoid unused imports such as 'java.util.Set'24

org/apache/geronimo/security/deploy/Role.java

ViolationLine
Avoid unused imports such as 'java.util.Map'21

org/apache/geronimo/security/keystore/FileKeystoreInstance.java

ViolationLine
Avoid empty catch blocks420 - 421
Avoid empty catch blocks457 - 459
Avoid empty catch blocks459 - 461
Avoid empty catch blocks461 - 463

org/apache/geronimo/security/keystore/FileKeystoreManager.java

ViolationLine
Avoid unused imports such as 'java.security.KeyManagementException'28
Avoid unused imports such as 'java.security.NoSuchProviderException'32
Avoid unused imports such as 'java.security.UnrecoverableKeyException'35
Avoid unused imports such as 'org.apache.geronimo.management.geronimo.KeyIsLocked'59
Avoid empty catch blocks356
Avoid empty catch blocks369

org/apache/geronimo/security/jaas/UPCredentialLoginModule.java

ViolationLine
Avoid empty catch blocks101 - 103

org/apache/geronimo/security/jaas/client/LoginModuleProxy.java

ViolationLine
Avoid unused imports such as 'org.apache.geronimo.security.jaas.server.JaasSessionId'22
Avoid unused imports such as 'org.apache.geronimo.security.jaas.server.JaasLoginServiceMBean'23

org/apache/geronimo/security/jaas/client/JaasLoginCoordinator.java

ViolationLine
These nested if statements could be combined229 - 231

org/apache/geronimo/security/jaas/server/JaasSecuritySession.java

ViolationLine
Avoid unused imports such as 'java.security.Principal'19
Avoid unused imports such as 'java.util.HashSet'20
Avoid unused imports such as 'java.util.Iterator'21
Avoid unused imports such as 'java.util.LinkedList'22
Avoid unused imports such as 'java.util.List'23
Avoid unused imports such as 'java.util.Set'25
Avoid unused imports such as 'org.apache.geronimo.security.DomainPrincipal'30
Avoid unused imports such as 'org.apache.geronimo.security.RealmPrincipal'31

org/apache/geronimo/security/jaas/server/JaasLoginServiceMBean.java

ViolationLine
Avoid unused imports such as 'java.util.Collection'20
Avoid unused imports such as 'org.apache.geronimo.common.GeronimoSecurityException'26

org/apache/geronimo/security/jaas/server/JaasLoginService.java

ViolationLine
Avoid empty catch blocks218 - 220
Avoid empty catch blocks223 - 225
Avoid empty catch blocks404 - 405
Avoid empty catch blocks405 - 406

org/apache/geronimo/security/jaas/NamedUPCredentialLoginModule.java

ViolationLine
Avoid empty catch blocks104 - 106

org/apache/geronimo/security/jaas/ConfiguredIdentityNamedUsernamePasswordLoginModule.java

ViolationLine
Avoid empty catch blocks78 - 80

org/apache/geronimo/security/ContextManager.java

ViolationLine
Avoid empty catch blocks376 - 377
Avoid empty catch blocks377 - 378

org/apache/geronimo/security/remoting/jmx/SerializableInvocation.java

ViolationLine
Avoid unused private fields such as 'proxy'40

org/apache/geronimo/security/jacc/ApplicationPrincipalRoleConfigurationManager.java

ViolationLine
Avoid unused imports such as 'java.security.Permission'19
Avoid unused imports such as 'java.security.PermissionCollection'20
Avoid unused imports such as 'java.security.Policy'21
Avoid unused imports such as 'java.util.Enumeration'22
Avoid unused imports such as 'java.util.HashMap'23
Avoid unused imports such as 'javax.security.auth.Subject'27
Avoid unused imports such as 'javax.security.jacc.PolicyConfiguration'28
Avoid unused imports such as 'javax.security.jacc.PolicyConfigurationFactory'29
Avoid unused imports such as 'org.apache.geronimo.gbean.GBeanLifecycle'34
Avoid unused imports such as 'org.apache.geronimo.security.ContextManager'36
Avoid unused imports such as 'org.apache.geronimo.security.IdentificationPrincipal'37
Avoid unused imports such as 'org.apache.geronimo.security.SubjectId'38

org/apache/geronimo/security/jacc/GeronimoPolicy.java

ViolationLine
Avoid empty catch blocks89 - 90

org/apache/geronimo/security/realm/providers/SQLLoginModule.java

ViolationLine
Avoid empty catch blocks125 - 127

org/apache/geronimo/security/realm/providers/GeronimoPasswordCredentialLoginModule.java

ViolationLine
Avoid empty catch blocks53 - 54

org/apache/geronimo/security/realm/providers/PasswordCallbackHandler.java

ViolationLine
Avoid unused imports such as 'javax.security.auth.callback.CallbackHandler'21

org/apache/geronimo/security/realm/providers/LDAPLoginModule.java

ViolationLine
Avoid unused imports such as 'java.util.List'26
No need to import a type that's in the same package56
No need to import a type that's in the same package57
Avoid unused private fields such as 'userPassword'90
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.130
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.131
Avoid empty if statements232
Avoid empty catch blocks260 - 262
Avoid unused local variables such as 'attr'334
Avoid unused private methods such as 'getAttributeValue(String,Attributes)'363

org/apache/geronimo/security/realm/providers/CertificateChainCallback.java

ViolationLine
Avoid unused imports such as 'java.security.cert.X509Certificate'19

org/apache/geronimo/security/realm/providers/CertificateChainLoginModule.java

ViolationLine
Avoid unused private fields such as 'log'53