/hg/release/icedtea-web-1.3: Fix PR975: Plugin should not includ...
ddadacha at icedtea.classpath.org
ddadacha at icedtea.classpath.org
Wed Aug 15 12:00:02 PDT 2012
changeset 684acbb6c35e in /hg/release/icedtea-web-1.3
details: http://icedtea.classpath.org/hg/release/icedtea-web-1.3?cmd=changeset;node=684acbb6c35e
author: Danesh Dadachanji <ddadacha at redhat.com>
date: Wed Aug 15 14:44:14 2012 -0400
Fix PR975: Plugin should not include classpaths specified in jar manifests when using jnlp_href
diffstat:
ChangeLog | 25 +
NEWS | 1 +
netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java | 6 +-
tests/reproducers/signed/Classpath.Manifest.Test.Helper/srcs/CheckForClasspath.java | 42 +++
tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestAppletTest.html | 48 +++
tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestAppletTest.jnlp | 61 ++++
tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestApplicationTest.jnlp | 56 ++++
tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestJNLPHrefTest.html | 46 +++
tests/reproducers/signed/ClasspathManifestTest/srcs/ClasspathManifest.java | 81 ++++++
tests/reproducers/signed/ClasspathManifestTest/srcs/META-INF/MANIFEST.MF | 3 +
tests/reproducers/signed/ClasspathManifestTest/testcases/ClasspathManifestTest.java | 134 ++++++++++
11 files changed, 502 insertions(+), 1 deletions(-)
diffs (truncated from 562 to 500 lines):
diff -r c7bf15bbc88e -r 684acbb6c35e ChangeLog
--- a/ChangeLog Tue Aug 14 16:07:38 2012 -0400
+++ b/ChangeLog Wed Aug 15 14:44:14 2012 -0400
@@ -1,3 +1,28 @@
+2012-08-14 Danesh Dadachanji <ddadacha at redhat.com>
+
+ Small addition to NEWS entry and bug annotation indicating the PR975 fix.
+ * NEWS: Add entry for PR975
+ * tests/reproducers/signed/ClasspathManifestTest/testcases/ClasspathManifestTest.java:
+ Add bug annotation for PR975.
+
+2012-08-14 Danesh Dadachanji <ddadacha at redhat.com>
+
+ Classpaths in jars' manifests are only considered when the applet is run
+ without using jnlp_href and a JNLP file.
+ * netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java (activateJars):
+ Add conditional check for use of jnlp_href.
+ * tests/reproducers/signed/Classpath.Manifest.Test.Helper/srcs/CheckForClasspath.java:
+ Applet whose jar is stored in a subdir under the test engine server.
+ * tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestAppletTest.html:
+ * tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestAppletTest.jnlp:
+ * tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestApplicationTest.jnlp:
+ * tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestJNLPHrefTest.html:
+ * tests/reproducers/signed/ClasspathManifestTest/srcs/ClasspathManifest.java:
+ * tests/reproducers/signed/ClasspathManifestTest/srcs/META-INF/MANIFEST.MF:
+ * tests/reproducers/signed/ClasspathManifestTest/testcases/ClasspathManifestTest.java:
+ Test if manifest entry is searched for classpath only when in the plugin
+ is run without using jnlp_href.
+
2012-08-14 Adam Domurad <adomurad at redhat.com>
Reproducer for allowing unsigned content in META-INF/ folder.
diff -r c7bf15bbc88e -r 684acbb6c35e NEWS
--- a/NEWS Tue Aug 14 16:07:38 2012 -0400
+++ b/NEWS Wed Aug 15 14:44:14 2012 -0400
@@ -25,6 +25,7 @@
- PR855: AppletStub getDocumentBase() doesn't return full URL
- PR1011: Folders treated as jar files in archive tag
- PR1106: Buffer overflow in plugin table
+ - PR975: Plugin should not include classpaths specified in jar manifests when using jnlp_href
* Common
- PR918: java applet windows uses a low resulution black/white icon
- RH838417: Disambiguate signed applet security prompt from certificate warning
diff -r c7bf15bbc88e -r 684acbb6c35e netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
--- a/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java Tue Aug 14 16:07:38 2012 -0400
+++ b/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java Wed Aug 15 14:44:14 2012 -0400
@@ -1286,7 +1286,11 @@
JarFile jarFile = new JarFile(localFile.getAbsolutePath());
Manifest mf = jarFile.getManifest();
- if (file instanceof PluginBridge) {
+ // Only check classpath if this is the plugin and there is no jnlp_href usage.
+ // Note that this is different from proprietary plugin behaviour.
+ // If jnlp_href is used, the app should be treated similarly to when
+ // it is run from javaws as a webstart.
+ if (file instanceof PluginBridge && !((PluginBridge) file).useJNLPHref()) {
classpaths.addAll(getClassPathsFromManifest(mf, jar.getLocation().getPath()));
}
diff -r c7bf15bbc88e -r 684acbb6c35e tests/reproducers/signed/Classpath.Manifest.Test.Helper/srcs/CheckForClasspath.java
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/reproducers/signed/Classpath.Manifest.Test.Helper/srcs/CheckForClasspath.java Wed Aug 15 14:44:14 2012 -0400
@@ -0,0 +1,42 @@
+/* CheckForClasspath.java
+Copyright (C) 2012 Red Hat, Inc.
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or
+modify it under the terms of the GNU General Public License as published by
+the Free Software Foundation, version 2.
+
+IcedTea is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to
+the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+ */
+
+public class CheckForClasspath {
+ public static void checkClasspathAndPrint() {
+ System.out.println("CheckForClasspath found on classpath.");
+ }
+}
diff -r c7bf15bbc88e -r 684acbb6c35e tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestAppletTest.html
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestAppletTest.html Wed Aug 15 14:44:14 2012 -0400
@@ -0,0 +1,48 @@
+<!--
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation; either version 2, or (at your option)
+any later version.
+
+IcedTea is distributed in the hope that it will be useful, but
+WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to the
+Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+
+ -->
+<html>
+ <head></head>
+ <body>
+ <applet code="ClasspathManifest.class"
+ archive="ClasspathManifestTest.jar"
+ codebase="."
+ width=800
+ height=600>
+ </applet>
+ </body>
+</html>
diff -r c7bf15bbc88e -r 684acbb6c35e tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestAppletTest.jnlp
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestAppletTest.jnlp Wed Aug 15 14:44:14 2012 -0400
@@ -0,0 +1,61 @@
+<!--
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation; either version 2, or (at your option)
+any later version.
+
+IcedTea is distributed in the hope that it will be useful, but
+WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to the
+Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+
+ -->
+<?xml version="1.0" encoding="utf-8"?>
+<jnlp spec="1.0" href="ClasspathManifestAppletTest.jnlp" codebase=".">
+ <information>
+ <title>Classpath Manifest Applet Test</title>
+ <vendor>IcedTea</vendor>
+ <homepage href="http://icedtea.classpath.org/wiki/IcedTea-Web#Testing_IcedTea-Web"/>
+ <description>ClasspathManifest</description>
+ <offline/>
+ </information>
+ <resources>
+ <j2se version="1.4+"/>
+ <jar href="ClasspathManifestTest.jar"/>
+ </resources>
+ <security>
+ <all-permissions />
+ </security>
+ <applet-desc
+ documentBase="."
+ name="ClasspathManifest"
+ main-class="ClasspathManifest"
+ width="100"
+ height="100">
+ </applet-desc>
+</jnlp>
diff -r c7bf15bbc88e -r 684acbb6c35e tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestApplicationTest.jnlp
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestApplicationTest.jnlp Wed Aug 15 14:44:14 2012 -0400
@@ -0,0 +1,56 @@
+<!--
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation; either version 2, or (at your option)
+any later version.
+
+IcedTea is distributed in the hope that it will be useful, but
+WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to the
+Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+
+ -->
+<?xml version="1.0" encoding="utf-8"?>
+<jnlp spec="1.0" href="ClasspathManifestApplicationTest.jnlp" codebase=".">
+ <information>
+ <title>ClasspathManifest</title>
+ <vendor>IcedTea</vendor>
+ <homepage href="http://icedtea.classpath.org/wiki/IcedTea-Web#Testing_IcedTea-Web"/>
+ <description>ClasspathManifest</description>
+ <offline/>
+ </information>
+ <resources>
+ <j2se version="1.4+"/>
+ <jar href="ClasspathManifestTest.jar"/>
+ </resources>
+ <security>
+ <all-permissions />
+ </security>
+ <application-desc main-class="ClasspathManifest">
+ </application-desc>
+</jnlp>
diff -r c7bf15bbc88e -r 684acbb6c35e tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestJNLPHrefTest.html
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/reproducers/signed/ClasspathManifestTest/resources/ClasspathManifestJNLPHrefTest.html Wed Aug 15 14:44:14 2012 -0400
@@ -0,0 +1,46 @@
+<!--
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation; either version 2, or (at your option)
+any later version.
+
+IcedTea is distributed in the hope that it will be useful, but
+WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to the
+Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+
+-->
+<html>
+ <head></head>
+ <body>
+ <applet code="ClasspathManifest.class" width=800 height=600>
+ <param name="jnlp_href" value="ClasspathManifestAppletTest.jnlp">
+ </applet>
+ </body>
+</html>
+
diff -r c7bf15bbc88e -r 684acbb6c35e tests/reproducers/signed/ClasspathManifestTest/srcs/ClasspathManifest.java
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/reproducers/signed/ClasspathManifestTest/srcs/ClasspathManifest.java Wed Aug 15 14:44:14 2012 -0400
@@ -0,0 +1,81 @@
+/* ClasspathManifest.java
+Copyright (C) 2012 Red Hat, Inc.
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or
+modify it under the terms of the GNU General Public License as published by
+the Free Software Foundation, version 2.
+
+IcedTea is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to
+the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+ */
+
+import java.applet.Applet;
+import java.lang.reflect.*;
+
+public class ClasspathManifest extends Applet {
+
+ private class Killer extends Thread {
+
+ public int n = 1000;
+
+ @Override
+ public void run() {
+ try {
+ Thread.sleep(n);
+ System.out.println("Applet killing himself after " + n + " ms of life");
+ System.exit(0);
+ } catch (Exception ex) {
+ }
+ }
+ }
+ private Killer killer;
+
+ public static void main(String[] args) {
+ searchForClasspath();
+ }
+
+ @Override
+ public void init() {
+ searchForClasspath();
+ killer = new Killer();
+ killer.start();
+ }
+
+ public static void searchForClasspath() {
+ System.out.println("Searching for CheckForClasspath.");
+ try {
+ Class checkClass = Class.forName("CheckForClasspath");
+ Method checkMethod = checkClass.getDeclaredMethod("checkClasspathAndPrint");
+ checkMethod.invoke((Object) null);
+ } catch (Exception ex) {
+ System.out.println("Exception was thrown, class not found on classpath.");
+ ex.printStackTrace();
+ }
+ }
+}
diff -r c7bf15bbc88e -r 684acbb6c35e tests/reproducers/signed/ClasspathManifestTest/srcs/META-INF/MANIFEST.MF
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/reproducers/signed/ClasspathManifestTest/srcs/META-INF/MANIFEST.MF Wed Aug 15 14:44:14 2012 -0400
@@ -0,0 +1,3 @@
+Manifest-Version: 1.0
+Class-Path: Classpath/Manifest/Test/Helper.jar
+
diff -r c7bf15bbc88e -r 684acbb6c35e tests/reproducers/signed/ClasspathManifestTest/testcases/ClasspathManifestTest.java
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/reproducers/signed/ClasspathManifestTest/testcases/ClasspathManifestTest.java Wed Aug 15 14:44:14 2012 -0400
@@ -0,0 +1,134 @@
+/* ClasspathManifestTest.java
+Copyright (C) 2012 Red Hat, Inc.
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or
+modify it under the terms of the GNU General Public License as published by
+the Free Software Foundation, version 2.
+
+IcedTea is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to
+the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+ */
+
+import java.util.ArrayList;
+import java.util.List;
+
+import net.sourceforge.jnlp.ServerAccess;
+import net.sourceforge.jnlp.ServerAccess.ProcessResult;
+import net.sourceforge.jnlp.annotations.Bug;
+import net.sourceforge.jnlp.annotations.KnownToFail;
+import net.sourceforge.jnlp.annotations.NeedsDisplay;
+import net.sourceforge.jnlp.annotations.TestInBrowsers;
+import net.sourceforge.jnlp.browsertesting.BrowserTest;
+import net.sourceforge.jnlp.browsertesting.Browsers;
+
+import org.junit.Assert;
+import org.junit.Test;
+
+ at Bug(id = "PR975")
+public class ClasspathManifestTest extends BrowserTest {
+
+ private static String s1 = "Searching for CheckForClasspath.";
+ private static String s2 = "CheckForClasspath found on classpath.";
+ private static String ss = "xception";
+
+ public void checkAppFails(ProcessResult pr, String testName) {
+ Assert.assertTrue("ClasspathManifest." + testName + " stdout should contain " + s1 + " but didn't", pr.stdout.contains(s1));
+ Assert.assertFalse("ClasspathManifest." + testName + " stdout should not contain " + s2 + " but did", pr.stdout.contains(s2));
+ Assert.assertTrue("ClasspathManifest." + testName + " stderr should contain " + ss + " but didn't", pr.stderr.contains(ss));
+ }
+
+ @NeedsDisplay
+ @Test
+ public void ApplicationJNLPRemoteTest() throws Exception {
+ ProcessResult pr = server.executeJavawsHeadless(null, "/ClasspathManifestApplicationTest.jnlp");
+ checkAppFails(pr, "ApplicationJNLPRemoteTest");
+ }
+
More information about the distro-pkg-dev
mailing list