feat(data): improve configuration vulnerability annotations for Issue #8#57
Open
Sam-Hui-dot wants to merge 1 commit into
Open
feat(data): improve configuration vulnerability annotations for Issue #8#57Sam-Hui-dot wants to merge 1 commit into
Sam-Hui-dot wants to merge 1 commit into
Conversation
Signed-off-by: Sam-Hui-dot <19303092837@163.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR improves the annotation semantics for configuration-class and non-taint vulnerabilities in VulnGym.
It updates the four OpenClaw Dockerfile entries covered by Issue #8:
entry-00241entry-00242entry-00243entry-00244The annotations now model configuration-state evidence rather than pretending that these samples contain a traditional attacker-controlled taint flow.
Changes
data/entries.jsonl.FROMas the configuration-state entry point for Dockerfiles that inherit the default root user without a laterUSERdirective.USERsamples:entry-00241:CMD ["bash"]entry-00242:RUN pnpm install --frozen-lockfileentry-00243: the test-imageENTRYPOINTNOPASSWD:ALLsudoers rule at line 18 as the critical operation forentry-00244.verify=0because the schema reservesverify=1for maintainer-confirmed human audit.reports/config_vuln_annotation.mdwith reusable annotation rules.reports/issue-8-config-annotations.diff.csvwith field-level before/after changes and reasons.scripts/apply_issue8_config_annotations.pyfor reproducible generation and validation.Evidence
GHSA-W7J5-J98M-W679c56fb7f353d63d6ea97028ee7d8a97bc4edf21c184281abd4bd1c717bb37a2de12694fe203827eeeThe validator confirms that:
{file, line, code}nodes match the vulnerable OpenClaw commit exactly.USERdirective.USER appuserto those three Dockerfiles.entry-00244is handled separately because it hasUSER appbut grants unrestricted passwordless sudo.Validation
USERDockerfiles passed.git diff --checkpassed.Scope
This PR changes only the four Issue #8 target entries and the supporting documentation, generator, diff artifact, and tests.
Closes #8