Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix sanitizer errors in testcases #1096

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix sanitizer errors in testcases #1096

wants to merge 2 commits into from

Commits on Jan 12, 2024

  1. Fix heap-use-after-free issues in TPGRxStatsTest 

    The tests uses `tpg_header` when validating the statistics
while its memory `m1` has been freed.
`tmp_buf` is not used in tests but is now correct in size.

Signed-off-by: Björn Svensson <bjorn.a.svensson@est.tech>
    @bjosv
    bjosv committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    50d86c1 View commit details
    Browse the repository at this point in the history

  2. Fix direct and indirect leaks in tests 

    Fixed a couple of testcases that triggered errors by the leak
sanitizer. All issues are due to missing actions in the testcases.

There are still 2 testcase left that are expected to leak:
 gt_astf_inter.astf_negative_2
 gt_astf_inter.astf_negative_3

so testruns needs to filter out these using:
./bp-sim-64 --ut --gtest_filter='-*.astf_negative*'

Signed-off-by: Björn Svensson <bjorn.a.svensson@est.tech>
    @bjosv
    bjosv committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    c5fb3ee View commit details
    Browse the repository at this point in the history