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.

Sign up for GitHub

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

Jump to bottom

lua: introduce fuzz introspector config #11040

Merged
merged 1 commit into from
Sep 29, 2023
Merged

lua: introduce fuzz introspector config #11040

merged 1 commit into from
Sep 29, 2023

Conversation

ligurio
Copy link
Contributor

@ligurio ligurio commented Sep 29, 2023 •
edited
Loading

Follows up #10931

@github-actions
Copy link

ligurio is either the primary contact or is in the CCs list of projects/lua.
ligurio has previously contributed to projects/lua. The previous PR was #10956

@ligurio
Copy link
Contributor Author

ligurio commented Sep 29, 2023

@DavidKorczynski David, could you review introduced config?
The idea is to exclude libprotobuf-mutator and protobuf sources from Fuzz Introspector report (https://storage.googleapis.com/oss-fuzz-introspector/lua/inspector-report/20230925/fuzz_report.html#Fuzzer:-luaL_loadbuffer_proto_test)

@DavidKorczynski
Copy link
Collaborator

I ran it locally and this is the result (see the number of functions at the button) Screenshot from 2023-09-29 11-28-31

Looks good?

@ligurio
Copy link
Contributor Author

ligurio commented Sep 29, 2023

Seems looks correct. The current number of functions is 9,022.

@DavidKorczynski DavidKorczynski merged commit 5de8c91 into google:master Sep 29, 2023
15 checks passed
@ligurio ligurio deleted the ligurio/fuzz-introspector-config branch September 29, 2023 11:41
@ligurio
Copy link
Contributor Author

ligurio commented Oct 3, 2023

After merging Fuzz Introspector config all tests except fuzz_lua has been removed from Fuzz Introspector report, see https://storage.googleapis.com/oss-fuzz-introspector/lua/inspector-report/20230930/fuzz_report.html
Seems removing testdir/tests with test's source code was superfluous.

@ligurio ligurio mentioned this pull request Oct 3, 2023
Merged
DavidKorczynski pushed a commit that referenced this pull request Oct 3, 2023
@ligurio
Fix fuzz introspector config (#11063)
b416f07 
Tarantool's Fuzz Introspector report is not generated at all, see [^1]
Lua's Fuzz Introspector config is missed all tests except `fuzz_lua`,
see [^2].
Proposed patches fixes that.

[^1]:
https://introspector.oss-fuzz.com/project-profile?project=tarantool
[^2]:
https://storage.googleapis.com/oss-fuzz-introspector/lua/inspector-report/20230930/fuzz_report.html

Follows up #11040
Follows up #11041
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DavidKorczynski DavidKorczynski DavidKorczynski approved these changes

Assignees
No one assigned
Labels
ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ligurio @DavidKorczynski