[PATCH] D17479: [sanitizer] Move recvmsg and recv interceptors to sanitizer_common.

Maxim Ostapenko via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 06:52:03 PST 2016


m.ostapenko added inline comments.

================
Comment at: compiler-rt/trunk/lib/sanitizer_common/sanitizer_common_interceptors.inc:5343
@@ +5342,3 @@
+  if (res > 0) {
+    COMMON_INTERCEPTOR_WRITE_RANGE(ctx, buf, len);
+  }
----------------
I've just noticed, that here and below in **recvfrom** we should pass **res** as third parameter, not **len**. Should I create a new revision on Phabricator?

================
Comment at: compiler-rt/trunk/lib/sanitizer_common/sanitizer_common_interceptors.inc:5359
@@ +5358,3 @@
+  if (res > 0) {
+    COMMON_INTERCEPTOR_WRITE_RANGE(ctx, buf, len);
+    if (srcaddr)
----------------
And here.


Repository:
  rL LLVM

http://reviews.llvm.org/D17479





More information about the llvm-commits mailing list