[all-commits] [llvm/llvm-project] 9dd2d5: [LLD] [COFF] Use the new encodeSectionName() helpe...

[Martin Storsjö via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9dd2d50984f5b02e19d3576812dc9b09ddcc9483
      https://github.com/llvm/llvm-project/commit/9dd2d50984f5b02e19d3576812dc9b09ddcc9483
  Author: Martin Storsjö <martin at martin.st>
  Date:   2022-03-01 (Tue, 01 Mar 2022)

  Changed paths:
    M lld/COFF/Writer.cpp

  Log Message:
  -----------
  [LLD] [COFF] Use the new encodeSectionName() helper for long section names

The previous code used an unbounded sprintf, which in theory can
overflow, writing either the null terminator or the last digits
into the next struct member.

In practice, in LLD, all long section names are written sequentially
first at the start of the string table, followed by all the long
symbol names. Due to this, even if the total string table would
end up large, the long section names have fairly short offsets,
which is why this hasn't been an issue in practice.

I don't think it's worth trying to write a test that produces an
executable with enough long section names to make the section names
themselves exceed 10^6 bytes, which is currently necessary to trigger
faults with the previous form.

Differential Revision: https://reviews.llvm.org/D120676