diff options
| author | Manna, Soumi <soumi.manna@intel.com> | 2023-05-19 19:41:27 -0700 |
|---|---|---|
| committer | Manna, Soumi <soumi.manna@intel.com> | 2023-05-19 19:41:38 -0700 |
| commit | 34d8cd153812c3e5e2db1a1e27f057db33b07b6e (patch) | |
| tree | e6b651b5d3c27bf16bb605f9c78068533ff64d76 /lldb/test/Shell/ScriptInterpreter/Python | |
| parent | 068e9889b1ba0856af10d8ba2e21d41d81554533 (diff) | |
| download | llvm-34d8cd153812c3e5e2db1a1e27f057db33b07b6e.zip llvm-34d8cd153812c3e5e2db1a1e27f057db33b07b6e.tar.gz llvm-34d8cd153812c3e5e2db1a1e27f057db33b07b6e.tar.bz2 | |
[NFC][CLANG] Fix issue with dereference null return value found by Coverity static analyzer tool
Reported by Coverity:
In clang::FunctionDecl::isReplaceableGlobalAllocationFunction(std::optional<unsigned int> *, bool *): Return value of function which returns null is dereferenced without checking
if (!IsSizedDelete && !Ty.isNull() && Ty->isEnumeralType()) {
QualType T = Ty;
//Condition TD, taking false branch.
while (const auto *TD = T->getAs<TypedefType>())
T = TD->getDecl()->getUnderlyingType();
//returned_null: getAs returns nullptr (checked 95 out of 97 times).
//Dereference null return value (NULL_RETURNS)
// dereference: Dereferencing a pointer that might be nullptr T->getAs() when calling getDecl.
IdentifierInfo *II = T->getAs<EnumType>()->getDecl()->getIdentifier();
if (II && II->isStr("__hot_cold_t"))
Consume();
}
This patch uses castAs instead of getAs which will assert if the type doesn't match.
Reviewed By: erichkeane
Differential Revision: https://reviews.llvm.org/D150968
Diffstat (limited to 'lldb/test/Shell/ScriptInterpreter/Python')
0 files changed, 0 insertions, 0 deletions
