diff options
| author | Kuba Mracek <mracek@apple.com> | 2017-11-29 19:43:11 +0000 |
|---|---|---|
| committer | Kuba Mracek <mracek@apple.com> | 2017-11-29 19:43:11 +0000 |
| commit | f1459c278658344f581330056f974ad1376dee78 (patch) | |
| tree | d3189a8bc799204441893bb8a9eecf0349bdb62c /lib | |
| parent | 676c2f0357ff8c3150bdbb6ed77457dfd1e32058 (diff) | |
[asan] Don't crash on fclose(NULL)
It's explicitly forbidden to call fclose with NULL, but at least on Darwin, this succeeds and doesn't segfault. To maintain binary compatibility, ASan should survice fclose(NULL) as well.
Differential Revision: https://reviews.llvm.org/D40053
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@319347 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/sanitizer_common/sanitizer_common_interceptors.inc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/sanitizer_common/sanitizer_common_interceptors.inc b/lib/sanitizer_common/sanitizer_common_interceptors.inc index 613774c72..c8a30bdc6 100644 --- a/lib/sanitizer_common/sanitizer_common_interceptors.inc +++ b/lib/sanitizer_common/sanitizer_common_interceptors.inc @@ -273,7 +273,7 @@ UNUSED static const FileMetadata *GetInterceptorMetadata( MetadataHashMap::Handle h(interceptor_metadata_map, (uptr)addr, /* remove */ false, /* create */ false); - if (h.exists()) { + if (addr && h.exists()) { CHECK(!h.created()); CHECK(h->type == CommonInterceptorMetadata::CIMT_FILE); return &h->file; |