Skip to content

fix: correct official names for multiple CWEs in mapped lists#937

Open
NureddinSoltan wants to merge 4 commits intoOWASP:masterfrom
NureddinSoltan:fix/cwe-298-299-211
Open

fix: correct official names for multiple CWEs in mapped lists#937
NureddinSoltan wants to merge 4 commits intoOWASP:masterfrom
NureddinSoltan:fix/cwe-298-299-211

Conversation

@NureddinSoltan
Copy link
Copy Markdown

Related Issue:

Related to #936

What was wrong?

Several CWEs were listed under the List of Mapped CWEs sections with outdated or incorrect names that did not match the official CWE dictionary

What was changed?

Updated the following entries to match the official CWE dictionary:

CWE ID Previous Name Correct Official Name
CWE-221 Information Loss of Omission Information Loss or Omission
CWE-298 Improper Validation of Certificate with Host Mismatch Improper Validation of Certificate Expiration
CWE-299 Improper Validation of Certificate with Host Mismatch Improper Check for Certificate Revocation

* [CWE-298 Improper Validation of Certificate Expiration](https://cwe.mitre.org/data/definitions/298.html)

* [CWE-299 Improper Validation of Certificate with Host Mismatch](https://cwe.mitre.org/data/definitions/299.html)
* [CWE-299 Improper Check for Certificate Revocation](https://cwe.mitre.org/data/definitions/299.html)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was already suggested in #923. Otherwise the change LGTM.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, I see. Alright, that's perfect. I didn't notice that. Also, I have CWE-299, so I can still keep that PR open. I’ve got another one for #931.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants