Differential D43202

targ: Handle errors from suword()
ClosedPublic
Actions

Authored by markj on Dec 26 2023, 11:57 PM.

Details

Reviewers

None

Group Reviewers

cam

Commits

rG185634d86d96: targ: Handle errors from suword()
rG904cd456f0f9: targ: Handle errors from suword()

Summary

In targstart() we are already handling an error and have no go way to
signal the failure to upper layers, so ignore the return value of
suword() there.

This is in preparation for annotating copyin() and related functions
with __result_use_check.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 55130
Build 52019: arc lint + arc unit

Event Timeline

markj created this revision.Dec 26 2023, 11:57 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 26 2023, 11:57 PM

markj requested review of this revision.Dec 26 2023, 11:57 PM

Harbormaster completed remote builds in B55130: Diff 131887.Dec 26 2023, 11:58 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jan 4 2024, 1:41 PM

Closed by commit rG904cd456f0f9: targ: Handle errors from suword() (authored by markj). · Explain Why

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG904cd456f0f9: targ: Handle errors from suword().

markj added a commit: rG185634d86d96: targ: Handle errors from suword().Jan 11 2024, 2:34 PM

Revision Contents
Changeset List

Path

Size

sys/

cam/

scsi/

scsi_target.c

9 lines

Diff 131887

View Options

sys/cam/scsi/scsi_target.c

targ: Handle errors from suword()ClosedPublicActions