diff options
author | Chris Liddell <chris.liddell@artifex.com> | 2017-04-27 13:21:31 +0100 |
---|---|---|
committer | Chris Liddell <chris.liddell@artifex.com> | 2017-04-27 16:28:50 +0100 |
commit | 04b37bbce174eed24edec7ad5b920eb93db4d47d (patch) | |
tree | e9c83141c70102a5d4a3a47ab0beba3dc90b7986 /psi/zfrsd.c | |
parent | 4f83478c88c2e05d6e8d79ca4557eb039354d2f3 (diff) | |
download | ghostpdl-04b37bbce174eed24edec7ad5b920eb93db4d47d.tar.gz |
Bug 697799: have .rsdparams check its parameters
The Ghostscript internal operator .rsdparams wasn't checking the number or
type of the operands it was being passed. Do so.
Diffstat (limited to 'psi/zfrsd.c')
-rw-r--r-- | psi/zfrsd.c | 22 |
1 files changed, 15 insertions, 7 deletions
diff --git a/psi/zfrsd.c b/psi/zfrsd.c index 191107d8a..950588d69 100644 --- a/psi/zfrsd.c +++ b/psi/zfrsd.c @@ -49,13 +49,20 @@ zrsdparams(i_ctx_t *i_ctx_p) ref *pFilter; ref *pDecodeParms; int Intent = 0; - bool AsyncRead; + bool AsyncRead = false; ref empty_array, filter1_array, parms1_array; uint i; - int code; + int code = 0; + + if (ref_stack_count(&o_stack) < 1) + return_error(gs_error_stackunderflow); + if (!r_has_type(op, t_dictionary) && !r_has_type(op, t_null)) { + return_error(gs_error_typecheck); + } make_empty_array(&empty_array, a_readonly); - if (dict_find_string(op, "Filter", &pFilter) > 0) { + if (r_has_type(op, t_dictionary) + && dict_find_string(op, "Filter", &pFilter) > 0) { if (!r_is_array(pFilter)) { if (!r_has_type(pFilter, t_name)) return_error(gs_error_typecheck); @@ -94,12 +101,13 @@ zrsdparams(i_ctx_t *i_ctx_p) return_error(gs_error_typecheck); } } - code = dict_int_param(op, "Intent", 0, 3, 0, &Intent); + if (r_has_type(op, t_dictionary)) + code = dict_int_param(op, "Intent", 0, 3, 0, &Intent); if (code < 0 && code != gs_error_rangecheck) /* out-of-range int is ok, use 0 */ return code; - if ((code = dict_bool_param(op, "AsyncRead", false, &AsyncRead)) < 0 - ) - return code; + if (r_has_type(op, t_dictionary)) + if ((code = dict_bool_param(op, "AsyncRead", false, &AsyncRead)) < 0) + return code; push(1); op[-1] = *pFilter; if (pDecodeParms) |