Blame SOURCES/ghostscript-cve-2020-16291.patch

0ab051
diff --git a/contrib/gdevdj9.c b/contrib/gdevdj9.c
0ab051
index eec1c77..a4e8e9c 100644
0ab051
--- a/contrib/gdevdj9.c
0ab051
+++ b/contrib/gdevdj9.c
0ab051
@@ -575,26 +575,55 @@ static int cdj_set_bpp(gx_device *, int, int);
0ab051
 static int
0ab051
 hp_colour_open(gx_device * pdev)
0ab051
 {
0ab051
-    int retCode;
0ab051
+    int retCode = 0;
0ab051
+
0ab051
+    /* Change the margins if necessary. */
0ab051
+    static const float dj_a4[4] = {
0ab051
+        DESKJET_MARGINS_A4
0ab051
+    };
0ab051
+
0ab051
+    static const float dj_letter[4] = {
0ab051
+        DESKJET_MARGINS_LETTER
0ab051
+    };
0ab051
+    const float *m = (float *)0;
0ab051
 
0ab051
     cdj970->PageCtr = 0;
0ab051
 
0ab051
+    /* quality setup */
0ab051
+    if (cdj970->quality == DRAFT) {
0ab051
+        gx_device_set_resolution((gx_device *) pdev, 300.0, 300.0);
0ab051
+        cdj970->xscal = 0;
0ab051
+        cdj970->yscal = 0;
0ab051
+        cdj970->intensities = 2;
0ab051
+    } else if (cdj970->quality == NORMAL) {
0ab051
+        gx_device_set_resolution((gx_device *) pdev, 600.0, 600.0);
0ab051
+        cdj970->xscal = 1;
0ab051
+        cdj970->yscal = 1;
0ab051
+        /* intensities = 4 from initialization */
0ab051
+    } else {                    /* quality == PRESENTATION */
0ab051
+        gx_device_set_resolution((gx_device *) pdev, 600.0, 600.0);
0ab051
+        cdj970->xscal = 0;
0ab051
+        cdj970->yscal = 0;
0ab051
+        /* intensities = 4 from initialization */
0ab051
+    }
0ab051
+
0ab051
+    m = (gdev_pcl_paper_size((gx_device *) pdev) ==
0ab051
+         PAPER_SIZE_A4 ? dj_a4 : dj_letter);
0ab051
+
0ab051
+    gx_device_set_margins((gx_device *) pdev, m, true);
0ab051
+
0ab051
     /* Set up colour params if put_params has not already done so */
0ab051
     if (pdev->color_info.num_components == 0) {
0ab051
-        int code = cdj_set_bpp(pdev, pdev->color_info.depth,
0ab051
+        retCode = cdj_set_bpp(pdev, pdev->color_info.depth,
0ab051
                                pdev->color_info.num_components);
0ab051
 
0ab051
-        if (code < 0)
0ab051
-            return code;
0ab051
+        if (retCode < 0)
0ab051
+            return retCode;
0ab051
     }
0ab051
 
0ab051
     retCode = gdev_prn_open(pdev);
0ab051
-    if (retCode < 0)
0ab051
-        return (retCode);
0ab051
-    else {
0ab051
+    if (retCode >= 0) {
0ab051
         retCode = gdev_prn_open_printer(pdev, true);
0ab051
-        if (retCode < 0)
0ab051
-            return (retCode);
0ab051
     }
0ab051
 
0ab051
     return 0;
0ab051
@@ -648,26 +677,25 @@ cdj970_put_params(gx_device * pdev, gs_param_list * plist)
0ab051
     int bpp = 0;
0ab051
     int code = 0;
0ab051
 
0ab051
-    code = cdj_put_param_int(plist, "BitsPerPixel", &bpp, 1, 32, code);
0ab051
-    code = cdj_put_param_int(plist, "Quality", &quality, 0, 2, code);
0ab051
-    code = cdj_put_param_int(plist, "Papertype", &papertype, 0, 4, code);
0ab051
-    code = cdj_put_param_int(plist, "Duplex", &duplex, 0, 2, code);
0ab051
-    code =
0ab051
-        cdj_put_param_float(plist, "MasterGamma", &mastergamma, 0.1, 9.0,
0ab051
-                            code);
0ab051
-    code =
0ab051
-        cdj_put_param_float(plist, "GammaValC", &gammavalc, 0.0, 9.0, code);
0ab051
-    code =
0ab051
-        cdj_put_param_float(plist, "GammaValM", &gammavalm, 0.0, 9.0, code);
0ab051
-    code =
0ab051
-        cdj_put_param_float(plist, "GammaValY", &gammavaly, 0.0, 9.0, code);
0ab051
-    code =
0ab051
-        cdj_put_param_float(plist, "GammaValK", &gammavalk, 0.0, 9.0, code);
0ab051
-    code =
0ab051
-        cdj_put_param_float(plist, "BlackCorrect", &blackcorrect, 0.0, 9.0,
0ab051
-                            code);
0ab051
-
0ab051
-    if (code < 0)
0ab051
+    if ((code = cdj_put_param_int(plist, "BitsPerPixel", &bpp, 1, 32, code)) < 0)
0ab051
+        return code;
0ab051
+    if ((code = cdj_put_param_int(plist, "Quality", &quality, 0, 2, code)) < 0)
0ab051
+        return code;
0ab051
+    if ((code = cdj_put_param_int(plist, "Papertype", &papertype, 0, 4, code)) < 0)
0ab051
+        return code;
0ab051
+    if ((code = cdj_put_param_int(plist, "Duplex", &duplex, 0, 2, code)) < 0)
0ab051
+        return code;
0ab051
+    if ((code = cdj_put_param_float(plist, "MasterGamma", &mastergamma, 0.1, 9.0, code)) < 0)
0ab051
+        return code;
0ab051
+    if ((code = cdj_put_param_float(plist, "GammaValC", &gammavalc, 0.0, 9.0, code)) < 0)
0ab051
+        return code;
0ab051
+    if ((code = cdj_put_param_float(plist, "GammaValM", &gammavalm, 0.0, 9.0, code)) < 0)
0ab051
+        return code;
0ab051
+    if ((code = cdj_put_param_float(plist, "GammaValY", &gammavaly, 0.0, 9.0, code)) < 0)
0ab051
+        return code;
0ab051
+    if ((code = cdj_put_param_float(plist, "GammaValK", &gammavalk, 0.0, 9.0, code)) < 0)
0ab051
+        return code;
0ab051
+    if ((code = cdj_put_param_float(plist, "BlackCorrect", &blackcorrect, 0.0, 9.0, code)) < 0)
0ab051
         return code;
0ab051
 
0ab051
     code = cdj_put_param_bpp(pdev, plist, bpp, bpp, 0);
0ab051
@@ -676,6 +704,12 @@ cdj970_put_params(gx_device * pdev, gs_param_list * plist)
0ab051
         return code;
0ab051
 
0ab051
     cdj970->quality = quality;
0ab051
+    if (cdj970->quality != quality) {
0ab051
+        if (pdev->is_open)
0ab051
+            gs_closedevice(pdev);      /* quality can change resolution, force re-open */
0ab051
+        cdj970->quality = quality;
0ab051
+    }
0ab051
+
0ab051
     cdj970->papertype = papertype;
0ab051
     cdj970->duplex = duplex;
0ab051
     cdj970->mastergamma = mastergamma;
0ab051
@@ -685,7 +719,7 @@ cdj970_put_params(gx_device * pdev, gs_param_list * plist)
0ab051
     cdj970->gammavalk = gammavalk;
0ab051
     cdj970->blackcorrect = blackcorrect;
0ab051
 
0ab051
-    return 0;
0ab051
+    return code;
0ab051
 }
0ab051
 
0ab051
 /**********************************************************************************/
0ab051
@@ -784,47 +818,6 @@ cdj970_terminate_page(gx_device_printer * pdev, FILE * prn_stream)
0ab051
     fputs("\033*rC\f\033&l-2H", prn_stream);    /* End Graphics, Reset */
0ab051
 }
0ab051
 
0ab051
-/* cdj970_one_time_initialisation:
0ab051
-----------------------------------------------------------------------------------*/
0ab051
-static void
0ab051
-cdj970_one_time_initialisation(gx_device_printer * pdev)
0ab051
-{
0ab051
-    /* Change the margins if necessary. */
0ab051
-    static const float dj_a4[4] = {
0ab051
-        DESKJET_MARGINS_A4
0ab051
-    };
0ab051
-
0ab051
-    static const float dj_letter[4] = {
0ab051
-        DESKJET_MARGINS_LETTER
0ab051
-    };
0ab051
-    const float *m = (float *)0;
0ab051
-
0ab051
-    /* quality setup */
0ab051
-    if (cdj970->quality == DRAFT) {
0ab051
-        gx_device_set_resolution((gx_device *) pdev, 300.0, 300.0);
0ab051
-        cdj970->xscal = 0;
0ab051
-        cdj970->yscal = 0;
0ab051
-        cdj970->intensities = 2;
0ab051
-    } else if (cdj970->quality == NORMAL) {
0ab051
-        gx_device_set_resolution((gx_device *) pdev, 600.0, 600.0);
0ab051
-        cdj970->xscal = 1;
0ab051
-        cdj970->yscal = 1;
0ab051
-        /* intensities = 4 from initialization */
0ab051
-    } else {                    /* quality == PRESENTATION */
0ab051
-        gx_device_set_resolution((gx_device *) pdev, 600.0, 600.0);
0ab051
-        cdj970->xscal = 0;
0ab051
-        cdj970->yscal = 0;
0ab051
-        /* intensities = 4 from initialization */
0ab051
-    }
0ab051
-
0ab051
-    m = (gdev_pcl_paper_size((gx_device *) pdev) ==
0ab051
-         PAPER_SIZE_A4 ? dj_a4 : dj_letter);
0ab051
-
0ab051
-    gx_device_set_margins((gx_device *) pdev, m, true);
0ab051
-
0ab051
-    cdj970_write_header((gx_device *) pdev, pdev->file);
0ab051
-}
0ab051
-
0ab051
 /* cdj970_print_page: Here comes the hp970 output routine
0ab051
 ----------------------------------------------------------------------------------*/
0ab051
 static int
0ab051
@@ -837,7 +830,7 @@ cdj970_print_page(gx_device_printer * pdev, FILE * prn_stream)
0ab051
     Gamma gamma;
0ab051
 
0ab051
     if (cdj970->PageCtr == 0 && cdj970->ptype == DJ970C) {
0ab051
-        cdj970_one_time_initialisation(pdev);
0ab051
+        cdj970_write_header((gx_device *)pdev, prn_stream);
0ab051
     }
0ab051
 
0ab051
     /* make a local writable copy of the Gamma tables */
0ab051
@@ -2280,6 +2273,11 @@ cdj_set_bpp(gx_device * pdev, int bpp, int ccomps)
0ab051
         ci->dither_colors = (bpp >= 8 ? 5 : bpp > 1 ? 2 : 0);
0ab051
     }
