The following document contains the results of SpotBugs
SpotBugs Version is 4.7.1
Threshold is medium
Effort is default
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 25 | 15 | 0 | 0 |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| nl.tudelft.simulation.naming.context.FileContextFactory.getInitialContext(Hashtable, String) may expose internal representation by returning FileContextFactory.context | MALICIOUS_CODE | EI_EXPOSE_REP | 55 | Medium |
| Static field "context" is modified by an instance level synchronized method. | MT_CORRECTNESS | SSD_DO_NOT_USE_INSTANCE_LOCK_ON_SHARED_STATIC_DATA | 42 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| nl.tudelft.simulation.naming.context.JVMContext.bindings() may expose internal representation by returning JVMContext.elements | MALICIOUS_CODE | EI_EXPOSE_REP | 417 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| nl.tudelft.simulation.naming.context.JVMContextFactory.getInitialContext(Hashtable, String) may expose internal representation by returning JVMContextFactory.context | MALICIOUS_CODE | EI_EXPOSE_REP | 30 | Medium |
| Static field "context" is modified by an instance level synchronized method. | MT_CORRECTNESS | SSD_DO_NOT_USE_INSTANCE_LOCK_ON_SHARED_STATIC_DATA | 28 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new nl.tudelft.simulation.naming.context.RemoteContext$RemoteChangeEventProducer(RemoteContext, String, int, String) may expose internal representation by storing an externally mutable object into RemoteContext$RemoteChangeEventProducer.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 446 | Medium |
| Should nl.tudelft.simulation.naming.context.RemoteContext$RemoteChangeEventProducer be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 446-460 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| nl.tudelft.simulation.naming.context.RemoteContextFactory.getInitialContext(Hashtable, String) may expose internal representation by returning RemoteContextFactory.context | MALICIOUS_CODE | EI_EXPOSE_REP | 43 | Medium |
| Static field "context" is modified by an instance level synchronized method. | MT_CORRECTNESS | SSD_DO_NOT_USE_INSTANCE_LOCK_ON_SHARED_STATIC_DATA | 100 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new nl.tudelft.simulation.naming.context.event.ContextEventProducerImpl$PatternListener(ContextEventProducerImpl, Pattern, EventListener) may expose internal representation by storing an externally mutable object into ContextEventProducerImpl$PatternListener.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 398 | Medium |
| nl.tudelft.simulation.naming.context.event.ContextEventProducerImpl$PatternListener is serializable and an inner class | BAD_PRACTICE | SE_INNER_CLASS | 398-419 | Medium |
| Should nl.tudelft.simulation.naming.context.event.ContextEventProducerImpl$PatternListener be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 398-419 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Hard coded reference to an absolute pathname in nl.tudelft.simulation.naming.context.event.InitialEventContext.buildEnvironment(Hashtable) | STYLE | DMI_HARDCODED_ABSOLUTE_FILENAME | 210 | Medium |
| Incorrect lazy initialization and update of static field nl.tudelft.simulation.naming.context.event.InitialEventContext.INSTANCE in nl.tudelft.simulation.naming.context.event.InitialEventContext.instantiate(Hashtable, String) | MT_CORRECTNESS | LI_LAZY_INIT_UPDATE_STATIC | 145-150 | High |
| Public static nl.tudelft.simulation.naming.context.event.InitialEventContext.instantiate(Hashtable, String) may expose internal representation by returning InitialEventContext.INSTANCE | MALICIOUS_CODE | MS_EXPOSE_REP | 147 | Medium |