From 3f3af95e0a7f421c84617ce21a2bae1c01368397 Mon Sep 17 00:00:00 2001
From: Even Rouault <even.rouault@spatialys.com>
Date: Mon, 13 Jan 2020 18:09:35 +0100
Subject: unitconvert with mjd time format: avoid potential integer overflow

Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=20072
---
 src/conversions/unitconvert.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'src/conversions/unitconvert.cpp')

diff --git a/src/conversions/unitconvert.cpp b/src/conversions/unitconvert.cpp
index 8472355d..f8439aee 100644
--- a/src/conversions/unitconvert.cpp
+++ b/src/conversions/unitconvert.cpp
@@ -164,7 +164,8 @@ static double decimalyear_to_mjd(double decimalyear) {
     double fractional_year;
     double mjd;
 
-    if( decimalyear < -10000 || decimalyear > 10000 )
+    // Written this way to deal with NaN input
+    if( !(decimalyear >= -10000 && decimalyear <= 10000) )
         return 0;
 
     year = lround(floor(decimalyear));
-- 
cgit v1.2.3