summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobin Watts <Robin.Watts@artifex.com>2022-02-17 16:39:41 +0000
committerRobin Watts <Robin.Watts@artifex.com>2022-02-17 16:39:41 +0000
commit7e8849c0c78d565836c0a7a6104264f6d56f19dd (patch)
tree9c0aec37dbac97c0e74c97eb187decb5afbcb01b
parentd2a362b30dadced457ed11e2151938df9ccbdd0f (diff)
downloadghostpdl-7e8849c0c78d565836c0a7a6104264f6d56f19dd.tar.gz
Coverity 375568: Tweak initialisation of px_args_t.
Because args.pv[1] is non-zero, and all the rest are zero, pxSetHalftoneMethod won't access args.source, but Coverity can't see this. For clarity, we update the init of args to clear the whole lot.
-rw-r--r--pcl/pxl/pxsessio.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/pcl/pxl/pxsessio.c b/pcl/pxl/pxsessio.c
index 25bce8b95..6448bcfd1 100644
--- a/pcl/pxl/pxsessio.c
+++ b/pcl/pxl/pxsessio.c
@@ -612,10 +612,9 @@ pxBeginPage(px_args_t * par, px_state_t * pxs)
* Set the default halftone method. We have to do this here,
* rather than earlier, so that the origin is set correctly.
*/
- px_args_t args;
+ px_args_t args = { 0 };
px_value_t device_matrix;
- memset(args.pv, 0, sizeof(args.pv));
args.pv[1] = &device_matrix; /* DeviceMatrix */
device_matrix.type = pxd_scalar | pxd_ubyte;
device_matrix.value.i = eDeviceBest;