[PATCH] D68431: [msan] Add interceptors: crypt, crypt_r.

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added inline comments.


================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_common_interceptors.inc:9593
+#if SANITIZER_INTERCEPT_CRYPT_R
+INTERCEPTOR(char *, crypt_r, char *key, char *salt, void *data) {
+  void *ctx;
----------------
eugenis wrote:
> vitalybuka wrote:
> > why you don't check data?
> we'd need a new kind of check for it, because it must all be addressable, but only one field of it is required to be initialized. I can add that...
maybe just write all?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68431