From 2366a528aa1bce3f9ccfea5940154dc7c6c947be Mon Sep 17 00:00:00 2001
From: Even Rouault <even.rouault@spatialys.com>
Date: Mon, 28 Oct 2019 15:12:20 +0100
Subject: createOperations(): avoid infinite recursion in a super weird case.
 Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=18587

---
 src/iso19111/coordinateoperation.cpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/iso19111/coordinateoperation.cpp b/src/iso19111/coordinateoperation.cpp
index 881a16fd..0f0e216c 100644
--- a/src/iso19111/coordinateoperation.cpp
+++ b/src/iso19111/coordinateoperation.cpp
@@ -12820,11 +12820,13 @@ CoordinateOperationFactory::Private::createOperations(
             dynamic_cast<const crs::GeographicCRS *>(hubSrc.get());
         auto geogCRSOfBaseOfBoundSrc =
             boundSrc->baseCRS()->extractGeographicCRS();
+        bool triedBoundCrsToGeogCRSSameAsHubCRS = false;
         // Is it: boundCRS to a geogCRS that is the same as the hubCRS ?
         if (hubSrcGeog && geogCRSOfBaseOfBoundSrc &&
             (hubSrcGeog->_isEquivalentTo(
                  geogDst, util::IComparable::Criterion::EQUIVALENT) ||
              hubSrcGeog->is2DPartOf3D(NN_NO_CHECK(geogDst)))) {
+            triedBoundCrsToGeogCRSSameAsHubCRS = true;
             if (boundSrc->baseCRS() == geogCRSOfBaseOfBoundSrc) {
                 // Optimization to avoid creating a useless concatenated
                 // operation
@@ -12938,7 +12940,8 @@ CoordinateOperationFactory::Private::createOperations(
             return res;
         }
 
-        if (hubSrcGeog && geogCRSOfBaseOfBoundSrc) {
+        if (!triedBoundCrsToGeogCRSSameAsHubCRS && hubSrcGeog &&
+            geogCRSOfBaseOfBoundSrc) {
             // This one should go to the above 'Is it: boundCRS to a geogCRS
             // that is the same as the hubCRS ?' case
             auto opsFirst = createOperations(sourceCRS, hubSrc, context);
-- 
cgit v1.2.3