/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */ /* * This file is part of the LibreOffice project. * * This Source Code Form is subject to the terms of the Mozilla Public * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ #include #include #include #include #include #include #include "plugin.hxx" #include "check.hxx" /** Look for calls where the param is bool but the call-site-arg is pointer. */ #ifndef LO_CLANG_SHARED_PLUGINS namespace { class PointerBool : public loplugin::FilteringPlugin { public: explicit PointerBool(loplugin::InstantiationData const& data) : FilteringPlugin(data) { } virtual void run() override { if (preRun()) TraverseDecl(compiler.getASTContext().getTranslationUnitDecl()); } bool VisitCallExpr(CallExpr const*); private: llvm::Optional getCallValue(const Expr* arg); }; bool PointerBool::VisitCallExpr(CallExpr const* callExpr) { if (ignoreLocation(callExpr)) return true; // TODO this doesn't currently work, the args and the params don't seem to line up if (isa(callExpr)) return true; const FunctionDecl* functionDecl; if (isa(callExpr)) { functionDecl = dyn_cast(callExpr)->getMethodDecl(); } else { functionDecl = callExpr->getDirectCallee(); } if (!functionDecl) return true; unsigned len = std::min(callExpr->getNumArgs(), functionDecl->getNumParams()); for (unsigned i = 0; i < len; ++i) { auto param = functionDecl->getParamDecl(i); auto paramTC = loplugin::TypeCheck(param->getType()); if (!paramTC.AnyBoolean()) continue; auto arg = callExpr->getArg(i)->IgnoreImpCasts(); auto argTC = loplugin::TypeCheck(arg->getType()); if (argTC.AnyBoolean()) continue; // sal_Bool is sometimes disguised if (isa(arg->getType())) if (arg->getType()->getUnqualifiedDesugaredType()->isSpecificBuiltinType( clang::BuiltinType::UChar)) continue; if (arg->getType()->isDependentType()) continue; if (arg->getType()->isIntegerType()) { auto ret = getCallValue(arg); if (ret.hasValue() && (ret.getValue() == 1 || ret.getValue() == 0)) continue; // something like: priv->m_nLOKFeatures & LOK_FEATURE_DOCUMENT_PASSWORD if (isa(arg->IgnoreParenImpCasts())) continue; // something like: pbEmbolden ? FcTrue : FcFalse if (isa(arg->IgnoreParenImpCasts())) continue; } report(DiagnosticsEngine::Warning, "possibly unwanted implicit conversion when calling bool param", arg->getExprLoc()) << arg->getSourceRange(); report(DiagnosticsEngine::Note, "method here", param->getLocation()) << param->getSourceRange(); } return true; } llvm::Optional PointerBool::getCallValue(const Expr* arg) { arg = arg->IgnoreParenCasts(); if (auto defArg = dyn_cast(arg)) { arg = defArg->getExpr()->IgnoreParenCasts(); } // ignore this, it seems to trigger an infinite recursion if (isa(arg)) { return llvm::Optional(); } APSInt x1; if (compat::EvaluateAsInt(arg, x1, compiler.getASTContext())) { return x1; } return llvm::Optional(); } loplugin::Plugin::Registration pointerbool("pointerbool"); } // namespace #endif // LO_CLANG_SHARED_PLUGINS /* vim:set shiftwidth=4 softtabstop=4 expandtab: */