SystemC-Components icon indicating copy to clipboard operation
SystemC-Components copied to clipboard

Fix reading across page boundary

Open cphurley82 opened this issue 2 months ago • 0 comments

Summary

  • Corrected memory::handle_operation to limit the second-page read to the remaining requested length, preventing buffer overwrite when crossing a page boundary.
  • Added a GoogleTest-based memory_boundary_test that reproduces and now validates the fixed boundary read.
  • Added unit test to CI.
  • Added TESTING.md.

Testing

  • Ran unit test in CI before fixing the memory::handle_operation, see: https://github.com/cphurley82/SystemC-Components/actions/runs/19548544658/job/55973618597#step:8:30
  • All tests are passing in CI now, see: https://github.com/cphurley82/SystemC-Components/actions/runs/19551895054

cphurley82 avatar Nov 20 '25 21:11 cphurley82