Page MenuHomeFreeBSD
Differential D38357

kboot: Assert errno is negative
ClosedPublic
Actions

Authored by imp on Feb 2 2023, 8:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 26, 10:15 AM
Unknown Object (File)
Wed, Mar 25, 5:09 PM
Unknown Object (File)
Wed, Mar 25, 4:53 PM
Unknown Object (File)
Mon, Mar 23, 4:22 PM
Unknown Object (File)
Thu, Mar 19, 2:51 PM
Unknown Object (File)
Thu, Mar 19, 4:23 AM
Unknown Object (File)
Wed, Mar 18, 11:21 PM
Unknown Object (File)
Wed, Mar 18, 8:59 PM
View All 89 Files
Subscribers
dab

Details

Reviewers
brooks
kevans
tsoome
manu
Commits
rG63c7a483e552: kboot: Assert errno is negative
Summary

When converting from a Linux error to a FreeBSD errno, assert that the
value passed in is negative, as is Linux's custom.

Suggested by: brooks
Sponsored by: Netflix

Diff Detail

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

Event Timeline

imp created this revision.Feb 2 2023, 8:27 PM
Herald added a reviewer: manu. · View Herald TranscriptFeb 2 2023, 8:27 PM
Herald added a subscriber: dab. · View Herald Transcript
imp requested review of this revision.Feb 2 2023, 8:27 PM
Harbormaster completed remote builds in B49490: Diff 116361.Feb 2 2023, 8:27 PM
brooks accepted this revision.Feb 2 2023, 8:32 PM

I'd be tempted to allow 0 so it could be used for standard syscall-like functions, but this is fine.

This revision is now accepted and ready to land.Feb 2 2023, 8:32 PM
tsoome accepted this revision.Feb 2 2023, 8:37 PM
Closed by commit rG63c7a483e552: kboot: Assert errno is negative (authored by imp). · Explain WhyFeb 2 2023, 9:12 PM
This revision was automatically updated to reflect the committed changes.
imp added a commit: rG63c7a483e552: kboot: Assert errno is negative.

Revision Contents
Changeset List

PathSize
stand/
kboot/
9 lines

Diff 116364

View Options

stand/kboot/host_syscall.h

Loading...