FindBugs Bug Detector Report
The following document contains the results of FindBugs
FindBugs Version is 2.0.3
Threshold is medium
Effort is min
Summary
| Classes |
Bugs |
Errors |
Missing Classes |
| 70 |
24 |
0 |
0 |
org.apache.maven.plugin.surefire.AbstractSurefireMojo
| Bug |
Category |
Details |
Line |
Priority |
| org.apache.maven.plugin.surefire.AbstractSurefireMojo.getAdditionalClasspathElements() may expose internal representation by returning AbstractSurefireMojo.additionalClasspathElements |
MALICIOUS_CODE |
EI_EXPOSE_REP |
3005 |
Medium |
| org.apache.maven.plugin.surefire.AbstractSurefireMojo.getClasspathDependencyExcludes() may expose internal representation by returning AbstractSurefireMojo.classpathDependencyExcludes |
MALICIOUS_CODE |
EI_EXPOSE_REP |
3015 |
Medium |
| org.apache.maven.plugin.surefire.AbstractSurefireMojo.getDependenciesToScan() may expose internal representation by returning AbstractSurefireMojo.dependenciesToScan |
MALICIOUS_CODE |
EI_EXPOSE_REP |
2959 |
Medium |
| org.apache.maven.plugin.surefire.AbstractSurefireMojo.setAdditionalClasspathElements(String[]) may expose internal representation by storing an externally mutable object into AbstractSurefireMojo.additionalClasspathElements |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
3010 |
Medium |
| org.apache.maven.plugin.surefire.AbstractSurefireMojo.setClasspathDependencyExcludes(String[]) may expose internal representation by storing an externally mutable object into AbstractSurefireMojo.classpathDependencyExcludes |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
3020 |
Medium |
| org.apache.maven.plugin.surefire.AbstractSurefireMojo.setDependenciesToScan(String[]) may expose internal representation by storing an externally mutable object into AbstractSurefireMojo.dependenciesToScan |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
2964 |
Medium |
| Exceptional return value of java.io.File.mkdirs() ignored in org.apache.maven.plugin.surefire.AbstractSurefireMojo.getForkConfiguration() |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
1783 |
Medium |
org.apache.maven.plugin.surefire.SurefireProperties
| Bug |
Category |
Details |
Line |
Priority |
| org.apache.maven.plugin.surefire.SurefireProperties doesn't override java.util.Hashtable.equals(Object) |
STYLE |
EQ_DOESNT_OVERRIDE_EQUALS |
857-886 |
Medium |
| org.apache.maven.plugin.surefire.SurefireProperties.copyProperties(Properties, Map) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
173 |
Medium |
| org.apache.maven.plugin.surefire.SurefireProperties.copyPropertiesFrom(Properties) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
103 |
Medium |
| org.apache.maven.plugin.surefire.SurefireProperties.copyTo(Map) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
187 |
Medium |
org.apache.maven.plugin.surefire.booterclient.ForkConfiguration
| Bug |
Category |
Details |
Line |
Priority |
| org.apache.maven.plugin.surefire.booterclient.ForkConfiguration.createJar(List, String) concatenates strings using + in a loop |
PERFORMANCE |
SBSC_USE_STRINGBUFFER_CONCATENATION |
274 |
Medium |
org.apache.maven.plugin.surefire.booterclient.MockReporter
| Bug |
Category |
Details |
Line |
Priority |
| Found reliance on default encoding in org.apache.maven.plugin.surefire.booterclient.MockReporter.writeTestOutput(byte[], int, int, boolean): new String(byte[], int, int) |
I18N |
DM_DEFAULT_ENCODING |
158 |
High |
org.apache.maven.plugin.surefire.booterclient.ProviderDetector
| Bug |
Category |
Details |
Line |
Priority |
| Found reliance on default encoding in org.apache.maven.plugin.surefire.booterclient.ProviderDetector.getReader(URL): new java.io.InputStreamReader(InputStream) |
I18N |
DM_DEFAULT_ENCODING |
125 |
High |
org.apache.maven.plugin.surefire.booterclient.lazytestprovider.OutputStreamFlushableCommandline$OutputStreamFlushReceiver
| Bug |
Category |
Details |
Line |
Priority |
| Should org.apache.maven.plugin.surefire.booterclient.lazytestprovider.OutputStreamFlushableCommandline$OutputStreamFlushReceiver be a _static_ inner class? |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
41-55 |
Medium |
org.apache.maven.plugin.surefire.booterclient.lazytestprovider.TestProvidingInputStream
| Bug |
Category |
Details |
Line |
Priority |
| Found reliance on default encoding in org.apache.maven.plugin.surefire.booterclient.lazytestprovider.TestProvidingInputStream.read(): String.getBytes() |
I18N |
DM_DEFAULT_ENCODING |
93 |
High |
| Inconsistent synchronization of org.apache.maven.plugin.surefire.booterclient.lazytestprovider.TestProvidingInputStream.flushReceiverProvider; locked 75% of time |
MT_CORRECTNESS |
IS2_INCONSISTENT_SYNC |
68 |
Medium |
org.apache.maven.plugin.surefire.report.ConsoleOutputFileReporter
| Bug |
Category |
Details |
Line |
Priority |
| Exceptional return value of java.io.File.mkdirs() ignored in org.apache.maven.plugin.surefire.report.ConsoleOutputFileReporter.writeTestOutput(byte[], int, int, boolean) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
89 |
Medium |
org.apache.maven.plugin.surefire.report.ConsoleReporter
| Bug |
Category |
Details |
Line |
Priority |
| Found reliance on default encoding in new org.apache.maven.plugin.surefire.report.ConsoleReporter(PrintStream): new java.io.OutputStreamWriter(OutputStream) |
I18N |
DM_DEFAULT_ENCODING |
50 |
High |
org.apache.maven.plugin.surefire.report.FileReporter
| Bug |
Category |
Details |
Line |
Priority |
| Found reliance on default encoding in org.apache.maven.plugin.surefire.report.FileReporter.testSetStarting(ReportEntry): new java.io.FileWriter(File) |
I18N |
DM_DEFAULT_ENCODING |
71 |
High |
| Exceptional return value of java.io.File.delete() ignored in org.apache.maven.plugin.surefire.report.FileReporter.testSetStarting(ReportEntry) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
66 |
Medium |
| Exceptional return value of java.io.File.mkdirs() ignored in org.apache.maven.plugin.surefire.report.FileReporter.testSetStarting(ReportEntry) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
61 |
Medium |
org.apache.maven.plugin.surefire.report.StatelessXmlReporter
| Bug |
Category |
Details |
Line |
Priority |
| Exceptional return value of java.io.File.mkdirs() ignored in org.apache.maven.plugin.surefire.report.StatelessXmlReporter.getOutputStream(WrappedReportEntry) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
364 |
Medium |
org.apache.maven.plugin.surefire.util.DependencyScanner$Matcher
| Bug |
Category |
Details |
Line |
Priority |
| Should org.apache.maven.plugin.surefire.util.DependencyScanner$Matcher be a _static_ inner class? |
PERFORMANCE |
SIC_INNER_SHOULD_BE_STATIC |
160-185 |
Medium |
