# HG changeset patch
# User Jiri Vanek <jvanek@redhat.com>
# Date 1437390049 -7200
# Node ID 7dcaaab8ea51667f8fa6f93536e133099d2407ee
# Parent  1204ef8dde15d2b395b34250337f2d58eb36e8fe
Fixed download service

* netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java: (fillInPartJars) for-each loop replaced by indexed loop to prevent ConcurrentModificationException

diff -r 1204ef8dde15 -r 7dcaaab8ea51 ChangeLog
--- a/ChangeLog	Thu Jun 25 12:17:56 2015 +0200
+++ b/ChangeLog	Mon Jul 20 13:00:49 2015 +0200
@@ -1,3 +1,9 @@
+2015-07-20  Jiri Vanek  <jvanek@redhat.com>
+
+	Fixed download service
+	* netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java: (fillInPartJars)
+	for-each loop replaced by indexed loop to prevent ConcurrentModificationException
+
 2015-06-25  Jiri Vanek  <jvanek@redhat.com>
 
 	Fixed to short buttons for localized text - RH1231441
diff -r 1204ef8dde15 -r 7dcaaab8ea51 NEWS
--- a/NEWS	Thu Jun 25 12:17:56 2015 +0200
+++ b/NEWS	Mon Jul 20 13:00:49 2015 +0200
@@ -9,6 +9,7 @@
 CVE-XXXX-YYYY: http://www.cve.mitre.org/cgi-bin/cvename.cgi?name=XXXX-YYYY
 
 New in release 1.5.3 (YYYY-MM-DD):
+* fixed DownloadService
 * RH1231441 Unable to read the text of the buttons of the security dialogue
 
 New in release 1.5.2 (2014-11-26):
diff -r 1204ef8dde15 -r 7dcaaab8ea51 netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
--- a/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java	Thu Jun 25 12:17:56 2015 +0200
+++ b/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java	Mon Jul 20 13:00:49 2015 +0200
@@ -1185,8 +1185,9 @@
      * in the same part).
      */
     protected void fillInPartJars(List<JARDesc> jars) {
-        for (JARDesc desc : jars) {
-            String part = desc.getPart();
+        //can not use iterator, will rise ConcurrentModificationException on jars.add(jar);
+        for (int x = 0 ; x< jars.size() ; x++) {
+            String part = jars.get(x).getPart();
 
             // "available" field can be affected by two different threads
             // working in loadClass(String)