Skip to content

Clarify that Security Advisory 2915720 is opt-in #1018

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: docs
Choose a base branch
from

Conversation

sigv
Copy link

@sigv sigv commented Jan 6, 2022

WinVerifyTrust and WinVerifyTrustEx functions (wintrust.h) have a note regarding WinVerifyTrust Signature Validation Vulnerability (CVE-2013-3900). This note leads to Security Advisory 2915720 which prompts administrators to configure EnableCertPaddingCheck registry key for additional validation.

The WinVerifyTrust{,Ex} function documentation states that registry key value will be set to "1" by default "on June 10, 2014" however this is not correct. The Advisory linked was amended, as to initially push back the defaults change, and then amended (V1.4) on July 29, 2014 as to clarify there are no active plans to set "1" as default.

These documents are being updated, as to avoid conflicting information and potential resulting confusion among system administrators, who are reading Microsoft documentation to understand the change.

@ghost
Copy link

ghost commented Jan 6, 2022

CLA assistant check
All CLA requirements met.

WinVerifyTrust and WinVerifyTrustEx functions (wintrust.h) have a
note regarding WinVerifyTrust Signature Validation Vulnerability
(CVE-2013-3900). This note leads to Security Advisory 2915720 which
prompts administrators to configure `EnableCertPaddingCheck` registry
key for additional validation.

The WinVerifyTrust{,Ex} function documentation states that registry
key value will be set to "1" by default "on June 10, 2014" however
this is not correct. The Advisory linked was amended, as to initially
push back the defaults change, and then amended (V1.4) on July 29, 2014
as to clarify there are no active plans to set "1" as default.

These documents are being updated, as to avoid conflicting information
and potential resulting confusion among system administrators, who are
reading Microsoft documentation to understand the change.
@sigv sigv force-pushed the advisory-2915720-optin branch from c7ee62e to d4b8ffd Compare January 6, 2022 07:37
@PRMerger3
Copy link

@sigv : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@sigv
Copy link
Author

sigv commented Jan 6, 2022

Force-pushed with updated author details, to comply with CLA requirements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants