|
 |
94503d |
commit 09bcd3ebb639af6cfd83ff2203ffeb80a59cc0eb
|
|
|
94503d |
Author: Jiri Vanek <jvanek@redhat.com>
|
|
|
94503d |
Date: Fri Jun 28 16:05:35 2019 +0200
|
|
|
94503d |
|
|
|
94503d |
All files, except signaturre files, are now checked for signatures
|
|
|
94503d |
|
|
|
94503d |
diff --git a/netx/net/sourceforge/jnlp/tools/JarCertVerifier.java b/netx/net/sourceforge/jnlp/tools/JarCertVerifier.java
|
|
|
94503d |
index 759bedfb..cabfb3c5 100644
|
|
|
94503d |
--- a/netx/net/sourceforge/jnlp/tools/JarCertVerifier.java
|
|
|
94503d |
+++ b/netx/net/sourceforge/jnlp/tools/JarCertVerifier.java
|
|
|
94503d |
@@ -41,6 +41,7 @@
|
|
|
94503d |
import java.util.Map;
|
|
|
94503d |
import java.util.Vector;
|
|
|
94503d |
import java.util.jar.JarEntry;
|
|
|
94503d |
+import java.util.regex.Pattern;
|
|
|
94503d |
|
|
|
94503d |
import net.sourceforge.jnlp.JARDesc;
|
|
|
94503d |
import net.sourceforge.jnlp.JNLPFile;
|
|
|
94503d |
@@ -67,6 +68,7 @@
|
|
|
94503d |
public class JarCertVerifier implements CertVerifier {
|
|
|
94503d |
|
|
|
94503d |
private static final String META_INF = "META-INF/";
|
|
|
94503d |
+ private static final Pattern SIG = Pattern.compile(".*" + META_INF + "SIG-.*");
|
|
|
94503d |
|
|
|
94503d |
// prefix for new signature-related files in META-INF directory
|
|
|
94503d |
private static final String SIG_PREFIX = META_INF + "SIG-";
|
|
|
94503d |
@@ -500,12 +502,20 @@
|
|
|
94503d |
|
|
|
94503d |
/**
|
|
|
94503d |
* Returns whether a file is in META-INF, and thus does not require signing.
|
|
|
94503d |
- *
|
|
|
94503d |
+ *
|
|
|
94503d |
* Signature-related files under META-INF include: . META-INF/MANIFEST.MF . META-INF/SIG-* . META-INF/*.SF . META-INF/*.DSA . META-INF/*.RSA
|
|
|
94503d |
*/
|
|
|
94503d |
static boolean isMetaInfFile(String name) {
|
|
|
94503d |
- String ucName = name.toUpperCase();
|
|
|
94503d |
- return ucName.startsWith(META_INF);
|
|
|
94503d |
+ if (name.endsWith("class")) {
|
|
|
94503d |
+ return false;
|
|
|
94503d |
+ }
|
|
|
94503d |
+ return name.startsWith(META_INF) && (
|
|
|
94503d |
+ name.endsWith(".MF") ||
|
|
|
94503d |
+ name.endsWith(".SF") ||
|
|
|
94503d |
+ name.endsWith(".DSA") ||
|
|
|
94503d |
+ name.endsWith(".RSA") ||
|
|
|
94503d |
+ SIG.matcher(name).matches()
|
|
|
94503d |
+ );
|
|
|
94503d |
}
|
|
|
94503d |
|
|
|
94503d |
/**
|
|
|
94503d |
diff --git a/tests/netx/unit/net/sourceforge/jnlp/tools/JarCertVerifierTest.java b/tests/netx/unit/net/sourceforge/jnlp/tools/JarCertVerifierTest.java
|
|
|
94503d |
index 4661fb87..44253e08 100644
|
|
|
94503d |
--- a/tests/netx/unit/net/sourceforge/jnlp/tools/JarCertVerifierTest.java
|
|
|
94503d |
+++ b/tests/netx/unit/net/sourceforge/jnlp/tools/JarCertVerifierTest.java
|
|
|
94503d |
@@ -58,9 +58,22 @@ public class JarCertVerifierTest {
|
|
|
94503d |
@Test
|
|
|
94503d |
public void testIsMetaInfFile() {
|
|
|
94503d |
final String METAINF = "META-INF";
|
|
|
94503d |
+ assertTrue(JarCertVerifier.isMetaInfFile(METAINF + "/file.MF"));
|
|
|
94503d |
+ assertTrue(JarCertVerifier.isMetaInfFile(METAINF + "/file.SF"));
|
|
|
94503d |
+ assertTrue(JarCertVerifier.isMetaInfFile(METAINF + "/file.DSA"));
|
|
|
94503d |
+ assertTrue(JarCertVerifier.isMetaInfFile(METAINF + "/file.RSA"));
|
|
|
94503d |
+ assertTrue(JarCertVerifier.isMetaInfFile(METAINF + "/SIG-blah.blah"));
|
|
|
94503d |
+
|
|
|
94503d |
+ assertFalse(JarCertVerifier.isMetaInfFile(METAINF + "/file.MF.class"));
|
|
|
94503d |
+ assertFalse(JarCertVerifier.isMetaInfFile(METAINF + "/file.SF.class"));
|
|
|
94503d |
+ assertFalse(JarCertVerifier.isMetaInfFile(METAINF + "/file.DSA.class"));
|
|
|
94503d |
+ assertFalse(JarCertVerifier.isMetaInfFile(METAINF + "/file.RSA.class"));
|
|
|
94503d |
+ assertFalse(JarCertVerifier.isMetaInfFile(METAINF + "/SIG-blah.blah.class"));
|
|
|
94503d |
+
|
|
|
94503d |
assertFalse(JarCertVerifier.isMetaInfFile("some_dir/" + METAINF + "/filename"));
|
|
|
94503d |
assertFalse(JarCertVerifier.isMetaInfFile(METAINF + "filename"));
|
|
|
94503d |
- assertTrue(JarCertVerifier.isMetaInfFile(METAINF + "/filename"));
|
|
|
94503d |
+ assertFalse(JarCertVerifier.isMetaInfFile(METAINF + "/filename"));
|
|
|
94503d |
+ assertFalse(JarCertVerifier.isMetaInfFile(METAINF + "/filename"));
|
|
|
94503d |
}
|
|
|
94503d |
|
|
|
94503d |
class JarCertVerifierEntry extends JarEntry {
|