Re: [PATCH 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-25 Thread Mike Stump
On Aug 25, 2014, at 11:03 AM, James Clarke wrote: > The __DARWIN_64_BIT_INO_T macro is used to indicate whether dirent.d_ino > is 64-bit or not, and so should be sized appropriately. Additionally, > dirent.d_seekoff should only be defined/checked when > __DARWIN_64_BIT_INO_T is defined. > > libsa

Re: [PATCH 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-25 Thread James Clarke
Please note that this fixes PR 61407 (forgot to add it to libsanitizer/ChangeLog). James Clarke > On 25 Aug 2014, at 19:03, James Clarke wrote: > > The __DARWIN_64_BIT_INO_T macro is used to indicate whether dirent.d_ino > is 64-bit or not, and so should be sized appropriately. Additionally, >

[PATCH 2/2] Fixed dirent sanitizer for 32-bit OS X builds

2014-08-25 Thread James Clarke
The __DARWIN_64_BIT_INO_T macro is used to indicate whether dirent.d_ino is 64-bit or not, and so should be sized appropriately. Additionally, dirent.d_seekoff should only be defined/checked when __DARWIN_64_BIT_INO_T is defined. libsanitizer/ChangeLog: * sanitizer_common/sanitizer_platform_l