[Libreoffice-commits] core.git: compilerplugins/clang

[Stephan Bergmann]

 compilerplugins/clang/derefnullptr.cxx |   44 +++++++++++++++++++++++++++++++++
 1 file changed, 44 insertions(+)

New commits:
commit 90d8b417cab85dc911fe6b69ddf6cca0679abd04
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Wed Jun 11 16:34:32 2014 +0200

    New loplugin:derefnullptr (all violations already fixed)
    
    Change-Id: I32cd8e740855699036c420a091c282029f8d4a08

diff --git a/compilerplugins/clang/derefnullptr.cxx b/compilerplugins/clang/derefnullptr.cxx
new file mode 100644
index 0000000..5663e4f
--- /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: */