From aa77c2ca585531023b9ed53b686200534a65e6ff Mon Sep 17 00:00:00 2001 From: Brian Gianforcaro Date: Mon, 3 Jan 2022 03:33:47 -0800 Subject: [PATCH] Kernel: Use unchecked_append in PerformanceEventBuffer `raw_backtrace()` We always validate the size before we append, so there is no reason to use the checked append. --- Kernel/PerformanceEventBuffer.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Kernel/PerformanceEventBuffer.cpp b/Kernel/PerformanceEventBuffer.cpp index c86ef04073..1a24f500cb 100644 --- a/Kernel/PerformanceEventBuffer.cpp +++ b/Kernel/PerformanceEventBuffer.cpp @@ -38,7 +38,7 @@ static Vector raw_backtrace(Fl { Vector backtrace; if (ip != 0) - backtrace.append(ip); + backtrace.unchecked_append(ip); FlatPtr stack_ptr_copy; FlatPtr stack_ptr = bp; // FIXME: Figure out how to remove this SmapDisabler without breaking profile stacks. @@ -63,7 +63,7 @@ static Vector raw_backtrace(Fl break; if (retaddr == 0) break; - backtrace.append(retaddr); + backtrace.unchecked_append(retaddr); if (backtrace.size() == PerformanceEvent::max_stack_frame_count) break; stack_ptr = stack_ptr_copy;