-
Home
- Project Documentation Project Reports 10
SpotBugs Bug Detector Report
The following document contains the results of SpotBugs
SpotBugs Version is 4.8.2
Threshold is medium
Effort is default
Summary
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 146 | 250 | 0 | 0 |
Files
org.bouncycastle.crypto.KeyGenerationParameters
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.bouncycastle.crypto.KeyGenerationParameters.getRandom() may expose internal representation by returning KeyGenerationParameters.random | MALICIOUS_CODE | EI_EXPOSE_REP | 58 | Medium |
| new org.bouncycastle.crypto.KeyGenerationParameters(SecureRandom, int) may expose internal representation by storing an externally mutable object into KeyGenerationParameters.random | MALICIOUS_CODE | EI_EXPOSE_REP2 | 46 | Medium |
org.bouncycastle.crypto.engines.AESEngine
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Complicated, subtle or wrong increment in for-loop org.bouncycastle.crypto.engines.AESEngine.generateWorkingKey(byte[], boolean) | STYLE | QF_QUESTIONABLE_FOR_LOOP | 308 | Medium |
org.bouncycastle.crypto.engines.DESEngine
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Vacuous iand operation on DESEngine.java:[line 448] in org.bouncycastle.crypto.engines.DESEngine.desFunc(int[], byte[], int, byte[], int) | STYLE | INT_VACUOUS_BIT_OPERATION | 448 | Medium |
org.bouncycastle.crypto.params.DESParameters
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class org.bouncycastle.crypto.params.DESParameters at new org.bouncycastle.crypto.params.DESParameters(byte[]) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 33 | Medium |
org.bouncycastle.crypto.params.KeyParameter
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.bouncycastle.crypto.params.KeyParameter.getKey() may expose internal representation by returning KeyParameter.key | MALICIOUS_CODE | EI_EXPOSE_REP | 50 | Medium |
org.sentrysoftware.snmp.client.SnmpClient
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class org.sentrysoftware.snmp.client.SnmpClient at new org.sentrysoftware.snmp.client.SnmpClient(String, int, int, int[], String, String, String, String, String, String, String, byte[]) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 99 | Medium |
| new org.sentrysoftware.snmp.client.SnmpClient(String, int, int, int[], String, String, String, String, String, String, String, byte[]) may expose internal representation by storing an externally mutable object into SnmpClient.contextEngineID | MALICIOUS_CODE | EI_EXPOSE_REP2 | 113 | Medium |
| new org.sentrysoftware.snmp.client.SnmpClient(String, int, int, int[], String, String, String, String, String, String, String, byte[]) may expose internal representation by storing an externally mutable object into SnmpClient.retryIntervals | MALICIOUS_CODE | EI_EXPOSE_REP2 | 105 | Medium |
uk.co.westhawk.snmp.beans.AnnexModemStatusBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.beans.AnnexModemStatusBean.update(Observable, Object) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 645 | Medium |
| uk.co.westhawk.snmp.beans.AnnexModemStatusBean.update(Observable, Object) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 546 | Medium |
| uk.co.westhawk.snmp.beans.AnnexModemStatusBean.getLastUpdateDate() may expose internal representation by returning AnnexModemStatusBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 298 | Medium |
| uk.co.westhawk.snmp.beans.AnnexModemStatusBean.setDefaultTreeModel(DefaultTreeModel) may expose internal representation by storing an externally mutable object into AnnexModemStatusBean.treeModel | MALICIOUS_CODE | EI_EXPOSE_REP2 | 682 | Medium |
| uk.co.westhawk.snmp.beans.AnnexModemStatusBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into AnnexModemStatusBean.pduGetNext | MALICIOUS_CODE | EI_EXPOSE_REP2 | 533 | Medium |
| uk.co.westhawk.snmp.beans.AnnexModemStatusBean.sig_state should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 225 | Medium |
| Unread field: uk.co.westhawk.snmp.beans.AnnexModemStatusBean.deviceType | PERFORMANCE | URF_UNREAD_FIELD | 243 | Medium |
| Unread field: uk.co.westhawk.snmp.beans.AnnexModemStatusBean.modemStatus | PERFORMANCE | URF_UNREAD_FIELD | 245 | Medium |
| Unread field: uk.co.westhawk.snmp.beans.AnnexModemStatusBean.openCount | PERFORMANCE | URF_UNREAD_FIELD | 244 | Medium |
uk.co.westhawk.snmp.beans.AnnexModemStatusBean$PortInfo
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should uk.co.westhawk.snmp.beans.AnnexModemStatusBean$PortInfo be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 707-800 | Medium |
uk.co.westhawk.snmp.beans.AscendActiveSessionBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.pdu.GetNextPdu_vec in uk.co.westhawk.snmp.beans.AscendActiveSessionBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 353 | Medium |
| uk.co.westhawk.snmp.beans.AscendActiveSessionBean.getLastUpdateDate() may expose internal representation by returning AscendActiveSessionBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 232 | Medium |
| uk.co.westhawk.snmp.beans.AscendActiveSessionBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into AscendActiveSessionBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 353 | Medium |
uk.co.westhawk.snmp.beans.DialogChannelStatusBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.pdu.GetNextPdu_vec in uk.co.westhawk.snmp.beans.DialogChannelStatusBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 524 | Medium |
| uk.co.westhawk.snmp.beans.DialogChannelStatusBean.update(Observable, Object) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 541 | Medium |
| uk.co.westhawk.snmp.beans.DialogChannelStatusBean.getLastUpdateDate() may expose internal representation by returning DialogChannelStatusBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 291 | Medium |
| uk.co.westhawk.snmp.beans.DialogChannelStatusBean.setDefaultTreeModel(DefaultTreeModel) may expose internal representation by storing an externally mutable object into DialogChannelStatusBean.treeModel | MALICIOUS_CODE | EI_EXPOSE_REP2 | 631 | Medium |
| uk.co.westhawk.snmp.beans.DialogChannelStatusBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into DialogChannelStatusBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 524 | Medium |
| uk.co.westhawk.snmp.beans.DialogChannelStatusBean.vch_status should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 204 | Medium |
uk.co.westhawk.snmp.beans.DialogChannelStatusBean$ChannelStatus
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should uk.co.westhawk.snmp.beans.DialogChannelStatusBean$ChannelStatus be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 655-744 | Medium |
uk.co.westhawk.snmp.beans.InterfaceIndexesBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.pdu.InterfaceGetNextPdu in uk.co.westhawk.snmp.beans.InterfaceIndexesBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 255 | Medium |
| uk.co.westhawk.snmp.beans.InterfaceIndexesBean.getLastUpdateDate() may expose internal representation by returning InterfaceIndexesBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 153 | Medium |
| uk.co.westhawk.snmp.beans.InterfaceIndexesBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into InterfaceIndexesBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 255 | Medium |
uk.co.westhawk.snmp.beans.IsHostReachableBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.pdu.UpSincePdu in uk.co.westhawk.snmp.beans.IsHostReachableBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 285 | Medium |
| uk.co.westhawk.snmp.beans.IsHostReachableBean.setReachable(boolean) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 189 | Medium |
| uk.co.westhawk.snmp.beans.IsHostReachableBean.getUpSinceDate() may expose internal representation by returning IsHostReachableBean.upSince | MALICIOUS_CODE | EI_EXPOSE_REP | 149 | Medium |
| uk.co.westhawk.snmp.beans.IsHostReachableBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into IsHostReachableBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 285 | Medium |
uk.co.westhawk.snmp.beans.NTPrintQBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.stack.GetNextPdu in uk.co.westhawk.snmp.beans.NTPrintQBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 280 | Medium |
| uk.co.westhawk.snmp.beans.NTPrintQBean.getLastUpdateDate() may expose internal representation by returning NTPrintQBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 156 | Medium |
| uk.co.westhawk.snmp.beans.NTPrintQBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into NTPrintQBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 280 | Medium |
uk.co.westhawk.snmp.beans.NTServiceNamesBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.stack.GetNextPdu in uk.co.westhawk.snmp.beans.NTServiceNamesBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 282 | Medium |
| uk.co.westhawk.snmp.beans.NTServiceNamesBean.getLastUpdateDate() may expose internal representation by returning NTServiceNamesBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 154 | Medium |
| uk.co.westhawk.snmp.beans.NTServiceNamesBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into NTServiceNamesBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 282 | Medium |
uk.co.westhawk.snmp.beans.NTSharedResBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.stack.GetNextPdu in uk.co.westhawk.snmp.beans.NTSharedResBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 285 | Medium |
| uk.co.westhawk.snmp.beans.NTSharedResBean.getLastUpdateDate() may expose internal representation by returning NTSharedResBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 154 | Medium |
| uk.co.westhawk.snmp.beans.NTSharedResBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into NTSharedResBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 285 | Medium |
uk.co.westhawk.snmp.beans.NTUserNamesBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.stack.GetNextPdu in uk.co.westhawk.snmp.beans.NTUserNamesBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 277 | Medium |
| uk.co.westhawk.snmp.beans.NTUserNamesBean.getLastUpdateDate() may expose internal representation by returning NTUserNamesBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 154 | Medium |
| uk.co.westhawk.snmp.beans.NTUserNamesBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into NTUserNamesBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 277 | Medium |
uk.co.westhawk.snmp.beans.NcdPerfDataBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Boxing/unboxing to parse a primitive uk.co.westhawk.snmp.beans.NcdPerfDataBean.setUpdateInterval(String) | PERFORMANCE | DM_BOXED_PRIMITIVE_FOR_PARSING | 236 | High |
| uk.co.westhawk.snmp.beans.NcdPerfDataBean.getLastUpdateDate() may expose internal representation by returning NcdPerfDataBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 252 | Medium |
uk.co.westhawk.snmp.beans.OneInterfaceBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.beans.OneInterfaceBean.getLastUpdateDate() may expose internal representation by returning OneInterfaceBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 199 | Medium |
uk.co.westhawk.snmp.beans.OneNTPrintQBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.pdu.GetPdu_vec in uk.co.westhawk.snmp.beans.OneNTPrintQBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 275 | Medium |
| uk.co.westhawk.snmp.beans.OneNTPrintQBean.getLastUpdateDate() may expose internal representation by returning OneNTPrintQBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 201 | Medium |
| uk.co.westhawk.snmp.beans.OneNTPrintQBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into OneNTPrintQBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 275 | Medium |
uk.co.westhawk.snmp.beans.OneNTServiceBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.pdu.GetPdu_vec in uk.co.westhawk.snmp.beans.OneNTServiceBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 345 | Medium |
| uk.co.westhawk.snmp.beans.OneNTServiceBean.getLastUpdateDate() may expose internal representation by returning OneNTServiceBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 267 | Medium |
| uk.co.westhawk.snmp.beans.OneNTServiceBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into OneNTServiceBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 345 | Medium |
| uk.co.westhawk.snmp.beans.OneNTServiceBean.msg_inst_state should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 118 | Medium |
| uk.co.westhawk.snmp.beans.OneNTServiceBean.msg_oper_state should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 127 | Medium |
uk.co.westhawk.snmp.beans.OneNTSharedResBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from java.util.Observable to uk.co.westhawk.snmp.pdu.GetPdu_vec in uk.co.westhawk.snmp.beans.OneNTSharedResBean.update(Observable, Object) | STYLE | BC_UNCONFIRMED_CAST | 288 | Medium |
| uk.co.westhawk.snmp.beans.OneNTSharedResBean.getLastUpdateDate() may expose internal representation by returning OneNTSharedResBean.lastUpdateDate | MALICIOUS_CODE | EI_EXPOSE_REP | 213 | Medium |
| uk.co.westhawk.snmp.beans.OneNTSharedResBean.update(Observable, Object) may expose internal representation by storing an externally mutable object into OneNTSharedResBean.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 288 | Medium |
uk.co.westhawk.snmp.beans.SNMPBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Boxing/unboxing to parse a primitive uk.co.westhawk.snmp.beans.SNMPBean.setPort(String) | PERFORMANCE | DM_BOXED_PRIMITIVE_FOR_PARSING | 265 | High |
| Questionable use of non-short-circuit logic in uk.co.westhawk.snmp.beans.SNMPBean.setCommunityName(String) | STYLE | NS_NON_SHORT_CIRCUIT | 289 | High |
uk.co.westhawk.snmp.beans.SNMPRunBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Boxing/unboxing to parse a primitive uk.co.westhawk.snmp.beans.SNMPRunBean.setUpdateInterval(String) | PERFORMANCE | DM_BOXED_PRIMITIVE_FOR_PARSING | 158 | High |
uk.co.westhawk.snmp.beans.UsmBeingDiscoveredBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.beans.UsmBeingDiscoveredBean at new uk.co.westhawk.snmp.beans.UsmBeingDiscoveredBean(SnmpContextv3, UsmAgent) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 121 | Medium |
| new uk.co.westhawk.snmp.beans.UsmBeingDiscoveredBean(SnmpContextv3, UsmAgent) may expose internal representation by storing an externally mutable object into UsmBeingDiscoveredBean.context | MALICIOUS_CODE | EI_EXPOSE_REP2 | 117 | Medium |
| new uk.co.westhawk.snmp.beans.UsmBeingDiscoveredBean(SnmpContextv3, UsmAgent) may expose internal representation by storing an externally mutable object into UsmBeingDiscoveredBean.usmAgent | MALICIOUS_CODE | EI_EXPOSE_REP2 | 116 | Medium |
uk.co.westhawk.snmp.beans.UsmDiscoveryBean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.beans.UsmDiscoveryBean at new uk.co.westhawk.snmp.beans.UsmDiscoveryBean(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 97 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.beans.UsmDiscoveryBean at new uk.co.westhawk.snmp.beans.UsmDiscoveryBean(String, int, 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 |
| Dead store to $L4 in uk.co.westhawk.snmp.beans.UsmDiscoveryBean.discoveryEngineId() | STYLE | DLS_DEAD_LOCAL_STORE | 295 | Medium |
| Dead store to vars in uk.co.westhawk.snmp.beans.UsmDiscoveryBean.discoveryTimeLine() | STYLE | DLS_DEAD_LOCAL_STORE | 351 | Medium |
| uk.co.westhawk.snmp.beans.UsmDiscoveryBean.setRetryIntervals(int[]) may expose internal representation by storing an externally mutable object into UsmDiscoveryBean.retry_intervals | MALICIOUS_CODE | EI_EXPOSE_REP2 | 181 | Medium |
uk.co.westhawk.snmp.beans.ethernet
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.beans.ethernet.update(Observable, Object) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 587 | Medium |
uk.co.westhawk.snmp.beans.memory
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.beans.memory.update(Observable, Object) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 882 | Medium |
uk.co.westhawk.snmp.beans.ncdPart
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unused field: uk.co.westhawk.snmp.beans.ncdPart.me | PERFORMANCE | UUF_UNUSED_FIELD | Not available | Medium |
uk.co.westhawk.snmp.beans.user
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to i in uk.co.westhawk.snmp.beans.user.update(Observable, Object) | STYLE | DLS_DEAD_LOCAL_STORE | 791 | Medium |
| Dead store to n in uk.co.westhawk.snmp.beans.user.update(Observable, Object) | STYLE | DLS_DEAD_LOCAL_STORE | 792 | Medium |
uk.co.westhawk.snmp.event.DecodedPduEvent
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.event.DecodedPduEvent.getPdu() may expose internal representation by returning DecodedPduEvent.pdu | MALICIOUS_CODE | EI_EXPOSE_REP | 110 | Medium |
| new uk.co.westhawk.snmp.event.DecodedPduEvent(Object, Pdu, int) may expose internal representation by storing an externally mutable object into DecodedPduEvent.pdu | MALICIOUS_CODE | EI_EXPOSE_REP2 | 85 | Medium |
uk.co.westhawk.snmp.event.RawPduEvent
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.event.RawPduEvent.getMessage() may expose internal representation by returning RawPduEvent.message | MALICIOUS_CODE | EI_EXPOSE_REP | 138 | Medium |
| new uk.co.westhawk.snmp.event.RawPduEvent(Object, int, String, byte[], int) may expose internal representation by storing an externally mutable object into RawPduEvent.message | MALICIOUS_CODE | EI_EXPOSE_REP2 | 95 | Medium |
uk.co.westhawk.snmp.event.RawPduReceivedSupport
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of uk.co.westhawk.snmp.event.RawPduReceivedSupport.pduListeners; locked 71% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 152 | Medium |
uk.co.westhawk.snmp.event.RequestPduReceivedSupport
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of uk.co.westhawk.snmp.event.RequestPduReceivedSupport.pduListeners; locked 71% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 144 | Medium |
uk.co.westhawk.snmp.event.TrapReceivedSupport
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of uk.co.westhawk.snmp.event.TrapReceivedSupport.trapListeners; locked 71% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 144 | Medium |
uk.co.westhawk.snmp.net.TCPSocket
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.net.TCPSocket.receive(int) ignores result of java.io.InputStream.read(byte[], int, int) | BAD_PRACTICE | RR_NOT_CHECKED | 223 | Medium |
uk.co.westhawk.snmp.pdu.BlockPdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new uk.co.westhawk.snmp.pdu.BlockPdu(SnmpContextBasisFace) may expose internal representation by storing an externally mutable object into BlockPdu.context | MALICIOUS_CODE | EI_EXPOSE_REP2 | 101 | Medium |
| uk.co.westhawk.snmp.pdu.BlockPdu.setRetryIntervals(int[]) may expose internal representation by storing an externally mutable object into BlockPdu.retry_intervals | MALICIOUS_CODE | EI_EXPOSE_REP2 | 230 | Medium |
| Inconsistent synchronization of uk.co.westhawk.snmp.pdu.BlockPdu.max_rep; locked 50% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 188 | Medium |
| Inconsistent synchronization of uk.co.westhawk.snmp.pdu.BlockPdu.non_rep; locked 50% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 187 | Medium |
| Inconsistent synchronization of uk.co.westhawk.snmp.pdu.BlockPdu.retry_intervals; locked 66% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 230 | Medium |
| Inconsistent synchronization of uk.co.westhawk.snmp.pdu.BlockPdu.type; locked 50% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 218 | Medium |
uk.co.westhawk.snmp.pdu.DiscoveryPdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new uk.co.westhawk.snmp.pdu.DiscoveryPdu(SnmpContextv3Face) may expose internal representation by storing an externally mutable object into DiscoveryPdu.context | MALICIOUS_CODE | EI_EXPOSE_REP2 | 79 | Medium |
uk.co.westhawk.snmp.pdu.InterfaceGetNextPdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to answered in uk.co.westhawk.snmp.pdu.InterfaceGetNextPdu.getIfNumber(SnmpContextBasisFace) | STYLE | DLS_DEAD_LOCAL_STORE | 150 | Medium |
uk.co.westhawk.snmp.pdu.InterfaceGetNextPduStub
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.pdu.InterfaceGetNextPduStub at new uk.co.westhawk.snmp.pdu.InterfaceGetNextPduStub(SnmpContextBasisFace, Observer) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 162 | Medium |
| Dead store to oid in uk.co.westhawk.snmp.pdu.InterfaceGetNextPduStub.new_value(int, varbind) | STYLE | DLS_DEAD_LOCAL_STORE | 237 | Medium |
| new uk.co.westhawk.snmp.pdu.InterfaceGetNextPduStub(SnmpContextBasisFace) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 138 | Medium |
| uk.co.westhawk.snmp.pdu.InterfaceGetNextPduStub.getIfOperStatusStr() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 420 | Medium |
| uk.co.westhawk.snmp.pdu.InterfaceGetNextPduStub.toString() invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 488 | Medium |
| uk.co.westhawk.snmp.pdu.InterfaceGetNextPduStub.all_oids should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 105 | Medium |
| uk.co.westhawk.snmp.pdu.InterfaceGetNextPduStub.col_oids should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 95 | Medium |
| uk.co.westhawk.snmp.pdu.InterfaceGetNextPduStub.scal_oids should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 89 | Medium |
uk.co.westhawk.snmp.pdu.InterfacePdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.pdu.InterfacePdu at new uk.co.westhawk.snmp.pdu.InterfacePdu(SnmpContextBasisFace, Observer, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 169 | Medium |
| Unused field: uk.co.westhawk.snmp.pdu.InterfacePdu.speed | PERFORMANCE | UUF_UNUSED_FIELD | Not available | Medium |
uk.co.westhawk.snmp.pdu.InterfacesPdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.pdu.InterfacesPdu at new uk.co.westhawk.snmp.pdu.InterfacesPdu(SnmpContextBasisFace, Observer, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 102 | Medium |
| uk.co.westhawk.snmp.pdu.InterfacesPdu.getInterfacePdus() may expose internal representation by returning InterfacesPdu.ifs | MALICIOUS_CODE | EI_EXPOSE_REP | 113 | Medium |
uk.co.westhawk.snmp.pdu.OneGetBulkPdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.pdu.OneGetBulkPdu.getVarbinds() may expose internal representation by returning OneGetBulkPdu.vars | MALICIOUS_CODE | EI_EXPOSE_REP | 107 | Medium |
uk.co.westhawk.snmp.pdu.OneGetNextPdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.pdu.OneGetNextPdu at new uk.co.westhawk.snmp.pdu.OneGetNextPdu(SnmpContextBasisFace, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 105 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.pdu.OneGetNextPdu at new uk.co.westhawk.snmp.pdu.OneGetNextPdu(SnmpContextBasisFace, String, Observer) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 119 | Medium |
uk.co.westhawk.snmp.pdu.OneGetPdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.pdu.OneGetPdu at new uk.co.westhawk.snmp.pdu.OneGetPdu(SnmpContextBasisFace, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 103 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.pdu.OneGetPdu at new uk.co.westhawk.snmp.pdu.OneGetPdu(SnmpContextBasisFace, String, Observer) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 117 | Medium |
uk.co.westhawk.snmp.pdu.OneInformPdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.pdu.OneInformPdu at new uk.co.westhawk.snmp.pdu.OneInformPdu(SnmpContextBasisFace, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 113 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.pdu.OneInformPdu at new uk.co.westhawk.snmp.pdu.OneInformPdu(SnmpContextBasisFace, String, Observer) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 127 | Medium |
uk.co.westhawk.snmp.pdu.OneIntPdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.pdu.OneIntPdu at new uk.co.westhawk.snmp.pdu.OneIntPdu(SnmpContextBasisFace, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 116 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.pdu.OneIntPdu at new uk.co.westhawk.snmp.pdu.OneIntPdu(SnmpContextBasisFace, String, Observer) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 130 | Medium |
| uk.co.westhawk.snmp.pdu.OneIntPdu.new_value(int, varbind) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 165 | Medium |
uk.co.westhawk.snmp.pdu.OneSetPdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.pdu.OneSetPdu at new uk.co.westhawk.snmp.pdu.OneSetPdu(SnmpContextBasisFace, String, AsnObject) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 107 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.pdu.OneSetPdu at new uk.co.westhawk.snmp.pdu.OneSetPdu(SnmpContextBasisFace, String, AsnObject, Observer) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 122 | Medium |
uk.co.westhawk.snmp.pdu.UpSincePdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.pdu.UpSincePdu at new uk.co.westhawk.snmp.pdu.UpSincePdu(SnmpContextBasisFace, Observer) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 89 | Medium |
| uk.co.westhawk.snmp.pdu.UpSincePdu.getDate() may expose internal representation by returning UpSincePdu.since | MALICIOUS_CODE | EI_EXPOSE_REP | 98 | Medium |
uk.co.westhawk.snmp.stack.AbstractSnmpContext
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.AbstractSnmpContext at new uk.co.westhawk.snmp.stack.AbstractSnmpContext(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 152 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.AbstractSnmpContext at new uk.co.westhawk.snmp.stack.AbstractSnmpContext(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 170 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.AbstractSnmpContext at new uk.co.westhawk.snmp.stack.AbstractSnmpContext(String, int, 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 | 96 | Medium |
| Inconsistent synchronization of uk.co.westhawk.snmp.stack.AbstractSnmpContext.isDestroyed; locked 50% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 522 | Medium |
| Inconsistent synchronization of uk.co.westhawk.snmp.stack.AbstractSnmpContext.me; locked 54% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 466 | Medium |
| Redundant nullcheck of sf, which is known to be non-null in uk.co.westhawk.snmp.stack.AbstractSnmpContext.getSocket(String) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 283 | Medium |
uk.co.westhawk.snmp.stack.AsnDecoderv3
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Return value of SnmpContextv3Basis.getUsmAgent() ignored, but method has no side effect | STYLE | RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT | 228 | Medium |
uk.co.westhawk.snmp.stack.AsnInteger
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnInteger at new uk.co.westhawk.snmp.stack.AsnInteger(InputStream, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 107 | Medium |
uk.co.westhawk.snmp.stack.AsnNull
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to anotherNull in uk.co.westhawk.snmp.stack.AsnNull.equals(Object) | STYLE | DLS_DEAD_LOCAL_STORE | 132 | Medium |
uk.co.westhawk.snmp.stack.AsnObject
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.AsnObject.debug isn't final and cannot be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 95 | Medium |
| Primitive field uk.co.westhawk.snmp.stack.AsnObject.debug 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 | 109 | Medium |
uk.co.westhawk.snmp.stack.AsnObjectId
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnObjectId at new uk.co.westhawk.snmp.stack.AsnObjectId(InputStream, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 137 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnObjectId at new uk.co.westhawk.snmp.stack.AsnObjectId(String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 191 | Medium |
| Dead store to oidArray in uk.co.westhawk.snmp.stack.AsnObjectId.toArrayOfLongs(String) | STYLE | DLS_DEAD_LOCAL_STORE | 216 | Medium |
uk.co.westhawk.snmp.stack.AsnOctets
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnOctets at new uk.co.westhawk.snmp.stack.AsnOctets(InputStream, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 371 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnOctets at new uk.co.westhawk.snmp.stack.AsnOctets(Inet4Address, byte) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 253 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnOctets at new uk.co.westhawk.snmp.stack.AsnOctets(InetAddress) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 218 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnOctets at new uk.co.westhawk.snmp.stack.AsnOctets(byte[]) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 143 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnOctets at new uk.co.westhawk.snmp.stack.AsnOctets(byte[], byte) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 131 | Medium |
| Dead store to minUTC in uk.co.westhawk.snmp.stack.AsnOctets.getCalendar() | STYLE | DLS_DEAD_LOCAL_STORE | 814 | Medium |
| Found reliance on default encoding in uk.co.westhawk.snmp.stack.AsnOctets.toDisplayString(): new String(byte[]) | I18N | DM_DEFAULT_ENCODING | 723 | High |
| new uk.co.westhawk.snmp.stack.AsnOctets(byte[], byte) may expose internal representation by storing an externally mutable object into AsnOctets.value | MALICIOUS_CODE | EI_EXPOSE_REP2 | 157 | Medium |
| uk.co.westhawk.snmp.stack.AsnOctets.HEX_PREFIX should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 106 | Medium |
| uk.co.westhawk.snmp.stack.AsnOctets.printableObject should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 126 | Medium |
| uk.co.westhawk.snmp.stack.AsnOctets.CALFORMAT isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 118 | High |
| Primitive field uk.co.westhawk.snmp.stack.AsnOctets.HEX_PREFIX 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 | 106 | Medium |
| Primitive field uk.co.westhawk.snmp.stack.AsnOctets.printableObject 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 | 126 | Medium |
| Self assignment of field AsnOctets.hash in uk.co.westhawk.snmp.stack.AsnOctets.hashCode() | CORRECTNESS | SA_FIELD_SELF_ASSIGNMENT | 982 | High |
| Call to method of static java.text.DateFormat in uk.co.westhawk.snmp.stack.AsnOctets.toCalendar() | MT_CORRECTNESS | STCAL_INVOKE_ON_STATIC_DATE_FORMAT_INSTANCE | 779 | Medium |
| uk.co.westhawk.snmp.stack.AsnOctets.CALFORMAT is a static field of type java.text.DateFormat, which isn't thread safe | MT_CORRECTNESS | STCAL_STATIC_SIMPLE_DATE_FORMAT_INSTANCE | Not available | Medium |
uk.co.westhawk.snmp.stack.AsnPduSequence
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnPduSequence at new uk.co.westhawk.snmp.stack.AsnPduSequence(InputStream, int, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 85 | Medium |
uk.co.westhawk.snmp.stack.AsnSequence
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnSequence at new uk.co.westhawk.snmp.stack.AsnSequence(InputStream, int, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 111 | Medium |
uk.co.westhawk.snmp.stack.AsnTrapPduv1Sequence
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnTrapPduv1Sequence at new uk.co.westhawk.snmp.stack.AsnTrapPduv1Sequence(InputStream, int, int) 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 |
uk.co.westhawk.snmp.stack.AsnUnsInteger
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnUnsInteger at new uk.co.westhawk.snmp.stack.AsnUnsInteger(InputStream, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 127 | Medium |
uk.co.westhawk.snmp.stack.AsnUnsInteger64
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.AsnUnsInteger64 at new uk.co.westhawk.snmp.stack.AsnUnsInteger64(InputStream, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 111 | Medium |
uk.co.westhawk.snmp.stack.DefaultAsnOctetsPrintable
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Found reliance on default encoding in uk.co.westhawk.snmp.stack.DefaultAsnOctetsPrintable.toInternationalDisplayString(byte[]): new String(byte[]) | I18N | DM_DEFAULT_ENCODING | 134 | High |
uk.co.westhawk.snmp.stack.DefaultTrapContext
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.DefaultTrapContext at new uk.co.westhawk.snmp.stack.DefaultTrapContext(int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 132 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.DefaultTrapContext at new uk.co.westhawk.snmp.stack.DefaultTrapContext(int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 153 | Medium |
| Public static uk.co.westhawk.snmp.stack.DefaultTrapContext.getInstance(int) may expose internal representation by returning DefaultTrapContext.current | MALICIOUS_CODE | MS_EXPOSE_REP | 168 | Medium |
| Public static uk.co.westhawk.snmp.stack.DefaultTrapContext.getInstance(int, String) may expose internal representation by returning DefaultTrapContext.current | MALICIOUS_CODE | MS_EXPOSE_REP | 183 | Medium |
uk.co.westhawk.snmp.stack.DefaultUsmAgent
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.DefaultUsmAgent.setSnmpContext(SnmpContextv3Basis) may expose internal representation by storing an externally mutable object into DefaultUsmAgent.context | MALICIOUS_CODE | EI_EXPOSE_REP2 | 175 | Medium |
uk.co.westhawk.snmp.stack.ListeningContextPool
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.ListeningContextPool.contextPool should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available | Medium |
uk.co.westhawk.snmp.stack.ListeningContextPool$Item
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should uk.co.westhawk.snmp.stack.ListeningContextPool$Item be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 458-498 | Medium |
uk.co.westhawk.snmp.stack.PassiveSnmpContext
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.PassiveSnmpContext at new uk.co.westhawk.snmp.stack.PassiveSnmpContext(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 96 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.PassiveSnmpContext at new uk.co.westhawk.snmp.stack.PassiveSnmpContext(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 114 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.PassiveSnmpContext at new uk.co.westhawk.snmp.stack.PassiveSnmpContext(String, int, 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 | 144 | Medium |
uk.co.westhawk.snmp.stack.PassiveSnmpContextv2c
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.PassiveSnmpContextv2c at new uk.co.westhawk.snmp.stack.PassiveSnmpContextv2c(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 97 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.PassiveSnmpContextv2c at new uk.co.westhawk.snmp.stack.PassiveSnmpContextv2c(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 115 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.PassiveSnmpContextv2c at new uk.co.westhawk.snmp.stack.PassiveSnmpContextv2c(String, int, 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 | 143 | Medium |
uk.co.westhawk.snmp.stack.Pdu
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.Pdu.getContext() may expose internal representation by returning Pdu.context | MALICIOUS_CODE | EI_EXPOSE_REP | 191 | Medium |
| uk.co.westhawk.snmp.stack.Pdu.getRetryIntervals() may expose internal representation by returning Pdu.retry_intervals | MALICIOUS_CODE | EI_EXPOSE_REP | 216 | Medium |
| new uk.co.westhawk.snmp.stack.Pdu(SnmpContextBasisFace) may expose internal representation by storing an externally mutable object into Pdu.context | MALICIOUS_CODE | EI_EXPOSE_REP2 | 163 | Medium |
| uk.co.westhawk.snmp.stack.Pdu.setRetryIntervals(int[]) may expose internal representation by storing an externally mutable object into Pdu.retry_intervals | MALICIOUS_CODE | EI_EXPOSE_REP2 | 211 | Medium |
| Naked notify in uk.co.westhawk.snmp.stack.Pdu.handleNoAnswer() | MT_CORRECTNESS | NN_NAKED_NOTIFY | 855 | Medium |
uk.co.westhawk.snmp.stack.SnmpContext
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.SnmpContext.clone() does not call super.clone() | BAD_PRACTICE | CN_IDIOM_NO_SUPER_CALL | 322-333 | High |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContext at new uk.co.westhawk.snmp.stack.SnmpContext(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 102 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContext at new uk.co.westhawk.snmp.stack.SnmpContext(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 91 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContext at new uk.co.westhawk.snmp.stack.SnmpContext(String, int, 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 | 91 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContext.processIncomingResponse(ByteArrayInputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 219 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContext.clone() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 326 | Medium |
| Redundant nullcheck of seqPdu, which is known to be non-null in uk.co.westhawk.snmp.stack.SnmpContext.processIncomingResponse(ByteArrayInputStream) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 216 | High |
| Redundant nullcheck of seqPdu, which is known to be non-null in uk.co.westhawk.snmp.stack.SnmpContext.processIncomingPdu(byte[]) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 258 | Medium |
uk.co.westhawk.snmp.stack.SnmpContextPool
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextPool at new uk.co.westhawk.snmp.stack.SnmpContextPool(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 112 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextPool at new uk.co.westhawk.snmp.stack.SnmpContextPool(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 132 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextPool at new uk.co.westhawk.snmp.stack.SnmpContextPool(String, int, 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 | 153 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextPool at new uk.co.westhawk.snmp.stack.SnmpContextPool(String, int, String, 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 | 101 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextPool.contextPool should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available | Medium |
uk.co.westhawk.snmp.stack.SnmpContextv2c
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.SnmpContextv2c.clone() does not call super.clone() | BAD_PRACTICE | CN_IDIOM_NO_SUPER_CALL | 258-269 | High |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv2c at new uk.co.westhawk.snmp.stack.SnmpContextv2c(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 88 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv2c at new uk.co.westhawk.snmp.stack.SnmpContextv2c(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 105 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv2c at new uk.co.westhawk.snmp.stack.SnmpContextv2c(String, int, 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 | 126 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv2c.processIncomingResponse(ByteArrayInputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 172 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv2c.clone() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 262 | Medium |
uk.co.westhawk.snmp.stack.SnmpContextv2cPool
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv2cPool at new uk.co.westhawk.snmp.stack.SnmpContextv2cPool(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 88 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv2cPool at new uk.co.westhawk.snmp.stack.SnmpContextv2cPool(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 107 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv2cPool at new uk.co.westhawk.snmp.stack.SnmpContextv2cPool(String, int, 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 | 127 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv2cPool at new uk.co.westhawk.snmp.stack.SnmpContextv2cPool(String, int, String, 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 | 149 | Medium |
uk.co.westhawk.snmp.stack.SnmpContextv3
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.SnmpContextv3.clone() does not call super.clone() | BAD_PRACTICE | CN_IDIOM_NO_SUPER_CALL | 484-495 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3 at new uk.co.westhawk.snmp.stack.SnmpContextv3(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 106 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3 at new uk.co.westhawk.snmp.stack.SnmpContextv3(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 95 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3 at new uk.co.westhawk.snmp.stack.SnmpContextv3(String, int, 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 | 95 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3.processIncomingPdu(byte[]) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 298 | Medium |
uk.co.westhawk.snmp.stack.SnmpContextv3Basis
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3Basis at new uk.co.westhawk.snmp.stack.SnmpContextv3Basis(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 146 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3Basis at new uk.co.westhawk.snmp.stack.SnmpContextv3Basis(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 163 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3Basis at new uk.co.westhawk.snmp.stack.SnmpContextv3Basis(String, int, 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 | 183 | Medium |
| Dead store to $L5 in new uk.co.westhawk.snmp.stack.SnmpContextv3Basis(String, int, String, String) | STYLE | DLS_DEAD_LOCAL_STORE | 187 | High |
| Dead store to rid in uk.co.westhawk.snmp.stack.SnmpContextv3Basis.processIncomingResponse(ByteArrayInputStream) | STYLE | DLS_DEAD_LOCAL_STORE | 956 | High |
| uk.co.westhawk.snmp.stack.SnmpContextv3Basis.addPdu(Pdu, boolean) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 610 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Basis.processIncomingResponse(ByteArrayInputStream) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 867 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Basis.cloneParameters(SnmpContextv3Face) invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 1076 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Basis.getContextEngineId() may expose internal representation by returning SnmpContextv3Basis.contextEngineId | MALICIOUS_CODE | EI_EXPOSE_REP | 489 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Basis.getUsmAgent() may expose internal representation by returning SnmpContextv3Basis.usmAgent | MALICIOUS_CODE | EI_EXPOSE_REP | 580 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Basis.setContextEngineId(byte[]) may expose internal representation by storing an externally mutable object into SnmpContextv3Basis.contextEngineId | MALICIOUS_CODE | EI_EXPOSE_REP2 | 474 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Basis.setUsmAgent(UsmAgent) may expose internal representation by storing an externally mutable object into SnmpContextv3Basis.usmAgent | MALICIOUS_CODE | EI_EXPOSE_REP2 | 570 | Medium |
uk.co.westhawk.snmp.stack.SnmpContextv3Discovery
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.SnmpContextv3Discovery.clone() does not call super.clone() | BAD_PRACTICE | CN_IDIOM_NO_SUPER_CALL | 210-221 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3Discovery at new uk.co.westhawk.snmp.stack.SnmpContextv3Discovery(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 96 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3Discovery at new uk.co.westhawk.snmp.stack.SnmpContextv3Discovery(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 112 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3Discovery at new uk.co.westhawk.snmp.stack.SnmpContextv3Discovery(String, int, 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 | 133 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Discovery.processIncomingPdu(byte[]) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead | PERFORMANCE | DM_NUMBER_CTOR | 196 | Medium |
uk.co.westhawk.snmp.stack.SnmpContextv3Face
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.SnmpContextv3Face.ProtocolNames should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 121 | Medium |
uk.co.westhawk.snmp.stack.SnmpContextv3Pool
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3Pool at new uk.co.westhawk.snmp.stack.SnmpContextv3Pool(String, int) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 122 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3Pool at new uk.co.westhawk.snmp.stack.SnmpContextv3Pool(String, int, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 141 | Medium |
| Exception thrown in class uk.co.westhawk.snmp.stack.SnmpContextv3Pool at new uk.co.westhawk.snmp.stack.SnmpContextv3Pool(String, int, 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 | 98 | Medium |
| Using .equals to compare two byte[]'s, (equivalent to ==) in uk.co.westhawk.snmp.stack.SnmpContextv3Pool.setContextEngineId(byte[]) | CORRECTNESS | EC_BAD_ARRAY_COMPARE | 375 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Pool.getContextEngineId() may expose internal representation by returning SnmpContextv3Pool.contextEngineId | MALICIOUS_CODE | EI_EXPOSE_REP | 390 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Pool.getUsmAgent() may expose internal representation by returning SnmpContextv3Pool.usmAgent | MALICIOUS_CODE | EI_EXPOSE_REP | 423 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Pool.setContextEngineId(byte[]) may expose internal representation by storing an externally mutable object into SnmpContextv3Pool.contextEngineId | MALICIOUS_CODE | EI_EXPOSE_REP2 | 377 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Pool.setUsmAgent(UsmAgent) may expose internal representation by storing an externally mutable object into SnmpContextv3Pool.usmAgent | MALICIOUS_CODE | EI_EXPOSE_REP2 | 416 | Medium |
| uk.co.westhawk.snmp.stack.SnmpContextv3Pool.contextPool should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available | Medium |
uk.co.westhawk.snmp.stack.TimeWindow
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Public static uk.co.westhawk.snmp.stack.TimeWindow.getCurrent() may expose internal representation by returning TimeWindow.current | MALICIOUS_CODE | MS_EXPOSE_REP | 125 | Medium |
| uk.co.westhawk.snmp.stack.TimeWindow.clearTimeWindow(String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 374 | Medium |
uk.co.westhawk.snmp.stack.TimeWindowNode
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Vacuous comparison of integer value uk.co.westhawk.snmp.stack.TimeWindowNode.incrementSnmpEngineTime(int) | STYLE | INT_VACUOUS_COMPARISON | 156 | Medium |
| Inconsistent synchronization of uk.co.westhawk.snmp.stack.TimeWindowNode.snmpEngineBoots; locked 50% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 124 | Medium |
| Inconsistent synchronization of uk.co.westhawk.snmp.stack.TimeWindowNode.snmpEngineTime; locked 71% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 146 | Medium |
uk.co.westhawk.snmp.stack.Transmitter
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of uk.co.westhawk.snmp.stack.Transmitter.pdu; locked 83% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 185 | Medium |
| Naked notify in uk.co.westhawk.snmp.stack.Transmitter.stand() | MT_CORRECTNESS | NN_NAKED_NOTIFY | 166 | Medium |
uk.co.westhawk.snmp.stack.TrapPduv1
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.TrapPduv1.getIpAddress() may expose internal representation by returning TrapPduv1.IpAddress | MALICIOUS_CODE | EI_EXPOSE_REP | 151 | Medium |
| uk.co.westhawk.snmp.stack.TrapPduv1.setIpAddress(byte[]) may expose internal representation by storing an externally mutable object into TrapPduv1.IpAddress | MALICIOUS_CODE | EI_EXPOSE_REP2 | 144 | Medium |
uk.co.westhawk.snmp.stack.TrapPduv2
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.TrapPduv2 at new uk.co.westhawk.snmp.stack.TrapPduv2(SnmpContextBasisFace) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 102 | Medium |
uk.co.westhawk.snmp.stack.usmStatsConstants
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| uk.co.westhawk.snmp.stack.usmStatsConstants.usmStatsOids should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 114 | Medium |
| uk.co.westhawk.snmp.stack.usmStatsConstants.usmStatsStrings should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 127 | Medium |
uk.co.westhawk.snmp.stack.varbind
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class uk.co.westhawk.snmp.stack.varbind at new uk.co.westhawk.snmp.stack.varbind(AsnSequence) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 160 | Medium |
| uk.co.westhawk.snmp.stack.varbind.getOid() may expose internal representation by returning varbind.name | MALICIOUS_CODE | EI_EXPOSE_REP | 187 | Medium |
| uk.co.westhawk.snmp.stack.varbind.getValue() may expose internal representation by returning varbind.value | MALICIOUS_CODE | EI_EXPOSE_REP | 197 | Medium |
| new uk.co.westhawk.snmp.stack.varbind(AsnObjectId, AsnObject) may expose internal representation by storing an externally mutable object into varbind.name | MALICIOUS_CODE | EI_EXPOSE_REP2 | 152 | Medium |
| new uk.co.westhawk.snmp.stack.varbind(AsnObjectId, AsnObject) may expose internal representation by storing an externally mutable object into varbind.value | MALICIOUS_CODE | EI_EXPOSE_REP2 | 153 | Medium |
| The class name uk.co.westhawk.snmp.stack.varbind doesn't start with an upper case letter | BAD_PRACTICE | NM_CLASS_NAMING_CONVENTION | 97-216 | Medium |
uk.co.westhawk.snmp.util.SnmpUtilities
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Random object created and used only once in uk.co.westhawk.snmp.util.SnmpUtilities.getSaltAES() | BAD_PRACTICE | DMI_RANDOM_USED_ONLY_ONCE | 562 | High |
| Random object created and used only once in uk.co.westhawk.snmp.util.SnmpUtilities.getSaltDES(int) | BAD_PRACTICE | DMI_RANDOM_USED_ONLY_ONCE | 529 | High |
| uk.co.westhawk.snmp.util.SnmpUtilities.toHexString(byte[]) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 232 | Medium |
Search Results for {{siteSearch | truncate:'50'}}
{{resultArray.length}}
No results.