OSDN Git Service

[lldb/COFF] Remove strtab zeroing hack
authorPavel Labath <pavel@labath.sk>
Wed, 15 Jul 2020 15:13:20 +0000 (17:13 +0200)
committerPavel Labath <pavel@labath.sk>
Fri, 17 Jul 2020 11:24:59 +0000 (13:24 +0200)
commitede7c02b38c0c3adf7fb9ee308bd0f6d92a0eb4e
tree88fed19aa0eeb063f55f1c7599849a8417306a70
parent6c348e4067b0826449caef2f77af2fb67c27040a
[lldb/COFF] Remove strtab zeroing hack

Summary:
This code (recently responsible for a unaligned access sanitizer
failure) claims that the string table offset zero should result in an
empty string.

I cannot find any mention of this detail in the Microsoft COFF
documentation, and the llvm COFF parser also does not handle offset zero
specially. This code was introduced in 0076e7159, which also does not go
into specifics, citing "various bugfixes".

Given that this is obviously a hack, and does not cause tests to fail, I
think we should just delete it.

Reviewers: amccarth, markmentovai

Subscribers: lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D83881
lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp