/hg/release/icedtea-web-1.5: Fixed download service

[jvanek at icedtea.classpath.org]

changeset 7dcaaab8ea51 in /hg/release/icedtea-web-1.5
details: http://icedtea.classpath.org/hg/release/icedtea-web-1.5?cmd=changeset;node=7dcaaab8ea51
author: Jiri Vanek <jvanek at redhat.com>
date: Mon Jul 20 13:00:49 2015 +0200

	Fixed download service

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


diffstat:

 ChangeLog                                              |  6 ++++++
 NEWS                                                   |  1 +
 netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java |  5 +++--
 3 files changed, 10 insertions(+), 2 deletions(-)

diffs (39 lines):

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 at 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 at 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)