add hasSuffix check

charles-zablit · charles-zablit · commit abfa26686e21 · 2025-06-19T17:08:38.000+01:00
diff --git a/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp b/lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
@@ -364,6 +364,10 @@ GetDemangledFunctionSuffix(const SymbolContext &sc) {
 
   auto [demangled_name, info] = *info_or_err;
 
+  if (!info.hasSuffix())
+    return llvm::createStringError("Suffix range for '%s' is invalid.",
+                                   demangled_name.data());
+
   return demangled_name.slice(info.SuffixRange.first, info.SuffixRange.second);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -364,6 +364,10 @@ GetDemangledFunctionSuffix(const SymbolContext &sc) {`
`364`	`364`
`365`	`365`	`auto [demangled_name, info] = *info_or_err;`
`366`	`366`
	`367`	`+ if (!info.hasSuffix())`
	`368`	`+ return llvm::createStringError("Suffix range for '%s' is invalid.",`
	`369`	`+ demangled_name.data());`
	`370`	`+`
`367`	`371`	`return demangled_name.slice(info.SuffixRange.first, info.SuffixRange.second);`
`368`	`372`	`}`
`369`	`373`