Skip to content

Conversation

@chrchr-github
Copy link
Collaborator

No description provided.

@chrchr-github
Copy link
Collaborator Author

Do we need to do something with the new id?

@firewave
Copy link
Collaborator

Do we need to do something with the new id?

Beside triggering it from getErrorMessages() (which you did) I am not aware of anything.

@sonarqubecloud
Copy link


void CheckExceptionSafety::entryPointThrowError(const Token * const tok)
{
reportError(tok, Severity::error, "throwInEntryPoint", "Exception thrown in function that is an entry point.", CWE398, Certainty::normal);
Copy link
Owner

Choose a reason for hiding this comment

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

I suggest something like:

Unhandled exception is thrown in function that is an entry point.

We only warn if it's unhandled right?

This is undefined behavior right? To throw an exception that is not catched..

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.

3 participants