Skip to content

Added rule to prevent trivial patching as a solution#135

Merged
xusheng6 merged 2 commits intomainfrom
chore/patch-rules
Jan 20, 2026
Merged

Added rule to prevent trivial patching as a solution#135
xusheng6 merged 2 commits intomainfrom
chore/patch-rules

Conversation

@3rdit
Copy link
Copy Markdown
Contributor

@3rdit 3rdit commented Jan 18, 2026

Adds rule to detail that patching only is not a solution. Closes #133

@3rdit 3rdit requested a review from xusheng6 January 18, 2026 17:23
@xusheng6
Copy link
Copy Markdown
Contributor

I do not think you did the best job to explain it. It is more like that the crackme should not allow patching as a solution unless patching is the theme, and it is not trivial. Users can still make e.g., a keygen that can be patched trivially, but the challenge description should not allow a patching to be considered a valid solution

The motivation for this change is that sometimes the challenge author allows a trivial patching when it should not, and another user would submit a patching solution. When we reject it citing patching, the user might be confused and/or feel mistreated. So we should not allow patching to be allowed in the beginning

@3rdit
Copy link
Copy Markdown
Contributor Author

3rdit commented Jan 20, 2026

@xusheng6 This should be better suited to what you expected?

@xusheng6
Copy link
Copy Markdown
Contributor

@xusheng6 This should be better suited to what you expected?

Exactly!

@xusheng6 xusheng6 merged commit ba502e2 into main Jan 20, 2026
@xusheng6 xusheng6 deleted the chore/patch-rules branch January 20, 2026 13:11
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.

Update crackmes submission rules that the authros should NOT allow patching unless patching is challenging

2 participants