From 0625fee9e49ebfeba0c805b00f7428e0a40ec75a Mon Sep 17 00:00:00 2001 From: Eyal Soha Date: Mon, 30 Oct 2023 10:46:38 -0600 Subject: [PATCH] Clear vbits after the test is done. https://bugs.kde.org/show_bug.cgi?id=417993 --- NEWS | 7 +++++-- memcheck/tests/vbit-test/valgrind.c | 5 +++++ 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 33da44e636..8791694e53 100644 --- a/NEWS +++ b/NEWS @@ -56,8 +56,11 @@ than mailing the developers (or mailing lists) directly -- bugs that are not entered into bugzilla tend to get forgotten about or ignored. 390871 ELF debug info reader confused with multiple .rodata* sections -426751 Valgrind reports "still reachable" memory using musl (alpine running inside docker) -432801 Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals +417993 vbit-test fail on s390x with Iop_Add32: spurious dependency on uninit +426751 Valgrind reports "still reachable" memory using musl + (alpine running inside docker) +432801 Valgrind 3.16.1 reports a jump based on uninitialized memory somehow + related to clang and signals 433857 Add validation to C++17 aligned new/delete alignment size 433859 Add mismatched detection to C++ 17 aligned new/delete 460192 Add epoll_pwait2 diff --git a/memcheck/tests/vbit-test/valgrind.c b/memcheck/tests/vbit-test/valgrind.c index 2597be8bc7..c24a06d649 100644 --- a/memcheck/tests/vbit-test/valgrind.c +++ b/memcheck/tests/vbit-test/valgrind.c @@ -138,4 +138,9 @@ valgrind_execute_test(const irop_t *op, test_data_t *data) print_opnd(stdout, &data->result); printf("\n"); } + + // Now that we have the vbits recorded, clear all the vbits. + for (i = 0; i < num_operands; ++i) { + VALGRIND_MAKE_MEM_DEFINED(&data->opnds[i].value, sizeof(data->opnds[i].value)); + } } -- 2.43.5