Skip to content

fixed #13440 - do not crash when whole program analysis addon execution fails#7114

Merged
firewave merged 1 commit into
cppcheck-opensource:mainfrom
firewave:addon-json
Jan 1, 2025
Merged

fixed #13440 - do not crash when whole program analysis addon execution fails#7114
firewave merged 1 commit into
cppcheck-opensource:mainfrom
firewave:addon-json

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

No description provided.

…on fails

Co-authored-by: Daniel Marjamäki <daniel.marjamaki@gmail.com>
Comment thread test/cli/other_test.py
print(stderr)
assert stderr.splitlines() == [
"{}:0:0: error: Bailing out from analysis: Checking file failed: Failed to execute addon 'addon.json' - exitcode is {} [internalError]".format(test_file, ec),
":0:0: error: Bailing out from analysis: Whole program analysis failed: Failed to execute addon 'addon.json' - exitcode is {} [internalError]".format(ec)
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be nofile instead of being empty. I will have a look later and file a ticket if it is the case.

@firewave
Copy link
Copy Markdown
Collaborator Author

Replaces #7113.

@firewave firewave merged commit c2313ad into cppcheck-opensource:main Jan 1, 2025
@firewave firewave deleted the addon-json branch January 1, 2025 16:36
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