Bug 239360 [sec] Unchecked SecurityException thrown when SignatureException makes more sense

2 files changed, 16 insertions, 0 deletions

diff --git a/bundles/org.eclipse.osgi.tests/src/org/eclipse/osgi/tests/security/SignedBundleTest.java b/bundles/org.eclipse.osgi.tests/src/org/eclipse/osgi/tests/security/SignedBundleTest.java
index f292fd9d1..927aad040 100644
--- a/bundles/org.eclipse.osgi.tests/src/org/eclipse/osgi/tests/security/SignedBundleTest.java
+++ b/bundles/org.eclipse.osgi.tests/src/org/eclipse/osgi/tests/security/SignedBundleTest.java
@@ -11,6 +11,7 @@
 package org.eclipse.osgi.tests.security;
 
 import java.io.File;
+import java.security.SignatureException;
 import java.security.cert.Certificate;
 import java.security.cert.X509Certificate;
 import junit.framework.Test;
@@ -797,4 +798,19 @@ public class SignedBundleTest extends BaseSecurityTest {
 		signedFile.delete();
 		assertFalse("File should not exist", signedFile.exists());
 	}
+
+	public void testBug236329_01() throws Exception {
+		File signedFile = copyEntryFile(getTestJarPath("signed_with_sf_corrupted"));
+
+		assertNotNull("Could not find signed file!", signedFile);
+		//getTrustEngine().addTrustAnchor(anchor, alias);
+
+		// get the signed content for the bundle
+		try {
+			getSignedContentFactory().getSignedContent(signedFile);
+			fail("Should have gotten a SignatureException for file: " + signedFile);
+		} catch (SignatureException e) {
+			// expected
+		}
+	}
 }
diff --git a/bundles/org.eclipse.osgi.tests/test_files/security/bundles/signed_with_sf_corrupted.jar b/bundles/org.eclipse.osgi.tests/test_files/security/bundles/signed_with_sf_corrupted.jar
new file mode 100644
index 000000000..99b955046
--- /dev/null
+++ b/bundles/org.eclipse.osgi.tests/test_files/security/bundles/signed_with_sf_corrupted.jar