[all-commits] [llvm/llvm-project] 00d3ed: [Reland] Detect against invalid variant index for ...

jeffreytan81 via All-commits all-commits at lists.llvm.org
Tue Oct 24 08:44:49 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 00d3ed6deadb1d647b70ab37ac66ada6a550c1ba
      https://github.com/llvm/llvm-project/commit/00d3ed6deadb1d647b70ab37ac66ada6a550c1ba
  Author: jeffreytan81 <jeffreytan at meta.com>
  Date:   2023-10-24 (Tue, 24 Oct 2023)

  Changed paths:
    M lldb/examples/synthetic/gnu_libstdcpp.py
    M lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
    M lldb/test/API/functionalities/data-formatter/data-formatter-stl/libstdcpp/variant/TestDataFormatterLibStdcxxVariant.py

  Log Message:
  -----------
  [Reland] Detect against invalid variant index for LibStdC++ std::variant data formatters (#69614)

This is relanding of https://github.com/llvm/llvm-project/pull/69253.
`TestTemplatePackArgs.py` is passing now.

https://github.com/llvm/llvm-project/pull/68012/files added new data
formatters for LibStdC++ std::variant.

However, this formatter can crash if std::variant's index field has
invalid value (exceeds the number of template arguments).
This can happen if the current IP stops at a place std::variant is not
initialized yet.

This patch fixes the crash by ensuring the index is a valid value and
fix GetNthTemplateArgument() to make sure it is not crashing.

Co-authored-by: jeffreytan81 <jeffreytan at fb.com>




More information about the All-commits mailing list