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)
Sat, Sep 5, 2:20 PM
Unknown Object (File)
Sat, Sep 5, 4:04 AM
Unknown Object (File)
Fri, Sep 4, 9:22 PM
Unknown Object (File)
Fri, Sep 4, 8:37 PM
Unknown Object (File)
Fri, Sep 4, 3:06 AM
Unknown Object (File)
Wed, Sep 2, 9:22 PM
Unknown Object (File)
Wed, Sep 2, 2:06 PM
Unknown Object (File)
Wed, Sep 2, 9:11 AM
View All 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 Skipped
Unit
Tests Skipped
Build Status
Buildable 49490
Build 46380: arc lint + arc unit

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 116361

View Options

stand/kboot/host_syscall.h

Loading...