[Lldb-commits] [PATCH] D80807: [lldb/Utility] Fix DecodeUUIDBytesFromString not to access past the input buffer

Mon Jun 1 02:05:47 PDT 2020

labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

In D80807#2063196 <https://reviews.llvm.org/D80807#2063196>, @friss wrote:

> I would have committed this right away if it weren't for the slight change in behavior I wanted to point out. With this patch, if an input string ends with a `-`, it won't be consumed anymore. I suppose it doesn't matter.

That looks fine. A trailing `-` looks like someone wanted to add more bytes but then pressed return too early, so rejecting that sounds perfectly reasonable.

Since you've obviously been looking at this code lately, I've also added you the to D80755 <https://reviews.llvm.org/D80755>, for the mac perspective of accepting "uuid"s with different lengths.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80807