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

Links to incomplete fuzz-introspector reports #7601

Closed
evverx opened this issue Apr 21, 2022 · 7 comments
Closed

Links to incomplete fuzz-introspector reports #7601

evverx opened this issue Apr 21, 2022 · 7 comments

Comments

@evverx
Copy link
Contributor

evverx commented Apr 21, 2022

https://oss-fuzz-introspector.storage.googleapis.com/index.html seems to point to incomplete reports and I think it's confusing: #7599

@evverx
Copy link
Contributor Author

evverx commented Apr 21, 2022

I think it should point to the latest complete reports instead

@Navidem
Copy link
Contributor

Navidem commented Apr 21, 2022

Is this only for the systemd case, or you may have other instances?
The index is to point to the latest successful fuzz introspector build, it does not have any sanity check on the generated reports. For that purpose something like #7593 (comment) is being considered.

@evverx
Copy link
Contributor Author

evverx commented Apr 21, 2022

The index is to point to the latest successful fuzz introspector build, it does not have any sanity check on the generated reports.

I think this issue can be closed then because the checks discussed in #7593 (comment) should prevent issues like that.

@evverx evverx closed this as completed Apr 21, 2022
@DavidKorczynski
Copy link
Collaborator

It looks like there also was a regression, e.g. this is the elfutils report a month earlier https://storage.googleapis.com/oss-fuzz-introspector/elfutils/inspector-report/20220321/fuzz_report.html and coverage looks more accurate. Am actually working on the coverage as I'm writing so will look at that too

@evverx
Copy link
Contributor Author

evverx commented Apr 21, 2022

I'm still puzzled as to why no functions were covered at runtime according to the latest reports. Looks like the fuzz targets weren't run at all.

@evverx
Copy link
Contributor Author

evverx commented Apr 29, 2022

I'm still puzzled as to why no functions were covered at runtime according to the latest reports. Looks like the fuzz targets weren't run at all

For the record It was reported once again and fixed in #7635

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

No branches or pull requests

3 participants