]> granicus.if.org Git - imagemagick/commitdiff
Fixed incorrect size of DICOM image.
authordirk <dirk@git.imagemagick.org>
Thu, 26 Sep 2013 01:22:44 +0000 (01:22 +0000)
committerdirk <dirk@git.imagemagick.org>
Thu, 26 Sep 2013 01:22:44 +0000 (01:22 +0000)
coders/dcm.c

index 64bb7f3da9a2c0d3c2d92f8edd258b6d38371a43..8cd2a333469d9e2a9ad6b1273d22103e79b05a85 100644 (file)
@@ -2815,9 +2815,10 @@ static Image *ReadDCMImage(const ImageInfo *image_info,ExceptionInfo *exception)
 
   MagickBooleanType
     explicit_file,
-    use_explicit,
     explicit_retry,
-    polarity;
+    polarity,
+    sequence,
+    use_explicit;
 
   MagickOffsetType
     offset;
@@ -2861,7 +2862,6 @@ static Image *ReadDCMImage(const ImageInfo *image_info,ExceptionInfo *exception)
     window_center,
     y;
 
-
   unsigned char
     *data;
 
@@ -2923,6 +2923,7 @@ static Image *ReadDCMImage(const ImageInfo *image_info,ExceptionInfo *exception)
   number_scenes=1;
   samples_per_pixel=1;
   scale=(Quantum *) NULL;
+  sequence=MagickFalse;
   signed_data=(~0UL);
   significant_bits=0;
   use_explicit=MagickFalse;
@@ -2930,7 +2931,8 @@ static Image *ReadDCMImage(const ImageInfo *image_info,ExceptionInfo *exception)
   width=0;
   window_center=0;
   window_width=0;
-  for (group=0; (group != 0x7FE0) || (element != 0x0010); )
+  for (group=0; (group != 0x7FE0) || (element != 0x0010) ||\r
+                (sequence != MagickFalse); )
   {
     /*
       Read a group.
@@ -2952,9 +2954,9 @@ static Image *ReadDCMImage(const ImageInfo *image_info,ExceptionInfo *exception)
       Check for "explicitness", but meta-file headers always explicit.
     */
     if ((explicit_file == MagickFalse) && (group != 0x0002))
-      explicit_file=(isupper((int) *explicit_vr) != MagickFalse) &&
-        (isupper((int) *(explicit_vr+1)) != MagickFalse) ? MagickTrue :
-        MagickFalse;
+      explicit_file=(isupper((unsigned char) *explicit_vr) != MagickFalse) &&
+        (isupper((unsigned char) *(explicit_vr+1)) != MagickFalse) ?
+        MagickTrue : MagickFalse;
     use_explicit=((group == 0x0002) && (explicit_retry == MagickFalse)) ||
       (explicit_file != MagickFalse) ? MagickTrue : MagickFalse;
     if ((use_explicit != MagickFalse) && (strcmp(implicit_vr,"xs") == 0))
@@ -3032,7 +3034,7 @@ static Image *ReadDCMImage(const ImageInfo *image_info,ExceptionInfo *exception)
           (void) FormatLocaleFile(stdout," %s",dicom_info[i].description);
         (void) FormatLocaleFile(stdout,": ");
       }
-    if ((group == 0x7FE0) && (element == 0x0010))
+    if ((sequence == MagickFalse) && (group == 0x7FE0) && (element == 0x0010))
       {
         if (image_info->verbose != MagickFalse)
           (void) FormatLocaleFile(stdout,"\n");
@@ -3070,7 +3072,23 @@ static Image *ReadDCMImage(const ImageInfo *image_info,ExceptionInfo *exception)
                      "InsufficientImageDataInFile");
                 }
               data[length*quantum]='\0';
-            }
+            }\r
+          else\r
+            if (datum == 0xFFFFFFFF)\r
+              {\r
+                sequence=MagickTrue;\r
+                continue;\r
+              }\r
+\r
+    if (((group << 16) | element) == 0xFFFEE0DD)\r
+      {\r
+        sequence=MagickFalse;\r
+        continue;\r
+      }\r
+\r
+    if (sequence == MagickTrue)\r
+      continue;\r
+
     switch (group)
     {
       case 0x0002: