diff options
| author | Michael Brown | 2014-11-24 15:55:07 +0100 |
|---|---|---|
| committer | Michael Brown | 2014-11-24 16:05:43 +0100 |
| commit | 5cbdc41778622c07429e00f5aee383b575532bf0 (patch) | |
| tree | fb2e74c0565b9725332b0511ebf0f9fc3f91d1a2 /src/drivers | |
| parent | [intel] Use autoloaded MAC address instead of EEPROM MAC address (diff) | |
| download | ipxe-5cbdc41778622c07429e00f5aee383b575532bf0.tar.gz ipxe-5cbdc41778622c07429e00f5aee383b575532bf0.tar.xz ipxe-5cbdc41778622c07429e00f5aee383b575532bf0.zip | |
[crypto] Fix parsing of OCSP responder ID key hash
We currently compare the entirety of the KeyHash object (including the
ASN.1 tag and length byte) against the raw SHA-1 hash of the
certificate's public key. This causes OCSP validation to fail for any
responses which identify the responder by key hash rather than by
name, and hence prevents the use of X.509 certificates where any
certificate in the chain has an OCSP responder which chooses to
identify itself via its key hash.
Fix by adding the missing asn1_enter() required to enter the ASN.1
octet string containing the key hash.
Also add a corresponding test case including an OCSP response where
the responder is identified by key hash, to ensure that this
functionality cannot be broken in future.
Debugged-by: Brian Rak <brak@gameservers.com>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
Diffstat (limited to 'src/drivers')
0 files changed, 0 insertions, 0 deletions