diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2014-06-11 16:34:32 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2014-06-11 16:34:55 +0200 |
commit | 90d8b417cab85dc911fe6b69ddf6cca0679abd04 (patch) | |
tree | 0e7ee556e5e851874c183bcfe10638bc51ba6c8e /compilerplugins | |
parent | aa6e78eeefac7af70bfa51892715fe03bea755db (diff) |
New loplugin:derefnullptr (all violations already fixed)
Change-Id: I32cd8e740855699036c420a091c282029f8d4a08
Diffstat (limited to 'compilerplugins')
-rw-r--r-- | compilerplugins/clang/derefnullptr.cxx | 44 |
1 files changed, 44 insertions, 0 deletions
diff --git a/compilerplugins/clang/derefnullptr.cxx b/compilerplugins/clang/derefnullptr.cxx new file mode 100644 index 000000000000..5663e4f27d46 --- /dev/null +++ b/compilerplugins/clang/derefnullptr.cxx @@ -0,0 +1,44 @@ +/* -*- 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 "plugin.hxx" + +namespace { + +class DerefNullPtr: + public RecursiveASTVisitor<DerefNullPtr>, public loplugin::Plugin +{ +public: + explicit DerefNullPtr(InstantiationData const & data): Plugin(data) {} + + void run() override + { TraverseDecl(compiler.getASTContext().getTranslationUnitDecl()); } + + bool VisitUnaryDeref(UnaryOperator const * op); +}; + +bool DerefNullPtr::VisitUnaryDeref(UnaryOperator const * op) { + if (!ignoreLocation(op) + && (op->getSubExpr()->IgnoreParenCasts()->isNullPointerConstant( + compiler.getASTContext(), Expr::NPC_NeverValueDependent/*TODO*/) + != Expr::NPCK_NotNull)) + { + report( + DiagnosticsEngine::Warning, "null pointer dereference", + op->getLocStart()) + << op->getSourceRange(); + } + return true; +} + +loplugin::Plugin::Registration<DerefNullPtr> X("derefnullptr"); + +} + +/* vim:set shiftwidth=4 softtabstop=4 expandtab: */ |