diff options
author | Daniel Sanders <daniel_l_sanders@apple.com> | 2025-07-22 17:45:13 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-07-22 17:45:13 -0700 |
commit | 726502d668d12d917411d48d0b2210f8592b23a9 (patch) | |
tree | de1aaad97dc3563234389456c9b47b38f308b8cd /llvm/lib/CodeGen/StackProtector.cpp | |
parent | e0dd22fab19df2c9370fc2e90bcf57eb0cf25bed (diff) | |
download | llvm-726502d668d12d917411d48d0b2210f8592b23a9.zip llvm-726502d668d12d917411d48d0b2210f8592b23a9.tar.gz llvm-726502d668d12d917411d48d0b2210f8592b23a9.tar.bz2 |
[lldb] Fix UB cast when encountering DW_LANG_* >= eNumLanguageTypes (#150132)
LanguageType has two kinds of enumerators in it. The first is
DWARF-assigned enumerators which must be consecutive and match DW_LANG
values. The second is the vendor-assigned enumerators which must be
unique and must follow on from the DWARF-assigned values (i.e. the first
one is currently eLanguageTypeMojo + 1) even if that collides with
DWARF-assigned values that lldb is not yet aware of
Only the DWARF-assigned enumerators may be static_cast from DW_LANG
since their values match. The vendor-assigned enumerators must be
explicitly converted since their values do not match. This needs to
handle new languages added to DWARF and not yet implemented in lldb.
This fixes a crash when encountering a DW_LANG value >=
eNumLanguageTypes and wrong behaviour when encountering DW_LANG values
that have not yet been added to LanguageType but happen to coincide with
a vendor-assigned enumerator due to the consecutive values requirement
described above.
Another way to fix the crash is to add the language to LanguageType (and
fill any preceeding gaps in the number space) so that the DW_LANG being
encountered is correctly handled but this just moves the problem to a
new subset of DW_LANG values.
Also fix an unnecessary static-cast from LanguageType to LanguageType.
Diffstat (limited to 'llvm/lib/CodeGen/StackProtector.cpp')
0 files changed, 0 insertions, 0 deletions