]> git.ipfire.org Git - thirdparty/cups.git/commitdiff
Check inheader before checking color space 595/head
authorRose <83477269+AtariDreams@users.noreply.github.com>
Fri, 20 Jan 2023 19:07:46 +0000 (14:07 -0500)
committerRose <83477269+AtariDreams@users.noreply.github.com>
Fri, 20 Jan 2023 19:07:46 +0000 (14:07 -0500)
filter/rastertopwg.c

index 6e0ccf2312d828e662f6336a3e06b75ec8af6a78..81c25ba9aa07518c6015a41c9d3577b8e0e775b2 100644 (file)
@@ -152,6 +152,23 @@ main(int  argc,                            /* I - Number of command-line args */
       return (1);
     }
 
+    if (inheader.cupsColorOrder != CUPS_ORDER_CHUNKED)
+    {
+      _cupsLangPrintFilter(stderr, "ERROR", _("Unsupported raster data."));
+      fprintf(stderr, "DEBUG: Unsupported cupsColorOrder %d on page %d.\n",
+              inheader.cupsColorOrder, page);
+      return (1);
+    }
+
+    if (inheader.cupsBitsPerPixel != 1 &&
+        inheader.cupsBitsPerColor != 8 && inheader.cupsBitsPerColor != 16)
+    {
+      _cupsLangPrintFilter(stderr, "ERROR", _("Unsupported raster data."));
+      fprintf(stderr, "DEBUG: Unsupported cupsBitsPerColor %d on page %d.\n",
+              inheader.cupsBitsPerColor, page);
+      return (1);
+    }
+
     switch (inheader.cupsColorSpace)
     {
       case CUPS_CSPACE_W :
@@ -189,23 +206,6 @@ main(int  argc,                            /* I - Number of command-line args */
          return (1);
     }
 
-    if (inheader.cupsColorOrder != CUPS_ORDER_CHUNKED)
-    {
-      _cupsLangPrintFilter(stderr, "ERROR", _("Unsupported raster data."));
-      fprintf(stderr, "DEBUG: Unsupported cupsColorOrder %d on page %d.\n",
-              inheader.cupsColorOrder, page);
-      return (1);
-    }
-
-    if (inheader.cupsBitsPerPixel != 1 &&
-        inheader.cupsBitsPerColor != 8 && inheader.cupsBitsPerColor != 16)
-    {
-      _cupsLangPrintFilter(stderr, "ERROR", _("Unsupported raster data."));
-      fprintf(stderr, "DEBUG: Unsupported cupsBitsPerColor %d on page %d.\n",
-              inheader.cupsBitsPerColor, page);
-      return (1);
-    }
-
     memcpy(&outheader, &inheader, sizeof(outheader));
     outheader.cupsWidth        = page_width;
     outheader.cupsHeight       = page_height;