Projects
Mega:24.03:SP1:Everything
ghostscript
_service:tar_scm:Bug-707510-review-printing-of-...
Sign Up
Log In
Username
Password
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File _service:tar_scm:Bug-707510-review-printing-of-pointers.patch of Package ghostscript
From ff1013a0ab485b66783b70145e342a82c670906a Mon Sep 17 00:00:00 2001 From: Ken Sharp <Ken.Sharp@artifex.com> Date: Thu, 25 Jan 2024 11:53:44 +0000 Subject: [PATCH 4/7] Bug 707510 - review printing of pointers http://www.ghostscript.com/cgi-bin/findgit.cgi?ff1013a0ab485b66783b70145e342a82c670906a This is for item 4 of the report, which is addressed by the change in gdevpdtb.c. That change uses a fixed name for fonts which have no name instead of using the pointer to the address of the font. The remaining changes are all due to reviewing the use of PRI_INTPTR. In general we only use that for debugging purposes but there were a few places which were printing pointers arbitrarily, even in a release build. We really don't want to do that so I've modified the places which were printing pointer unconditionally so that they only do so if DEBUG is set at compile time, or a specific debug flag is set. --- base/gsfont.c | 2 +- base/gsicc_cache.c | 6 +++--- base/gsmalloc.c | 2 +- base/gxclmem.c | 3 +-- base/gxcpath.c | 4 ++++ base/gxpath.c | 6 ++++++ base/szlibc.c | 2 ++ devices/gdevupd.c | 5 +++++ devices/vector/gdevpdtb.c | 2 +- psi/ialloc.c | 2 +- psi/igc.c | 4 ++-- psi/igcstr.c | 4 ++-- psi/iinit.c | 4 ++++ psi/imainarg.c | 3 ++- psi/isave.c | 2 +- psi/iutil.c | 4 ++++ 16 files changed, 40 insertions(+), 15 deletions(-) diff --git a/base/gsfont.c b/base/gsfont.c index 8e2015b..cc9af15 100644 --- a/base/gsfont.c +++ b/base/gsfont.c @@ -791,7 +791,7 @@ gs_purge_font(gs_font * pfont) else if (pdir->scaled_fonts == pfont) pdir->scaled_fonts = next; else { /* Shouldn't happen! */ - lprintf1("purged font "PRI_INTPTR" not found\n", (intptr_t)pfont); + if_debug1m('u', pfont->memory, "purged font "PRI_INTPTR" not found\n", (intptr_t)pfont); } /* Purge the font from the scaled font cache. */ diff --git a/base/gsicc_cache.c b/base/gsicc_cache.c index 13eb003..8dcdb71 100644 --- a/base/gsicc_cache.c +++ b/base/gsicc_cache.c @@ -151,7 +151,7 @@ icc_linkcache_finalize(const gs_memory_t *mem, void *ptr) while (link_cache->head != NULL) { if (link_cache->head->ref_count != 0) { - emprintf2(mem, "link at "PRI_INTPTR" being removed, but has ref_count = %d\n", + if_debug2m(gs_debug_flag_icc, mem, "link at "PRI_INTPTR" being removed, but has ref_count = %d\n", (intptr_t)link_cache->head, link_cache->head->ref_count); link_cache->head->ref_count = 0; /* force removal */ } @@ -573,7 +573,7 @@ gsicc_findcachelink(gsicc_hashlink_t hash, gsicc_link_cache_t *icc_link_cache, /* that was building it failed to be able to complete building it. Try this only a limited number of times before we bail. */ if (curr->valid == false) { - emprintf1(curr->memory, "link "PRI_INTPTR" lock released, but still not valid.\n", (intptr_t)curr); /* Breakpoint here */ + if_debug1m(gs_debug_flag_icc, curr->memory, "link "PRI_INTPTR" lock released, but still not valid.\n", (intptr_t)curr); /* Breakpoint here */ } gx_monitor_enter(icc_link_cache->lock); /* re-enter to loop and check */ } @@ -600,7 +600,7 @@ gsicc_remove_link(gsicc_link_t *link, const gs_memory_t *memory) /* NOTE: link->ref_count must be 0: assert ? */ gx_monitor_enter(icc_link_cache->lock); if (link->ref_count != 0) { - emprintf2(memory, "link at "PRI_INTPTR" being removed, but has ref_count = %d\n", (intptr_t)link, link->ref_count); + if_debug2m(gs_debug_flag_icc, memory, "link at "PRI_INTPTR" being removed, but has ref_count = %d\n", (intptr_t)link, link->ref_count); } curr = icc_link_cache->head; prev = NULL; diff --git a/base/gsmalloc.c b/base/gsmalloc.c index 63c8b6b..3182b56 100644 --- a/base/gsmalloc.c +++ b/base/gsmalloc.c @@ -420,7 +420,7 @@ gs_heap_resize_string(gs_memory_t * mem, byte * data, size_t old_num, size_t new client_name_t cname) { if (gs_heap_object_type(mem, data) != &st_bytes) - lprintf2("%s: resizing non-string "PRI_INTPTR"!\n", + if_debug2m('a', mem, "%s: resizing non-string "PRI_INTPTR"!\n", client_name_string(cname), (intptr_t)data); return gs_heap_resize_object(mem, data, new_num, cname); } diff --git a/base/gxclmem.c b/base/gxclmem.c index 1905a43..933cb4e 100644 --- a/base/gxclmem.c +++ b/base/gxclmem.c @@ -490,8 +490,7 @@ memfile_fclose(clist_file_ptr cf, const char *fname, bool delete) /* leaks if other users of the memfile don't 'fclose with delete=true */ if (f->openlist != NULL || ((f->base_memfile != NULL) && f->base_memfile->is_open)) { /* TODO: do the cleanup rather than just giving an error */ - emprintf1(f->memory, - "Attempt to delete a memfile still open for read: "PRI_INTPTR"\n", + if_debug1(':', "Attempt to delete a memfile still open for read: "PRI_INTPTR"\n", (intptr_t)f); return_error(gs_error_invalidfileaccess); } else { diff --git a/base/gxcpath.c b/base/gxcpath.c index 437b065..a8a5504 100644 --- a/base/gxcpath.c +++ b/base/gxcpath.c @@ -175,8 +175,10 @@ gx_cpath_init_contained_shared(gx_clip_path * pcpath, { if (shared) { if (shared->path.segments == &shared->path.local_segments) { +#ifdef DEBUG lprintf1("Attempt to share (local) segments of clip path "PRI_INTPTR"!\n", (intptr_t)shared); +#endif return_error(gs_error_Fatal); } *pcpath = *shared; @@ -233,8 +235,10 @@ gx_cpath_init_local_shared_nested(gx_clip_path * pcpath, if (shared) { if ((shared->path.segments == &shared->path.local_segments) && !safely_nested) { +#ifdef DEBUG lprintf1("Attempt to share (local) segments of clip path "PRI_INTPTR"!\n", (intptr_t)shared); +#endif return_error(gs_error_Fatal); } pcpath->path = shared->path; diff --git a/base/gxpath.c b/base/gxpath.c index e700729..0e9dba8 100644 --- a/base/gxpath.c +++ b/base/gxpath.c @@ -137,8 +137,10 @@ gx_path_init_contained_shared(gx_path * ppath, const gx_path * shared, { if (shared) { if (shared->segments == &shared->local_segments) { +#ifdef DEBUG lprintf1("Attempt to share (local) segments of path "PRI_INTPTR"!\n", (intptr_t)shared); +#endif return_error(gs_error_Fatal); } *ppath = *shared; @@ -172,8 +174,10 @@ gx_path_alloc_shared(const gx_path * shared, gs_memory_t * mem, ppath->procs = &default_path_procs; if (shared) { if (shared->segments == &shared->local_segments) { +#ifdef DEBUG lprintf1("Attempt to share (local) segments of path "PRI_INTPTR"!\n", (intptr_t)shared); +#endif gs_free_object(mem, ppath, cname); return 0; } @@ -203,8 +207,10 @@ gx_path_init_local_shared(gx_path * ppath, const gx_path * shared, { if (shared) { if (shared->segments == &shared->local_segments) { +#ifdef DEBUG lprintf1("Attempt to share (local) segments of path "PRI_INTPTR"!\n", (intptr_t)shared); +#endif return_error(gs_error_Fatal); } *ppath = *shared; diff --git a/base/szlibc.c b/base/szlibc.c index 0be3338..35a2fce 100644 --- a/base/szlibc.c +++ b/base/szlibc.c @@ -110,7 +110,9 @@ s_zlib_free(void *zmem, void *data) gs_free_object(mem, data, "s_zlib_free(data)"); for (; ; block = block->next) { if (block == 0) { +#ifdef DEBUG lprintf1("Freeing unrecorded data "PRI_INTPTR"!\n", (intptr_t)data); +#endif return; } if (block->data == data) diff --git a/devices/gdevupd.c b/devices/gdevupd.c index 7826507..12dfbc0 100644 --- a/devices/gdevupd.c +++ b/devices/gdevupd.c @@ -1040,8 +1040,13 @@ upd_print_page(gx_device_printer *pdev, gp_file *out) */ if(!upd || B_OK4GO != (upd->flags & (B_OK4GO | B_ERROR))) { #if UPD_MESSAGES & (UPD_M_ERROR | UPD_M_TOPCALLS) +#ifdef DEBUG errprintf(pdev->memory, "CALL-REJECTED upd_print_page(" PRI_INTPTR "," PRI_INTPTR ")\n", (intptr_t)udev,(intptr_t) out); +#else + errprintf(pdev->memory, "CALL-REJECTED upd_print_page\n", + (intptr_t)udev,(intptr_t) out); +#endif #endif return_error(gs_error_undefined); } diff --git a/devices/vector/gdevpdtb.c b/devices/vector/gdevpdtb.c index 5c8fb8d..aacbca2 100644 --- a/devices/vector/gdevpdtb.c +++ b/devices/vector/gdevpdtb.c @@ -371,7 +371,7 @@ pdf_base_font_alloc(gx_device_pdf *pdev, pdf_base_font_t **ppbfont, font_name.size -= SUBSET_PREFIX_SIZE; } } else { - gs_snprintf(fnbuf, sizeof(fnbuf), ".F" PRI_INTPTR, (intptr_t)copied); + gs_snprintf(fnbuf, sizeof(fnbuf), "Anonymous"); font_name.data = (byte *)fnbuf; font_name.size = strlen(fnbuf); } diff --git a/psi/ialloc.c b/psi/ialloc.c index d84ec00..85e36ac 100644 --- a/psi/ialloc.c +++ b/psi/ialloc.c @@ -386,7 +386,7 @@ gs_free_ref_array(gs_ref_memory_t * mem, ref * parr, client_name_t cname) size = num_refs * sizeof(ref); break; default: - lprintf3("Unknown type 0x%x in free_ref_array(%u,"PRI_INTPTR")!", + if_debug3('A', "Unknown type 0x%x in free_ref_array(%u,"PRI_INTPTR")!", r_type(parr), num_refs, (intptr_t)obj); return; } diff --git a/psi/igc.c b/psi/igc.c index 373cdcc..137783c 100644 --- a/psi/igc.c +++ b/psi/igc.c @@ -1062,7 +1062,7 @@ gc_extend_stack(gc_mark_stack * pms, gc_state_t * pstate) if (cp == 0) { /* We were tracing outside collectible */ /* storage. This can't happen. */ - lprintf1("mark stack overflowed while outside collectible space at "PRI_INTPTR"!\n", + if_debug1('6', "mark stack overflowed while outside collectible space at "PRI_INTPTR"!\n", (intptr_t)cptr); gs_abort(pstate->heap); } @@ -1291,7 +1291,7 @@ igc_reloc_struct_ptr(const void /*obj_header_t */ *obj, gc_state_t * gcst) if (cp != 0 && cp->cbase <= (byte *)obj && (byte *)obj <cp->ctop) { if (back > (cp->ctop - cp->cbase) >> obj_back_shift) { - lprintf2("Invalid back pointer %u at "PRI_INTPTR"!\n", + if_debug2('6', "Invalid back pointer %u at "PRI_INTPTR"!\n", back, (intptr_t)obj); gs_abort(NULL); } diff --git a/psi/igcstr.c b/psi/igcstr.c index 0e10f74..1bc7e7a 100644 --- a/psi/igcstr.c +++ b/psi/igcstr.c @@ -152,7 +152,7 @@ gc_string_mark(const byte * ptr, uint size, bool set, gc_state_t * gcst) return false; #ifdef DEBUG if (ptr < cp->ctop) { - lprintf4("String pointer "PRI_INTPTR"[%u] outside ["PRI_INTPTR".."PRI_INTPTR")\n", + if_debug4('6', "String pointer "PRI_INTPTR"[%u] outside ["PRI_INTPTR".."PRI_INTPTR")\n", (intptr_t)ptr, size, (intptr_t)cp->ctop, (intptr_t)cp->climit); return false; } else if (ptr + size > cp->climit) { /* @@ -171,7 +171,7 @@ gc_string_mark(const byte * ptr, uint size, bool set, gc_state_t * gcst) while (ptr == scp->climit && scp->outer != 0) scp = scp->outer; if (ptr + size > scp->climit) { - lprintf4("String pointer "PRI_INTPTR"[%u] outside ["PRI_INTPTR".."PRI_INTPTR")\n", + if_debug4('6', "String pointer "PRI_INTPTR"[%u] outside ["PRI_INTPTR".."PRI_INTPTR")\n", (intptr_t)ptr, size, (intptr_t)scp->ctop, (intptr_t)scp->climit); return false; diff --git a/psi/iinit.c b/psi/iinit.c index e347129..3371979 100644 --- a/psi/iinit.c +++ b/psi/iinit.c @@ -395,8 +395,12 @@ zop_init(i_ctx_t *i_ctx_p) if (def->proc != 0) { code = def->proc(i_ctx_p); if (code < 0) { +#ifdef DEBUG lprintf2("op_init proc "PRI_INTPTR" returned error %d!\n", (intptr_t)def->proc, code); +#else + lprintf("op_init proc returned error !\n"); +#endif return code; } } diff --git a/psi/imainarg.c b/psi/imainarg.c index aaf41b6..3b9efdc 100644 --- a/psi/imainarg.c +++ b/psi/imainarg.c @@ -229,7 +229,8 @@ gs_main_init_with_args01(gs_main_instance * minst, int argc, char *argv[]) if (gs_debug[':'] && !have_dumped_args) { int i; - dmprintf1(minst->heap, "%% Args passed to instance "PRI_INTPTR": ", + if (gs_debug_c(gs_debug_flag_init_details)) + dmprintf1(minst->heap, "%% Args passed to instance "PRI_INTPTR": ", (intptr_t)minst); for (i=1; i<argc; i++) dmprintf1(minst->heap, "%s ", argv[i]); diff --git a/psi/isave.c b/psi/isave.c index f0f3db0..d5f1448 100644 --- a/psi/isave.c +++ b/psi/isave.c @@ -487,7 +487,7 @@ alloc_save_change_in(gs_ref_memory_t *mem, const ref * pcont, else if (r_is_struct(pcont)) cp->offset = (byte *) where - (byte *) pcont->value.pstruct; else { - lprintf3("Bad type %u for save! pcont = "PRI_INTPTR", where = "PRI_INTPTR"\n", + if_debug3('u', "Bad type %u for save! pcont = "PRI_INTPTR", where = "PRI_INTPTR"\n", r_type(pcont), (intptr_t) pcont, (intptr_t) where); gs_abort((const gs_memory_t *)mem); } diff --git a/psi/iutil.c b/psi/iutil.c index 39a0a31..e24bbfd 100644 --- a/psi/iutil.c +++ b/psi/iutil.c @@ -537,7 +537,11 @@ other: break; } /* Internal operator, no name. */ +#if DEBUG gs_snprintf(buf, sizeof(buf), "@"PRI_INTPTR, (intptr_t) op->value.opproc); +#else + gs_snprintf(buf, sizeof(buf), "@anonymous_operator", (intptr_t) op->value.opproc); +#endif break; } case t_real: -- 2.43.0
Locations
Projects
Search
Status Monitor
Help
Open Build Service
OBS Manuals
API Documentation
OBS Portal
Reporting a Bug
Contact
Mailing List
Forums
Chat (IRC)
Twitter
Open Build Service (OBS)
is an
openSUSE project
.
浙ICP备2022010568号-2