env to dso inits.
ClosedPublic
Actions

Authored by kib on Sep 7 2020, 11:39 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Dec 14, 9:20 AM

	Unknown Object (File)
	Nov 20 2024, 2:57 AM

	Unknown Object (File)
	Sep 26 2024, 7:20 PM

	Unknown Object (File)
	Sep 8 2024, 9:56 PM

	Unknown Object (File)
	Sep 7 2024, 11:41 PM

	Unknown Object (File)
	Sep 7 2024, 4:49 AM

	Unknown Object (File)
	Sep 3 2024, 6:13 PM

	Unknown Object (File)
	Sep 2 2024, 2:05 PM

View All 33 Files

Subscribers

bdragon

imp

Details

Reviewers

emaste
dim

Commits

rS365432: rtld: pass argc/argv/env to dso inits.

Summary

This is consistent with how array inits are called, and also makes us more compatible with glibc environment.

Requested by: Alex S <iwtcex@gmail.com>
PR: 249162

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 33429

Event Timeline

kib created this revision.Sep 7 2020, 11:39 AM

Herald added subscribers: bdragon, imp. · View Herald TranscriptSep 7 2020, 11:39 AM

Harbormaster completed remote builds in B33429: Diff 76734.Sep 7 2020, 11:39 AM

kib requested review of this revision.Sep 7 2020, 11:39 AM

emaste accepted this revision.Sep 7 2020, 6:02 PM

This revision is now accepted and ready to land.Sep 7 2020, 6:02 PM

LGTM, I don't see any downsides. I sort of feel like it's asymmetric that the arguments aren't passed to the fini functions, but glibc also doesn't do that. :)

Closed by commit rS365432: rtld: pass argc/argv/env to dso inits. (authored by kib). · Explain WhySep 7 2020, 9:32 PM

This revision was automatically updated to reflect the committed changes.

kib added a commit: rS365432: rtld: pass argc/argv/env to dso inits..

Revision Contents
Changeset List

Path

Size

libexec/

rtld-elf/

rtld.c

2 lines

Diff 76734

View Options

rtld: pass argc/argv/env to dso inits.ClosedPublicActions