http://blog.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel hourly 1 1901-01-01T00:00+00:00 http://gmane.org/img/gmane-25t.png http://gmane.org http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18709 <pre>http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=1012 Jon VanAlten &lt;jon.vanalten-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Component|Thermostat |IcedTea Version|unspecified |6-1.10.1 Assignee|jon.vanalten-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org |unassigned-HMOWzH9t1bryS2n3IP37Vg&lt; at &gt;public.gmane.org | |h.org Product|Thermostat |IcedTea Target Milestone|--- |6-1.10.1 </pre> bugzilla-daemon-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E< at >public.gmane.org 2012-05-24T16:07:21 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18708 <pre> "This is not the classloader you are looking for" :) The code is at (line 1160) http://java-game-lib.svn.sourceforge.net/viewvc/java-game-lib/trunk/LWJGL/src/java/org/lwjgl/util/applet/AppletLoader.java?revision=3635&amp;view=markup#1160 The code uses a custom classloader (which does know about the urls) to load 3rd-party jars. The custom classloader delegates to the parent classloader to verify permissions and check if the 3rd-party jar is signed with a good certificate or not. The custom classloader relies on the parent classloader to grant the code the appropriate permissions (minimial permissions if the jar is unsigned, or full permissions if the jar is signed/trusted). Omair </pre> Omair Majid 2012-05-24T15:58:20 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18707 <pre>http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=1010 Deepak Bhole &lt;dbhole-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|dbhole-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org |omajid-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org </pre> bugzilla-daemon-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E< at >public.gmane.org 2012-05-24T15:52:47 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18706 <pre>* Jiri Vanek &lt;jvanek-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; [2012-05-24 09:40]: Hi Jiri, This is still not solving the root issue -- why does the resource not have an entry in the map? Whatever is getting that resource should be adding it. getCodeSourceSecurity should only be doing a simple look up on security and returning what is already known. Cheers, Deepak </pre> Deepak Bhole 2012-05-24T15:28:47 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18705 <pre>http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=1011 Deepak Bhole &lt;dbhole-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|dbhole-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org |adomurad-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org </pre> bugzilla-daemon-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E< at >public.gmane.org 2012-05-24T15:16:37 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18704 <pre>Hey all. This is a response to comments from Pavel on a patch that I created earlier. I decided to re-post with [rfc] in the name to be clear. This addresses the issue of magic numbers being used for the buffer sizes, and the somewhat unclear 'sizeof' usage (that I originally used) to get the buffer size. As well it avoids using snprintf for emptying a string, opting for the somewhat clearer strcpy. Changelog: 2012-05-23 Adam Domurad &lt;adomurad-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; Removed instances of snprintf where buffer size was not known. Added buffer size constant for allocating buffers for numeric conversions. * plugin/icedteanp/IcedTeaNPPlugin.cc: Removed usage of snprintf for simple blanking of strings. Buffer size was misguided previously. Used NUM_STR_BUFFER_SIZE constant to replace magic numbers. * plugin/icedteanp/IcedTeaPluginUtils.cc: Made NPVariantToString(NPVariant variant, std::string* result) use space indentation. Used NUM_STR_BUFFER_SIZE constant to replace magic numbers. * plugin/icedteanp/IcedTeaPluginUtils.h: Added constant, NUM_STR_BUFFER_SIZE. </pre> Adam Domurad 2012-05-24T14:59:18 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18703 <pre>changeset 8d326a855021 in /hg/MauveTestCoverage details: http://icedtea.classpath.org/hg/MauveTestCoverage?cmd=changeset;node=8d326a855021 author: Pavel Tisnovsky &lt;ptisnovs-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; date: Thu May 24 16:55:09 2012 +0200 * src/ReportGenerator.java: Fixed an exception occuring when class list contains an empty line (usually at the end of file). diffstat: ChangeLog | 7 +++++++ src/ReportGenerator.java | 10 +++++++++- 2 files changed, 16 insertions(+), 1 deletions(-) diffs (34 lines): diff -r b6c8372f5723 -r 8d326a855021 ChangeLog --- a/ChangeLogWed May 16 10:59:45 2012 +0200 +++ b/ChangeLogThu May 24 16:55:09 2012 +0200 &lt; at &gt;&lt; at &gt; -1,3 +1,10 &lt; at &gt;&lt; at &gt; +2012-05-24 Pavel Tisnovsky &lt;ptisnovs-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; + +* src/ReportGenerator.java: +Fixed an exception occuring when class list +contains an empty line (usually at the end +of file). + 2012-05-16 Pavel Tisnovsky &lt;ptisnovs-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; * src/ClassInfo.java: diff -r b6c8372f5723 -r 8d326a855021 src/ReportGenerator.java --- a/src/ReportGenerator.javaWed May 16 10:59:45 2012 +0200 +++ b/src/ReportGenerator.javaThu May 24 16:55:09 2012 +0200 &lt; at &gt;&lt; at &gt; -71,7 +71,15 &lt; at &gt;&lt; at &gt; // set of classes should be sorted Set&lt;String&gt; allClasses = new TreeSet&lt;String&gt;(); // add all lines read from a file to a set (and sort them) - allClasses.addAll(fileContent); + // we have to exclude all empty lines + for (String className : fileContent) + { + // add only non empty lines + if (className != null &amp;&amp; !className.trim().isEmpty()) + { + allClasses.add(className.trim()); + } + } return allClasses; } </pre> ptisnovs-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E< at >public.gmane.org 2012-05-24T14:52:36 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18702 <pre>I hope sincerely I properly backed-out and re-applied. Everything looks good when I do hg diff -r392:f6eddd071004 to me. Sorry for the mess! At least I learned some stuff about mercurial in the process. On Thu, 2012-05-24 at 14:34 +0000, adomurad-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E&lt; at &gt;public.gmane.org wrote: </pre> Adam Domurad 2012-05-24T14:40:34 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18701 <pre>changeset f19299dd8144 in /hg/icedtea-web details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=f19299dd8144 author: Adam Domurad &lt;adomurad-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; date: Thu May 24 10:21:14 2012 -0400 Backed out changeset 43a78616b1dd changeset bb68413d52fb in /hg/icedtea-web details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=bb68413d52fb author: Adam Domurad &lt;adomurad-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; date: Thu May 24 10:28:47 2012 -0400 Finished commit rollbacks. changeset 2c4154754aef in /hg/icedtea-web details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=2c4154754aef author: Adam Domurad &lt;adomurad-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; date: Thu May 24 10:31:44 2012 -0400 Reapplied the patches, sorry for the mess! diffstat: ChangeLog | 58 +++++++++++++++++++++++++++++++++++++++++++++++++--------- 1 files changed, 49 insertions(+), 9 deletions(-) diffs (75 lines): diff -r 43a78616b1dd -r 2c4154754aef ChangeLog --- a/ChangeLogThu May 24 09:12:38 2012 -0400 +++ b/ChangeLogThu May 24 10:31:44 2012 -0400 &lt; at &gt;&lt; at &gt; -2,6 +2,46 &lt; at &gt;&lt; at &gt; * plugin/icedteanp/IcedTeaPluginUtils.cc: Fix two typos. +2012-05-23 Deepak Bhole &lt;dbhole-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; + +* AUTHORS: Added Martin Olsson to list. + +2012-05-23 Martin Olsson &lt;martin-Gwph8TgvYLDLoDKTGw+V6w&lt; at &gt;public.gmane.org&gt; + +* plugin/icedteanp/IcedTeaNPPlugin.cc: +Use g_mutex_free instead of g_free to free appletviewer_mutex (fixes +crash). + +2012-05-23 Deepak Bhole &lt;dbhole-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; + +* ChangeLog: Converted spaces to tabs in an older entry + +2012-05-23 Jiri Vanek &lt;jvanek-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; + +* netx/net/sourceforge/jnlp/resources/Messages.properties: fixed error +in PBadNonrelativeUrl + +2012-05-23 Jiri Vanek &lt;jvanek-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; + +Added more debugging outputs +* netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java: +(getCodeSourceSecurity): added output message when no SecurityDesc is found +for some url/resource +* netx/net/sourceforge/jnlp/resources/Messages.properties: added LNoSecInstance +and LCertFoundIn values +* netx/net/sourceforge/jnlp/security/KeyStores.java: (getPathToKeystore): +new method, able to search for file used for creating of KeyStore if possible +* netx/net/sourceforge/jnlp/security/CertificateUtils.java: (inKeyStores) +using getPathToKeystore for debug output + +2012-05-23 Jiri Vanek &lt;jvanek-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; + +* netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java: (getPermissions): +Any exception from this method is consumed somewhere. I have cough exception, +reprint it in debug mode and re-throw (to be lost). Main condition in this +method had several possible NullPointer exceptions. Separated and thrown before +this condition. + 2012-05-23 Jiri Vanek &lt;jvanek-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; Enhanced about dialog &lt; at &gt;&lt; at &gt; -22,15 +62,15 &lt; at &gt;&lt; at &gt; 2012-05-22 Adam Domurad &lt;adomurad-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; - Changed allocation of small, fixed-size buffers to stack-based - allocations. Changed occurences of sprintf to the safer function - snprintf, added buffer information. While unlikely to change - functionality, snprintf adds an extra check to prevent buffer - overflows. - * plugin/icedteanp/IcedTeaNPPlugin.cc: Allocation of small buffers - using malloc changed to stack allocation &amp; changed sprintf calls to - buffer-size aware snprintf calls. - * plugin/icedteanp/IcedTeaPluginUtils.cc: Same as above. +Changed allocation of small, fixed-size buffers to stack-based +allocations. Changed occurences of sprintf to the safer function +snprintf, added buffer information. While unlikely to change +functionality, snprintf adds an extra check to prevent buffer +overflows. +* plugin/icedteanp/IcedTeaNPPlugin.cc: Allocation of small buffers +using malloc changed to stack allocation &amp; changed sprintf calls to +buffer-size aware snprintf calls. +* plugin/icedteanp/IcedTeaPluginUtils.cc: Same as above. 2012-05-22 Jiri Vanek &lt;jvanek-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; </pre> adomurad-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E< at >public.gmane.org 2012-05-24T14:34:28 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18700 <pre> I elaborated little bit above sources of jmol (http://jmol.sourceforge.net/demo/atoms/) and I have NOT FOUND where they are injecting theirs jars :-/ So I have prepared testing fix (the WRONG one) which was checkigg if somebody is using addURL by reflection (which I consider as best way to do and I have used in my reproducer - (http://mail.openjdk.java.net/pipermail/distro-pkg-dev/2012-May/018357.html). However it appeared that not jmol nor geogebra are using this approach. Today I have traversed through jmol code and have not find how tehy are getting the sources in:-/ They are using for preloading Class.forName in few first lines of app, but in thsi time there IS already requested jar which is not declared on classpath of applet (well.. object generated byjavascript :-/) tag. But it can be anything. Even direct reflection to field holding urls in UrlClassLaoder or some JS as it looks like in jmol.... In this case everything hook upon any method is in vain. But My original solution WILL work for all this cases O:) J. The improved patch is fixing possible exception thrown and repeated reloading of resources in case of failure. 2012-05-24 Jiri Vanek &lt;jvanek-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; Fix for RH816592 * netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java: (getCodeSourceSecurity): will now try to download and verify resource which was downloaded outside of netx. (alreadyTried) set for memory of once tried resources to not try again </pre> Jiri Vanek 2012-05-24T13:41:29 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18699 <pre>hi! It does not look like your push have done what was described in changelogs! It looks like you have repush my and Deepak's recent changes.... Please be very careful next time and verify your hg diff to match *exactly* what was apporved and that nothinx nasty have not mixed in. J. On 05/24/2012 03:12 PM, adomurad-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E&lt; at &gt;public.gmane.org wrote: </pre> Jiri Vanek 2012-05-24T13:20:12 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18698 <pre>changeset cb6d6d2e086a in /hg/icedtea-web details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=cb6d6d2e086a author: Adam Domurad &lt;adomurad-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; date: Thu May 24 09:10:41 2012 -0400 Added Martin Olsson's typo fix changeset 43a78616b1dd in /hg/icedtea-web details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=43a78616b1dd author: Adam Domurad &lt;adomurad-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; date: Thu May 24 09:12:38 2012 -0400 fixing conflict in ChangeLog diffstat: AUTHORS | 1 + ChangeLog | 4 + netx/net/sourceforge/jnlp/resources/Messages.properties | 6 +- netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java | 89 ++++++++++----- netx/net/sourceforge/jnlp/security/CertificateUtils.java | 3 +- netx/net/sourceforge/jnlp/security/KeyStores.java | 15 ++ plugin/icedteanp/IcedTeaNPPlugin.cc | 4 +- plugin/icedteanp/IcedTeaPluginUtils.cc | 4 +- 8 files changed, 88 insertions(+), 38 deletions(-) diffs (267 lines): diff -r 7041304bfc62 -r 43a78616b1dd AUTHORS --- a/AUTHORSWed May 23 14:56:30 2012 +0200 +++ b/AUTHORSThu May 24 09:12:38 2012 -0400 &lt; at &gt;&lt; at &gt; -17,6 +17,7 &lt; at &gt;&lt; at &gt; Jon A. Maxwell &lt;jmaxwell-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f&lt; at &gt;public.gmane.org&gt; Thomas Meyer &lt;thomas-VsYtu1Qij5c&lt; at &gt;public.gmane.org&gt; Saad Mohammad &lt;smohammad-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; +Martin Olsson &lt;martin-Gwph8TgvYLDLoDKTGw+V6w&lt; at &gt;public.gmane.org&gt; Andrew Su &lt;asu-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; Joshua Sumali &lt;jsumali-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; Jiri Vanek &lt;jvanek-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; diff -r 7041304bfc62 -r 43a78616b1dd ChangeLog --- a/ChangeLogWed May 23 14:56:30 2012 +0200 +++ b/ChangeLogThu May 24 09:12:38 2012 -0400 &lt; at &gt;&lt; at &gt; -1,3 +1,7 &lt; at &gt;&lt; at &gt; +2012-05-24 Martin Olsson &lt;martin-Gwph8TgvYLDLoDKTGw+V6w&lt; at &gt;public.gmane.org&gt; +* plugin/icedteanp/IcedTeaPluginUtils.cc: +Fix two typos. + 2012-05-23 Jiri Vanek &lt;jvanek-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; Enhanced about dialog diff -r 7041304bfc62 -r 43a78616b1dd netx/net/sourceforge/jnlp/resources/Messages.properties --- a/netx/net/sourceforge/jnlp/resources/Messages.propertiesWed May 23 14:56:30 2012 +0200 +++ b/netx/net/sourceforge/jnlp/resources/Messages.propertiesThu May 24 09:12:38 2012 -0400 &lt; at &gt;&lt; at &gt; -81,7 +81,9 &lt; at &gt;&lt; at &gt; LSignedAppJarUsingUnsignedJar=Signed application using unsigned jars. LSignedAppJarUsingUnsignedJarInfo=The main application jar is signed, but some of the jars it is using aren't. LSignedJNLPFileDidNotMatch=The signed JNLP file did not match the launching JNLP file. - +LNoSecInstance=Error: No security instance for {0}. The application may have trouble continuing +LCertFoundIn={0} found in cacerts ({1}) + JNotApplet=File is not an applet. JNotApplication=File is not an application. JNotComponent=File is not a component. &lt; at &gt;&lt; at &gt; -122,7 +124,7 &lt; at &gt;&lt; at &gt; PBadHeight=Invalid applet height. PUrlNotInCodebase=Relative URL does not specify a subdirectory of the codebase. (node={0}, href={1}, base={2}) PBadRelativeUrl=Invalid relative URL (node={0}, href={1}, base={2}) -PBadNonrelativeUrl=Invalid non-relative URL (node={0}, href={0}). +PBadNonrelativeUrl=Invalid non-relative URL (node={0}, href={1}) PNeedsAttribute=The {0} element must specify a {1} attribute. PBadXML=Invalid XML document syntax. PBadHeapSize=Invalid value for heap size ({0}) diff -r 7041304bfc62 -r 43a78616b1dd netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java --- a/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.javaWed May 23 14:56:30 2012 +0200 +++ b/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.javaThu May 24 09:12:38 2012 -0400 &lt; at &gt;&lt; at &gt; -895,44 +895,66 &lt; at &gt;&lt; at &gt; * Returns the permissions for the CodeSource. */ protected PermissionCollection getPermissions(CodeSource cs) { - Permissions result = new Permissions(); + try { + Permissions result = new Permissions(); - // should check for extensions or boot, automatically give all - // access w/o security dialog once we actually check certificates. + // should check for extensions or boot, automatically give all + // access w/o security dialog once we actually check certificates. - // copy security permissions from SecurityDesc element - if (security != null) { - // Security desc. is used only to track security settings for the - // application. However, an application may comprise of multiple - // jars, and as such, security must be evaluated on a per jar basis. + // copy security permissions from SecurityDesc element + if (security != null) { + // Security desc. is used only to track security settings for the + // application. However, an application may comprise of multiple + // jars, and as such, security must be evaluated on a per jar basis. - // set default perms - PermissionCollection permissions = security.getSandBoxPermissions(); + // set default perms + PermissionCollection permissions = security.getSandBoxPermissions(); - // If more than default is needed: - // 1. Code must be signed - // 2. ALL or J2EE permissions must be requested (note: plugin requests ALL automatically) - if (cs.getCodeSigners() != null &amp;&amp; - (getCodeSourceSecurity(cs.getLocation()).getSecurityType().equals(SecurityDesc.ALL_PERMISSIONS) || - getCodeSourceSecurity(cs.getLocation()).getSecurityType().equals(SecurityDesc.J2EE_PERMISSIONS))) { + // If more than default is needed: + // 1. Code must be signed + // 2. ALL or J2EE permissions must be requested (note: plugin requests ALL automatically) + if (cs == null) { + throw new RuntimeException("Code source was null"); + } + if (cs.getLocation() == null) { + throw new RuntimeException("Code source location was null"); + } + if (getCodeSourceSecurity(cs.getLocation()) == null) { + throw new RuntimeException("Code source security was null"); + } + if (getCodeSourceSecurity(cs.getLocation()).getSecurityType() == null) { + throw new RuntimeException("Code source security type was null"); + } + if (cs.getCodeSigners() != null + &amp;&amp; (getCodeSourceSecurity(cs.getLocation()).getSecurityType().equals(SecurityDesc.ALL_PERMISSIONS) + || getCodeSourceSecurity(cs.getLocation()).getSecurityType().equals(SecurityDesc.J2EE_PERMISSIONS))) { - permissions = getCodeSourceSecurity(cs.getLocation()).getPermissions(cs); + permissions = getCodeSourceSecurity(cs.getLocation()).getPermissions(cs); + } + + Enumeration&lt;Permission&gt; e = permissions.elements(); + while (e.hasMoreElements()) { + result.add(e.nextElement()); + } } - Enumeration&lt;Permission&gt; e = permissions.elements(); - while (e.hasMoreElements()) - result.add(e.nextElement()); + // add in permission to read the cached JAR files + for (int i = 0; i &lt; resourcePermissions.size(); i++) { + result.add(resourcePermissions.get(i)); + } + + // add in the permissions that the user granted. + for (int i = 0; i &lt; runtimePermissions.size(); i++) { + result.add(runtimePermissions.get(i)); + } + + return result; + } catch (RuntimeException ex) { + if (JNLPRuntime.isDebug()) { + ex.printStackTrace(); + } + throw ex; } - - // add in permission to read the cached JAR files - for (int i = 0; i &lt; resourcePermissions.size(); i++) - result.add(resourcePermissions.get(i)); - - // add in the permissions that the user granted. - for (int i = 0; i &lt; runtimePermissions.size(); i++) - result.add(runtimePermissions.get(i)); - - return result; } protected void addPermission(Permission p) { &lt; at &gt;&lt; at &gt; -1732,7 +1754,12 &lt; at &gt;&lt; at &gt; */ protected SecurityDesc getCodeSourceSecurity(URL source) { - return jarLocationSecurityMap.get(source); + SecurityDesc sec=jarLocationSecurityMap.get(source); + if (sec == null){ + System.out.println(Translator.R("LNoSecInstance",source.toString())); + } + return sec; + } /** diff -r 7041304bfc62 -r 43a78616b1dd netx/net/sourceforge/jnlp/security/CertificateUtils.java --- a/netx/net/sourceforge/jnlp/security/CertificateUtils.javaWed May 23 14:56:30 2012 +0200 +++ b/netx/net/sourceforge/jnlp/security/CertificateUtils.javaThu May 24 09:12:38 2012 -0400 &lt; at &gt;&lt; at &gt; -60,6 +60,7 &lt; at &gt;&lt; at &gt; import java.util.Random; import net.sourceforge.jnlp.runtime.JNLPRuntime; +import net.sourceforge.jnlp.runtime.Translator; import net.sourceforge.jnlp.util.replacements.BASE64Encoder; import sun.security.provider.X509Factory; &lt; at &gt;&lt; at &gt; -173,7 +174,7 &lt; at &gt;&lt; at &gt; if (c.equals(keyStores[i].getCertificate(alias))) { if (JNLPRuntime.isDebug()) { - System.out.println(c.getSubjectX500Principal().getName() + " found in cacerts"); + System.out.println(Translator.R("LCertFoundIn", c.getSubjectX500Principal().getName(), KeyStores.getPathToKeystore(keyStores[i].hashCode()))); } return true; diff -r 7041304bfc62 -r 43a78616b1dd netx/net/sourceforge/jnlp/security/KeyStores.java --- a/netx/net/sourceforge/jnlp/security/KeyStores.javaWed May 23 14:56:30 2012 +0200 +++ b/netx/net/sourceforge/jnlp/security/KeyStores.javaThu May 24 09:12:38 2012 -0400 &lt; at &gt;&lt; at &gt; -47,7 +47,9 &lt; at &gt;&lt; at &gt; import java.security.NoSuchAlgorithmException; import java.security.cert.CertificateException; import java.util.ArrayList; +import java.util.HashMap; import java.util.List; +import java.util.Map; import java.util.StringTokenizer; import net.sourceforge.jnlp.config.DeploymentConfiguration; &lt; at &gt;&lt; at &gt; -75,6 +77,8 &lt; at &gt;&lt; at &gt; CLIENT_CERTS, } + public static final Map&lt;Integer,String&gt; keystoresPaths=new HashMap&lt;Integer, String&gt;(); + private static DeploymentConfiguration config = null; private static final String KEYSTORE_TYPE = "JKS"; &lt; at &gt;&lt; at &gt; -133,12 +137,23 &lt; at &gt;&lt; at &gt; KeyStore ks = null; try { ks = createKeyStoreFromFile(new File(location), create, DEFAULT_PASSWORD); + //hashcode is used instead of instance so when no references are left + //to keystore, then this will not be blocker for garbage collection + keystoresPaths.put(ks.hashCode(),location); } catch (Exception e) { e.printStackTrace(); } return ks; } + public static String getPathToKeystore(int k) { + String s = keystoresPaths.get(k); + if (s == null) { + return "unknown keystore location"; + } + return s; + } + /** * Returns an array of KeyStore that contain certificates that are trusted. * The KeyStores contain certificates from different sources. diff -r 7041304bfc62 -r 43a78616b1dd plugin/icedteanp/IcedTeaNPPlugin.cc --- a/plugin/icedteanp/IcedTeaNPPlugin.ccWed May 23 14:56:30 2012 +0200 +++ b/plugin/icedteanp/IcedTeaNPPlugin.ccThu May 24 09:12:38 2012 -0400 &lt; at &gt;&lt; at &gt; -427,7 +427,7 &lt; at &gt;&lt; at &gt; goto cleanup_done; cleanup_appletviewer_mutex: - g_free (data-&gt;appletviewer_mutex); + g_mutex_free (data-&gt;appletviewer_mutex); data-&gt;appletviewer_mutex = NULL; // cleanup_instance_string: &lt; at &gt;&lt; at &gt; -2000,7 +2000,7 &lt; at &gt;&lt; at &gt; tofree-&gt;window_width = 0; // cleanup_appletviewer_mutex: - g_free (tofree-&gt;appletviewer_mutex); + g_mutex_free (tofree-&gt;appletviewer_mutex); tofree-&gt;appletviewer_mutex = NULL; // cleanup_instance_string: diff -r 7041304bfc62 -r 43a78616b1dd plugin/icedteanp/IcedTeaPluginUtils.cc --- a/plugin/icedteanp/IcedTeaPluginUtils.ccWed May 23 14:56:30 2012 +0200 +++ b/plugin/icedteanp/IcedTeaPluginUtils.ccThu May 24 09:12:38 2012 -0400 &lt; at &gt;&lt; at &gt; -289,7 +289,7 &lt; at &gt;&lt; at &gt; * * &lt; at &gt;param str The string to split * &lt; at &gt;param The delimiters to split on - * &lt; at &gt;return A string vector containing the aplit components + * &lt; at &gt;return A string vector containing the split components */ std::vector&lt;std::string*&gt;* &lt; at &gt;&lt; at &gt; -1068,7 +1068,7 &lt; at &gt;&lt; at &gt; if(ret) PLUGIN_DEBUG("Error: Unable to initialize message queue mutex: %d\n", ret); -PLUGIN_DEBUG("Mutexs %p and %p initialized\n", &amp;subscriber_mutex, &amp;msg_queue_mutex); +PLUGIN_DEBUG("Mutexes %p and %p initialized\n", &amp;subscriber_mutex, &amp;msg_queue_mutex); } /** </pre> adomurad-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E< at >public.gmane.org 2012-05-24T13:12:42 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18697 <pre>I have to get around reviewing a patch sometime, so I figured I'd start with this one : ) Thanks for all the patches, Martin! Patch looks OK to me, I'll go ahead and commit this one to HEAD seeing as it's minor. </pre> Adam Domurad 2012-05-24T13:04:16 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18696 <pre>http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=1011 Priority: P3 Bug ID: 1011 CC: unassigned-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E&lt; at &gt;public.gmane.org Assignee: dbhole-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org Summary: ZipException when a folder is in the classpath Severity: normal Classification: Unclassified OS: Linux Reporter: damieng-whniv8GeeGkdnm+yROfE0A&lt; at &gt;public.gmane.org Hardware: x86_64 Status: NEW Version: 1.2 Component: Plugin Product: IcedTea-Web I use an applet with the following archive attribute : archive="MyApplet.jar,config/" With the config folder in the classpath, it can load configuration files that are not in a jar archive. This works well when Apache does not use the Indexes option, there is just a warning about not finding config/. But when a web page is generated for the folder, IcedTea throws the following ZipException because it is trying to unzip the HTML as if it were a jar archive. The applet never loads. java.util.zip.ZipException: error in opening zip file at java.util.zip.ZipFile.open(Native Method) at java.util.zip.ZipFile.&lt;init&gt;(ZipFile.java:214) at java.util.zip.ZipFile.&lt;init&gt;(ZipFile.java:144) at java.util.jar.JarFile.&lt;init&gt;(JarFile.java:152) at java.util.jar.JarFile.&lt;init&gt;(JarFile.java:103) at net.sourceforge.jnlp.tools.JarSigner.verifyJar(JarSigner.java:241) at net.sourceforge.jnlp.tools.JarSigner.verifyJars(JarSigner.java:201) at net.sourceforge.jnlp.runtime.JNLPClassLoader.verifyJars(JNLPClassLoader.java:1286) at net.sourceforge.jnlp.runtime.JNLPClassLoader.initializeResources(JNLPClassLoader.java:476) at net.sourceforge.jnlp.runtime.JNLPClassLoader.&lt;init&gt;(JNLPClassLoader.java:201) at net.sourceforge.jnlp.runtime.JNLPClassLoader.getInstance(JNLPClassLoader.java:320) at net.sourceforge.jnlp.Launcher.createApplet(Launcher.java:701) at net.sourceforge.jnlp.Launcher.getApplet(Launcher.java:676) at net.sourceforge.jnlp.Launcher$TgThread.run(Launcher.java:886) java.util.zip.ZipException: error in opening zip file at java.util.zip.ZipFile.open(Native Method) at java.util.zip.ZipFile.&lt;init&gt;(ZipFile.java:214) at java.util.zip.ZipFile.&lt;init&gt;(ZipFile.java:144) at java.util.jar.JarFile.&lt;init&gt;(JarFile.java:152) at java.util.jar.JarFile.&lt;init&gt;(JarFile.java:103) at net.sourceforge.jnlp.tools.JarSigner.verifyJar(JarSigner.java:241) at net.sourceforge.jnlp.tools.JarSigner.verifyJars(JarSigner.java:201) at net.sourceforge.jnlp.runtime.JNLPClassLoader.verifyJars(JNLPClassLoader.java:1286) at net.sourceforge.jnlp.runtime.JNLPClassLoader.initializeResources(JNLPClassLoader.java:476) at net.sourceforge.jnlp.runtime.JNLPClassLoader.&lt;init&gt;(JNLPClassLoader.java:201) at net.sourceforge.jnlp.runtime.JNLPClassLoader.getInstance(JNLPClassLoader.java:320) at net.sourceforge.jnlp.Launcher.createApplet(Launcher.java:701) at net.sourceforge.jnlp.Launcher.getApplet(Launcher.java:676) at net.sourceforge.jnlp.Launcher$TgThread.run(Launcher.java:886) net.sourceforge.jnlp.LaunchException: Fatal: Initialization Error: Could not initialize applet. at net.sourceforge.jnlp.Launcher.createApplet(Launcher.java:735) at net.sourceforge.jnlp.Launcher.getApplet(Launcher.java:676) at net.sourceforge.jnlp.Launcher$TgThread.run(Launcher.java:886) Caused by: net.sourceforge.jnlp.LaunchException: Fatal: Initialization Error: A fatal error occurred while trying to verify jars. at net.sourceforge.jnlp.runtime.JNLPClassLoader.initializeResources(JNLPClassLoader.java:482) at net.sourceforge.jnlp.runtime.JNLPClassLoader.&lt;init&gt;(JNLPClassLoader.java:201) at net.sourceforge.jnlp.runtime.JNLPClassLoader.getInstance(JNLPClassLoader.java:320) at net.sourceforge.jnlp.Launcher.createApplet(Launcher.java:701) ... 2 more Caused by: net.sourceforge.jnlp.LaunchException: Fatal: Initialization Error: A fatal error occurred while trying to verify jars. at net.sourceforge.jnlp.runtime.JNLPClassLoader.initializeResources(JNLPClassLoader.java:482) at net.sourceforge.jnlp.runtime.JNLPClassLoader.&lt;init&gt;(JNLPClassLoader.java:201) at net.sourceforge.jnlp.runtime.JNLPClassLoader.getInstance(JNLPClassLoader.java:320) at net.sourceforge.jnlp.Launcher.createApplet(Launcher.java:701) at net.sourceforge.jnlp.Launcher.getApplet(Launcher.java:676) at net.sourceforge.jnlp.Launcher$TgThread.run(Launcher.java:886) java.lang.NullPointerException at net.sourceforge.jnlp.NetxPanel.runLoader(NetxPanel.java:154) at sun.applet.AppletPanel.run(AppletPanel.java:379) at java.lang.Thread.run(Thread.java:722) java.lang.NullPointerException at sun.applet.AppletPanel.run(AppletPanel.java:429) at java.lang.Thread.run(Thread.java:722) </pre> bugzilla-daemon-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E< at >public.gmane.org 2012-05-24T12:36:00 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18695 <pre>http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=1010 Priority: P3 Bug ID: 1010 CC: unassigned-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E&lt; at &gt;public.gmane.org Assignee: dbhole-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org Summary: accessEventQueue exception when displaying a modal dialog in the event queue Severity: major Classification: Unclassified OS: Linux Reporter: damieng-whniv8GeeGkdnm+yROfE0A&lt; at &gt;public.gmane.org Hardware: x86_64 Status: NEW Version: 1.2 Component: Plugin Product: IcedTea-Web Displaying a modal dialog in the event queue with JOptionPane results in the following exception being raised. Firefox does not quit properly after the java process is killed. I was using firefox 12. No problem occurs with Oracle JRE and plugin. Exception in thread "AWT-EventQueue-1" java.security.AccessControlException: access denied ("java.awt.AWTPermission" "accessEventQueue") at java.security.AccessControlContext.checkPermission(AccessControlContext.java:366) at java.security.AccessController.checkPermission(AccessController.java:555) at java.lang.SecurityManager.checkPermission(SecurityManager.java:549) at net.sourceforge.jnlp.runtime.JNLPSecurityManager.checkPermission(JNLPSecurityManager.java:284) at java.lang.SecurityManager.checkAwtEventQueueAccess(SecurityManager.java:1415) at net.sourceforge.jnlp.runtime.JNLPSecurityManager.checkAwtEventQueueAccess(JNLPSecurityManager.java:466) at java.awt.Toolkit.getSystemEventQueue(Toolkit.java:1721) at java.awt.Dialog.show(Dialog.java:1070) at javax.swing.JOptionPane.showOptionDialog(JOptionPane.java:871) at javax.swing.JOptionPane.showMessageDialog(JOptionPane.java:668) at javax.swing.JOptionPane.showMessageDialog(JOptionPane.java:639) </pre> bugzilla-daemon-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E< at >public.gmane.org 2012-05-24T09:46:57 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18694 <pre>changeset ab51d1f4d8ba in /hg/gfx-test details: http://icedtea.classpath.org/hg/gfx-test?cmd=changeset;node=ab51d1f4d8ba author: Pavel Tisnovsky &lt;ptisnovs-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; date: Thu May 24 11:41:36 2012 +0200 * src/org/gfxtest/framework/PrintTest.java: Added new constants required by tests. * src/org/gfxtest/testsuites/PrintTestArcs.java: Basic functionality of this test suite. diffstat: ChangeLog | 7 + src/org/gfxtest/framework/PrintTest.java | 10 + src/org/gfxtest/testsuites/PrintTestArcs.java | 146 +++++++++++++++++++++++++- 3 files changed, 162 insertions(+), 1 deletions(-) diffs (202 lines): diff -r 9d3dfd9343a9 -r ab51d1f4d8ba ChangeLog --- a/ChangeLogTue May 22 12:03:36 2012 +0200 +++ b/ChangeLogThu May 24 11:41:36 2012 +0200 &lt; at &gt;&lt; at &gt; -1,3 +1,10 &lt; at &gt;&lt; at &gt; +2012-05-24 Pavel Tisnovsky &lt;ptisnovs-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; + +* src/org/gfxtest/framework/PrintTest.java: +Added new constants required by tests. +* src/org/gfxtest/testsuites/PrintTestArcs.java: +Basic functionality of this test suite. + 2012-05-22 Pavel Tisnovsky &lt;ptisnovs-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; * src/org/gfxtest/testsuites/PrintTestCircles.java: diff -r 9d3dfd9343a9 -r ab51d1f4d8ba src/org/gfxtest/framework/PrintTest.java --- a/src/org/gfxtest/framework/PrintTest.javaTue May 22 12:03:36 2012 +0200 +++ b/src/org/gfxtest/framework/PrintTest.javaThu May 24 11:41:36 2012 +0200 &lt; at &gt;&lt; at &gt; -70,6 +70,16 &lt; at &gt;&lt; at &gt; protected static final int CIRCLE_RADIUS_STEP = 10; /** + * Offset between two arcs. + */ + protected static final int ARC_RADIUS_STEP = 10; + + /** + * Minimum radius of a circle or an arc. + */ + protected static final int MINIMUM_RADIUS = 10; + + /** * Horizontal distance between two lines. */ protected static final int HORIZONTAL_STEP = 10; diff -r 9d3dfd9343a9 -r ab51d1f4d8ba src/org/gfxtest/testsuites/PrintTestArcs.java --- a/src/org/gfxtest/testsuites/PrintTestArcs.javaTue May 22 12:03:36 2012 +0200 +++ b/src/org/gfxtest/testsuites/PrintTestArcs.javaThu May 24 11:41:36 2012 +0200 &lt; at &gt;&lt; at &gt; -40,7 +40,14 &lt; at &gt;&lt; at &gt; package org.gfxtest.testsuites; +import java.awt.Color; +import java.awt.Graphics2D; + + + import org.gfxtest.framework.PrintTest; +import org.gfxtest.framework.TestImage; +import org.gfxtest.framework.TestResult; import org.gfxtest.framework.annotations.GraphicsPrimitive; import org.gfxtest.framework.annotations.GraphicsPrimitives; import org.gfxtest.framework.annotations.RenderStyle; &lt; at &gt;&lt; at &gt; -65,7 +72,86 &lt; at &gt;&lt; at &gt; &lt; at &gt;Zoom(1) public class PrintTestArcs extends PrintTest { - + + /** + * Method which renders set of arcs using various colors and + * stroke styles. For each arc, the callback function/method is called to + * perform all required setup. + * + * &lt; at &gt;param image + * image to which arcs are to be drawn + * &lt; at &gt;param graphics2d + * graphics canvas + * &lt; at &gt;param radiusStep + * between two near arcs + * &lt; at &gt;param arcDrawCallback + * class containing set of callback methods + */ + private void drawArcs(TestImage image, Graphics2D graphics2d, int radiusStep, CommonArcDrawCallbacks arcDrawCallback) + { + // setup rendering + arcDrawCallback.setup(image, graphics2d); + + // image width and height + final int width = image.getWidth(); + final int height = image.getHeight(); + + // horizontal coordinates of arc center + final int xc = width &gt;&gt; 1; + final int yc = height &gt;&gt; 1; + + // maximum radius + final int maxRadius = Math.min(width, height) - BORDER; + + // index to color palette + int colorIndex = 0; + + // arc start angle + int startAngle = 0; + + // draw all arcs onto a paper + for (int radius = MINIMUM_RADIUS; radius &lt; maxRadius; radius += radiusStep) + { + // setup can be made for each arc + arcDrawCallback.iterationCallBack(xc, yc, radius, maxRadius, colorIndex++); + startAngle += 4; + // render the arc + graphics2d.drawArc(xc - radius, yc - radius, radius &lt;&lt; 1, radius &lt;&lt; 1, startAngle, 270); + } + + // cleanup rendering + arcDrawCallback.cleanup(); + } + + /** + * Test basic behavior of method Graphics.drawOval(). + * Color of all rendered arcs are set to black. + * + * &lt; at &gt;param image + * image to which arcs are to be drawn + * &lt; at &gt;param graphics2d + * graphics canvas + * &lt; at &gt;return test result status - PASSED, FAILED or ERROR + */ + public TestResult testDrawArcBasicStyle(TestImage image, Graphics2D graphics2d) + { + drawArcs(image, graphics2d, ARC_RADIUS_STEP, new CommonArcDrawCallbacks() + { + /** + * Callback function called before each arc is rendered. + */ + &lt; at &gt;Override + public void iterationCallBack(int x, int y, int radius, int maxRadius, int index) + { + // nothing need to be changed + return; + } + }); + + // test return value + return TestResult.PASSED; + } + /** * Entry point to the test suite. * &lt; at &gt;&lt; at &gt; -77,3 +163,61 &lt; at &gt;&lt; at &gt; new PrintTestArcs().runTestSuite(args); } } + +/** + * Class representing set of callback methods called for each rendered arc. + * + * &lt; at &gt;author Pavel Tisnovsky + */ +abstract class CommonArcDrawCallbacks +{ + /** + * Image to which arcs are to be drawn. + */ + protected TestImage image; + + /** + * Graphics canvas. + */ + protected Graphics2D graphics; + + /** + * Setup phase. + * + * &lt; at &gt;param image + * image to which arcs are to be drawn + * &lt; at &gt;param graphics2d + * graphics canvas + */ + public void setup(TestImage image, Graphics2D graphics) + { + this.image = image; + this.graphics = graphics; + // set drawing color + graphics.setColor(Color.BLACK); + } + + /** + * This method is called for each rendered horizontal arc. + * + * &lt; at &gt;param x + * horizontal coordinate of a arc center + * &lt; at &gt;param y + * vertical coordinate of a arc center + * &lt; at &gt;param radius + * arc radius + * &lt; at &gt;param maxRadius + * maximum allowable arc radius + * &lt; at &gt;param colorIndex + * color index + */ + public abstract void iterationCallBack(int x, int y, int radius, int maxRadius, int colorIndex); + + /** + * Cleanup phase. + */ + public void cleanup() + { + return; + } +} </pre> ptisnovs-HMOWzH9t1bryS2n3IP37Vh1hho49/T1E< at >public.gmane.org 2012-05-24T09:38:54 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18693 <pre> Hmmm.. I have look inside this, and I do not feel guilty. Actually it looks like another occurrence of RH816592. The fact that we see why it is failing does not mean that it caused the app to fail. AFAIK my debbuging outputs have not changed the behaviour at all. Adding sout printing before returning null and rethrowing "exception which would be thrown anyway". So the just printing WILL change older behaviour. Although I addmit that I have one small mistake - fixed in attached patch. However this did not help. The original pathc for RH816592 is solving this issue, but 1.2 is working as well! So I think this reproducer had to be broken much longer ago! I'm going to attach enhanced patch for RH816592 later today. OpenJDK Runtime Environment (IcedTea6 1.11.1) (fedora-65.1.11.1.fc16-x86_64) OpenJDK 64-Bit Server VM (build 20.0-b12, mixed mode) ClassPreloader - javax.vecmath.Vector3f+ ClassPreloader - javax.vecmath.Point3i+ ClassPreloader - org.jmol.g3d.Graphics3D ClassPreloader - javax.vecmath.Point3f+ ClassPreloader - javax.vecmath.Matrix3f+ ClassPreloader - org.jmol.g3d.Sphere3D Jmol applet jmolApplet0__752170922377234__ initializing AppletRegistry.checkIn(jmolApplet0__752170922377234__) ClassPreloader - org.jmol.g3d.Line3D ClassPreloader - org.jmol.g3d.Cylinder3D ClassPreloader - org.jmol.g3d.Colix3D ClassPreloader - org.jmol.g3d.Shade3D ClassPreloader - org.jmol.adapter.smarter.SmarterJmolAdapter ClassPreloader - org.jmol.adapter.smarter.Atom ClassPreloader - org.jmol.adapter.smarter.Bond ClassPreloader - org.jmol.adapter.smarter.AtomSetCollection ClassPreloader - org.jmol.adapter.smarter.AtomSetCollectionReader ClassPreloader - org.jmol.adapter.smarter.Resolver ClassPreloader - org.jmol.popup.JmolPopup urlImage=jar:http://jmol.sourceforge.net/jmol/JmolApplet0.jar!/jmol75x29x8.gif *Error: No security instance for http://jmol.sourceforge.net/jmol/JmolApplet0_Popup.jar. The application may have trouble continuing* injected jar error preloading org.jmol.popup.JmolPopup: Code source security was null net.sourceforge.jnlp.runtime.JNLPClassLoader - 923 - getPermissions java.security.SecureClassLoader - 210 - getProtectionDomain java.security.SecureClassLoader - 142 - defineClass java.net.URLClassLoader - 277 - defineClass java.net.URLClassLoader - 73 - access$000 java.net.URLClassLoader$1 - 212 - run java.security.AccessController - -2 - doPrivileged java.net.URLClassLoader - 205 - findClass net.sourceforge.jnlp.runtime.JNLPClassLoader - 1521 - findClass net.sourceforge.jnlp.runtime.JNLPClassLoader - 1548 - loadClassExt net.sourceforge.jnlp.runtime.JNLPClassLoader - 1363 - loadClass java.lang.Class - -2 - forName0 java.lang.Class - 186 - forName org.jmol.applet.ClassPreloader - -1 - run 1822 script command tokens applet context: -applet appletDocumentBase=http://jmol.sourceforge.net/demo/atoms/ appletCodeBase=http://jmol.sourceforge.net/jmol/ (C) 2009 Jmol Development Jmol Version: 12.2.2 2011-10-11 22:29 java.vendor: Sun Microsystems Inc. java.version: 1.6.0_24 os.name: Linux memory: 7.0/123.7 processors available: 4 useCommandThread: false appletId:jmolApplet0__752170922377234__ defaults = "Jmol" backgroundColor = "black" language=en_US Jmol applet jmolApplet0__752170922377234__ ready FileManager.getAtomSetCollectionFromFile(caffeine.xyz.gz) FileManager opening http://jmol.sourceforge.net/demo/atoms/caffeine.xyz.gz *The Resolver thinks Xyz* thsi is also wrong, ony http://jmol.sourceforge.net/demo/atoms/caffeine.xyz.gz exists *Error: No security instance for http://jmol.sourceforge.net/jmol/JmolApplet0_ReadersMolXyz.jar. * injected jar. The application may have trouble continuing File reader was not found:Xyz openFile(caffeine.xyz.gz): 743 ms eval ERROR: File reader was not found:Xyz ---- load &gt;&gt; "caffeine.xyz.gz" &lt;&lt; J. diff -r f6eddd071004 ChangeLog --- a/ChangeLogWed May 23 13:02:58 2012 -0400 +++ b/ChangeLogThu May 24 11:08:44 2012 +0200 &lt; at &gt;&lt; at &gt; -1,3 +1,9 &lt; at &gt;&lt; at &gt; +2012-05-24 Jiri Vanek &lt;jvanek-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; + +* netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java: (getPermissions): +Originally NullPointerException was changed to RuntimeException. +Rewerting back. + 2012-05-23 Deepak Bhole &lt;dbhole-H+wXaHxf7aLQT0dZR+AlfA&lt; at &gt;public.gmane.org&gt; * AUTHORS: Added Martin Olsson to list. diff -r f6eddd071004 netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java --- a/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.javaWed May 23 13:02:58 2012 -0400 +++ b/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.javaThu May 24 11:08:44 2012 +0200 &lt; at &gt;&lt; at &gt; -914,16 +914,16 &lt; at &gt;&lt; at &gt; // 1. Code must be signed // 2. ALL or J2EE permissions must be requested (note: plugin requests ALL automatically) if (cs == null) { - throw new RuntimeException("Code source was null"); + throw new NullPointerException("Code source was null"); } if (cs.getLocation() == null) { - throw new RuntimeException("Code source location was null"); + throw new NullPointerException("Code source location was null"); } if (getCodeSourceSecurity(cs.getLocation()) == null) { - throw new RuntimeException("Code source security was null"); + throw new NullPointerException("Code source security was null"); } if (getCodeSourceSecurity(cs.getLocation()).getSecurityType() == null) { - throw new RuntimeException("Code source security type was null"); + throw new NullPointerException("Code source security type was null"); } if (cs.getCodeSigners() != null &amp;&amp; (getCodeSourceSecurity(cs.getLocation()).getSecurityType().equals(SecurityDesc.ALL_PERMISSIONS) </pre> Jiri Vanek 2012-05-24T09:16:34 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18692 <pre> ----- Original Message ----- snip... I realise we've slipped on this quite a bit... the sync was done a while ago but it's taken time to be in a state where a 2.2 branch of IcedTea is appropriate. This is now available: http://icedtea.classpath.org/hg/release/icedtea7-2.2/ I propose we do the aforementioned release next Wednesday, the 30th of May, at 18h00 UTC. In the meantime, please test and report any regressions or bugs. </pre> Andrew Hughes 2012-05-23T21:37:48 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18691 <pre>Hi Deepak, On 08/03/12 05:17 PM, Deepak Bhole wrote: Extra "/" there Looks like this fell through the cracks. You've taken care of PR518[1] as well. Can you update the ChangeLog to include this and NEWS entries for the various branches? would you mind also making sure none of the recent changesets have thrown any of this off? Sorry for that, this should have gone in ages ago! The patch itself looks good to me though, thanks for taking care of the bugs! Cheers, Danesh </pre> Danesh Dadachanji 2012-05-23T21:28:35 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18690 <pre>Deepak Bhole escreveu: You're welcome :-) Personally I do not like C++ code with static variables initialized by a function call, but if it works, not much of an issue, instead of checking and if NULL initializing it at runtime. My original patch did not cover NP_shutdown, so it should have its own problem in case the plugin is somehow unloaded or restarted. I remember I added it because there were conditions there would be no listeners, and it would leak memory, but not sure if the condition of no listeners was common, or side effect of some other problem. I have also the patch at http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=621 but I did not need it anymore since firefox-3.6.13 so the change of a few types from int to long and atoi to atol should no longer be required in x86_64. But may be worth checking as this may cause issues again. Paulo </pre> pcpa-4qZELD6Fgxg39yzSjRtAkw< at >public.gmane.org 2012-05-23T21:26:40 http://permalink.gmane.org/gmane.comp.java.openjdk.distro-packaging.devel/18689 <pre></pre> Martin Olsson 2012-05-23T21:13:23 Search the mailing list at Gmane query http://search.gmane.org/?group=$group=gmane.comp.java.openjdk.distro-packaging.devel