diff options
author | Ken Sharp <ken.sharp@artifex.com> | 2018-02-05 16:09:28 +0000 |
---|---|---|
committer | Ken Sharp <ken.sharp@artifex.com> | 2018-02-05 16:09:28 +0000 |
commit | 650aae633131396b802b2cae6e09afbd5aae6ad6 (patch) | |
tree | f0f443a20038911aae0ed5c985525b79a74485d6 /psi/zcrd.c | |
parent | d7a5f0a14e873c7f5f557c4b635f4b4f166b41d2 (diff) | |
download | ghostpdl-650aae633131396b802b2cae6e09afbd5aae6ad6.tar.gz |
Coverity ID #261212
This probably showed up because of Ray's work adding return value
checking. Previously there were probably enough places not checking
the return code for Coverity to decide it was deliberate, now it is
raising a warning.
Add a check and take action on an error return.
Diffstat (limited to 'psi/zcrd.c')
-rw-r--r-- | psi/zcrd.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/psi/zcrd.c b/psi/zcrd.c index 1c39730f4..7993b1544 100644 --- a/psi/zcrd.c +++ b/psi/zcrd.c @@ -291,7 +291,10 @@ cie_cache_joint(i_ctx_t *i_ctx_p, const ref_cie_render_procs * pcrprocs, return code; /* When we're done, deallocate the procs and complete the caches. */ check_estack(3); - cie_cache_push_finish(i_ctx_p, cie_tpqr_finish, imem, pgs); + code = cie_cache_push_finish(i_ctx_p, cie_tpqr_finish, imem, pgs); + if (code < 0) + return code; + *++esp = pqr_procs; space = r_space(&pqr_procs); for (i = 0; i < 3; i++) { |