Skip to content

OSS-Fuzz: Add new fuzzer targets ows - #7525

Merged
geographika merged 3 commits into
MapServer:mainfrom
arthurscchan:new-fuzzer
Aug 21, 2026
Merged

geographika merged 3 commits into
MapServer:mainfrom
arthurscchan:new-fuzzer

Conversation

@arthurscchan

Copy link
Copy Markdown
Contributor

This PR creates a new OSS-Fuzz fuzzer targets OWS.

Comment thread fuzzers/build.sh Outdated
@rouault

rouault commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

@arthurscchan Who is going to fix the issues that might come from that ? Do you have identified funding ?

@arthurscchan

Copy link
Copy Markdown
Contributor Author

Hi @rouault, The CIFuzz failing should be a leak discovery from the fuzzer. Not sure if you want to handle it first?

@rouault

rouault commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

you

left as an exercise to the contributor...

@geographika

Copy link
Copy Markdown
Member

@arthurscchan - now #7539 has been merged, could you rebase this PR against the main branch, and we'll see if the first issue is resolved, thanks.

Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
@arthurscchan

arthurscchan commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor Author

Finally, after fixing the 2 memory leaking issues, the CI passed now. I think this new fuzzer is ready then. Sorry for leaving this PR for a bit long.

@geographika
geographika merged commit a8ae991 into MapServer:main Aug 21, 2026
17 checks passed
@geographika

Copy link
Copy Markdown
Member

Thanks @arthurscchan for seeing this through to completion.

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.

5 participants