From: Dirk Lemstra <dirk@git.imagemagick.org>
Date: Sat, 12 Aug 2017 08:44:24 +0000 (+0200)
Subject: Fixed buffer overflow reported in #670
X-Git-Tag: 7.0.6-7~4
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=5304ae14655a67b9a3db00563fe44d9abd6de4f0;p=imagemagick

Fixed buffer overflow reported in #670
---

diff --git a/coders/sfw.c b/coders/sfw.c
index 5d2cc60c2..4a778ea22 100644
--- a/coders/sfw.c
+++ b/coders/sfw.c
@@ -126,16 +126,15 @@ static unsigned char *SFWScan(const unsigned char *p,const unsigned char *q,
   register ssize_t
     i;
 
-  if ((p+length) < q)
-    while (p < q)
-    {
-      for (i=0; i < (ssize_t) length; i++)
-        if (p[i] != target[i])
-          break;
-      if (i == (ssize_t) length)
-        return((unsigned char *) p);
-      p++;
-    }
+  while ((p+length) < q)
+  {
+    for (i=0; i < (ssize_t) length; i++)
+      if (p[i] != target[i])
+        break;
+    if (i == (ssize_t) length)
+      return((unsigned char *) p);
+    p++;
+  }
   return((unsigned char *) NULL);
 }