Bug 1301683 - openssl: X509_verify_cert() ignores EKU extension of trust anchors
Summary: openssl: X509_verify_cert() ignores EKU extension of trust anchors
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Security Response
Classification: Other
Component: vulnerability
Version: unspecified
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Red Hat Product Security
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1301692
TreeView+ depends on / blocked
 
Reported: 2016-01-25 16:52 UTC by Adam Mariš
Modified: 2021-06-01 15:37 UTC (History)
25 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-03-24 06:45:46 UTC
Embargoed:

Attachments (Terms of Use)
Proposed patch from upstream (1.82 KB, patch)
2016-02-15 16:20 UTC, Adam Mariš 		no flags Details | Diff
Proposed patch from upstream (1.84 KB, patch)
2016-02-15 16:21 UTC, Adam Mariš 		no flags Details | Diff
Master/mitaka patch (11.24 KB, patch)
2016-02-15 16:21 UTC, Adam Mariš 		no flags Details | Diff
Proposed patch from upstream (6.88 KB, patch)
2016-02-15 16:25 UTC, Adam Mariš 		no flags Details | Diff
View All

Description Adam Mariš 2016-01-25 16:52:53 UTC 
It was reported that X509_verify_cert() ignores the extended key usage extension of trust anchors. Only EKU extensions of intermediate and leaf certs are checked.
Comment 3 Adam Mariš 2016-01-27 09:50:07 UTC 
Quoting maintainer:

"This is a known behaviour, that is already planned to be addressed
in OpenSSL 1.1.0. In fact the EKU is only checked for certificates
from the peer, and not those from the trust store, so if your CAfile
or CApath contains intermediate certificates that get used to build
the chain, those won't be checked either."
Comment 4 Adam Mariš 2016-02-15 16:20:40 UTC 
Created attachment 1127327 [details]
Proposed patch from upstream
Comment 5 Adam Mariš 2016-02-15 16:21:27 UTC 
Created attachment 1127328 [details]
Proposed patch from upstream
Comment 6 Adam Mariš 2016-02-15 16:21:47 UTC 
Created attachment 1127329 [details]
Master/mitaka patch

Proposed patch from upstream
//ignore "master/mitaka" name
Comment 7 Adam Mariš 2016-02-15 16:25:24 UTC 
Created attachment 1127330 [details]
Proposed patch from upstream
Comment 9 Huzaifa S. Sidhpurwala 2017-03-24 06:34:28 UTC 
This patch was applied to upstream master branch to fix this issue:

https://github.com/openssl/openssl/commit/33cc5dde478ba5ad79f8fd4acd8737f0e60e236e
Note You need to log in before you can comment on or make changes to this bug.