createOperations(): avoid harmless floating-point division by zero if conversion factor of target unit is 0. Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=39969

1 files changed, 7 insertions, 0 deletions

diff --git a/src/iso19111/operation/coordinateoperationfactory.cpp b/src/iso19111/operation/coordinateoperationfactory.cpp
index b59eeb91..e9bd3cfe 100644
--- a/src/iso19111/operation/coordinateoperationfactory.cpp
+++ b/src/iso19111/operation/coordinateoperationfactory.cpp
@@ -4347,6 +4347,9 @@ void CoordinateOperationFactory::Private::createOperationsBoundToGeog(
                     const bool heightDepthReversal =
                         ((srcIsUp && dstIsDown) || (srcIsDown && dstIsUp));
 
+                    if (convDst == 0)
+                        throw InvalidOperation(
+                            "Conversion factor of target unit is 0");
                     const double factor = convSrc / convDst;
                     auto conv = Conversion::createChangeVerticalUnit(
                         util::PropertyMap().set(
@@ -4460,6 +4463,8 @@ void CoordinateOperationFactory::Private::createOperationsVertToVert(
     const bool heightDepthReversal =
         ((srcIsUp && dstIsDown) || (srcIsDown && dstIsUp));
 
+    if (convDst == 0)
+        throw InvalidOperation("Conversion factor of target unit is 0");
     const double factor = convSrc / convDst;
     if (!equivalentVDatum) {
         auto name = buildTransfName(sourceCRS->nameStr(), targetCRS->nameStr());
@@ -4557,6 +4562,8 @@ void CoordinateOperationFactory::Private::createOperationsVertToGeogBallpark(
     const bool heightDepthReversal =
         ((srcIsUp && dstIsDown) || (srcIsDown && dstIsUp));
 
+    if (convDst == 0)
+        throw InvalidOperation("Conversion factor of target unit is 0");
     const double factor = convSrc / convDst;
 
     const auto &sourceCRSExtent = getExtent(sourceCRS);