The following document contains the results of FindBugs Report
FindBugs Version is 1.3.9
Threshold is medium
Effort is min
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.jclouds.cim.VirtualSystemSettingData.getCreationTime() may expose internal representation by returning VirtualSystemSettingData.creationTime | MALICIOUS_CODE | EI_EXPOSE_REP | 468 | Medium |
| new org.jclouds.cim.VirtualSystemSettingData(String, String, String, String, VirtualSystemSettingData$AutomaticRecoveryAction, VirtualSystemSettingData$AutomaticShutdownAction, VirtualSystemSettingData$AutomaticStartupAction, Long, Integer, URI, URI, String, Date, URI, URI, URI, URI, URI, String, Iterable, String) may expose internal representation by storing an externally mutable object into VirtualSystemSettingData.creationTime | MALICIOUS_CODE | EI_EXPOSE_REP2 | 386 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.jclouds.cim.VirtualSystemSettingData$Builder.creationTime(Date) may expose internal representation by storing an externally mutable object into VirtualSystemSettingData$Builder.creationTime | MALICIOUS_CODE | EI_EXPOSE_REP2 | 126 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| integral division result cast to double or float in org.jclouds.cim.functions.HardwareBuilderFromResourceAllocations.apply(ResourceAllocationSettingData) | STYLE | ICAST_IDIV_CAST_TO_DOUBLE | 71 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.jclouds.cim.xml.ResourceAllocationSettingDataHandler.endElement(String, String, String) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 70 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class name org.jclouds.compute.Utils shadows the simple name of implemented interface org.jclouds.rest.Utils | BAD_PRACTICE | NM_SAME_SIMPLE_NAME_AS_INTERFACE | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Field RunScriptOnNodeAsInitScriptUsingSsh.logger masks field in superclass org.jclouds.compute.callables.SudoAwareInitManager | CORRECTNESS | MF_CLASS_MASKS_FIELD | 54 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| instanceof will always return true in org.jclouds.compute.domain.Processor.compareTo(Processor), since all org.jclouds.compute.domain.Processor are instances of org.jclouds.compute.domain.Processor | STYLE | BC_VACUOUS_INSTANCEOF | 42 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.jclouds.compute.domain.internal.HardwareImpl doesn't override ComputeMetadataImpl.equals(Object) | STYLE | EQ_DOESNT_OVERRIDE_EQUALS | 47 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.jclouds.compute.domain.internal.ImageImpl defines non-transient non-serializable instance field defaultCredentials | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Class org.jclouds.compute.domain.internal.ImageImpl defines non-transient non-serializable instance field operatingSystem | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.jclouds.compute.domain.internal.NodeMetadataImpl defines non-transient non-serializable instance field credentials | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Class org.jclouds.compute.domain.internal.NodeMetadataImpl defines non-transient non-serializable instance field os | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.jclouds.compute.internal.ComputeServiceContextImpl.equals(Object) checks for operand being a org.jclouds.rest.RestContext | BAD_PRACTICE | EQ_CHECK_FOR_OPERAND_NOT_COMPATIBLE_WITH_THIS | 90 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class name org.jclouds.compute.internal.UtilsImpl shadows the simple name of the superclass org.jclouds.rest.internal.UtilsImpl | BAD_PRACTICE | NM_SAME_SIMPLE_NAME_AS_SUPERCLASS | 56 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.jclouds.compute.options.TemplateOptions.clone() does not call super.clone() | BAD_PRACTICE | CN_IDIOM_NO_SUPER_CALL | 57 | High |
| org.jclouds.compute.options.TemplateOptions.getInboundPorts() may expose internal representation by returning TemplateOptions.inboundPorts | MALICIOUS_CODE | EI_EXPOSE_REP | 319 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.jclouds.compute.options.TemplateOptions$ImmutableTemplateOptions.clone() does not call super.clone() | BAD_PRACTICE | CN_IDIOM_NO_SUPER_CALL | 93 | Medium |
| org.jclouds.compute.options.TemplateOptions$ImmutableTemplateOptions doesn't override TemplateOptions.equals(Object) | STYLE | EQ_DOESNT_OVERRIDE_EQUALS | 93 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Possible null pointer dereference of toMatch in org.jclouds.compute.predicates.OperatingSystemPredicates$2.apply(OperatingSystem) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 81 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to extensionHandlers in org.jclouds.ovf.xml.internal.BaseVirtualSystemHandler.setExtensionHandlers(Map) | STYLE | DLS_DEAD_LOCAL_STORE | 72 | High |
