Page MenuHomeFreeBSD

libsys: link symbolic
ClosedPublic

Authored by kib on Feb 22 2024, 9:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 7, 10:51 AM
Unknown Object (File)
Thu, Nov 7, 10:50 AM
Unknown Object (File)
Tue, Nov 5, 4:50 PM
Unknown Object (File)
Tue, Nov 5, 4:38 PM
Unknown Object (File)
Mon, Oct 28, 2:07 PM
Unknown Object (File)
Tue, Oct 22, 3:54 PM
Unknown Object (File)
Oct 1 2024, 4:58 AM
Unknown Object (File)
Sep 30 2024, 11:30 AM

Details

Summary

so that libc/libthr do not preempt libsys symbols, esp. errno-related

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib requested review of this revision.Feb 22 2024, 9:14 PM

To confirm I understand this:

  • Linking libsys.so without -Bsymbolic means the linker deems the various symbols preemptible
  • If libc.so is earlier in the link map then libsys's unfiltered symbols will be preempted by libc.so's at run time

I can conform this fixes the issues I've seen with an WITH_BIND_NOW jail.

This revision is now accepted and ready to land.Feb 22 2024, 11:53 PM

To confirm I understand this:

  • Linking libsys.so without -Bsymbolic means the linker deems the various symbols preemptible
  • If libc.so is earlier in the link map then libsys's unfiltered symbols will be preempted by libc.so's at run time

It also means that symbols looked up not _from the filter_ are preempted. This is the cause of the problem

This revision was automatically updated to reflect the committed changes.

This change breaks legacy programs that declare extern int errno; instead of including errno.h. Maybe we want to find a different solution.

In D44027#1066388, @fuz wrote:

This change breaks legacy programs that declare extern int errno; instead of including errno.h. Maybe we want to find a different solution.

See D46780 for a change that completes the removal of the errno symbol.