[PATCH] D77924: ld128 demangle: allow space for 'L' suffix.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 30 17:19:54 PDT 2020


efriedma added a comment.

> Yes. And maybe also change the code to use snprintf, so we never end up with a buffer overflow.

Err, I guess it is using snprintf, just not in a way that actually prevents the buffer overflow.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77924/new/

https://reviews.llvm.org/D77924



More information about the llvm-commits mailing list