Skip to content

Add analyzer detecting let! _ = ... .#12

Draft
fkj wants to merge 1 commit intomasterfrom
add-let-wildcard-result-analyzer
Draft

Add analyzer detecting let! _ = ... .#12
fkj wants to merge 1 commit intomasterfrom
add-let-wildcard-result-analyzer

Conversation

@fkj
Copy link
Collaborator

@fkj fkj commented Nov 21, 2025

This detects a pattern that makes it easy to accidentally introduce subtle security bugs.

This detects a pattern that makes it easy to accidentally introduce
subtle security bugs.
@fkj
Copy link
Collaborator Author

fkj commented Nov 21, 2025

Results of running on the Verify solution

64 occurrences detected, none of which were false positives.
Most occurrences were in test code or in the management API. Most occurrences were validation code which is the target of the analyzer.
Almost of the occurrences look like they could be fixed with minor rewrites.

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