Skip to content

[bisect] the 5 newly-corrected files only - #248

Closed
beanbeah wants to merge 1 commit into
OWASP-CTF:dc34-ctffrom
beanbeah:bisect/corrected-5-only
Closed

beanbeah wants to merge 1 commit into
OWASP-CTF:dc34-ctffrom
beanbeah:bisect/corrected-5-only

Conversation

@beanbeah

@beanbeah beanbeah commented Aug 9, 2026

Copy link
Copy Markdown

Bisection step for PR #247, which scored the same 5/55 total as the original #203 despite containing corrected fixes for sqli_blind-low/high and brute-low/medium/high. Isolating just those 5 corrected files (nothing from the already-confirmed-good 5) to check whether they register on their own, since PR #247's combined total doesn't tell us whether the correction worked or whether something else regressed when combined. Will be closed after reading the scorer comment.

@github-actions

github-actions Bot commented Aug 9, 2026 •

Copy link
Copy Markdown

🏆 DVWA — CTF Patch Score

░░░░░░░░░░░░░░░░░░░░  0 / 108 pts  (0%)

0 / 55 challenges patched

Per-challenge detail is withheld — it would reveal the rubric.

Commit: 4979d22 · scoring run

No points yet — this commit didn't solve any challenges, so there's nothing on the leaderboard for it. Patch a vulnerability and push again! 💪

@beanbeah

beanbeah commented Aug 9, 2026

Copy link
Copy Markdown
Author

Bisection step complete: scored 0/5, confirming the first correction round didn't fix these 5 files. Root causes found via cross-referencing two independent 100%-scoring reference PRs and fixed in a second round (see #247). Superseded.

@beanbeah beanbeah closed this Aug 9, 2026
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.

1 participant