Page MenuHomeFreeBSD
Differential D43101

freebsd32: Report errors when copying out oldlenp in __sysctl
ClosedPublic
Actions

Authored by markj on Dec 19 2023, 3:14 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jul 5, 11:18 PM
Unknown Object (File)
Thu, Jul 2, 2:04 AM
Unknown Object (File)
Thu, Jun 25, 9:00 AM
Unknown Object (File)
Wed, Jun 24, 10:44 PM
Unknown Object (File)
Tue, Jun 23, 11:46 AM
Unknown Object (File)
May 14 2026, 9:07 AM
Unknown Object (File)
May 13 2026, 1:04 AM
Unknown Object (File)
Apr 23 2026, 3:33 AM
View All 97 Files
Subscribers
imp

Details

Reviewers
kib
Commits
rG573b117f6e30: freebsd32: Report errors when copying out oldlenp in __sysctl
rG8fa808af6fb9: freebsd32: Report errors when copying out oldlenp in __sysctl
rG81eb7baa69e9: freebsd32: Report errors when copying out oldlenp in __sysctl
Summary

This matches the native implementation's behaviour.

MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 55031
Build 51920: arc lint + arc unit

Event Timeline

markj created this revision.Dec 19 2023, 3:14 AM
Herald added a subscriber: imp. · View Herald TranscriptDec 19 2023, 3:14 AM
markj requested review of this revision.Dec 19 2023, 3:14 AM
Harbormaster completed remote builds in B55007: Diff 131552.Dec 19 2023, 3:14 AM
kib added inline comments.Dec 19 2023, 8:02 AM
sys/compat/freebsd32/freebsd32_misc.c
2679

Shouldn't this be converted from -1 to EFAULT?

markj updated this revision to Diff 131581.Dec 19 2023, 3:59 PM
markj marked an inline comment as done.

Fix error handling

Harbormaster completed remote builds in B55031: Diff 131581.Dec 19 2023, 3:59 PM
markj added inline comments.Dec 19 2023, 3:59 PM
sys/compat/freebsd32/freebsd32_misc.c
2679

Indeed, thank you. sysctlbyname had the same bug. In fact, I found several examples with a quick search.

kib accepted this revision.Dec 19 2023, 4:22 PM
This revision is now accepted and ready to land.Dec 19 2023, 4:22 PM
Closed by commit rG81eb7baa69e9: freebsd32: Report errors when copying out oldlenp in __sysctl (authored by markj). · Explain WhyDec 26 2023, 2:04 AM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG81eb7baa69e9: freebsd32: Report errors when copying out oldlenp in __sysctl.
markj added a commit: rG8fa808af6fb9: freebsd32: Report errors when copying out oldlenp in __sysctl.Jan 2 2024, 12:38 AM
markj added a commit: rG573b117f6e30: freebsd32: Report errors when copying out oldlenp in __sysctl.Jan 2 2024, 1:14 AM

Revision Contents
Changeset List

PathSize
sys/
compat/
freebsd32/
11 lines

Diff 131581

View Options

sys/compat/freebsd32/freebsd32_misc.c

Loading...