Skip to content
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

Convert cwes list to markdown with fewer rows and columns #737

Merged
merged 6 commits into from
Mar 6, 2025

Conversation

sei-renae
Copy link
Contributor

@sei-renae sei-renae commented Mar 5, 2025

Resolves #529
Resolves #530
Resolves #713

The CWEs list on reference/decision_points/exploitation/#cwe-ids-for-poc has many rows wherein the 'Possible POC' = 'no'; therefore, these rows are irrelevant to Exploitation. In rows where 'Possible POC' = 'no', 'In NVD's CWE slice' is also always 'no' and information about the exploit is nan.
Furthermore, #529 describes a hypothetical list of CWEs, but was not updated after this list was obtained.

This PR cleans the CWEs list to that:

The new table will look like this:
image

The PR also updates the language described in SSVC/docs/topics/information_sources.md to resolve #529 .

…ide scrolling, add hyperlinks, and create a markdown version of the csv per issues 530 and 713.
@sei-renae
Copy link
Contributor Author

I didn't find where the link checker failed - would appreciate a second set of eyes.

@sei-renae sei-renae marked this pull request as ready for review March 6, 2025 13:57
Since CWE classifies clusters of related vulnerabilities, the community could likely curate a list of CWE-IDs for which this condition of well known exploit technique is satisfied.
Once that list were curated, it could be used to automatically populate a CVE-ID as [*proof of concept*](../reference/decision_points/exploitation.md) if the CWE-ID of which it is an instance is on the list.
Such a check could not be exhaustive, since there are other conditions that satisfy [*proof of concept*](../reference/decision_points/exploitation.md).
We provide a non-exhaustive [list of CWE-IDs with known proofs of concept](../reference/decision_points/exploitation/#cwe-ids-for-poc). This is list is non-exhaustive becuase there are other conditions that satisfy [*proof of concept*](../reference/decision_points/exploitation.md).
Copy link
Contributor

Choose a reason for hiding this comment

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

link checker is choking on the first url in this line. I think it needs to end in exploitation.md#cwe-ids-for-poc instead

@ahouseholder
Copy link
Contributor

I'd also request that this PR remove the data/csvs/cwe directory and the data/csvs/cwe/possible-cwe-with-poc-examples.csv file it contains. That way we can be clearer on where the canonical information resides (in the new markdown table already included in this PR).

@sei-renae sei-renae requested a review from ahouseholder March 6, 2025 20:54
@ahouseholder ahouseholder merged commit affa90f into CERTCC:main Mar 6, 2025
3 checks passed
@sei-renae sei-renae deleted the 713 branch March 6, 2025 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants