Skip to content

Commit 45ee02a

Browse files
committed
SKI is expected to be always octet wrapped - do not check for length
resolves GH-131
1 parent 06431cd commit 45ee02a

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

src/main/java/org/jruby/ext/openssl/x509store/X509Utils.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -231,10 +231,7 @@ public static int checkIfIssuedBy(final X509AuxCertificate issuer,
231231
if ( sakid.getKeyIdentifier() != null ) {
232232
if ( issuer.getExtensionValue("2.5.29.14") != null ) {
233233
DEROctetString der = (DEROctetString) get(issuer.getExtensionValue("2.5.29.14"));
234-
if ( der.getOctets().length > 20 ) {
235-
der = (DEROctetString) get(der.getOctets());
236-
}
237-
SubjectKeyIdentifier iskid = SubjectKeyIdentifier.getInstance(der);
234+
SubjectKeyIdentifier iskid = SubjectKeyIdentifier.getInstance(get(der.getOctets()));
238235
if ( iskid.getKeyIdentifier() != null ) {
239236
if ( ! Arrays.equals( sakid.getKeyIdentifier(), iskid.getKeyIdentifier() ) ) {
240237
return V_ERR_AKID_SKID_MISMATCH;

0 commit comments

Comments
 (0)