DBTAdapter.printDeviceLists(): Avoid ConcurrentModificationException by using a shallow copy of ArrayList

Actually this was the intention here, but due to mixed C++/Java programming I missed this nuance.

In C++ the simple assignment is sufficient, not in Java.

1 files changed, 2 insertions, 1 deletions

diff --git a/java/jau/direct_bt/DBTAdapter.java b/java/jau/direct_bt/DBTAdapter.java
index dc50c80b..09aec7e1 100644
--- a/java/jau/direct_bt/DBTAdapter.java
+++ b/java/jau/direct_bt/DBTAdapter.java
@@ -537,7 +537,8 @@ public class DBTAdapter extends DBTObject implements BTAdapter
         printDeviceListsImpl();
         List<WeakReference<BTDevice>> _discoveredDevices;
         synchronized(discoveredDevicesLock) {
-            _discoveredDevices = discoveredDevices;
+            // Shallow (but expensive) copy to avoid java.util.ConcurrentModificationException while iterating: debug mode only
+            _discoveredDevices = new ArrayList<WeakReference<BTDevice>>(discoveredDevices);
         }
         final int sz = _discoveredDevices.size();
         BTUtils.fprintf_td(System.err, "- BTAdapter::DiscoveredDevicesJ: %d elements%s", sz, System.lineSeparator());