Index: modules/archive/src/test/java/org/apache/harmony/archive/tests/java/util/jar/JarEntryTest.java =================================================================== --- modules/archive/src/test/java/org/apache/harmony/archive/tests/java/util/jar/JarEntryTest.java (revision 422694) +++ modules/archive/src/test/java/org/apache/harmony/archive/tests/java/util/jar/JarEntryTest.java (working copy) @@ -137,6 +137,9 @@ } assertEquals(2,certs_bob.size()); assertEquals(1,certs_alice.size()); + + assertNull("getCodeSigners() of a primitive JarEntry should return null", + new JarEntry("aaa").getCodeSigners()); } /** Index: modules/archive/src/main/java/java/util/jar/JarEntry.java =================================================================== --- modules/archive/src/main/java/java/util/jar/JarEntry.java (revision 422694) +++ modules/archive/src/main/java/java/util/jar/JarEntry.java (working copy) @@ -134,6 +134,9 @@ } private CodeSigner[] getCodeSigners(Certificate[] certs) { + if(null == certs) { + return null; + } X500Principal prevIssuer = null; ArrayList list = new ArrayList(certs.length);