From d7fd254a798775d7d4208c6a8ded7780cd8632b8 Mon Sep 17 00:00:00 2001
From: hjk <hjk@qt.io>
Date: Thu, 7 Jul 2022 16:33:08 +0200
Subject: [PATCH] Utils: Fix maximum size computation for backtraces on abort

Change-Id: Ia974343cdeb6d54d2e7331735c9cdd063f445af1
Reviewed-by: Christian Stenger <christian.stenger@qt.io>
---
 src/libs/utils/qtcassert.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/libs/utils/qtcassert.cpp b/src/libs/utils/qtcassert.cpp
index 560df43a196..51bb529427a 100644
--- a/src/libs/utils/qtcassert.cpp
+++ b/src/libs/utils/qtcassert.cpp
@@ -38,11 +38,12 @@ namespace Utils {
 
 void dumpBacktrace(int maxdepth)
 {
-    if (maxdepth == -1)
-        maxdepth = 1000;
+    const int ArraySize = 1000;
+    if (maxdepth < 0 || maxdepth > ArraySize)
+        maxdepth = ArraySize;
 #if defined(Q_OS_UNIX)
-    void *bt[1000] = {nullptr};
-    int size = backtrace(bt, sizeof(bt) / sizeof(bt[0]));
+    void *bt[ArraySize] = {nullptr};
+    int size = backtrace(bt, maxdepth);
     char **lines = backtrace_symbols(bt, size);
     for (int i = 0; i < size; ++i)
         qDebug() << "0x" + QByteArray::number(quintptr(bt[i]), 16) << lines[i];