arkouda icon indicating copy to clipboard operation
arkouda copied to clipboard

Resolve CSV Asan "use after free" memory issues

Open ShreyasKhandekar opened this issue 7 months ago • 1 comments

The recent changes to CSV reads were causing issues with ASAN testing where we were getting a "heap use after free" error message in both lscsv and get_info.

The offending code pattern was similar in both cases, even though the pattern itself looks pretty innocent.

This PR fixes the ASAN issues by rewriting the code to be a little more verbose and avoids the offending pattern.

When I have more time, I will try to see if we can reproduce this ASAN issue outside of Arkouda (i.e. if it's a problem on the Chapel side)

  • [X] tests/io_test.py::IOTest::test_csv_read_write in ASAN+gasnet
  • [x] tests/io_test.py::IOTest::test_csv_read_write in gasnet
  • [x] tests/io_test.py::IOTest::test_csv_read_write in default

ShreyasKhandekar avatar Jul 05 '24 01:07 ShreyasKhandekar