The following document contains the results of FindBugs
FindBugs Version is 3.0.1
Threshold is medium
Effort is min
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.Activator.mc isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 49 | Medium |
org.universAAL.tools.logmonitor.Activator.bc should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Medium | |
org.universAAL.tools.logmonitor.Activator.contentSerializer should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 33 | Medium |
org.universAAL.tools.logmonitor.Activator.lm should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Medium | |
Possible null pointer dereference of b in org.universAAL.tools.logmonitor.Activator.getResource(String) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 76 | Medium |
Possible null pointer dereference in org.universAAL.tools.logmonitor.Activator.getResource(String) due to return value of called method | STYLE | NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE | 77 | Medium |
Write to static field org.universAAL.tools.logmonitor.Activator.lm from instance method org.universAAL.tools.logmonitor.Activator.start() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 50 | High |
Write to static field org.universAAL.tools.logmonitor.Activator.bc from instance method org.universAAL.tools.logmonitor.Activator.start(BundleContext) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 41 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Write to static field org.universAAL.tools.logmonitor.BusMemberMonitor.instance from instance method new org.universAAL.tools.logmonitor.BusMemberMonitor() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 26 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.LogMonitor.busMemberMonitor should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 21 | Medium |
Write to static field org.universAAL.tools.logmonitor.LogMonitor.busMemberMonitor from instance method new org.universAAL.tools.logmonitor.LogMonitor() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 41 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.MainGui.tabbedPane isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Medium | |
Write to static field org.universAAL.tools.logmonitor.MainGui.tabbedPane from instance method new org.universAAL.tools.logmonitor.MainGui() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 30 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unread public/protected field: org.universAAL.tools.logmonitor.MemberData.origin | STYLE | URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD | 60 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.Utils.getDateString(Date) invokes inefficient new String() constructor | PERFORMANCE | DM_STRING_VOID_CTOR | 27 | Medium |
Switch statement found in org.universAAL.tools.logmonitor.Utils.getLevel(int) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 48 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unread public/protected field: org.universAAL.tools.logmonitor.all_log.LogEntry.t | STYLE | URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD | 31 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unread public/protected field: org.universAAL.tools.logmonitor.msgflow.MemberDataEx.idxEnd | STYLE | URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD | 31 | Medium |
Unread public/protected field: org.universAAL.tools.logmonitor.msgflow.MemberDataEx.idxStart | STYLE | URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD | 30 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.msgflow.gui.MsgFlowPanel.instance isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Medium | |
org.universAAL.tools.logmonitor.msgflow.gui.MsgFlowPanel.busColor should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 96 | Medium |
org.universAAL.tools.logmonitor.msgflow.gui.MsgFlowPanel.fmNormal should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Medium | |
org.universAAL.tools.logmonitor.msgflow.gui.MsgFlowPanel.fontBold isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 76 | High |
org.universAAL.tools.logmonitor.msgflow.gui.MsgFlowPanel.fontNormal isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 71 | High |
java.awt.BasicStroke stored into non-transient field MsgFlowPanel.dashed | BAD_PRACTICE | SE_BAD_FIELD_STORE | 90 | Medium |
java.awt.BasicStroke stored into non-transient field MsgFlowPanel.thick | BAD_PRACTICE | SE_BAD_FIELD_STORE | 88 | Medium |
java.awt.BasicStroke stored into non-transient field MsgFlowPanel.thin | BAD_PRACTICE | SE_BAD_FIELD_STORE | 89 | Medium |
Unread field: org.universAAL.tools.logmonitor.msgflow.gui.MsgFlowPanel.DST; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 130 | Medium |
Unread field: org.universAAL.tools.logmonitor.msgflow.gui.MsgFlowPanel.SRC; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 129 | Medium |
Write to static field org.universAAL.tools.logmonitor.msgflow.gui.MsgFlowPanel.instance from instance method new org.universAAL.tools.logmonitor.msgflow.gui.MsgFlowPanel(JScrollBar, JScrollBar) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 219 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Unread public/protected field: org.universAAL.tools.logmonitor.msgflow.gui.MyScrollBarUI.defaultUI | STYLE | URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD | 37 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.ontology.gui.OntologyPane.createOverviewHTML(StringBuilder) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 106 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.rdfvis.Edge defines compareTo(Object) and uses Object.equals() | BAD_PRACTICE | EQ_COMPARETO_USE_OBJECT_EQUALS | 230 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.rdfvis.LogMonitor.log(int, String, String, String, String, Object[], Throwable) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 39 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.rdfvis.gui.GraphPanel.fmBold isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Medium | |
org.universAAL.tools.logmonitor.rdfvis.gui.GraphPanel.fmNormal isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Medium | |
org.universAAL.tools.logmonitor.rdfvis.gui.GraphPanel.selectedNode isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Medium | |
org.universAAL.tools.logmonitor.rdfvis.gui.GraphPanel.fontBold isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 48 | High |
org.universAAL.tools.logmonitor.rdfvis.gui.GraphPanel.fontNormal isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 43 | High |
org.universAAL.tools.logmonitor.rdfvis.gui.GraphPanel.nodeBorder isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 38 | Medium |
Write to static field org.universAAL.tools.logmonitor.rdfvis.gui.GraphPanel.fmBold from instance method org.universAAL.tools.logmonitor.rdfvis.gui.GraphPanel.paintComponent(Graphics) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 237 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Found reliance on default encoding in org.universAAL.tools.logmonitor.rdfvis.gui.RDFVis.saveAll(): new java.io.FileWriter(File, boolean) | I18N | DM_DEFAULT_ENCODING | 183 | High |
org.universAAL.tools.logmonitor.rdfvis.gui.RDFVis.saveAll() may fail to clean up java.io.Writer on checked exception | EXPERIMENTAL | OBL_UNSATISFIED_OBLIGATION_EXCEPTION_EDGE | 183 | Medium |
Write to static field org.universAAL.tools.logmonitor.rdfvis.gui.RDFVis.instance from instance method new org.universAAL.tools.logmonitor.rdfvis.gui.RDFVis() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 151 | High |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.rdfvis.gui.RDFVis$Message.getDateString() invokes inefficient new String() constructor | PERFORMANCE | DM_STRING_VOID_CTOR | 93 | Medium |
Should org.universAAL.tools.logmonitor.rdfvis.gui.RDFVis$Message be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 54 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.service_bus_matching.LogMonitor.instance isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Medium | |
org.universAAL.tools.logmonitor.service_bus_matching.LogMonitor.checkModule isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 36 | Medium |
Write to static field org.universAAL.tools.logmonitor.service_bus_matching.LogMonitor.instance from instance method new org.universAAL.tools.logmonitor.service_bus_matching.LogMonitor() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 52 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Nullcheck of srv at line 114 of value previously dereferenced in org.universAAL.tools.logmonitor.service_bus_matching.LogMonitor_Util.getServiceString(Service, boolean) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 113 | Medium |
org.universAAL.tools.logmonitor.service_bus_matching.LogMonitor_Util.getPP(PropertyPath, boolean) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 238 | Medium |
org.universAAL.tools.logmonitor.service_bus_matching.LogMonitor_Util.getProfileString(ServiceProfile, boolean) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 193 | Medium |
org.universAAL.tools.logmonitor.service_bus_matching.LogMonitor_Util.getRequestString(ServiceRequest, boolean) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 224 | Medium |
org.universAAL.tools.logmonitor.service_bus_matching.LogMonitor_Util.getServiceString(Service, boolean) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 120 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.service_bus_matching.Matchmaking.getDateString() invokes inefficient new String() constructor | PERFORMANCE | DM_STRING_VOID_CTOR | 130 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.service_bus_matching.Matchmaking$SingleMatching.processStandardMessage(Object[]) may expose internal representation by storing an externally mutable object into Matchmaking$SingleMatching.msgPart | MALICIOUS_CODE | EI_EXPOSE_REP2 | 82 | Medium |
Should org.universAAL.tools.logmonitor.service_bus_matching.Matchmaking$SingleMatching be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 25 | Medium |
Unread public/protected field: org.universAAL.tools.logmonitor.service_bus_matching.Matchmaking$SingleMatching.outputURI | STYLE | URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD | 135 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.universAAL.tools.logmonitor.service_bus_matching.Sparul.getEffects(StringBuilder, StringBuilder, ServiceRequest) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 148 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Switch statement found in org.universAAL.tools.logmonitor.service_bus_matching.gui.MatchmakingPane.getOverviewHTML(StringBuilder, List) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 58 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Redundant nullcheck of lst, which is known to be non-null in org.universAAL.tools.logmonitor.util.HTMLBusOperationsPane.getCEPSerialization(ContextEventPattern, String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 95 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.universAAL.tools.logmonitor.util.HTMLPaneBase.getPropPathHTML(PropertyPath, boolean) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 122 | Medium |