Skip to content

Commit

Permalink
(unittests/concretization/test_concretization_unmapped) fix: for ephe…
Browse files Browse the repository at this point in the history
…meral runs, it's okay if two different runs re-use the same VA
  • Loading branch information
bensimner committed Mar 7, 2024
1 parent 60b1735 commit 5d874c5
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions unittests/tests/concretization/test_generic.c
Original file line number Diff line number Diff line change
Expand Up @@ -282,11 +282,9 @@ void __test_concretization_unmapped(concretize_type_t conc_type) {
u64 xpage = PAGE(x);
for (var_idx_t varidx = 0; varidx < ctx.cfg->no_heap_vars; varidx++) {
if (varidx != idx_from_varname(&ctx, "x")) {
for (run_idx_t r2 = 0; r2 < ctx.no_runs; r2++) {
u64* v2 = ctx.heap_vars[varidx].values[r2];
u64 v2page = PAGE(v2);
ASSERT(v2page != xpage, "x (%p) and %s (%p) had same page", x, ctx.heap_vars[varidx].name, v2);
}
u64* v2 = ctx.heap_vars[varidx].values[r];
u64 v2page = PAGE(v2);
ASSERT(v2page != xpage, "x (%p) and %s (%p) had same page", x, ctx.heap_vars[varidx].name, v2);
}
}
}
Expand Down

0 comments on commit 5d874c5

Please sign in to comment.