diff options
author | Henry Stiles <henry.stiles@artifex.com> | 2008-08-02 21:09:12 +0000 |
---|---|---|
committer | Henry Stiles <henry.stiles@artifex.com> | 2008-08-02 21:09:12 +0000 |
commit | f22a9d62e53400923f79a023b9722383e4737c12 (patch) | |
tree | 3fd3eff6b6ca30520f1c20b6a50bf7fd3ef84421 /pxl | |
parent | 23b75b8e68bbfab36743e738dda1aef0296e2794 (diff) |
Cleanup logical errors which could result in a null dereference. In
fact if any of the affected allocations failed the system would not
function properly, so the change is mostly cosmetic.
git-svn-id: http://svn.ghostscript.com/ghostscript/trunk@8924 a1074d23-0009-0410-80fe-cf8c14f379e6
Diffstat (limited to 'pxl')
-rw-r--r-- | pxl/pxtop.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/pxl/pxtop.c b/pxl/pxtop.c index 85f15123a..67a48ae74 100644 --- a/pxl/pxtop.c +++ b/pxl/pxtop.c @@ -223,13 +223,13 @@ pxl_impl_allocate_interp_instance( px_state_t *pxs = px_state_alloc(mem); /* inits interp state, potentially expensive */ /* If allocation error, deallocate & return */ if (!pxli || !pgs || !st || !pxs) { - if (!pxli) + if (pxli) gs_free_object(mem, pxli, "pxl_impl_allocate_interp_instance(pxl_interp_instance_t)"); - if (!pgs) + if (pgs) gs_state_free(pgs); - if (!st) + if (st) px_process_release(st); - if (!pxs) + if (pxs) px_state_release(pxs); return gs_error_VMerror; } |