The following document contains the results of FindBugs Report
FindBugs Version is 1.2.0
Threshold is Low
Effort is Default
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 747 | 219 | 0 | 0 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Write to static field net.sf.bluecove.Configuration.storage from instance method net.sf.bluecove.BlueCoveTestCanvas.BlueCoveTestCanvas() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 110 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Should net.sf.bluecove.BlueCoveTestConfigurationForm$ConfigurationComponent be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sf.bluecove.CommunicationData.byteAray should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 39 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 425 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 367 |
| net.sf.bluecove.CommunicationTester.clientConnectionOpenRetry isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 51 |
| net.sf.bluecove.CommunicationTester.dataOutputStreamFlush isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 49 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to sequenceSize in net.sf.bluecove.CommunicationTesterL2CAP.maxMTUSend(int, ConnectionHolderL2CAP) | STYLE | DLS_DEAD_LOCAL_STORE | 213 |
| Dead store to sequenceSize in net.sf.bluecove.CommunicationTesterL2CAP.sequenceSend(int, ConnectionHolderL2CAP) | STYLE | DLS_DEAD_LOCAL_STORE | 116 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 216 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 233 |
| net.sf.bluecove.Configuration.hasManyDevices isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 192 |
| net.sf.bluecove.Configuration.logTimeStamp isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 196 |
| net.sf.bluecove.Configuration.screenSizeSmall isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 198 |
| net.sf.bluecove.Configuration.storage isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Not available |
| net.sf.bluecove.Configuration.tgSize isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 133 |
| net.sf.bluecove.Configuration.tgSleep isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 131 |
| net.sf.bluecove.Configuration.threadLocalBluetoothStack isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Not available |
| net.sf.bluecove.Configuration.clientContinuousDiscoveryDevices should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 90 |
| net.sf.bluecove.Configuration.isConfigured should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 178 |
| net.sf.bluecove.Configuration.stackWIDCOMM should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 190 |
| net.sf.bluecove.Configuration.supportL2CAP should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 194 |
| net.sf.bluecove.Configuration.STERSS_TEST_CASE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 111 |
| net.sf.bluecove.Configuration.TEST_CASE_FIRST isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 107 |
| net.sf.bluecove.Configuration.TEST_CASE_L2CAP_FIRST isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 115 |
| net.sf.bluecove.Configuration.TEST_CASE_L2CAP_LAST isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 117 |
| net.sf.bluecove.Configuration.TEST_CASE_LAST isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 109 |
| net.sf.bluecove.Configuration.authenticate isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 99 |
| net.sf.bluecove.Configuration.authenticateOBEX isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 125 |
| net.sf.bluecove.Configuration.bluecovepsm isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | Not available |
| net.sf.bluecove.Configuration.clientContinuous isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 86 |
| net.sf.bluecove.Configuration.clientContinuousDiscovery isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 88 |
| net.sf.bluecove.Configuration.deviceClassFilter isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 48 |
| net.sf.bluecove.Configuration.discoverDevicesComputers isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 50 |
| net.sf.bluecove.Configuration.discoverDevicesPhones isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 52 |
| net.sf.bluecove.Configuration.discoveryGetDeviceFriendlyName isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 66 |
| net.sf.bluecove.Configuration.discoveryUUID isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 68 |
| net.sf.bluecove.Configuration.encrypt isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 101 |
| net.sf.bluecove.Configuration.ignoreDevices isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 76 |
| net.sf.bluecove.Configuration.listedDevicesOnly isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 61 |
| net.sf.bluecove.Configuration.serverAcceptWhileConnected isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 80 |
| net.sf.bluecove.Configuration.testAllServiceAttributes isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 129 |
| net.sf.bluecove.Configuration.testDeviceNames isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 74 |
| net.sf.bluecove.Configuration.testIgnoreNotWorkingServiceAttributes isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 142 |
| net.sf.bluecove.Configuration.testL2CAP isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 113 |
| net.sf.bluecove.Configuration.testRFCOMM isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 105 |
| net.sf.bluecove.Configuration.testServerForceDiscoverable isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 144 |
| net.sf.bluecove.Configuration.testServerOBEX_TCP isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 121 |
| net.sf.bluecove.Configuration.testServiceAttributes isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 127 |
| net.sf.bluecove.Configuration.useDevices isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 78 |
| net.sf.bluecove.Configuration.useServiceClassExtUUID isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 72 |
| net.sf.bluecove.Configuration.useShortUUID isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 70 |
| net.sf.bluecove.Configuration.authorize isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 103 |
| net.sf.bluecove.Configuration.canCloseServer isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 176 |
| net.sf.bluecove.Configuration.clientContinuousServicesSearch isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 92 |
| net.sf.bluecove.Configuration.clientSleepBetweenConnections isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 148 |
| net.sf.bluecove.Configuration.clientSleepOnConnectionRetry isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 160 |
| net.sf.bluecove.Configuration.clientSleepOnDeviceInquiryError isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 162 |
| net.sf.bluecove.Configuration.clientTestConnections isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 94 |
| net.sf.bluecove.Configuration.clientTestConnectionsMultipleThreads isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 97 |
| net.sf.bluecove.Configuration.clientTestStopOnErrorCount isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 154 |
| net.sf.bluecove.Configuration.clientTestTimeOutSec isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 152 |
| net.sf.bluecove.Configuration.discoverySearchOnlyBluecoveUuid isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 56 |
| net.sf.bluecove.Configuration.initializeLocalDevice isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 146 |
| net.sf.bluecove.Configuration.linux isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 186 |
| net.sf.bluecove.Configuration.macOSx isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 188 |
| net.sf.bluecove.Configuration.searchOnlyBluecoveUuid isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 54 |
| net.sf.bluecove.Configuration.serverAcceptWhileConnectedOnJavaSE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 82 |
| net.sf.bluecove.Configuration.serverContinuous isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 84 |
| net.sf.bluecove.Configuration.serverMAXTimeSec isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 158 |
| net.sf.bluecove.Configuration.serverSleepB4ClosingConnection isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 150 |
| net.sf.bluecove.Configuration.serverTestTimeOutSec isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 156 |
| net.sf.bluecove.Configuration.testServerOBEXObjectPush isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 123 |
| net.sf.bluecove.Configuration.tgDurationMin isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 135 |
| net.sf.bluecove.Configuration.windows isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 180 |
| net.sf.bluecove.Configuration.windowsCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 184 |
| net.sf.bluecove.Configuration.windowsXP isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 182 |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sf.bluecove.Consts.TEST_SERVICE_ATTRIBUTE_BYTES_VALUE should be moved out of an interface and made package protected | MALICIOUS_CODE | MS_OOI_PKGPROTECT | 86 |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sf.bluecove.OBEXTestAuthenticator.onAuthenticationChallenge(String, boolean, boolean) invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 45 |
| net.sf.bluecove.OBEXTestAuthenticator.onAuthenticationResponse(byte[]) invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 50 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 81 |
| net.sf.bluecove.RemoteDeviceInfo.services isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 41 |
| net.sf.bluecove.RemoteDeviceInfo.devices should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 39 |
| net.sf.bluecove.RemoteDeviceInfo.allServiceSearch isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 59 |
| net.sf.bluecove.RemoteDeviceInfo.deviceInquiryDuration isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 55 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 628 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 333 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Result of integer multiplication cast to long in net.sf.bluecove.Switcher.run() | STYLE | ICAST_INTEGER_MULTIPLY_CAST_TO_LONG | 153 |
| net.sf.bluecove.Switcher.client should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available |
| net.sf.bluecove.Switcher.clientStartCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 51 |
| net.sf.bluecove.Switcher.server should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available |
| net.sf.bluecove.Switcher.serverStartCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 53 |
| Naked notify in net.sf.bluecove.Switcher.shutdown() | MT_CORRECTNESS | NN_NAKED_NOTIFY | 192 |
| Naked notify in net.sf.bluecove.Switcher.yield(TestResponderClient) | MT_CORRECTNESS | NN_NAKED_NOTIFY | 86 |
| Exception is caught when Exception is not thrown in net.sf.bluecove.Switcher.run() | STYLE | REC_CATCH_EXCEPTION | 154 |
| Write to static field net.sf.bluecove.Switcher.instance from instance method net.sf.bluecove.Switcher.Switcher() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 74 |
| Write to static field net.sf.bluecove.Switcher.instance from instance method net.sf.bluecove.Switcher.shutdown() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 194 |
| Unread field: net.sf.bluecove.Switcher.runningServers | PERFORMANCE | URF_UNREAD_FIELD | 49 |
| Unconditional wait in net.sf.bluecove.Switcher.run() | MT_CORRECTNESS | UW_UNCOND_WAIT | 130 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 310 |
| Useless control flow in net.sf.bluecove.TestClientBluetoothInquirer.servicesDiscovered(int, ServiceRecord[]) | STYLE | UCF_USELESS_CONTROL_FLOW | 422 |
| Unconditional wait in net.sf.bluecove.TestClientBluetoothInquirer.runDeviceInquiry() | MT_CORRECTNESS | UW_UNCOND_WAIT | 200 |
| Unconditional wait in net.sf.bluecove.TestClientBluetoothInquirer.startServicesSearch() | MT_CORRECTNESS | UW_UNCOND_WAIT | 370 |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sf.bluecove.TestResponderClient.countSuccess isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 57 |
| net.sf.bluecove.TestResponderClient.connectionCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 63 |
| net.sf.bluecove.TestResponderClient.countConnectionThreads should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 71 |
| net.sf.bluecove.TestResponderClient.discoveryCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 61 |
| net.sf.bluecove.TestResponderClient.discoveryDryCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 65 |
| net.sf.bluecove.TestResponderClient.discoverySuccessCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 67 |
| net.sf.bluecove.TestResponderClient.lastSuccessfulDiscovery should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available |
| net.sf.bluecove.TestResponderClient.concurrentStatistic isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 81 |
| net.sf.bluecove.TestResponderClient.connectionDuration isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 83 |
| net.sf.bluecove.TestResponderClient.connectionRetyStatistic isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 85 |
| net.sf.bluecove.TestResponderClient.failure isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 59 |
| net.sf.bluecove.TestResponderClient.recentDeviceNames isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 101 |
| Naked notify in net.sf.bluecove.TestResponderClient.connectAndTest(String, String, IntVar, IntVar, TestResponderClientConnection) | MT_CORRECTNESS | NN_NAKED_NOTIFY | 203 |
| Write to static field net.sf.bluecove.TestResponderClient.sdAttrRetrievableMax from instance method net.sf.bluecove.TestResponderClient.TestResponderClient(boolean) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 130 |
| Write to static field net.sf.bluecove.Configuration.clientContinuousDiscoveryDevices from instance method net.sf.bluecove.TestResponderClient.run() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 437 |
| Write to static field net.sf.bluecove.TestResponderClient.discoveryDryCount from instance method net.sf.bluecove.TestResponderClient.run() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 487 |
| Write to static field net.sf.bluecove.TestResponderClient.lastSuccessfulDiscovery from instance method net.sf.bluecove.TestResponderClient.run() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 489 |
| Unconditional wait in net.sf.bluecove.TestResponderClient.connectAndTest(String, String, IntVar, IntVar, TestResponderClientConnection) | MT_CORRECTNESS | UW_UNCOND_WAIT | 247 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Redundant nullcheck of message, which is known to be non-null in net.sf.bluecove.TestResponderClientRFCOMM.replySuccess(String, int, TestStatus) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE | 85 |
| TestResponderClientRFCOMM.c not initialized in constructor | STYLE | UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 144 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 135 |
| net.sf.bluecove.TestResponderCommon.receiveMTU_max isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 43 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to ATTR_BROWSE_GRP_LIST in net.sf.bluecove.TestResponderServer.buildServiceRecordPub(ServiceRecord) | STYLE | DLS_DEAD_LOCAL_STORE | 668 |
| Dead store to UUID_PUBLICBROWSE_GROUP in net.sf.bluecove.TestResponderServer.buildServiceRecordPub(ServiceRecord) | STYLE | DLS_DEAD_LOCAL_STORE | 667 |
| net.sf.bluecove.TestResponderServer.countSuccess isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 58 |
| net.sf.bluecove.TestResponderServer.concurrentConnectionsMax should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 68 |
| net.sf.bluecove.TestResponderServer.connectorOpenTime should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 84 |
| net.sf.bluecove.TestResponderServer.countConnection should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 64 |
| net.sf.bluecove.TestResponderServer.countRunningConnections should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 66 |
| net.sf.bluecove.TestResponderServer.discoverableStartTime should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 82 |
| net.sf.bluecove.TestResponderServer.discoverable should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 80 |
| net.sf.bluecove.TestResponderServer.allServerDuration isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 60 |
| net.sf.bluecove.TestResponderServer.concurrentStatistic isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 96 |
| net.sf.bluecove.TestResponderServer.connectionDuration isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 98 |
| net.sf.bluecove.TestResponderServer.failure isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 62 |
| Write to static field net.sf.bluecove.TestResponderServer.connectorOpenTime from instance method net.sf.bluecove.TestResponderServer.run() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 336 |
| Useless control flow in net.sf.bluecove.TestResponderServer.run() | STYLE | UCF_USELESS_CONTROL_FLOW | 380 |
| Unconditional wait in net.sf.bluecove.TestResponderServer.run() | MT_CORRECTNESS | UW_UNCOND_WAIT | 411 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Naked notify in net.sf.bluecove.TestResponderServer$ServerConnectionRunnable.run() | MT_CORRECTNESS | NN_NAKED_NOTIFY | 287 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to ATTR_BROWSE_GRP_LIST in net.sf.bluecove.TestResponderServerOBEX.run() | STYLE | DLS_DEAD_LOCAL_STORE | 141 |
| Dead store to ATTR_SUPPORTED_FORMAT_LIST_LIST in net.sf.bluecove.TestResponderServerOBEX.run() | STYLE | DLS_DEAD_LOCAL_STORE | 134 |
| Dead store to UUID_PUBLICBROWSE_GROUP in net.sf.bluecove.TestResponderServerOBEX.run() | STYLE | DLS_DEAD_LOCAL_STORE | 140 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Should net.sf.bluecove.TestResponderServerOBEX$NoTimeWrapper be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.TestResponderServerOBEX$NoTimeWrapper$1 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Result of integer multiplication cast to long in net.sf.bluecove.TestTimeOutMonitor.run() | STYLE | ICAST_INTEGER_MULTIPLY_CAST_TO_LONG | 77 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Class net.sf.bluecove.awt.ClientConnectionDialog defines non-transient non-serializable instance field monitorTimer | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Class net.sf.bluecove.awt.ClientConnectionDialog defines non-transient non-serializable instance field thread | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Switch statement found in net.sf.bluecove.awt.ClientConnectionDialog.send() where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | 508-510 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Method net.sf.bluecove.awt.ClientConnectionThread.ClientConnectionThread(String) creates a thread using the default empty run method | MT_CORRECTNESS | DM_USELESS_THREAD | 100 |
| Inconsistent synchronization of net.sf.bluecove.awt.ClientConnectionThread.fileOut; locked 72% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 307 |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.ConfigurationDialog$3 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Should net.sf.bluecove.awt.ConfigurationDialog$ConfigurationComponent be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Method net.sf.bluecove.awt.Main.keyPressed(int) uses the same code for two switch clauses | STYLE | DB_DUPLICATE_SWITCH_CLAUSES | 566 |
| net.sf.bluecove.awt.Main.quit() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 639 |
| net.sf.bluecove.awt.Main.Main() invokes java.lang.Thread.start() | MT_CORRECTNESS | SC_START_IN_CTOR | 497 |
| Write to static field net.sf.bluecove.Configuration.hasManyDevices from instance method net.sf.bluecove.awt.Main.Main() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 434 |
| Write to static field net.sf.bluecove.Configuration.screenSizeSmall from instance method net.sf.bluecove.awt.Main.Main() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 516 |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$14 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$18 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$19 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$20 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$26 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$27 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$28 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$29 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$3 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$30 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$31 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$32 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$33 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$34 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$35 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$36 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$4 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$42 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$43 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$6 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.awt.Main$7 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Class net.sf.bluecove.awt.ObexClientConnectionDialog defines non-transient non-serializable instance field monitorTimer | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Class net.sf.bluecove.awt.ObexClientConnectionDialog defines non-transient non-serializable instance field thread | BAD_PRACTICE | SE_BAD_FIELD | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Method net.sf.bluecove.awt.ServiceRecords.populateChoice(Choice, boolean) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 65 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Write to static field net.sf.bluecove.se.BlueCoveLoggerAppender.blueCoveLoggerAppender from instance method net.sf.bluecove.se.BlueCoveLoggerAppender.BlueCoveLoggerAppender(LoggerAppender) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 40 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 55 |
| net.sf.bluecove.se.Console.quit() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 131 |
| net.sf.bluecove.se.Console.readCommand() invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 110 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 104 |
| Possible null pointer dereference of out in net.sf.bluecove.se.FileStorage.storeData(String, String) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 93 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Method net.sf.bluecove.se.Main.printSystemProperties() makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 123 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to clientStart in net.sf.bluecove.swing.Main.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 117 |
| Dead store to clientStop in net.sf.bluecove.swing.Main.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 124 |
| Dead store to serverStart in net.sf.bluecove.swing.Main.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 104 |
| Dead store to serverStop in net.sf.bluecove.swing.Main.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 111 |
| Dead store to stop in net.sf.bluecove.swing.Main.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 183 |
| Dead store to tckStart in net.sf.bluecove.swing.Main.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 132 |
| net.sf.bluecove.swing.Main.quit() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 311 |
| Write to static field net.sf.bluecove.Configuration.screenSizeSmall from instance method net.sf.bluecove.swing.Main.Main() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 250 |
| Useless control flow in net.sf.bluecove.swing.Main.addMenu(JMenu, String, ActionListener, int) | STYLE | UCF_USELESS_CONTROL_FLOW | 278 |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.swing.Main$13 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.swing.Main$17 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.swing.Main$18 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.swing.Main$19 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.swing.Main$3 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.swing.Main$5 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| The class net.sf.bluecove.swing.Main$6 could be refactored into a named _static_ inner class | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC_ANON | Not available |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to sendTime in net.sf.bluecove.tests.L2TrafficGenerator.trafficGeneratorRead(ConnectionHolderL2CAP, byte[], TestStatus) | STYLE | DLS_DEAD_LOCAL_STORE | 267 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 482 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to comma in net.sf.bluecove.util.BluetoothTypesInfo$DeviceClassConsts.toString(DeviceClass) | STYLE | DLS_DEAD_LOCAL_STORE | 1024 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 82 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 180 |
| Use of non-localized String.toUpperCase() or String.toLowerCase | I18N | DM_CONVERT_CASE | 197 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Primitive boxed just to call toString in net.sf.bluecove.util.BooleanVar.toString() | PERFORMANCE | DM_BOXED_PRIMITIVE_TOSTRING | 60 |
| net.sf.bluecove.util.BooleanVar defines equals and uses Object.hashCode() | BAD_PRACTICE | HE_EQUALS_USE_HASHCODE | 52-55 |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sf.bluecove.util.IntVar defines equals and uses Object.hashCode() | BAD_PRACTICE | HE_EQUALS_USE_HASHCODE | 55-58 |
| Bug | Category | Details | Line |
|---|---|---|---|
| Dead store to $L5 in net.sf.bluecove.util.StorageRMS.storeData(String, String) | STYLE | DLS_DEAD_LOCAL_STORE | 68 |
| Bug | Category | Details | Line |
|---|---|---|---|
| net.sf.bluecove.util.StringVar defines equals and uses Object.hashCode() | BAD_PRACTICE | HE_EQUALS_USE_HASHCODE | 47-50 |