Skip to content

Commit aa728ff

Browse files
committed
[lldb] Fix stack-use-after-scope issue in ScriptedInterface.h
This patch should fix a `stack-use-after-scope` in the helper function `ScriptedInterface::ErrorWithMessage`. Signed-off-by: Med Ismail Bennani <[email protected]>
1 parent fbe91fe commit aa728ff

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

lldb/include/lldb/Interpreter/ScriptedInterface.h

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,16 @@ class ScriptedInterface {
4040
LLDBLog log_caterogy = LLDBLog::Process) {
4141
LLDB_LOGF(GetLog(log_caterogy), "%s ERROR = %s", caller_name.data(),
4242
error_msg.data());
43-
llvm::Twine err = llvm::Twine(caller_name + llvm::Twine(" ERROR = ") +
44-
llvm::Twine(error_msg));
43+
std::string full_error_message =
44+
llvm::Twine(caller_name + llvm::Twine(" ERROR = ") +
45+
llvm::Twine(error_msg))
46+
.str();
4547
if (const char *detailed_error = error.AsCString())
46-
err.concat(llvm::Twine(" (") + llvm::Twine(detailed_error) +
47-
llvm::Twine(")"));
48-
error.SetErrorString(err.str());
48+
full_error_message +=
49+
llvm::Twine(llvm::Twine(" (") + llvm::Twine(detailed_error) +
50+
llvm::Twine(")"))
51+
.str();
52+
error.SetErrorString(full_error_message);
4953
return {};
5054
}
5155

0 commit comments

Comments
 (0)