The following document contains the results of FindBugs Report
FindBugs Version is 2.0.1
Threshold is medium
Effort is min
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Load of known null value in org.deegree.commons.config.AbstractResourceManager.processResourceConfig(File) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 286 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.config.DeegreeWorkspace.getModulesInfo() is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 131 | High |
org.deegree.commons.config.DeegreeWorkspace.initManagers() makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 218 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.dataaccess.CSVReader.parseLine() may expose internal representation by returning CSVReader.columnNames | MALICIOUS_CODE | EI_EXPOSE_REP | 164 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.deegree.commons.index.QTree.bboxForSon(int) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 587 | Medium |
Switch statement found in org.deegree.commons.index.QTree.getDotVertex(int, int, boolean) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 870 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.index.QTree$Entry is serializable and an inner class | BAD_PRACTICE | SE_INNER_CLASS | 762 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.jdbc.UpdateRow.performUpdate(Connection) may fail to clean up java.sql.Statement on checked exception | EXPERIMENTAL | OBL_UNSATISFIED_OBLIGATION_EXCEPTION_EDGE | 112 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.jdbc.param.JDBCParamsManager.shutdown() might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 181 | Medium |
Exception is caught when Exception is not thrown in org.deegree.commons.jdbc.param.JDBCParamsManager.shutdown() | STYLE | REC_CATCH_EXCEPTION | 169 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Exception is caught when Exception is not thrown in org.deegree.commons.mail.MailHelper.createAndSendMail(MailMessage, Session, File[], String[]) | STYLE | REC_CATCH_EXCEPTION | 276 | Medium |
Exception is caught when Exception is not thrown in org.deegree.commons.mail.MailHelper.createAndSendMail(MailMessage, Session, Object[], String[]) | STYLE | REC_CATCH_EXCEPTION | 214 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.modules.ModuleInfo.extractModulesInfo(Set) is or uses a map or set of URLs, which can be a performance hog | PERFORMANCE | DMI_COLLECTION_OF_URLS | 121 | High |
org.deegree.commons.modules.ModuleInfo.extractModuleInfo(URL) creates a java.net.URLClassLoader classloader, which should be performed within a doPrivileged block | MALICIOUS_CODE | DP_CREATE_CLASSLOADER_INSIDE_DO_PRIVILEGED | 160 | Medium |
org.deegree.commons.modules.ModuleInfo defines equals and uses Object.hashCode() | BAD_PRACTICE | HE_EQUALS_USE_HASHCODE | 206 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unused field: org.deegree.commons.ows.metadata.Description.langToKeywords | PERFORMANCE | UUF_UNUSED_FIELD | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unwritten public or protected field: org.deegree.commons.struct.Tree.value | STYLE | UWF_UNWRITTEN_PUBLIC_OR_PROTECTED_FIELD | 57 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
The class name org.deegree.commons.tom.Object shadows the simple name of the superclass java.lang.Object | BAD_PRACTICE | NM_SAME_SIMPLE_NAME_AS_SUPERCLASS | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Inconsistent synchronization of org.deegree.commons.tom.Reference.object; locked 40% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 121 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.tom.array.TypedObjectNodeArray.getElements() may expose internal representation by returning TypedObjectNodeArray.elements | MALICIOUS_CODE | EI_EXPOSE_REP | 68 | Medium |
new org.deegree.commons.tom.array.TypedObjectNodeArray(TypedObjectNode[]) may expose internal representation by storing an externally mutable object into TypedObjectNodeArray.elements | MALICIOUS_CODE | EI_EXPOSE_REP2 | 61 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.tom.datetime.Date inherits equals and uses Object.hashCode() | BAD_PRACTICE | HE_INHERITS_EQUALS_USE_HASHCODE | 52 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.tom.datetime.DateTime inherits equals and uses Object.hashCode() | BAD_PRACTICE | HE_INHERITS_EQUALS_USE_HASHCODE | 52 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.tom.datetime.Time inherits equals and uses Object.hashCode() | BAD_PRACTICE | HE_INHERITS_EQUALS_USE_HASHCODE | 51 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.tom.datetime.TimeInstant defines equals and uses Object.hashCode() | BAD_PRACTICE | HE_EQUALS_USE_HASHCODE | 167 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.tom.gml.GenericGMLObjectType.toString() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 87 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.tom.ows.StringOrRef.equals(Object) checks for operand being a String | BAD_PRACTICE | EQ_CHECK_FOR_OPERAND_NOT_COMPATIBLE_WITH_THIS | 95 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
instanceof will always return false in org.deegree.commons.tom.primitive.PrimitiveValue.makeComparable(Object, Object), since a String can't be a Boolean | CORRECTNESS | BC_IMPOSSIBLE_INSTANCEOF | 250 | Medium |
instanceof will always return false in org.deegree.commons.tom.primitive.PrimitiveValue.makeComparable(Object, Object), since a String can't be a org.deegree.commons.tom.ows.CodeType | CORRECTNESS | BC_IMPOSSIBLE_INSTANCEOF | 258 | Medium |
instanceof will always return false in org.deegree.commons.tom.primitive.PrimitiveValue.makeComparable(Object, Object), since a String can't be a org.deegree.commons.uom.Measure | CORRECTNESS | BC_IMPOSSIBLE_INSTANCEOF | 261 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.tools.CommandUtils.HELP_TEXT_WIDTH isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 59 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.tools.ToolBox$ToolInfo.toString() may return null | BAD_PRACTICE | NP_TOSTRING_COULD_RETURN_NULL | 435 | Medium |
Should org.deegree.commons.tools.ToolBox$ToolInfo be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 380 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
exceptional return value of java.io.File.mkdirs() ignored in org.deegree.commons.utils.ConfigManager.<static initializer>() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 64 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.deegree.commons.utils.DeegreeAALogoUtils.print(OutputStream): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 117 | High |
org.deegree.commons.utils.DeegreeAALogoUtils.<static initializer>() may fail to close stream | BAD_PRACTICE | OS_OPEN_STREAM | 69 | Medium |
Exception is caught when Exception is not thrown in org.deegree.commons.utils.DeegreeAALogoUtils.<static initializer>() | STYLE | REC_CATCH_EXCEPTION | 74 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.utils.EncodingGuesser.guess(Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 146 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.deegree.commons.utils.FileUtils.writeFile(File, String): new java.io.FileWriter(File) | I18N | DM_DEFAULT_ENCODING | 178 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.utils.JDBCUtils.determinePostGISVersion(Connection, Logger) may fail to clean up java.sql.ResultSet | EXPERIMENTAL | OBL_UNSATISFIED_OBLIGATION | 166 | Medium |
org.deegree.commons.utils.JDBCUtils.determinePostGISVersion(Connection, Logger) may fail to clean up java.sql.Statement | EXPERIMENTAL | OBL_UNSATISFIED_OBLIGATION | 165 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.deegree.commons.utils.ProxyUtils.openURLConnection(URL, String, String): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 338 | High |
Found reliance on default encoding in org.deegree.commons.utils.ProxyUtils.openURLConnection(URL, String, String, String, String): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 310 | High |
Found reliance on default encoding in org.deegree.commons.utils.ProxyUtils.openURLConnection(URL, String, String, String, String): String.getBytes() | I18N | DM_DEFAULT_ENCODING | 315 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.utils.RequestUtils.replaceParameters(Map, Map, Map, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 78 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
exceptional return value of java.io.File.mkdir() ignored in org.deegree.commons.utils.TempFileManager.init(String) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 86 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
exceptional return value of java.io.File.delete() ignored in org.deegree.commons.utils.io.StreamBufferStore.reset() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 209 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.utils.io.Zip.unzip(InputStream, File, boolean) may fail to clean up java.io.OutputStream on checked exception | EXPERIMENTAL | OBL_UNSATISFIED_OBLIGATION_EXCEPTION_EDGE | 101 | Medium |
exceptional return value of java.io.File.mkdir() ignored in org.deegree.commons.utils.io.Zip.unzip(InputStream, File, boolean) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 72 | Medium |
exceptional return value of java.io.File.mkdirs() ignored in org.deegree.commons.utils.io.Zip.unzip(InputStream, File, boolean) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 100 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.deegree.commons.utils.jaxb.CreateExternalBindings.matchSCDType(String, String): new java.io.FileReader(String) | I18N | DM_DEFAULT_ENCODING | 232 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.deegree.commons.utils.kvp.KVPUtils.readFileIntoMap(URL): new java.io.InputStreamReader(InputStream) | I18N | DM_DEFAULT_ENCODING | 353 | High |
org.deegree.commons.utils.kvp.KVPUtils.readFileIntoMap(URL) may fail to close stream | BAD_PRACTICE | OS_OPEN_STREAM | 353 | Medium |
org.deegree.commons.utils.kvp.KVPUtils.getNormalizedKVPMap(String, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 422 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.utils.net.HttpUtils.get(HttpUtils$Worker, String, Map, String, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 473 | Medium |
org.deegree.commons.utils.net.HttpUtils.getFullResponse(HttpUtils$Worker, String, Map, String, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 509 | Medium |
org.deegree.commons.utils.net.HttpUtils.getFullResponse(HttpUtils$Worker, String, Map, String, String, int) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 593 | Medium |
org.deegree.commons.utils.net.HttpUtils.post(HttpUtils$Worker, String, File, Map, String, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 420 | Medium |
org.deegree.commons.utils.net.HttpUtils.post(HttpUtils$Worker, String, InputStream, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 288 | Medium |
org.deegree.commons.utils.net.HttpUtils.post(HttpUtils$Worker, String, Map, Map, int) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 329 | Medium |
org.deegree.commons.utils.net.HttpUtils.postFullResponse(HttpUtils$Worker, String, Map, Map, int) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 372 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.deegree.commons.utils.test.TestProperties.<static initializer>(): new java.io.FileReader(File) | I18N | DM_DEFAULT_ENCODING | 67 | High |
org.deegree.commons.utils.test.TestProperties.<static initializer>() may fail to clean up java.io.Reader | EXPERIMENTAL | OBL_UNSATISFIED_OBLIGATION | 67 | Medium |
org.deegree.commons.utils.test.TestProperties.<static initializer>() may fail to close stream | BAD_PRACTICE | OS_OPEN_STREAM | 67 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.deegree.commons.xml.schema.ErrorHandler.handleError(DOMError) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 580 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.deegree.commons.xml.schema.GrammarPoolManager.createGrammarPool(String[]) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 130 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.deegree.commons.xml.stax.StAXExportingHelper.writeStartElement(XMLStreamWriter, QName) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 86 | Medium |