From c021126079afa2e77416a673818abac6512eba69 Mon Sep 17 00:00:00 2001
From: Even Rouault <even.rouault@spatialys.com>
Date: Sun, 7 Mar 2021 19:18:58 +0100
Subject: pipeline: avoid CLang Static Analyzer 10.0 likely false positive
 about dereferencing a nullptr

---
 src/pipeline.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/pipeline.cpp b/src/pipeline.cpp
index b1989824..88793027 100644
--- a/src/pipeline.cpp
+++ b/src/pipeline.cpp
@@ -465,8 +465,8 @@ PJ *OPERATION(pipeline,0) {
         return destructor (P, PROJ_ERR_OTHER /*ENOMEM*/);
 
     /* Do some syntactical sanity checking */
-    for (i = 0;  i < argc;  i++) {
-        if (0==strcmp (argv_sentinel, argv[i])) {
+    for (i = 0;  i < argc && argv[i] != nullptr;  i++) {
+        if ( 0==strcmp (argv_sentinel, argv[i])) {
             if (-1==i_pipeline) {
                 proj_log_error (P, _("Pipeline: +step before +proj=pipeline"));
                 return destructor (P, PROJ_ERR_INVALID_OP_WRONG_SYNTAX);
-- 
cgit v1.2.3