Skip to content

CWE mapping of stlIfStrFind, stlcstrReturn, stlcstrParam, stlSize, StlMissingComparison, redundantIfRemove.#801

Merged
PKEuS merged 1 commit intocppcheck-opensource:masterfrom
boos:cwe-mapping-5
Jun 5, 2016
Merged

CWE mapping of stlIfStrFind, stlcstrReturn, stlcstrParam, stlSize, StlMissingComparison, redundantIfRemove.#801
PKEuS merged 1 commit intocppcheck-opensource:masterfrom
boos:cwe-mapping-5

Conversation

@boos
Copy link
Copy Markdown
Contributor

@boos boos commented Jun 3, 2016

CWE mapping of stlIfStrFind, stlcstrReturn, stlcstrParam, stlSize, StlMissingComparison, redundantIfRemove

Comment thread lib/checkstl.cpp Outdated
"you should compare with std::string::npos.", CWE597, false);
else
reportError(tok, Severity::warning, "stlIfFind", "Suspicious condition. The result of find() is an iterator, but it is not properly checked.");
reportError(tok, Severity::warning, "stlIfFind", "Suspicious condition. The result of find() is an iterator, but it is not properly checked.", CWE597, false);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don't think that is a matching CWE entry

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Right.

@boos boos force-pushed the cwe-mapping-5 branch from 979d0ff to bd8f1e7 Compare June 4, 2016 10:26
@PKEuS PKEuS merged commit f1c39db into cppcheck-opensource:master Jun 5, 2016
boos added a commit to boos/cppcheck that referenced this pull request Jun 18, 2016
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