Page MenuHomeFreeBSD
Differential D32088

nvme: Sanity check completion id
ClosedPublic
Actions

Authored by imp on Sep 24 2021, 12:11 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Aug 15, 3:09 AM
Unknown Object (File)
Fri, Aug 14, 8:47 PM
Unknown Object (File)
Wed, Aug 12, 11:18 PM
Unknown Object (File)
Wed, Aug 12, 1:59 PM
Unknown Object (File)
Wed, Aug 12, 2:25 AM
Unknown Object (File)
Tue, Aug 11, 7:34 AM
Unknown Object (File)
Mon, Aug 10, 2:09 PM
Unknown Object (File)
Mon, Aug 10, 1:52 PM
View All Files
Subscribers
dab

Details

Reviewers
chs
mav
chuck
Commits
rG1397c8ebb7ef: nvme: Sanity check completion id
rG36a87d0c6fe9: nvme: Sanity check completion id
Summary

Make sure the completion ID is in the range of [0..num_trackers) since
the values past the end of the act_tr array are never going to be valid
trackers and will lead to pain and suffering if we try to dereference
them to get the tracker or to set the tracker back to NULL as we
complete the I/O.

Sponsored by: Netflix

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 41685
Build 38574: arc lint + arc unit

Revision Contents
Changeset List

PathSize
sys/
dev/
nvme/
8 lines

Diff 95592

View Options

sys/dev/nvme/nvme_qpair.c

Loading...