[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 11:24:09 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);
+ }
----------------
eugenis wrote:
> m.ostapenko wrote:
> > 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?
> I think you can submit such an obvious fix directly.
>
Thanks, I've already done it (approved by Dmitry): http://reviews.llvm.org/D17608.
Repository:
rL LLVM
http://reviews.llvm.org/D17479
More information about the llvm-commits
mailing list