richardphibel / rpms / systemd

Forked from rpms/systemd 2 years ago
Clone
17aa40
From af6eac25456d4ca7e8233e00aec7531e640f17af Mon Sep 17 00:00:00 2001
17aa40
From: Lennart Poettering <lennart@poettering.net>
17aa40
Date: Fri, 5 Apr 2019 15:31:34 +0200
17aa40
Subject: [PATCH] lgtm: warn about strerror() use
17aa40
17aa40
(cherry picked from commit 9ff46eded2b99d244455467eb55c0ff3f51c5362)
17aa40
17aa40
Related: #2017033
17aa40
---
17aa40
 .lgtm/cpp-queries/PotentiallyDangerousFunction.ql | 3 +++
17aa40
 1 file changed, 3 insertions(+)
17aa40
17aa40
diff --git a/.lgtm/cpp-queries/PotentiallyDangerousFunction.ql b/.lgtm/cpp-queries/PotentiallyDangerousFunction.ql
17aa40
index cd0284b37a..96712cf1c6 100644
17aa40
--- a/.lgtm/cpp-queries/PotentiallyDangerousFunction.ql
17aa40
+++ b/.lgtm/cpp-queries/PotentiallyDangerousFunction.ql
17aa40
@@ -38,6 +38,9 @@ predicate potentiallyDangerousFunction(Function f, string message) {
17aa40
   ) or (
17aa40
     f.getQualifiedName() = "ntohs" and
17aa40
     message = "Call to ntohs() is confusing. Use be16toh() instead."
17aa40
+  ) or (
17aa40
+    f.getQualifiedName() = "strerror" and
17aa40
+    message = "Call to strerror() is not thread-safe. Use strerror_r() or printf()'s %m format string instead."
17aa40
   )
17aa40
 }
17aa40