Skip to content

fix(fuzz): avoid keywords in identifiers#239

Merged
apoelstra merged 1 commit intoBlockstreamResearch:masterfrom
gerau:simc/fuzz-tests-fix
Mar 16, 2026
Merged

fix(fuzz): avoid keywords in identifiers#239
apoelstra merged 1 commit intoBlockstreamResearch:masterfrom
gerau:simc/fuzz-tests-fix

Conversation

@gerau
Copy link
Contributor

@gerau gerau commented Mar 16, 2026

Fixes #236.

Sometimes the fuzzer would generate keywords in identifiers and parser would reject this. This PR simply pads any keyword with _, like it was done with other built-ins.

@gerau gerau requested a review from delta1 as a code owner March 16, 2026 13:20
@gerau gerau force-pushed the simc/fuzz-tests-fix branch from c7a7228 to 6384adc Compare March 16, 2026 13:22
@gerau
Copy link
Contributor Author

gerau commented Mar 16, 2026

cc @KyrylR

Copy link
Contributor

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK 6384adc; successfully ran local tests

@apoelstra apoelstra merged commit baff7fa into BlockstreamResearch:master Mar 16, 2026
10 checks passed
@gerau gerau deleted the simc/fuzz-tests-fix branch March 17, 2026 09:25
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.

Fuzz failure on master

2 participants