diff options
author | Jason Molenda <jmolenda@apple.com> | 2024-01-09 15:20:06 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-01-09 15:20:06 -0800 |
commit | 5f71aa9270c3d680babfbc6e766773d113c2a79a (patch) | |
tree | c7292b4f2526b6a225c7484b51f9b4079e46975a /flang/lib/Frontend/CompilerInvocation.cpp | |
parent | b932f03bda5a88f699d33d118ca2735da3c66677 (diff) | |
download | llvm-5f71aa9270c3d680babfbc6e766773d113c2a79a.zip llvm-5f71aa9270c3d680babfbc6e766773d113c2a79a.tar.gz llvm-5f71aa9270c3d680babfbc6e766773d113c2a79a.tar.bz2 |
[lldb] [Mach-O] don't strip the end of the "kern ver str" LC_NOTE (#77538)
The "kern ver str" LC_NOTE gives lldb a kernel version string -- with a
UUID and/or a load address (stext) to load it at. The LC_NOTE specifies
a size of the identifier string in bytes. In
ObjectFileMachO::GetIdentifierString, I copy that number of bytes into a
std::string, and in case there were additional nul characters at the end
of the sting for padding reasons, I tried to shrink the std::string to
not include these extra nul's.
However, I did this resizing without handling the case of an empty
identifier string. I don't know why any corefile creator would do that,
but of course at least one does. This patch removes the resizing
altogether; I was solving something that hasn't ever shown to be a
problem. I also added a test case for this, to check that lldb doesn't
crash when given one of these corefiles.
rdar://120390199
Diffstat (limited to 'flang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions