Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fuzz_dlltool issue 387378117 #13064

Merged
merged 1 commit into from
Feb 19, 2025
Merged

fuzz_dlltool issue 387378117 #13064

merged 1 commit into from
Feb 19, 2025

Conversation

amodra
Copy link
Contributor

@amodra amodra commented Feb 19, 2025

This fixes a dlltool leak that is only seen under oss-fuzz, ie. it is not seen as a direct leak when running dlltool stand-alone. The fix to defparse.y which I'd commit upstream if we didn't fuss about posix compliance, is instead edited by build.sh. This patch also updates fuzz_dlltool.c for upstream changes, and removes an unnecessary sed command in build.sh.

This fixes a dlltool leak that is only seen under oss-fuzz, ie. it is
not seen as a direct leak when running dlltool stand-alone.  The fix
to defparse.y which I'd commit upstream if we didn't fuss about posix
compliance, is instead edited by build.sh.  This patch also updates
fuzz_dlltool.c for upstream changes, and removes an unnecessary sed
command in build.sh.
Copy link

amodra is either the primary contact or is in the CCs list of projects/binutils.
amodra has previously contributed to projects/binutils. The previous PR was #12849

@DavidKorczynski DavidKorczynski merged commit 9af5384 into google:master Feb 19, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants