[PATCH] D43702: [asan] Fix bug where suppression of overlapping accesses was ignored.

Dan Liew via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 8 08:48:30 PST 2018


delcypher added a comment.

In https://reviews.llvm.org/D43702#1030327, @kubamracek wrote:

> LGTM, thanks!


I decided that I should probably add tests for the other functions where we do this overlap check. Could you check you're still happy with this patch?

Sidenote: It looks like we are missing interceptors for

- `stpcpy()`
- `strlcpy()`
- `strlcat()`

Fixing that is out of scope for this patch though.


https://reviews.llvm.org/D43702





More information about the llvm-commits mailing list