From: Cristy <urban-warrior@imagemagick.org>
Date: Thu, 25 Jan 2018 12:15:55 +0000 (-0500)
Subject: Eliminate Use-of-uninitialized-value in PerceptibleReciprocal
X-Git-Tag: 7.0.7-23~279
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3afd43e265b53a8b1c6ff973978a1053d4b574e6;p=imagemagick

Eliminate Use-of-uninitialized-value in PerceptibleReciprocal
Credit OSS Fuzz
---

diff --git a/MagickCore/geometry.c b/MagickCore/geometry.c
index 7cc3b06a5..686ccb6df 100644
--- a/MagickCore/geometry.c
+++ b/MagickCore/geometry.c
@@ -869,6 +869,7 @@ MagickExport MagickStatusType ParseGeometry(const char *geometry,
     Remove whitespaces meta characters from geometry specification.
   */
   assert(geometry_info != (GeometryInfo *) NULL);
+  (void) ResetMagickMemory(geometry_info,0,sizeof(geometry_info));
   flags=NoValue;
   if ((geometry == (char *) NULL) || (*geometry == '\0'))
     return(flags);
@@ -1119,7 +1120,8 @@ MagickExport MagickStatusType ParseGeometry(const char *geometry,
       /*
         Normalize sampling factor (e.g. 4:2:2 => 2x1).
       */
-      geometry_info->rho*=PerceptibleReciprocal(geometry_info->sigma);
+      if ((flags & SigmaValue) != 0)
+        geometry_info->rho*=PerceptibleReciprocal(geometry_info->sigma);
       geometry_info->sigma=1.0;
       if (geometry_info->xi == 0.0)
         geometry_info->sigma=2.0;