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 |
|---|---|---|---|
| 49 | 37 | 0 | 0 |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| com.intel.bluetooth.BluetoothEmulator.isRemoteDeviceAuthenticated(long) has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 239 | Medium |
| com.intel.bluetooth.BluetoothEmulator.isRemoteDeviceTrusted(long) has Boolean return type and returns explicit null | BAD_PRACTICE | NP_BOOLEAN_RETURN_NULL | 235 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| com.intel.bluetooth.EmulatorCommandReceiver.execute(DeviceCommand) invokes System.exit(...), which shuts down the entire virtual machine | BAD_PRACTICE | DM_EXIT | 84 | Medium |
| Can't close out since it is always null in com.intel.bluetooth.EmulatorCommandReceiver.threadDump(boolean) | CORRECTNESS | NP_CLOSING_NULL | 126 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Incorrect lazy initialization of static field com.intel.bluetooth.EmulatorDeviceInquiry.rnd in com.intel.bluetooth.EmulatorDeviceInquiry.randomWait(long, int) | MT_CORRECTNESS | LI_LAZY_INIT_STATIC | 147-148 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method com.intel.bluetooth.EmulatorLocalDevice.createL2CAPClient(long) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 264 | Medium |
| Method com.intel.bluetooth.EmulatorLocalDevice.createL2CAPService(int) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 243 | Medium |
| Method com.intel.bluetooth.EmulatorLocalDevice.createRFCOMMClient(long) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 233 | Medium |
| Method com.intel.bluetooth.EmulatorLocalDevice.createRFCOMMService() invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 213 | Medium |
| Method com.intel.bluetooth.EmulatorLocalDevice.getConnection(long) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 182 | Medium |
| Method com.intel.bluetooth.EmulatorLocalDevice.removeConnection(EmulatorConnection) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 190 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of com.intel.bluetooth.emu.ConnectedInputStream.available; locked 93% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 147 | Medium |
| Inconsistent synchronization of com.intel.bluetooth.emu.ConnectedInputStream.receiverClosed; locked 75% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 151 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| com.intel.bluetooth.emu.Device.setDevicePower(boolean) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 208 | Medium |
| Method com.intel.bluetooth.emu.Device.addConnectionBuffer(long, ConnectionBuffer) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 171 | Medium |
| Method com.intel.bluetooth.emu.Device.closeConnection(long) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 195 | Medium |
| Method com.intel.bluetooth.emu.Device.getConnectedTo() invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 86 | Medium |
| Method com.intel.bluetooth.emu.Device.getConnectionBuffer(long) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 176 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| com.intel.bluetooth.emu.DeviceCommand.getParameters() may expose internal representation by returning DeviceCommand.parameters | MALICIOUS_CODE | EI_EXPOSE_REP | 62 | Medium |
| com.intel.bluetooth.emu.DeviceCommand.setParameters(Object[]) may expose internal representation by storing an externally mutable object into DeviceCommand.parameters | MALICIOUS_CODE | EI_EXPOSE_REP2 | 66 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method com.intel.bluetooth.emu.DeviceManagerServiceImpl.createNewDevice(String, String) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 88 | Medium |
| Method com.intel.bluetooth.emu.DeviceManagerServiceImpl.getDevice(long) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 116 | Medium |
| Method com.intel.bluetooth.emu.DeviceManagerServiceImpl.releaseDevice(long) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 100 | Medium |
| Result of integer multiplication cast to long in com.intel.bluetooth.emu.DeviceManagerServiceImpl.isDiscoverable(DeviceDescriptor) | STYLE | ICAST_INTEGER_MULTIPLY_CAST_TO_LONG | 201 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method com.intel.bluetooth.emu.DeviceSDP.getServicesDescriptor(long) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 69 | Medium |
| Method com.intel.bluetooth.emu.DeviceSDP.removeServiceRecord(long) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 62 | Medium |
| Method com.intel.bluetooth.emu.DeviceSDP.updateServiceRecord(long, ServicesDescriptor) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 49 | Medium |
| Bug | Category | Details | Line | Priority |
|---|
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Result of integer multiplication cast to long in com.intel.bluetooth.emu.EmulatorUtils.getNextAvailable(Long[], long, int) | STYLE | ICAST_INTEGER_MULTIPLY_CAST_TO_LONG | 54 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| com.intel.bluetooth.emu.MonitorDevice.getConnectedTo() may expose internal representation by returning MonitorDevice.connectedTo | MALICIOUS_CODE | EI_EXPOSE_REP | 82 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unconditional wait in com.intel.bluetooth.emu.ServiceListener.accept(Device, boolean, boolean, int) | MT_CORRECTNESS | UW_UNCOND_WAIT | 94 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| com.intel.bluetooth.emu.ServicesDescriptor.getSdpBinary() may expose internal representation by returning ServicesDescriptor.sdpBinary | MALICIOUS_CODE | EI_EXPOSE_REP | 55 | Medium |
| com.intel.bluetooth.emu.ServicesDescriptor.getUuidSet() may expose internal representation by returning ServicesDescriptor.uuidSet | MALICIOUS_CODE | EI_EXPOSE_REP | 51 | Medium |
| new com.intel.bluetooth.emu.ServicesDescriptor(String[], byte[], int) may expose internal representation by storing an externally mutable object into ServicesDescriptor.sdpBinary | MALICIOUS_CODE | EI_EXPOSE_REP2 | 46 | Medium |
| new com.intel.bluetooth.emu.ServicesDescriptor(String[], byte[], int) may expose internal representation by storing an externally mutable object into ServicesDescriptor.uuidSet | MALICIOUS_CODE | EI_EXPOSE_REP2 | 45 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| instanceof will always return false in com.intel.bluetooth.rmi.Server.startRMIService(), since a com.intel.bluetooth.rmi.RemoteServiceImpl can't be a java.rmi.server.UnicastRemoteObject | CORRECTNESS | BC_IMPOSSIBLE_INSTANCEOF | 112 | Medium |
| Incorrect lazy initialization and update of static field com.intel.bluetooth.rmi.Server.server in com.intel.bluetooth.rmi.Server.start(String) | MT_CORRECTNESS | LI_LAZY_INIT_UPDATE_STATIC | 78-81 | Medium |