FindBugs Bug Detector Report
The following document contains the results of FindBugs Report
FindBugs Version is 1.3.2
Threshold is Low
Effort is Max
Summary
Classes | Bugs | Errors | Missing Classes |
---|---|---|---|
637 | 78 | 0 | 0 |
Files
com.gridsystems.config.Configurator
Bug | Category | Details | Line |
---|---|---|---|
Method com.gridsystems.config.Configurator.getView() uses the same code for two switch clauses | STYLE | DB_DUPLICATE_SWITCH_CLAUSES | 233-234 |
com.gridsystems.config.app.ConfigFrame
Bug | Category | Details | Line |
---|---|---|---|
new com.gridsystems.config.app.ConfigFrame() might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 131 |
Exception is caught when Exception is not thrown in new com.gridsystems.config.app.ConfigFrame() | STYLE | REC_CATCH_EXCEPTION | 131 |
com.gridsystems.config.app.ConfigNode
Bug | Category | Details | Line |
---|---|---|---|
com.gridsystems.config.app.ConfigNode defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 192-193 |
com.gridsystems.config.app.ConfigPanel
Bug | Category | Details | Line |
---|---|---|---|
Class com.gridsystems.config.app.ConfigPanel defines non-transient non-serializable instance field cfg | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Field only ever set to null: com.gridsystems.config.app.ConfigPanel.progressBar | CORRECTNESS | UWF_NULL_FIELD | 123 |
com.gridsystems.config.app.ConfigTree
Bug | Category | Details | Line |
---|---|---|---|
Incorrect lazy initialization and update of static field com.gridsystems.config.app.ConfigTree.root in com.gridsystems.config.app.ConfigTree.buildNodes() | MT_CORRECTNESS | LI_LAZY_INIT_UPDATE_STATIC | 69-70 |
com.gridsystems.config.app.ConsoleUI
Bug | Category | Details | Line |
---|---|---|---|
com.gridsystems.config.app.ConsoleUI.execute() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 101 |
Exception is caught when Exception is not thrown in com.gridsystems.config.app.ConsoleUI.showMenu(ConfigNode, String) | STYLE | REC_CATCH_EXCEPTION | 238 |
com.gridsystems.config.app.NullUI
Bug | Category | Details | Line |
---|---|---|---|
com.gridsystems.config.app.NullUI.execute() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 79 |
com.gridsystems.config.app.PatternFormatter
Bug | Category | Details | Line |
---|---|---|---|
com.gridsystems.config.app.PatternFormatter.format(LogRecord) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 127 |
Exception is caught when Exception is not thrown in com.gridsystems.config.app.PatternFormatter.format(LogRecord) | STYLE | REC_CATCH_EXCEPTION | 127 |
com.gridsystems.config.app.ReflectionTools
Bug | Category | Details | Line |
---|---|---|---|
com.gridsystems.config.app.ReflectionTools.close(Object) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 52 |
Exception is caught when Exception is not thrown in com.gridsystems.config.app.ReflectionTools.close(Object) | STYLE | REC_CATCH_EXCEPTION | 52 |
com.gridsystems.config.app.SwingUI$1
Bug | Category | Details | Line |
---|---|---|---|
The class com.gridsystems.config.app.SwingUI$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 47-55 |
com.gridsystems.config.app.UI
Bug | Category | Details | Line |
---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 460 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 462 |
com.gridsystems.config.app.UI.execute(int) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 363 |
com.gridsystems.config.app.UI.register(Map, Configurator) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 413 |
Incorrect lazy initialization of static field com.gridsystems.config.app.UI.configurators in com.gridsystems.config.app.UI.getConfigurators() | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 170-211 |
public static com.gridsystems.config.app.UI.getConfigurators() may expose internal representation by returning UI.configurators | MALICIOUS_CODE | MS_EXPOSE_REP | 213 |
com.gridsystems.config.app.UI.configurators should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 125 |
Exception is caught when Exception is not thrown in com.gridsystems.config.app.UI.execute(int) | STYLE | REC_CATCH_EXCEPTION | 361 |
Exception is caught when Exception is not thrown in com.gridsystems.config.app.UI.newConfigurator(String) | STYLE | REC_CATCH_EXCEPTION | 435 |
com.gridsystems.config.modules.jvm.JVMConfigModel
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in com.gridsystems.config.modules.jvm.JVMConfigModel.load() | STYLE | REC_CATCH_EXCEPTION | 311 |
com.gridsystems.config.modules.jvm.JVMConfigModel.store() ignores exceptional return value of java.io.File.mkdir() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 324 |
com.gridsystems.config.modules.jvm.JVMConsoleView
Bug | Category | Details | Line |
---|---|---|---|
Unchecked/unconfirmed cast from com.gridsystems.config.ConfiguratorModel to com.gridsystems.config.modules.jvm.JVMConfigModel in com.gridsystems.config.modules.jvm.JVMConsoleView.getValues(ConfiguratorModel) | STYLE | BC_UNCONFIRMED_CAST | 125 |
Unchecked/unconfirmed cast from com.gridsystems.config.ConfiguratorModel to com.gridsystems.config.modules.jvm.JVMConfigModel in com.gridsystems.config.modules.jvm.JVMConsoleView.setValues(ConfiguratorModel) | STYLE | BC_UNCONFIRMED_CAST | 146 |
com.gridsystems.config.modules.jvm.JVMSwingView
Bug | Category | Details | Line |
---|---|---|---|
Unchecked/unconfirmed cast from com.gridsystems.config.ConfiguratorModel to com.gridsystems.config.modules.jvm.JVMConfigModel in com.gridsystems.config.modules.jvm.JVMSwingView.getValues(ConfiguratorModel) | STYLE | BC_UNCONFIRMED_CAST | 161 |
Unchecked/unconfirmed cast from com.gridsystems.config.ConfiguratorModel to com.gridsystems.config.modules.jvm.JVMConfigModel in com.gridsystems.config.modules.jvm.JVMSwingView.setValues(ConfiguratorModel) | STYLE | BC_UNCONFIRMED_CAST | 193 |
com.gridsystems.config.modules.tomcat.Connector
Bug | Category | Details | Line |
---|---|---|---|
com.gridsystems.config.modules.tomcat.Connector.loadFromFile(File) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 751 |
com.gridsystems.config.modules.tomcat.Connector.saveToFile(File, List) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 791 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 861 |
Comparison of String parameter using == or != in com.gridsystems.config.modules.tomcat.Connector.minCompression(String, String) | BAD_PRACTICE | ES_COMPARING_PARAMETER_STRING_WITH_EQ | 680 |
Exception is caught when Exception is not thrown in com.gridsystems.config.modules.tomcat.Connector.toString() | STYLE | REC_CATCH_EXCEPTION | 529 |
com.gridsystems.config.modules.tomcat.ConnectorEditor
Bug | Category | Details | Line |
---|---|---|---|
Class com.gridsystems.config.modules.tomcat.ConnectorEditor defines non-transient non-serializable instance field changeCallback | BAD_PRACTICE | SE_BAD_FIELD | Not available |
com.gridsystems.config.modules.tomcat.ConnectorEditor$ConnectorCellRenderer$1
Bug | Category | Details | Line |
---|---|---|---|
The class com.gridsystems.config.modules.tomcat.ConnectorEditor$ConnectorCellRenderer$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | 505-515 |
com.gridsystems.config.modules.tomcat.ConnectorEditor$ConnectorModel
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in com.gridsystems.config.modules.tomcat.ConnectorEditor$ConnectorModel.setValueAt(Object, int, int) | STYLE | REC_CATCH_EXCEPTION | 386 |
com.gridsystems.config.modules.tomcat.ConnectorEditor$ConnectorSelectionListener
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in com.gridsystems.config.modules.tomcat.ConnectorEditor$ConnectorSelectionListener.valueChanged(ListSelectionEvent) | STYLE | REC_CATCH_EXCEPTION | 688 |
com.gridsystems.config.modules.tomcat.ConnectorField
Bug | Category | Details | Line |
---|---|---|---|
Unchecked/unconfirmed cast from com.gridsystems.config.ConsoleConfiguratorView to com.gridsystems.config.modules.tomcat.TomcatConsoleView in com.gridsystems.config.modules.tomcat.ConnectorField.execute(ConsoleConfiguratorView) | STYLE | BC_UNCONFIRMED_CAST | 128 |
com.gridsystems.config.modules.tomcat.ConnectorField.execute(ConsoleConfiguratorView) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 155 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 134 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 147 |
com.gridsystems.config.modules.tomcat.ConnectorGroupField
Bug | Category | Details | Line |
---|---|---|---|
Unchecked/unconfirmed cast from com.gridsystems.config.ConsoleConfiguratorView to com.gridsystems.config.modules.tomcat.TomcatConsoleView in com.gridsystems.config.modules.tomcat.ConnectorGroupField.execute(ConsoleConfiguratorView) | STYLE | BC_UNCONFIRMED_CAST | 97 |
com.gridsystems.config.modules.tomcat.IncompatiblePortException
Bug | Category | Details | Line |
---|---|---|---|
Class com.gridsystems.config.modules.tomcat.IncompatiblePortException defines non-transient non-serializable instance field c | BAD_PRACTICE | SE_BAD_FIELD | Not available |
com.gridsystems.config.modules.tomcat.KeystoreUtils
Bug | Category | Details | Line |
---|---|---|---|
Possible null pointer dereference of exitCode in com.gridsystems.config.modules.tomcat.KeystoreUtils.execCommand(String[]) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 274 |
com.gridsystems.config.modules.tomcat.KeystoreUtils.createKeystores(File, Properties) ignores exceptional return value of java.io.File.mkdirs() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 66 |
com.gridsystems.config.modules.tomcat.PortInUseException
Bug | Category | Details | Line |
---|---|---|---|
Class com.gridsystems.config.modules.tomcat.PortInUseException defines non-transient non-serializable instance field c | BAD_PRACTICE | SE_BAD_FIELD | Not available |
com.gridsystems.config.modules.tomcat.TomcatConfigModel
Bug | Category | Details | Line |
---|---|---|---|
Usage of GetResource in com.gridsystems.config.modules.tomcat.TomcatConfigModel.applyTemplate(Properties, String, String, String) may be unsafe if class is extended | BAD_PRACTICE | UI_INHERITANCE_UNSAFE_GETRESOURCE | 386 |
com.gridsystems.config.modules.tomcat.TomcatConsoleView
Bug | Category | Details | Line |
---|---|---|---|
Unchecked/unconfirmed cast from com.gridsystems.config.ConfiguratorModel to com.gridsystems.config.modules.tomcat.TomcatConfigModel in com.gridsystems.config.modules.tomcat.TomcatConsoleView.getValues(ConfiguratorModel) | STYLE | BC_UNCONFIRMED_CAST | 158 |
Unchecked/unconfirmed cast from com.gridsystems.config.ConfiguratorModel to com.gridsystems.config.modules.tomcat.TomcatConfigModel in com.gridsystems.config.modules.tomcat.TomcatConsoleView.setValues(ConfiguratorModel) | STYLE | BC_UNCONFIRMED_CAST | 180 |
com.gridsystems.config.modules.tomcat.TomcatConsoleView.setValues(ConfiguratorModel) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 190 |
Method com.gridsystems.config.modules.tomcat.TomcatConsoleView.getValues(ConfiguratorModel) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 170 |
com.gridsystems.config.modules.tomcat.TomcatSwingView
Bug | Category | Details | Line |
---|---|---|---|
Unchecked/unconfirmed cast from com.gridsystems.config.ConfiguratorModel to com.gridsystems.config.modules.tomcat.TomcatConfigModel in com.gridsystems.config.modules.tomcat.TomcatSwingView.getValues(ConfiguratorModel) | STYLE | BC_UNCONFIRMED_CAST | 236 |
Unchecked/unconfirmed cast from com.gridsystems.config.ConfiguratorModel to com.gridsystems.config.modules.tomcat.TomcatConfigModel in com.gridsystems.config.modules.tomcat.TomcatSwingView.setValues(ConfiguratorModel) | STYLE | BC_UNCONFIRMED_CAST | 254 |
com.gridsystems.config.tools.IpVerifier
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in com.gridsystems.config.tools.IpVerifier.verify(String) | STYLE | REC_CATCH_EXCEPTION | 215 |
com.gridsystems.config.tools.NumberVerifier
Bug | Category | Details | Line |
---|---|---|---|
Possible null pointer dereference of text in com.gridsystems.config.tools.NumberVerifier.verify(String) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 119 |
com.gridsystems.config.tools.Template
Bug | Category | Details | Line |
---|---|---|---|
com.gridsystems.config.tools.Template.close(Object) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 218 |
Exception is caught when Exception is not thrown in com.gridsystems.config.tools.Template.close(Object) | STYLE | REC_CATCH_EXCEPTION | 218 |
com.gridsystems.config.tools.console.AbstractConsoleView
Bug | Category | Details | Line |
---|---|---|---|
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 198 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 139 |
Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 228 |
com.gridsystems.config.tools.console.ConsoleTools
Bug | Category | Details | Line |
---|---|---|---|
com.gridsystems.config.tools.console.ConsoleTools.g0(StringBuffer) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 370 |
com.gridsystems.config.tools.console.ConsoleTools.g1(StringBuffer) might ignore java.lang.Exception | BAD_PRACTICE | DE_MIGHT_IGNORE | 397 |
Incorrect lazy initialization of static field com.gridsystems.config.tools.console.ConsoleTools.reader in com.gridsystems.config.tools.console.ConsoleTools.readLine(String) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 196-197 |
Incorrect lazy initialization of static field com.gridsystems.config.tools.console.ConsoleTools.acsmap in com.gridsystems.config.tools.console.ConsoleTools.setup(boolean, boolean) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 160-168 |
Exception is caught when Exception is not thrown in com.gridsystems.config.tools.console.ConsoleTools.g0(StringBuffer) | STYLE | REC_CATCH_EXCEPTION | 370 |
Exception is caught when Exception is not thrown in com.gridsystems.config.tools.console.ConsoleTools.g1(StringBuffer) | STYLE | REC_CATCH_EXCEPTION | 397 |
com.gridsystems.config.tools.console.SelectionField
Bug | Category | Details | Line |
---|---|---|---|
new com.gridsystems.config.tools.console.SelectionField(ResourceBundle, String, String, String[]) may expose internal representation by storing an externally mutable object into SelectionField.values | MALICIOUS_CODE | EI_EXPOSE_REP2 | 77 |
com.gridsystems.config.tools.swing.BasePanel
Bug | Category | Details | Line |
---|---|---|---|
Class com.gridsystems.config.tools.swing.BasePanel defines non-transient non-serializable instance field bundle | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Class com.gridsystems.config.tools.swing.BasePanel defines non-transient non-serializable instance field config | BAD_PRACTICE | SE_BAD_FIELD | Not available |
com.gridsystems.config.tools.swing.PasswordField
Bug | Category | Details | Line |
---|---|---|---|
Method com.gridsystems.config.tools.swing.PasswordField.setValue(String) invokes toString() method on a String | PERFORMANCE | DM_STRING_TOSTRING | 167 |
Null pointer dereference of resourceBundle in new com.gridsystems.config.tools.swing.PasswordField(ResourceBundle, String) | CORRECTNESS | NP_ALWAYS_NULL | 99 |
Load of known null value in new com.gridsystems.config.tools.swing.PasswordField(ResourceBundle, String) | STYLE | NP_LOAD_OF_KNOWN_NULL_VALUE | 99 |
Class com.gridsystems.config.tools.swing.PasswordField defines non-transient non-serializable instance field resourceBundle | BAD_PRACTICE | SE_BAD_FIELD | Not available |
com.gridsystems.config.tools.swing.ReflectedAction
Bug | Category | Details | Line |
---|---|---|---|
Exception is caught when Exception is not thrown in com.gridsystems.config.tools.swing.ReflectedAction.getMethod(Class, String, Class[]) | STYLE | REC_CATCH_EXCEPTION | 137 |
Class com.gridsystems.config.tools.swing.ReflectedAction defines non-transient non-serializable instance field method | BAD_PRACTICE | SE_BAD_FIELD | Not available |
Unwritten field: com.gridsystems.config.tools.swing.ReflectedAction.sendEvent | CORRECTNESS | UWF_UNWRITTEN_FIELD | 147 |