Skip to content

fuzz: initial integration#117

Merged
Kludex merged 3 commits into
Kludex:masterfrom
manunio:fuzz-initial-integration
Mar 15, 2024
Merged

fuzz: initial integration#117
Kludex merged 3 commits into
Kludex:masterfrom
manunio:fuzz-initial-integration

Conversation

@manunio

@manunio manunio commented Feb 19, 2024

Copy link
Copy Markdown
Contributor

This pr adds few initial fuzz_targets for oss-fuzz integration.
See #48 for more details.

@manunio manunio marked this pull request as draft February 19, 2024 11:34
@manunio manunio force-pushed the fuzz-initial-integration branch from d7fde24 to 0a308ef Compare February 19, 2024 12:10
@manunio manunio marked this pull request as ready for review February 19, 2024 12:10
@manunio

manunio commented Feb 29, 2024

Copy link
Copy Markdown
Contributor Author

@Kludex ping :)

@Kludex

Kludex commented Feb 29, 2024

Copy link
Copy Markdown
Owner

@manunio pong :)

@manunio

manunio commented Mar 11, 2024

Copy link
Copy Markdown
Contributor Author

Hey @Kludex, I hope this message finds you well. Would you be available to take a look at this PR whenever you have a moment?

Once this PR is merged and I have your email address, I can then reach out to the OSS-Fuzz team to inquire about their decision on this project's acceptance.

Thanks :)

@Kludex

Kludex commented Mar 15, 2024

Copy link
Copy Markdown
Owner

atheris doesn't support Python 3.12?

@manunio

manunio commented Mar 15, 2024

Copy link
Copy Markdown
Contributor Author

atheris doesn't support Python 3.12?

According to their readme

Atheris supports Linux (32- and 64-bit) and Mac OS X, Python versions 3.6-3.10.

it only supports 3.6-3.10, while they are testing it against 3.11 in their CI, there is no support for 3.12 yet.

@Kludex

Kludex commented Mar 15, 2024

Copy link
Copy Markdown
Owner

Wanna wait for support, or don't run the fuzz on 3.12?

@manunio

manunio commented Mar 15, 2024

Copy link
Copy Markdown
Contributor Author

Wanna wait for support, or don't run the fuzz on 3.12?

I think it's better to not run it on 3.12, Will you be okay with skipping atheris installation for 3.12?

@Kludex

Kludex commented Mar 15, 2024

Copy link
Copy Markdown
Owner

yes

@manunio

manunio commented Mar 15, 2024

Copy link
Copy Markdown
Contributor Author

yes

Done :)

Comment thread .github/workflows/main.yml Outdated
@manunio manunio force-pushed the fuzz-initial-integration branch from 644b511 to 94ddbb0 Compare March 15, 2024 11:45
@Kludex Kludex merged commit 29829f7 into Kludex:master Mar 15, 2024
@Kludex

Kludex commented Mar 15, 2024

Copy link
Copy Markdown
Owner

Thanks 👍

@manunio

manunio commented Mar 15, 2024

Copy link
Copy Markdown
Contributor Author

Thanks 👍

Should I use your Gmail id for oss-fuzz integration?

@manunio manunio deleted the fuzz-initial-integration branch March 15, 2024 12:07
@Kludex

Kludex commented Mar 15, 2024

Copy link
Copy Markdown
Owner

Yes, please. 👍

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.

2 participants