From fb916bc870d42cb7980c244b337edcda4be82c36 Mon Sep 17 00:00:00 2001 From: Rene Gollent Date: Mon, 16 Sep 2013 20:22:24 +0200 Subject: [PATCH] Debugger: Fix data copy issue. - B_PRIx8 unfortunately doesn't necessarily strictly stay within the bounds of an 8-bit hex constant, so force the size passed to snprintf to make it that way. --- .../user_interface/gui/inspector_window/MemoryView.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/apps/debugger/user_interface/gui/inspector_window/MemoryView.cpp b/src/apps/debugger/user_interface/gui/inspector_window/MemoryView.cpp index 4c945d01ea..4f2979d951 100644 --- a/src/apps/debugger/user_interface/gui/inspector_window/MemoryView.cpp +++ b/src/apps/debugger/user_interface/gui/inspector_window/MemoryView.cpp @@ -802,7 +802,8 @@ MemoryView::_GetSelectedText(BString& text) char buffer[32]; for (int32 i = 0; i < count; i++) { - _GetNextHexBlock(buffer, sizeof(buffer), data); + _GetNextHexBlock(buffer, std::min(_GetHexDigitsPerBlock() + 1, + (int32)32), data); data += blockSize; text << buffer; if (i < count - 1)