The following document contains the results of FindBugs Report
FindBugs Version is 1.3.9
Threshold is medium
Effort is min
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 75 | 14 | 0 | 0 |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to LP9 in org.kuali.student.common.messagebuilder.booleanmessage.ast.parsers.BooleanFunctionParser.atom() | STYLE | DLS_DEAD_LOCAL_STORE | 385 | Medium |
| Dead store to RP11 in org.kuali.student.common.messagebuilder.booleanmessage.ast.parsers.BooleanFunctionParser.atom() | STYLE | DLS_DEAD_LOCAL_STORE | 392 | Medium |
| org.kuali.student.common.messagebuilder.booleanmessage.ast.parsers.BooleanFunctionParser.getTokenNames() may expose internal representation by returning BooleanFunctionParser.tokenNames | MALICIOUS_CODE | EI_EXPOSE_REP | 68 | Medium |
| org.kuali.student.common.messagebuilder.booleanmessage.ast.parsers.BooleanFunctionParser.tokenNames should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 33 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.util.ModPropertyPlaceholderConfigurer.setLocations(Resource[]) may expose internal representation by storing an externally mutable object into ModPropertyPlaceholderConfigurer.locations | MALICIOUS_CODE | EI_EXPOSE_REP2 | 54 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.common.util.PropertiesFactory.getProperties(String, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 150 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| instanceof will always return true in org.kuali.student.common.util.SimpleExceptionLoggingAdvice.afterThrowing(JoinPoint, Throwable), since all Throwable are instances of Throwable | STYLE | BC_VACUOUS_INSTANCEOF | 100 | Medium |
| Switch statement found in org.kuali.student.common.util.SimpleExceptionLoggingAdvice.afterThrowing(JoinPoint, Throwable) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | 101-104 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception is caught when Exception is not thrown in org.kuali.student.common.util.jpa.LoadJpaBeanFileListener.onApplicationEvent(ApplicationEvent) | STYLE | REC_CATCH_EXCEPTION | 100 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.common.util.security.UserWithId doesn't override org.springframework.security.userdetails.User.equals(Object) | STYLE | EQ_DOESNT_OVERRIDE_EQUALS | 28-37 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of org.kuali.student.common.ws.beans.JaxWsClientFactoryBean.serviceUrl; locked 60% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 116 | Medium |
| Inconsistent synchronization of org.kuali.student.common.ws.beans.JaxWsClientFactoryBean.wsdlDocumentLocation; locked 60% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 93 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception is caught when Exception is not thrown in org.kuali.student.common.ws.security.AuthenticationService.<static initializer>() | STYLE | REC_CATCH_EXCEPTION | 54 | Medium |
