CodeQL / Analyze (actions) (push) Canceled after 0s
CodeQL / Analyze (python) (push) Canceled after 0s
pre-commit / pre-commit (push) Canceled after 0s
Multi-Arch CI / setup (push) Canceled after 0s
Publish build_manylinux_x86_64 images / publish_build_manylinux_x86_64 (push) Canceled after 0s
Setup (Multi-Arch) / setup (push) Canceled after 0s
Multi-Arch CI / Linux::${{ fromJSON(needs.setup.outputs.linux_build_config || '{}').build_variant_label || 'skip' }} (push) Canceled after 0s
Publish TheRock Docker image / build-and-push-image (push) Canceled after 0s
Publish no_rocm_image_ubi10 image / publish_no_rocm_image_ubi10 (push) Canceled after 0s
Multi-Arch CI / Windows::${{ fromJSON(needs.setup.outputs.windows_build_config || '{}').build_variant_label || 'skip' }} (push) Canceled after 0s
Publish TheRock Docker image / build-and-push-image (push) Canceled after 0s
Multi-Arch CI / Manifest Diff (push) Canceled after 0s
TheRock Manifest Diff Report / Generate Manifest Diff Report (push) Canceled after 0s
Multi-Arch CI / CI Summary (push) Canceled after 0s
Publish no_rocm_image_ubuntu24_04 images / publish_no_rocm_image_ubuntu24_04 (push) Canceled after 0s
Publish no_rocm_image_ubuntu24_04_ocl_rt images / publish_no_rocm_image_ubuntu24_04_ocl_rt (push) Canceled after 0s
Publish TheRock Docker image / build-and-push-image (push) Canceled after 0s
Publish TheRock Docker image / build-and-push-image (push) Canceled after 0s
Publish no_rocm_image_ubuntu24_04 images / publish_no_rocm_image_ubuntu24_04_rocgdb (push) Canceled after 0s
Publish no_rocm_image_ubuntu24_04_rocgdb images / publish_no_rocm_image_ubuntu24_04_rocgdb (push) Canceled after 0s
Publish no_rocm_image_ubuntu24_04 images / publish_no_rocm_image_ubuntu24_04_ocl_rt (push) Canceled after 0s
Publish TheRock Docker image / build-and-push-image (push) Canceled after 0s
Unit Tests / Unit Tests :: ubuntu-24.04 (push) Canceled after 0s
Unit Tests / Unit Tests :: windows-2022 (push) Canceled after 0s
Publish no_rocm_image_ubuntu24_04 images / publish_no_rocm_image_ubuntu24_04_media (push) Canceled after 0s
Unit Tests / Unit Tests Summary (push) Canceled after 0s
Publish no_rocm_image_ubuntu24_04 images / publish_no_rocm_image_ubuntu24_04_openmpi (push) Canceled after 0s
85 lines
3.0 KiB
C++
85 lines
3.0 KiB
C++
//==- ObjCPropertyChecker.cpp - Check ObjC properties ------------*- C++ -*-==//
|
|
//
|
|
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
|
|
// See https://llvm.org/LICENSE.txt for license information.
|
|
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
|
|
//
|
|
//===----------------------------------------------------------------------===//
|
|
//
|
|
// This checker finds issues with Objective-C properties.
|
|
// Currently finds only one kind of issue:
|
|
// - Find synthesized properties with copy attribute of mutable NS collection
|
|
// types. Calling -copy on such collections produces an immutable copy,
|
|
// which contradicts the type of the property.
|
|
//
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
#include "clang/StaticAnalyzer/Checkers/BuiltinCheckerRegistration.h"
|
|
#include "clang/StaticAnalyzer/Core/BugReporter/BugReporter.h"
|
|
#include "clang/StaticAnalyzer/Core/Checker.h"
|
|
|
|
using namespace clang;
|
|
using namespace ento;
|
|
|
|
namespace {
|
|
class ObjCPropertyChecker
|
|
: public Checker<check::ASTDecl<ObjCPropertyDecl>> {
|
|
void checkCopyMutable(const ObjCPropertyDecl *D, BugReporter &BR) const;
|
|
|
|
public:
|
|
void checkASTDecl(const ObjCPropertyDecl *D, AnalysisManager &Mgr,
|
|
BugReporter &BR) const;
|
|
};
|
|
} // end anonymous namespace.
|
|
|
|
void ObjCPropertyChecker::checkASTDecl(const ObjCPropertyDecl *D,
|
|
AnalysisManager &Mgr,
|
|
BugReporter &BR) const {
|
|
checkCopyMutable(D, BR);
|
|
}
|
|
|
|
void ObjCPropertyChecker::checkCopyMutable(const ObjCPropertyDecl *D,
|
|
BugReporter &BR) const {
|
|
if (D->isReadOnly() || D->getSetterKind() != ObjCPropertyDecl::Copy)
|
|
return;
|
|
|
|
QualType T = D->getType();
|
|
if (!T->isObjCObjectPointerType())
|
|
return;
|
|
|
|
const std::string &PropTypeName(T->getPointeeType().getCanonicalType()
|
|
.getUnqualifiedType()
|
|
.getAsString());
|
|
if (!StringRef(PropTypeName).starts_with("NSMutable"))
|
|
return;
|
|
|
|
const ObjCImplDecl *ImplD = nullptr;
|
|
if (const ObjCInterfaceDecl *IntD =
|
|
dyn_cast<ObjCInterfaceDecl>(D->getDeclContext())) {
|
|
ImplD = IntD->getImplementation();
|
|
} else if (auto *CatD = dyn_cast<ObjCCategoryDecl>(D->getDeclContext())) {
|
|
ImplD = CatD->getClassInterface()->getImplementation();
|
|
}
|
|
|
|
if (!ImplD || ImplD->HasUserDeclaredSetterMethod(D))
|
|
return;
|
|
|
|
SmallString<128> Str;
|
|
llvm::raw_svector_ostream OS(Str);
|
|
OS << "Property of mutable type '" << PropTypeName
|
|
<< "' has 'copy' attribute; an immutable object will be stored instead";
|
|
|
|
BR.EmitBasicReport(
|
|
D, this, "Objective-C property misuse", "Logic error", OS.str(),
|
|
PathDiagnosticLocation::createBegin(D, BR.getSourceManager()),
|
|
D->getSourceRange());
|
|
}
|
|
|
|
void ento::registerObjCPropertyChecker(CheckerManager &Mgr) {
|
|
Mgr.registerChecker<ObjCPropertyChecker>();
|
|
}
|
|
|
|
bool ento::shouldRegisterObjCPropertyChecker(const CheckerManager &mgr) {
|
|
return true;
|
|
}
|