0ab051
 
0ab051
+    if (ci->depth != ((bpp > 1) && (bpp < 8) ? 8 : bpp)) {
0ab051
+        if (pdev->is_open)
0ab051
+            gs_closedevice(pdev);  /* depth changed, make sure we re-open */
0ab051
+    }
0ab051
+
0ab051
     ci->depth = ((bpp > 1) && (bpp < 8) ? 8 : bpp);
0ab051
 
0ab051
     return (0);
0ab051
@@ -2598,16 +2596,15 @@ cdj_put_param_bpp(gx_device * pdev,
0ab051
                   gs_param_list * plist,
0ab051
                   int new_bpp, int real_bpp, int ccomps)
0ab051
 {
0ab051
-    if (new_bpp == 0 && ccomps == 0)
0ab051
-        return gdev_prn_put_params(pdev, plist);
0ab051
-    else {
0ab051
-        gx_device_color_info save_info;
0ab051
-        int save_bpp;
0ab051
-        int code;
0ab051
-
0ab051
-        save_info = pdev->color_info;
0ab051
-        save_bpp = save_info.depth;
0ab051
+    int code = 0;
0ab051
+    int save_bpp;
0ab051
+    gx_device_color_info save_info;
0ab051
+    save_info = pdev->color_info;
0ab051
+    save_bpp = save_info.depth;
0ab051
 
0ab051
+    if (new_bpp == 0 && ccomps == 0) {
0ab051
+       code = gdev_prn_put_params(pdev, plist);
0ab051
+    } else {
0ab051
         if (save_bpp == 8 && save_ccomps == 3 && !cprn_device->cmyk)
0ab051
             save_bpp = 3;
0ab051
 
0ab051
@@ -2631,12 +2628,22 @@ cdj_put_param_bpp(gx_device * pdev,
0ab051
         if ((cdj970->color_info.depth != save_bpp
0ab051
              || (ccomps != 0 && ccomps != save_ccomps))
0ab051
             && pdev->is_open)
0ab051
-            return (gs_closedevice(pdev));
0ab051
+            gs_closedevice(pdev);
0ab051
+    }
0ab051
+
0ab051
+    /* check for valid resolutions */
0ab051
+    if (pdev->HWResolution[0] != pdev->HWResolution[1] ||
0ab051
+        (pdev->HWResolution[0] != 300.0 && pdev->HWResolution[0] != 600.0) ) {
0ab051
+        param_signal_error(plist, "HWResolution", gs_error_rangecheck);
0ab051
+        emprintf1(pdev->memory, "\ncdj970: Invalid resolution: '%f'. Only 300 or 600 supported.\n\n",
0ab051
+                  pdev->HWResolution[0]);
0ab051
+        cdj_set_bpp(pdev, save_bpp, save_ccomps);
0ab051
+        return gs_error_rangecheck;
0ab051
+    }
0ab051
+    return code;
0ab051
 
0ab051
-        return (0);
0ab051
 
0ab051
 #undef save_ccomps
0ab051
-    }
0ab051
 }
0ab051
 
0ab051
 /* cdj970_write_header: