Skip to content

Commit fd388c1

Browse files
dotnwatvbotbuildovich
authored andcommitted
utils: fix container-overflow in tracking allocator
================================================================= ==524==ERROR: AddressSanitizer: container-overflow on address 0x52100028a100 at pc 0x7f29c0bf51fd bp 0x7ffda75397d0 sp 0x7ffda75397c8 READ of size 4 at 0x52100028a100 thread T0 #0 0x7f29c0bf51fc in util::mem_tracker::pretty_print_ascii() const /var/lib/buildkite-agent/builds/buildkite-amd64-builders-i-024f8676eb7b139e9-1/redpanda/vtools/src/v/utils/tracking_allocator.cc:75:31 redpanda-data#1 0x55be1ac7eb7b in mem_tracker_pretty_printing::test_method() /var/lib/buildkite-agent/builds/buildkite-amd64-builders-i-024f8676eb7b139e9-1/redpanda/vtools/src/v/utils/tests/tracking_allocator_tests.cc:82:5 redpanda-data#2 0x55be1ac7d651 in mem_tracker_pretty_printing_invoker() /var/lib/buildkite-agent/builds/buildkite-amd64-builders-i-024f8676eb7b139e9-1/redpanda/vtools/src/v/utils/tests/tracking_allocator_tests.cc:50:1 Signed-off-by: Noah Watkins <[email protected]> (cherry picked from commit 71c352c)
1 parent 4fd4c65 commit fd388c1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/v/utils/tracking_allocator.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,8 @@ ss::sstring mem_tracker::pretty_print_ascii() const {
7171
// rooted at the current tracker to avoid an O(n^2) loop.
7272
result << pretty_print_ascii_level(
7373
top.first, tracker._label, tracker._consumption);
74-
stack.pop();
7574
auto next_level = top.first + 1;
75+
stack.pop();
7676
for (const auto& child : tracker._children) {
7777
stack.push(std::pair(next_level, std::cref(child)));
7878
}

0 commit comments

Comments
 (0)