[Lldb-commits] [PATCH] D59854: [Host] Add LibraryLoader abstraction around dlopen/LoadLibrary

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

In D59854#1443904 <https://reviews.llvm.org/D59854#1443904>, @JDevlieghere wrote:

> In D59854#1443891 <https://reviews.llvm.org/D59854#1443891>, @jingham wrote:
>
> > It isn't safe to pass "data()" of a StringRef to dlsym.
>
>
> Why?


StringRef's aren't guaranteed to be NULL terminated.


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

https://reviews.llvm.org/D59854