]> granicus.if.org Git - imagemagick/commitdiff
...
authorCristy <urban-warrior@imagemagick.org>
Thu, 9 Aug 2018 22:36:55 +0000 (18:36 -0400)
committerCristy <urban-warrior@imagemagick.org>
Thu, 9 Aug 2018 22:36:55 +0000 (18:36 -0400)
ChangeLog
coders/xbm.c

index ff2c7d48e9d911d009ed125732f8bc56699bbf5b..cd4523255674e01cdd28e51a25bbac0e3857a290 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2018-08-08  7.0.8-10 Cristy  <quetzlzacatenango@image...>
+  * ThumbnailImage function no longer reveals sensitive information (reference
+    https://github.com/ImageMagick/ImageMagick/issues/1243).
+
 2018-08-06  7.0.8-9 Cristy  <quetzlzacatenango@image...>
   * Release ImageMagick version 7.0.8-9, GIT revision 14618:a3663c3dc:20180805.
 
index 0cc82339f1a5b37a5fe8c6aae62076dcb0e41c6a..a974825ad8801621fac98869a41fad8423c923d3 100644 (file)
@@ -132,18 +132,18 @@ static MagickBooleanType IsXBM(const unsigned char *magick,const size_t length)
 */
 
 static int XBMInteger(Image *image,short int *hex_digits)
-{ 
+{
   int
     c;
-  
+
   unsigned int
     value;
-  
+
   /*
     Skip any leading whitespace.
   */
   do
-  { 
+  {
     c=ReadBlobByte(image);
     if (c == EOF)
       return(-1);
@@ -153,14 +153,14 @@ static int XBMInteger(Image *image,short int *hex_digits)
   */
   value=0;
   do
-  { 
-    if (value > (unsigned int) (INT_MAX/10))
-      break;
-    value*=16;
-    c&=0xff;
-    if (value > (unsigned int) (INT_MAX-hex_digits[c]))
-      break;
-    value+=hex_digits[c];
+  {
+    if (value <= (unsigned int) (INT_MAX/16))
+      {
+        value*=16;
+        c&=0xff;
+        if (value <= (unsigned int) (INT_MAX-hex_digits[c]))
+          value+=hex_digits[c];
+      }
     c=ReadBlobByte(image);
     if (c == EOF)
       return(-1);