The following document contains the results of FindBugs Report
FindBugs Version is 1.3.9
Threshold is medium
Effort is min
| Classes | Bugs | Errors | Missing Classes |
|---|---|---|---|
| 193 | 149 | 0 | 0 |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field net.sf.bluecove.Configuration.storage from instance method new net.sf.bluecove.BlueCoveTestCanvas() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 110 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should net.sf.bluecove.BlueCoveTestConfigurationForm$ConfigurationComponent be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 55 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.CommunicationData.byteAray should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 39 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.CommunicationTester.clientConnectionOpenRetry isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 51 | Medium |
| net.sf.bluecove.CommunicationTester.dataOutputStreamFlush isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 49 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.Configuration.hasManyDevices isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 192 | Medium |
| net.sf.bluecove.Configuration.logTimeStamp isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 196 | Medium |
| net.sf.bluecove.Configuration.screenSizeSmall isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 198 | Medium |
| net.sf.bluecove.Configuration.storage isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Not available | Medium |
| net.sf.bluecove.Configuration.tgSize isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 133 | Medium |
| net.sf.bluecove.Configuration.tgSleep isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 131 | Medium |
| net.sf.bluecove.Configuration.threadLocalBluetoothStack isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | Not available | Medium |
| net.sf.bluecove.Configuration.clientContinuousDiscoveryDevices should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 90 | Medium |
| net.sf.bluecove.Configuration.isConfigured should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 178 | Medium |
| net.sf.bluecove.Configuration.stackWIDCOMM should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 190 | Medium |
| net.sf.bluecove.Configuration.supportL2CAP should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 194 | Medium |
| net.sf.bluecove.Configuration.STERSS_TEST_CASE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 111 | High |
| net.sf.bluecove.Configuration.TEST_CASE_FIRST isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 107 | High |
| net.sf.bluecove.Configuration.TEST_CASE_L2CAP_FIRST isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 115 | High |
| net.sf.bluecove.Configuration.TEST_CASE_L2CAP_LAST isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 117 | High |
| net.sf.bluecove.Configuration.TEST_CASE_LAST isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 109 | High |
| net.sf.bluecove.Configuration.authenticate isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 99 | High |
| net.sf.bluecove.Configuration.authenticateOBEX isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 125 | High |
| net.sf.bluecove.Configuration.bluecovepsm isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | Not available | High |
| net.sf.bluecove.Configuration.clientContinuous isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 86 | High |
| net.sf.bluecove.Configuration.clientContinuousDiscovery isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 88 | High |
| net.sf.bluecove.Configuration.deviceClassFilter isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 48 | High |
| net.sf.bluecove.Configuration.discoverDevicesComputers isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 50 | High |
| net.sf.bluecove.Configuration.discoverDevicesPhones isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 52 | High |
| net.sf.bluecove.Configuration.discoveryGetDeviceFriendlyName isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 66 | High |
| net.sf.bluecove.Configuration.discoveryUUID isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 68 | High |
| net.sf.bluecove.Configuration.encrypt isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 101 | High |
| net.sf.bluecove.Configuration.ignoreDevices isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 76 | High |
| net.sf.bluecove.Configuration.listedDevicesOnly isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 61 | High |
| net.sf.bluecove.Configuration.serverAcceptWhileConnected isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 80 | High |
| net.sf.bluecove.Configuration.testAllServiceAttributes isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 129 | High |
| net.sf.bluecove.Configuration.testDeviceNames isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 74 | High |
| net.sf.bluecove.Configuration.testIgnoreNotWorkingServiceAttributes isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 142 | High |
| net.sf.bluecove.Configuration.testL2CAP isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 113 | High |
| net.sf.bluecove.Configuration.testRFCOMM isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 105 | High |
| net.sf.bluecove.Configuration.testServerForceDiscoverable isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 144 | High |
| net.sf.bluecove.Configuration.testServerOBEX_TCP isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 121 | High |
| net.sf.bluecove.Configuration.testServiceAttributes isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 127 | High |
| net.sf.bluecove.Configuration.useDevices isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 78 | High |
| net.sf.bluecove.Configuration.useServiceClassExtUUID isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 72 | High |
| net.sf.bluecove.Configuration.useShortUUID isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 70 | High |
| net.sf.bluecove.Configuration.authorize isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 103 | Medium |
| net.sf.bluecove.Configuration.canCloseServer isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 176 | Medium |
| net.sf.bluecove.Configuration.clientContinuousServicesSearch isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 92 | Medium |
| net.sf.bluecove.Configuration.clientSleepBetweenConnections isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 148 | Medium |
| net.sf.bluecove.Configuration.clientSleepOnConnectionRetry isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 160 | Medium |
| net.sf.bluecove.Configuration.clientSleepOnDeviceInquiryError isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 162 | Medium |
| net.sf.bluecove.Configuration.clientTestConnections isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 94 | Medium |
| net.sf.bluecove.Configuration.clientTestConnectionsMultipleThreads isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 97 | Medium |
| net.sf.bluecove.Configuration.clientTestStopOnErrorCount isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 154 | Medium |
| net.sf.bluecove.Configuration.clientTestTimeOutSec isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 152 | Medium |
| net.sf.bluecove.Configuration.discoverySearchOnlyBluecoveUuid isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 56 | Medium |
| net.sf.bluecove.Configuration.initializeLocalDevice isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 146 | Medium |
| net.sf.bluecove.Configuration.linux isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 186 | Medium |
| net.sf.bluecove.Configuration.macOSx isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 188 | Medium |
| net.sf.bluecove.Configuration.searchOnlyBluecoveUuid isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 54 | Medium |
| net.sf.bluecove.Configuration.serverAcceptWhileConnectedOnJavaSE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 82 | Medium |
| net.sf.bluecove.Configuration.serverContinuous isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 84 | Medium |
| net.sf.bluecove.Configuration.serverMAXTimeSec isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 158 | Medium |
| net.sf.bluecove.Configuration.serverSleepB4ClosingConnection isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 150 | Medium |
| net.sf.bluecove.Configuration.serverTestTimeOutSec isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 156 | Medium |
| net.sf.bluecove.Configuration.testServerOBEXObjectPush isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 123 | Medium |
| net.sf.bluecove.Configuration.tgDurationMin isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 135 | Medium |
| net.sf.bluecove.Configuration.windows isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 180 | Medium |
| net.sf.bluecove.Configuration.windowsCE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 184 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| 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 | 87 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.OBEXTestAuthenticator.onAuthenticationChallenge(String, boolean, boolean) invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 45 | Medium |
| net.sf.bluecove.OBEXTestAuthenticator.onAuthenticationResponse(byte[]) invokes inefficient new String(String) constructor | PERFORMANCE | DM_STRING_CTOR | 50 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.RemoteDeviceInfo.services isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 41 | Medium |
| net.sf.bluecove.RemoteDeviceInfo.devices should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 39 | Medium |
| net.sf.bluecove.RemoteDeviceInfo.allServiceSearch isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 59 | High |
| net.sf.bluecove.RemoteDeviceInfo.deviceInquiryDuration isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 55 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Incorrect lazy initialization of static field net.sf.bluecove.Switcher.client in net.sf.bluecove.Switcher.clientShutdown() | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 444-446 | Medium |
| Incorrect lazy initialization of static field net.sf.bluecove.Switcher.client in net.sf.bluecove.Switcher.createClient(boolean) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 268-270 | Medium |
| Incorrect lazy initialization of static field net.sf.bluecove.Switcher.server in net.sf.bluecove.Switcher.serverShutdownForce() | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 504-506 | Medium |
| Incorrect lazy initialization of static field net.sf.bluecove.Switcher.server in net.sf.bluecove.Switcher.startServer() | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 457-458 | Medium |
| net.sf.bluecove.Switcher.client should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available | Medium |
| net.sf.bluecove.Switcher.clientStartCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 51 | Medium |
| net.sf.bluecove.Switcher.server should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available | Medium |
| net.sf.bluecove.Switcher.serverStartCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 53 | Medium |
| Naked notify in net.sf.bluecove.Switcher.shutdown() | MT_CORRECTNESS | NN_NAKED_NOTIFY | 192 | Medium |
| Naked notify in net.sf.bluecove.Switcher.yield(TestResponderClient) | MT_CORRECTNESS | NN_NAKED_NOTIFY | 86 | Medium |
| Write to static field net.sf.bluecove.Switcher.instance from instance method new net.sf.bluecove.Switcher() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 74 | High |
| 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 | Medium |
| Unconditional wait in net.sf.bluecove.Switcher.run() | MT_CORRECTNESS | UW_UNCOND_WAIT | 130 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unconditional wait in net.sf.bluecove.TestClientBluetoothInquirer.runDeviceInquiry() | MT_CORRECTNESS | UW_UNCOND_WAIT | 200 | Medium |
| Unconditional wait in net.sf.bluecove.TestClientBluetoothInquirer.startServicesSearch() | MT_CORRECTNESS | UW_UNCOND_WAIT | 370 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.TestResponderClient.countSuccess isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 57 | Medium |
| net.sf.bluecove.TestResponderClient.connectionCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 63 | Medium |
| net.sf.bluecove.TestResponderClient.countConnectionThreads should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 71 | Medium |
| net.sf.bluecove.TestResponderClient.discoveryCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 61 | Medium |
| net.sf.bluecove.TestResponderClient.discoveryDryCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 65 | Medium |
| net.sf.bluecove.TestResponderClient.discoverySuccessCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 67 | Medium |
| net.sf.bluecove.TestResponderClient.lastSuccessfulDiscovery should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | Not available | Medium |
| net.sf.bluecove.TestResponderClient.concurrentStatistic isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 81 | High |
| net.sf.bluecove.TestResponderClient.connectionDuration isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 83 | High |
| net.sf.bluecove.TestResponderClient.connectionRetyStatistic isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 85 | High |
| net.sf.bluecove.TestResponderClient.failure isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 59 | High |
| net.sf.bluecove.TestResponderClient.recentDeviceNames isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 101 | High |
| Naked notify in net.sf.bluecove.TestResponderClient.connectAndTest(String, String, IntVar, IntVar, TestResponderClientConnection) | MT_CORRECTNESS | NN_NAKED_NOTIFY | 203 | Medium |
| Write to static field net.sf.bluecove.TestResponderClient.sdAttrRetrievableMax from instance method new net.sf.bluecove.TestResponderClient(boolean) | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 130 | Medium |
| 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 | Medium |
| 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 | Medium |
| 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 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.TestResponderCommon.receiveMTU_max isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 43 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.TestResponderServer.countSuccess isn't final and can't be protected from malicious code | MALICIOUS_CODE | MS_CANNOT_BE_FINAL | 60 | Medium |
| net.sf.bluecove.TestResponderServer.concurrentConnectionsMax should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 70 | Medium |
| net.sf.bluecove.TestResponderServer.connectorOpenTime should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 86 | Medium |
| net.sf.bluecove.TestResponderServer.countConnection should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 66 | Medium |
| net.sf.bluecove.TestResponderServer.countRunningConnections should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 68 | Medium |
| net.sf.bluecove.TestResponderServer.discoverableStartTime should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 84 | Medium |
| net.sf.bluecove.TestResponderServer.updateVariableCount should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 102 | Medium |
| net.sf.bluecove.TestResponderServer.allServerDuration isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 62 | High |
| net.sf.bluecove.TestResponderServer.concurrentStatistic isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 98 | High |
| net.sf.bluecove.TestResponderServer.connectionDuration isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 100 | High |
| net.sf.bluecove.TestResponderServer.failure isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 64 | High |
| 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 | 340 | Medium |
| Unconditional wait in net.sf.bluecove.TestResponderServer.run() | MT_CORRECTNESS | UW_UNCOND_WAIT | 415 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Naked notify in net.sf.bluecove.TestResponderServer$ServerConnectionRunnable.run() | MT_CORRECTNESS | NN_NAKED_NOTIFY | 291 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should net.sf.bluecove.TestResponderServerOBEX$NoTimeWrapper be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 260-282 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of net.sf.bluecove.awt.ClientConnectionThread.fileOut; locked 61% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 325 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should net.sf.bluecove.awt.ConfigurationDialog$ConfigurationComponent be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 74 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.awt.Main.quit() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 671 | Medium |
| new net.sf.bluecove.awt.Main() invokes Thread.start() | MT_CORRECTNESS | SC_START_IN_CTOR | 528 | Medium |
| Write to static field net.sf.bluecove.Configuration.hasManyDevices from instance method new net.sf.bluecove.awt.Main() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 465 | Medium |
| Write to static field net.sf.bluecove.Configuration.screenSizeSmall from instance method new net.sf.bluecove.awt.Main() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 548 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method net.sf.bluecove.awt.ServiceRecords.populateChoice(Choice, boolean) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 65 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| 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 | High |
| net.sf.bluecove.se.FileStorage.homePath() ignores exceptional return value of java.io.File.delete() | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 129 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method net.sf.bluecove.se.JavaSECommon.logSystemProperties() makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 161 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method net.sf.bluecove.se.Main.printSystemProperties() makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 136 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to clientStart in new net.sf.bluecove.swing.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 117 | Medium |
| Dead store to clientStop in new net.sf.bluecove.swing.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 124 | Medium |
| Dead store to serverStart in new net.sf.bluecove.swing.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 104 | Medium |
| Dead store to serverStop in new net.sf.bluecove.swing.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 111 | Medium |
| Dead store to stop in new net.sf.bluecove.swing.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 183 | Medium |
| Dead store to tckStart in new net.sf.bluecove.swing.Main() | STYLE | DLS_DEAD_LOCAL_STORE | 132 | Medium |
| net.sf.bluecove.swing.Main.quit() invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 311 | Medium |
| Write to static field net.sf.bluecove.Configuration.screenSizeSmall from instance method new net.sf.bluecove.swing.Main() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 250 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.util.BooleanVar defines equals and uses Object.hashCode() | BAD_PRACTICE | HE_EQUALS_USE_HASHCODE | 52-55 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.util.IntVar defines equals and uses Object.hashCode() | BAD_PRACTICE | HE_EQUALS_USE_HASHCODE | 55-58 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to recordId in net.sf.bluecove.util.StorageRMS.storeData(String, String) | STYLE | DLS_DEAD_LOCAL_STORE | 68 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| net.sf.bluecove.util.StringVar.equals(Object) checks for operand being a String | BAD_PRACTICE | EQ_CHECK_FOR_OPERAND_NOT_COMPATIBLE_WITH_THIS | 50 | High |
| net.sf.bluecove.util.StringVar defines equals and uses Object.hashCode() | BAD_PRACTICE | HE_EQUALS_USE_HASHCODE | 47-50 | High |