Page MenuHomeFreeBSD
Differential D41101

amd64 crt1: Explicitly use a PLT entry for main in the PIC case.
ClosedPublic
Actions

Authored by jhb on Jul 19 2023, 6:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 2, 10:43 AM
Unknown Object (File)
Thu, May 2, 10:43 AM
Unknown Object (File)
Thu, May 2, 10:43 AM
Unknown Object (File)
Thu, May 2, 9:08 AM
Unknown Object (File)
Thu, May 2, 1:58 AM
Unknown Object (File)
Thu, May 2, 1:58 AM
Unknown Object (File)
Fri, Apr 26, 1:14 AM
Unknown Object (File)
Thu, Apr 25, 4:17 PM
View All 16 Files
Subscribers
imp
jlduran_gmail.com

Details

Reviewers
kib
dchagin
Commits
rGe5c4737955d5: amd64 crt1: Explicitly use a PLT entry for main in the PIC case.
Summary

This ensures GNU as generates a R_X86_64_PLT32 relocation instead of
R_X86_64_32.

Fixes: c969310c992a csu: Implement _start using as to satisfy unwinders on x86_64

Diff Detail

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

Event Timeline

jhb created this revision.Jul 19 2023, 6:31 PM
Herald added a subscriber: imp. · View Herald TranscriptJul 19 2023, 6:31 PM
jhb requested review of this revision.Jul 19 2023, 6:31 PM
Harbormaster completed remote builds in B52728: Diff 124884.Jul 19 2023, 6:31 PM
jhb added a comment.Jul 19 2023, 6:32 PM

This fixes linking dynamic binaries again with GCC 12.

kib accepted this revision.Jul 19 2023, 6:40 PM
This revision is now accepted and ready to land.Jul 19 2023, 6:40 PM
Closed by commit rGe5c4737955d5: amd64 crt1: Explicitly use a PLT entry for main in the PIC case. (authored by jhb). · Explain WhyJul 19 2023, 6:54 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rGe5c4737955d5: amd64 crt1: Explicitly use a PLT entry for main in the PIC case..
jlduran_gmail.com added a subscriber: jlduran_gmail.com.EditedJul 19 2023, 9:26 PM

Thank you!
I also had to add:

LDFLAGS+=       -Wl,-znoexecstack

to secure/lib/libcrypto/**/**/Makefile.

EDIT: https://github.com/freebsd/freebsd-src/pull/800

jhb added a comment.Jul 20 2023, 5:45 PM
In D41101#935894, @jlduran_gmail.com wrote:

Thank you!
I also had to add:

LDFLAGS+=       -Wl,-znoexecstack

to secure/lib/libcrypto/**/**/Makefile.

EDIT: https://github.com/freebsd/freebsd-src/pull/800

I have a separate review for this: D41102

Revision Contents
Changeset List

PathSize
lib/
csu/
amd64/
7 lines

Diff 124886

View Options

lib/csu/amd64/crt1_s.S

Loading...