[PATCH] D72272: [Sanitizers] Elf mapping update of FreeBSD offset

[Dimitry Andric via Phabricator via llvm-commits]

dim added inline comments.


================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_platform_limits_freebsd.h:27
 # define GET_LINK_MAP_BY_DLOPEN_HANDLE(handle) \
-    ((link_map*)((handle) == nullptr ? nullptr : ((char*)(handle) + 560)))
+    ((link_map*)((handle) == nullptr ? nullptr : ((char*)(handle) + 568)))
 // Get sys/_types.h, because that tells us whether 64-bit inodes are
----------------
eugenis wrote:
> dim wrote:
> > MaskRay wrote:
> > > Can you use `offsetof` instead of the magic number? In the description you can add a reference to the commit that causes the change.
> > Ideally we shouldn't include any system headers here, right?  Otherwise it is not possible to use offsetof, and we should choose the value based on e.g. `__FreeBSD_version`.  @emaste any thoughts?
> > 
> If you'd like, you could include a system header in the corresponding .cpp file, store the offset value in a global and use that global in this macro.
> 
The relevant header file, rtld.h is not exported, but private to the dynamic linker.  Officially, you can get the linkmap by calling dlinfo(3) with `RTLD_DI_LINKMAP` as the request, and this method has been supported for a very long time now.

But maybe it is not feasible to call this here?  Are we allow to call the non-intercepted `dlinfo()` at this point?



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72272/new/

https://reviews.llvm.org/D72272