-
Notifications
You must be signed in to change notification settings - Fork 59
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
[Java-Integration]: Coverage report #612
Conversation
Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
Two separate items to be done in this PR.
|
Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
Fix bug for file long function handling Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
d138b3e
to
a328a26
Compare
Signed-off-by: Arthur Chan <[email protected]>
a328a26
to
9a28282
Compare
Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
Signed-off-by: Arthur Chan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
# "First node in calltree seems to be non-fuzzer function" | ||
# ) | ||
|
||
if not profile.func_is_entrypoint(demangled_name): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one we should watch out for as I'm not sure the condition will always be true. I'm thinking with regards to Python here.
# Sort function and lines numbers for each coverage file. | ||
# Store in function_internals. | ||
logger.debug("Function intervals") | ||
logger.debug("Geting function start and end line") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: typo
Generate Jacoco coverage report and link to the fuzz-introspector call tree.
Related to Steps 5~6 of issue #536