Ticket #1121 (closed bug: invalid)

Opened 5 years ago

Last modified 4 years ago

system.c trace_mem_block small bug

Reported by: jessevdam Owned by:
Priority: normal Milestone:
Component: GC Version: 1.6.0
Severity: medium Keywords: memory stack gc
Cc: Language:
Patch status: Platform: all

Description

In system.c trace_mem_block is a small but nasty bug. Which could cause difficult to debug errors.

prefix = mask & buffer_min;

should become

prefix = mask & (buffer_min < pmc_min ? buffer_min : pmc_min);

since the mask is also ditermined like

const size_t mask =

find_common_mask(interp,

buffer_min < pmc_min ? buffer_min : pmc_min, buffer_max > pmc_max ? buffer_max : pmc_max);

Change History

Changed 4 years ago by nwellnhof

  • status changed from new to closed
  • resolution set to invalid

No, we could use any of buffer_min, buffer_max, pmc_min or pmc_max to compute the prefix because mask is exactly the mask of the common prefix.

Note: See TracTickets for help on using tickets.