diff options
author | Ken Sharp <ken.sharp@artifex.com> | 2018-02-06 10:30:32 +0000 |
---|---|---|
committer | Ken Sharp <ken.sharp@artifex.com> | 2018-02-06 10:30:32 +0000 |
commit | 16688a8b833cf4e8950265e933604c32b1f2392e (patch) | |
tree | 6c10311dbce4d9ee8eb113ebbef5e381c8fa3282 /base/gsdparam.c | |
parent | 650aae633131396b802b2cae6e09afbd5aae6ad6 (diff) | |
download | ghostpdl-16688a8b833cf4e8950265e933604c32b1f2392e.tar.gz |
Address more scanbuild/gcc/clang warnings
gp_os2.c free buffer and close file on memory full (as per other
error conditions).
gscicach.c remove unnecessary assignment
gsdparam.c " " " "
gxfapi.c " " " "
gscscie.c check a return code
zicc.c " " " "
zpcolor.c " " " "
mkromfs.c If realloc should fail in process_path, free working memory
print an error and exit as per other error conditions.
zcolor.c ensure depth cannot be 0. This should not be possible, no
existing code causes this, but its best to be safe and it prevents a
possible unassigned pointer dereference (possibly NULL) which is what
scanbuild complains of. Best to catch this in the interpreter.
Diffstat (limited to 'base/gsdparam.c')
-rw-r--r-- | base/gsdparam.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/base/gsdparam.c b/base/gsdparam.c index 71115c878..2c29c4142 100644 --- a/base/gsdparam.c +++ b/base/gsdparam.c @@ -1538,7 +1538,6 @@ e: param_signal_error(plist, param_name, ecode);\ if (param_read_null(plist, "LeadingEdge") == 0) { /* if param is null, clear explicitly-set flag */ leadingedge &= ~LEADINGEDGE_SET_MASK; - code = 0; } else { ecode = code; } |