Add null-checks around getter/setter before using them.

clang static analyzer fixit.

llvm-svn: 220024
This commit is contained in:
Jason Molenda 2014-10-17 01:55:08 +00:00
parent 93a13de304
commit 9bdc4e1a59
1 changed files with 10 additions and 4 deletions

View File

@ -5370,9 +5370,12 @@ ClangASTType::AddObjCClassProperty (const char *property_name,
if (getter && metadata)
ClangASTContext::SetMetadata(m_ast, getter, *metadata);
getter->setMethodParams(*m_ast, llvm::ArrayRef<clang::ParmVarDecl*>(), llvm::ArrayRef<clang::SourceLocation>());
if (getter)
{
getter->setMethodParams(*m_ast, llvm::ArrayRef<clang::ParmVarDecl*>(), llvm::ArrayRef<clang::SourceLocation>());
class_interface_decl->addDecl(getter);
class_interface_decl->addDecl(getter);
}
}
if (!setter_sel.isNull() && !class_interface_decl->lookupInstanceMethod(setter_sel))
@ -5417,9 +5420,12 @@ ClangASTType::AddObjCClassProperty (const char *property_name,
clang::SC_Auto,
nullptr));
setter->setMethodParams(*m_ast, llvm::ArrayRef<clang::ParmVarDecl*>(params), llvm::ArrayRef<clang::SourceLocation>());
if (setter)
{
setter->setMethodParams(*m_ast, llvm::ArrayRef<clang::ParmVarDecl*>(params), llvm::ArrayRef<clang::SourceLocation>());
class_interface_decl->addDecl(setter);
class_interface_decl->addDecl(setter);
}
}
return true;