Page MenuHomeFreeBSD
Differential D40148

linux(4): Add a dedicated linux_exec_copyin_args()
ClosedPublic
Actions

Authored by dchagin on May 18 2023, 6:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 13, 1:46 AM
Unknown Object (File)
Sun, Oct 12, 2:41 PM
Unknown Object (File)
Sun, Oct 12, 2:41 PM
Unknown Object (File)
Sun, Oct 12, 2:41 PM
Unknown Object (File)
Sun, Oct 12, 3:34 AM
Unknown Object (File)
Mon, Sep 22, 3:26 AM
Unknown Object (File)
Sep 17 2025, 7:29 AM
Unknown Object (File)
Sep 15 2025, 9:15 PM
View All 86 Files
Subscribers
andrew
emaste
imp

Details

Reviewers
manu
brooks
Commits
rG8340b03425cf: linux(4): Add a dedicated linux_exec_copyin_args()
Summary

Because Linux allows to exec binaries with 0 argc.

MFC after: 1 week

Diff Detail

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

Event Timeline

dchagin created this revision.May 18 2023, 6:42 PM
Herald added a reviewer: manu. · View Herald TranscriptMay 18 2023, 6:42 PM
Herald added a reviewer: brooks. · View Herald Transcript
Herald added subscribers: emaste, andrew, imp. · View Herald Transcript
dchagin requested review of this revision.May 18 2023, 6:42 PM
Harbormaster completed remote builds in B51538: Diff 122084.May 18 2023, 6:42 PM
emaste added a comment.EditedMay 18 2023, 7:33 PM

Are we aware of any actual Linux binary use cases for argc=0? (I don't object to this change, just wondering.)

brooks accepted this revision.May 18 2023, 7:41 PM
This revision is now accepted and ready to land.May 18 2023, 7:41 PM
dchagin added a comment.May 18 2023, 8:46 PM
In D40148#914234, @emaste wrote:

Are we aware of any actual Linux binary use cases for argc=0? (I don't object to this change, just wondering.)

well, I dont know, I’m not use Linux since 1997:)
I try to fix Glibc posix_spawn(), for which some glibc tests fails due to argc=0, also see comment in the linux_exec_copyin_args() explaining this change

Closed by commit rG8340b03425cf: linux(4): Add a dedicated linux_exec_copyin_args() (authored by dchagin). · Explain WhyMay 29 2023, 9:19 AM
This revision was automatically updated to reflect the committed changes.
dchagin added a commit: rG8340b03425cf: linux(4): Add a dedicated linux_exec_copyin_args().

Revision Contents
Changeset List

PathSize
sys/
amd64/
linux/
4 lines
linux32/
14 lines
4 lines
arm64/
linux/
4 lines
compat/
linux/
87 lines
i386/
linux/
4 lines

Diff 122559

View Options

sys/amd64/linux/syscalls.master

Loading...
View Options

sys/amd64/linux32/linux32_machdep.c

Loading...
View Options

sys/amd64/linux32/syscalls.master

Loading...
View Options

sys/arm64/linux/syscalls.master

Loading...
View Options

sys/compat/linux/linux_misc.c

Loading...
View Options

sys/i386/linux/syscalls.master

Loading...