Skip to content

adding fuzzing harnesses for OSS-Fuzz - #1559

Open
mindlaess wants to merge 1 commit into
PowerGridModel:mainfrom
mindlaess:add-fuzzing-harnesses
Open

mindlaess wants to merge 1 commit into
PowerGridModel:mainfrom
mindlaess:add-fuzzing-harnesses

Conversation

@mindlaess

Copy link
Copy Markdown

Adds tests/fuzzer/ — a coverage-guided harness for the C API JSON deserialization path (deserialize -> build model -> power-flow / short-circuit), plus a seed corpus and dictionary. Gated behind PGM_ENABLE_FUZZER (default OFF, independent of the dev/test build) and runs under AFL++ by default; libFuzzer via -DPGM_FUZZER_ENGINE=libfuzzer.

Once the harness is merged into main, a PR will be performed to Google's OSS-Fuzz to onboard the power-grid-model prohect.

@TonyXiang8787 TonyXiang8787 added the improvement Improvement on internal implementation label Aug 31, 2026
@nitbharambe

Copy link
Copy Markdown
Member

Thanks for creating the fuzz test PR. Would you have time to give a small demo on it and also provide a reviewing and maintenance methodology and recommendations on it?

@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo added the do-not-merge This should not be merged temporarily label Sep 7, 2026
@Jerry-Jinfeng-Guo

Jerry-Jinfeng-Guo commented Sep 7, 2026

Copy link
Copy Markdown
Member

Added a do-not-merge label following an meeting with contributor. Pending further addition to the feature. Label added to avoid accidental merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge This should not be merged temporarily improvement Improvement on internal implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants