[Lldb-commits] [lldb] [Reland] Detect against invalid variant index for LibStdC++ std::variant data formatters (PR #69614)

Alex Langford via lldb-commits lldb-commits at lists.llvm.org
Fri Oct 20 09:59:56 PDT 2023


================
@@ -7183,7 +7183,8 @@ GetNthTemplateArgument(const clang::ClassTemplateSpecializationDecl *decl,
   // (including the ones preceding the parameter pack).
   const auto &pack = args[last_idx];
   const size_t pack_idx = idx - last_idx;
-  assert(pack_idx < pack.pack_size() && "parameter pack index out-of-bounds");
+  if (pack_idx >= pack.pack_size())
+    return nullptr;
----------------
bulbazord wrote:

+1, we should still include this change. We shouldn't crash if the `idx` argument is too high.

https://github.com/llvm/llvm-project/pull/69614


More information about the lldb-commits mailing list