Synchronize the loop and break out of it before fetching arbitrary data files. Hopeful fix for ConcurrentModificationException, and maybe a potential deadlock.

src/main/java/org/qortal/controller/arbitrary/ArbitraryDataFileManager.java

@@ -84,6 +84,13 @@ public class ArbitraryDataFileManager extends Thread {
     private void processFileHashes(Long now) {
         try (final Repository repository = RepositoryManager.getRepository()) {
 
+            ArbitraryTransactionData arbitraryTransactionData = null;
+            byte[] signature = null;
+            byte[] hash = null;
+            Peer peer = null;
+            boolean shouldProcess = false;
+
+            synchronized (arbitraryDataFileHashResponses) {
                 for (String hash58 : arbitraryDataFileHashResponses.keySet()) {
                     if (isStopping) {
                         return;
@@ -91,7 +98,7 @@ public class ArbitraryDataFileManager extends Thread {
 
                     Triple<Peer, String, Long> value = arbitraryDataFileHashResponses.get(hash58);
                     if (value != null) {
-                    Peer peer = value.getA();
+                        peer = value.getA();
                         String signature58 = value.getB();
                         Long timestamp = value.getC();
 
@@ -100,20 +107,35 @@ public class ArbitraryDataFileManager extends Thread {
                             continue;
                         }
 
-                    byte[] hash = Base58.decode(hash58);
+                        hash = Base58.decode(hash58);
-                    byte[] signature = Base58.decode(signature58);
+                        signature = Base58.decode(signature58);
 
                         // Fetch the transaction data
-                    ArbitraryTransactionData arbitraryTransactionData = ArbitraryTransactionUtils.fetchTransactionData(repository, signature);
+                        arbitraryTransactionData = ArbitraryTransactionUtils.fetchTransactionData(repository, signature);
                         if (arbitraryTransactionData == null) {
                             continue;
                         }
 
-                    LOGGER.debug("Fetching file {} from peer {} via response queue...", hash58, peer);
+                        // We want to process this file
-                    this.fetchArbitraryDataFiles(repository, peer, signature, arbitraryTransactionData, Arrays.asList(hash));
+                        shouldProcess = true;
+                        break;
+                    }
+                }
+            }
+
+            if (!shouldProcess) {
+                // Nothing to do
+                return;
             }
+
+            if (signature == null || hash == null || peer == null || arbitraryTransactionData == null) {
+                return;
             }
 
+            String hash58 = Base58.encode(hash);
+            LOGGER.debug("Fetching file {} from peer {} via response queue...", hash58, peer);
+            this.fetchArbitraryDataFiles(repository, peer, signature, arbitraryTransactionData, Arrays.asList(hash));
+
         } catch (DataException e) {
             LOGGER.info("Unable to process file hashes: {}", e.getMessage());
         }