SpotBugs Bug Detector Report
The following document contains the results of SpotBugs
SpotBugs Version is 4.9.8
Threshold is medium
Effort is default
Summary
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 351 | 67 | 0 | 0 |
Files
com.jsql.view.swing.JFrameView
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class com.jsql.view.swing.JFrameView at new com.jsql.view.swing.JFrameView(InjectionModel) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 70 | Medium |
com.jsql.view.swing.JFrameView$1
| Bug | Category | Details | Line | Priority |
|---|
com.jsql.view.swing.action.ActionSaveTab
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in new com.jsql.view.swing.action.ActionSaveTab() | STYLE | DCN_NULLPOINTER_EXCEPTION | 49 | Medium |
com.jsql.view.swing.console.JTextPaneAppender
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| com.jsql.view.swing.console.JTextPaneAppender.register(SimpleConsoleAdapter) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.console.JTextPaneAppender.consoleTextPane | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 97 | Medium |
| com.jsql.view.swing.console.JTextPaneAppender.registerJavaConsole(SimpleConsoleAdapter) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.console.JTextPaneAppender.javaTextPane | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 90 | Medium |
com.jsql.view.swing.dialog.DialogAbout
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class com.jsql.view.swing.dialog.DialogAbout at new com.jsql.view.swing.dialog.DialogAbout() will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 74 | Medium |
com.jsql.view.swing.interaction.MessageHeader
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.interaction.MessageHeader.execute() | STYLE | DCN_NULLPOINTER_EXCEPTION | 76 | Medium |
com.jsql.view.swing.list.AbstractListTransfertHandler
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.list.AbstractListTransfertHandler.parseStringPaste(DnDList, DefaultListModel, Transferable) | STYLE | DCN_NULLPOINTER_EXCEPTION | 147 | Medium |
com.jsql.view.swing.list.DnDList
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.list.DnDList.addItems(List, int) | STYLE | DCN_NULLPOINTER_EXCEPTION | 239 | Medium |
| Do not catch NullPointerException like in com.jsql.view.swing.list.DnDList.removeSelectedItem() | STYLE | DCN_NULLPOINTER_EXCEPTION | 159 | Medium |
com.jsql.view.swing.list.MenuActionNewValue
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.list.MenuActionNewValue.actionPerformed(ActionEvent) | STYLE | DCN_NULLPOINTER_EXCEPTION | 89 | Medium |
com.jsql.view.swing.list.MouseAdapterMenuAction
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.list.MouseAdapterMenuAction.lambda$initMenu$1(JFileChooser, MouseEvent, ActionEvent) | STYLE | DCN_NULLPOINTER_EXCEPTION | 133 | Medium |
com.jsql.view.swing.manager.ManagerBruteForce
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.manager.ManagerBruteForce.lambda$initThirdLine$2(ModelSpinner) | STYLE | DCN_NULLPOINTER_EXCEPTION | 210 | Medium |
com.jsql.view.swing.manager.util.ActionBruteForce
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Shared primitive variable "isStopped" in one thread may not yield the value of the most recent write from another thread | MT_CORRECTNESS | AT_STALE_THREAD_WRITE_OF_PRIMITIVE | 48 | Medium |
| Shared primitive variable "isStopped" in one thread may not yield the value of the most recent write from another thread | MT_CORRECTNESS | AT_STALE_THREAD_WRITE_OF_PRIMITIVE | 117 | Medium |
com.jsql.view.swing.menubar.AppMenubar
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.menubar.AppMenubar.switchLocale(Locale, Locale, boolean) | STYLE | DCN_NULLPOINTER_EXCEPTION | 344 | Medium |
com.jsql.view.swing.panel.PanelConsoles
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Shared primitive variable "dividerLocation" in one thread may not yield the value of the most recent write from another thread | MT_CORRECTNESS | AT_STALE_THREAD_WRITE_OF_PRIMITIVE | 332 | Medium |
| Do not catch NullPointerException like in com.jsql.view.swing.panel.PanelConsoles.messageBinary(String) | STYLE | DCN_NULLPOINTER_EXCEPTION | 319 | Medium |
| Do not catch NullPointerException like in com.jsql.view.swing.panel.PanelConsoles.messageChunk(String) | STYLE | DCN_NULLPOINTER_EXCEPTION | 308 | Medium |
| Do not catch NullPointerException like in com.jsql.view.swing.panel.PanelConsoles.reset() | STYLE | DCN_NULLPOINTER_EXCEPTION | 210 | Medium |
| Primitive field com.jsql.view.swing.panel.PanelConsoles.networkSplitPane is public and set from inside the class, which makes it too exposed. Consider making it private to limit external accessibility. | BAD_PRACTICE | PA_PUBLIC_PRIMITIVE_ATTRIBUTE | 132 | Medium |
com.jsql.view.swing.panel.consoles.TabbedPaneNetworkTab
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.panel.consoles.TabbedPaneNetworkTab.changeTextNetwork(HttpHeader) | STYLE | DCN_NULLPOINTER_EXCEPTION | 129 | Medium |
| Do not catch NullPointerException like in com.jsql.view.swing.panel.consoles.TabbedPaneNetworkTab.reset() | STYLE | DCN_NULLPOINTER_EXCEPTION | 193 | Medium |
com.jsql.view.swing.panel.preferences.PanelExploit$1
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.panel.preferences.PanelExploit$1.process() | STYLE | DCN_NULLPOINTER_EXCEPTION | 34 | Medium |
com.jsql.view.swing.popupmenu.JPopupMenuComponent$PopupMenuOrientedListener
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.popupmenu.JPopupMenuComponent$PopupMenuOrientedListener.popupMenuWillBecomeVisible(PopupMenuEvent) | STYLE | DCN_NULLPOINTER_EXCEPTION | 101 | Medium |
com.jsql.view.swing.sql.SqlEngine
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field com.jsql.view.swing.sql.SqlEngine.modelYaml from instance method new com.jsql.view.swing.sql.SqlEngine() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 254 | Medium |
com.jsql.view.swing.tab.TabbedPaneMouseWheelListener
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.tab.TabbedPaneMouseWheelListener.mouseWheelMoved(MouseWheelEvent) | STYLE | DCN_NULLPOINTER_EXCEPTION | 38 | Medium |
com.jsql.view.swing.table.ComparatorColumn
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| com.jsql.view.swing.table.ComparatorColumn implements Comparator but not Serializable | BAD_PRACTICE | SE_COMPARATOR_SHOULD_BE_SERIALIZABLE | 19-54 | Medium |
com.jsql.view.swing.table.PanelTable
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.table.PanelTable.lambda$initRenderer$1(DefaultTableCellRenderer, JTable, Object, boolean, boolean, int, int) | STYLE | DCN_NULLPOINTER_EXCEPTION | 137 | Medium |
com.jsql.view.swing.terminal.AbstractExploit
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class com.jsql.view.swing.terminal.AbstractExploit at new com.jsql.view.swing.terminal.AbstractExploit(UUID, String, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 81 | Medium |
| Exception thrown in class com.jsql.view.swing.terminal.AbstractExploit at new com.jsql.view.swing.terminal.AbstractExploit(UUID, String, String, boolean) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 90 | Medium |
com.jsql.view.swing.terminal.ExploitReverseShell
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class com.jsql.view.swing.terminal.ExploitReverseShell at new com.jsql.view.swing.terminal.ExploitReverseShell(UUID, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 37 | Medium |
com.jsql.view.swing.terminal.ServerInputConnection
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class com.jsql.view.swing.terminal.ServerInputConnection at new com.jsql.view.swing.terminal.ServerInputConnection(ExploitReverseShell, Socket, ServerInput) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 31 | Medium |
| Found reliance on default encoding in new com.jsql.view.swing.terminal.ServerInputConnection(ExploitReverseShell, Socket, ServerInput): new java.io.InputStreamReader(InputStream) | I18N | DM_DEFAULT_ENCODING | 31 | High |
com.jsql.view.swing.terminal.util.KeyAdapterTerminal
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Operation on the "indexCommandsHistory" shared variable in "KeyAdapterTerminal" class is not atomic | MT_CORRECTNESS | AT_NONATOMIC_OPERATIONS_ON_SHARED_VARIABLE | 141 | Medium |
| Operation on the "indexCommandsHistory" shared variable in "KeyAdapterTerminal" class is not atomic | MT_CORRECTNESS | AT_NONATOMIC_OPERATIONS_ON_SHARED_VARIABLE | 164 | Medium |
| Shared primitive variable "indexCommandsHistory" in one thread may not yield the value of the most recent write from another thread | MT_CORRECTNESS | AT_STALE_THREAD_WRITE_OF_PRIMITIVE | 194 | Medium |
com.jsql.view.swing.text.JTextAreaPlaceholder
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.text.JTextAreaPlaceholder.paint(Graphics) | STYLE | DCN_NULLPOINTER_EXCEPTION | 44 | Medium |
com.jsql.view.swing.text.SyntaxTextArea
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.text.SyntaxTextArea.paint(Graphics) | STYLE | DCN_NULLPOINTER_EXCEPTION | 56 | Medium |
com.jsql.view.swing.tree.model.AbstractNodeModel
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Do not catch NullPointerException like in com.jsql.view.swing.tree.model.AbstractNodeModel.initLabel(boolean, boolean, boolean) | STYLE | DCN_NULLPOINTER_EXCEPTION | 297 | Medium |
com.jsql.view.swing.util.MediatorHelper
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| com.jsql.view.swing.util.MediatorHelper.register(InjectionModel) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.model | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 154 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(JFrameView) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.frame | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 101 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(ManagerAdminPage) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.managerAdminPage | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 135 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(ManagerBruteForce) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.managerBruteForce | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 150 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(ManagerExploit) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.managerExploit | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 142 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(ManagerFile) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.managerFile | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 139 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(ManagerScan) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.managerScan | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 146 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(AppMenubar) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.appMenubar | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 105 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(PanelAddressBar) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.panelAddressBar | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 109 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(PanelConsoles) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.panelConsoles | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 117 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(TabManagersCards) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.tabManagersCards | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 129 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(TabResults) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.tabResults | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 125 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(TabbedPaneWheeled) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.tabConsoles | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 113 | Medium |
| com.jsql.view.swing.util.MediatorHelper.register(TreeDatabase) may expose internal static state by storing a mutable object into a static field com.jsql.view.swing.util.MediatorHelper.treeDatabase | MALICIOUS_CODE | EI_EXPOSE_STATIC_REP2 | 121 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.frame() may expose internal representation by returning MediatorHelper.frame | MALICIOUS_CODE | MS_EXPOSE_REP | 58 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.managerAdminPage() may expose internal representation by returning MediatorHelper.managerAdminPage | MALICIOUS_CODE | MS_EXPOSE_REP | 78 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.managerBruteForce() may expose internal representation by returning MediatorHelper.managerBruteForce | MALICIOUS_CODE | MS_EXPOSE_REP | 94 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.managerExploit() may expose internal representation by returning MediatorHelper.managerExploit | MALICIOUS_CODE | MS_EXPOSE_REP | 86 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.managerFile() may expose internal representation by returning MediatorHelper.managerFile | MALICIOUS_CODE | MS_EXPOSE_REP | 82 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.managerScan() may expose internal representation by returning MediatorHelper.managerScan | MALICIOUS_CODE | MS_EXPOSE_REP | 90 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.menubar() may expose internal representation by returning MediatorHelper.appMenubar | MALICIOUS_CODE | MS_EXPOSE_REP | 62 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.model() may expose internal representation by returning MediatorHelper.model | MALICIOUS_CODE | MS_EXPOSE_REP | 42 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.panelAddressBar() may expose internal representation by returning MediatorHelper.panelAddressBar | MALICIOUS_CODE | MS_EXPOSE_REP | 66 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.panelConsoles() may expose internal representation by returning MediatorHelper.panelConsoles | MALICIOUS_CODE | MS_EXPOSE_REP | 74 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.tabConsoles() may expose internal representation by returning MediatorHelper.tabConsoles | MALICIOUS_CODE | MS_EXPOSE_REP | 70 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.tabManagersCards() may expose internal representation by returning MediatorHelper.tabManagersCards | MALICIOUS_CODE | MS_EXPOSE_REP | 54 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.tabResults() may expose internal representation by returning MediatorHelper.tabResults | MALICIOUS_CODE | MS_EXPOSE_REP | 50 | Medium |
| Public static com.jsql.view.swing.util.MediatorHelper.treeDatabase() may expose internal representation by returning MediatorHelper.treeDatabase | MALICIOUS_CODE | MS_EXPOSE_REP | 46 | Medium |