Page MenuHomeFreeBSD
Differential D6099

Move check for (done_ccb == NULL)
ClosedPublic
Actions

Authored by slm on Apr 25 2016, 10:41 PM.
Tags
None
Referenced Files
F81652886: D6099.id15601.diff
Fri, Apr 19, 12:19 PM
F81613205: D6099.id15601.diff
Thu, Apr 18, 11:33 PM
Unknown Object (File)
Mon, Apr 15, 10:24 PM
Unknown Object (File)
Thu, Apr 11, 12:12 AM
Unknown Object (File)
Feb 21 2024, 11:57 PM
Unknown Object (File)
Jan 24 2024, 3:32 AM
Unknown Object (File)
Jan 16 2024, 6:34 PM
Unknown Object (File)
Dec 22 2023, 8:12 AM
View All 51 Files
Subscribers
imp

Details

Reviewers
scottl
ken
ambrisko
allanjude
asomers
mav
Commits
rS299272: done_ccb pointer can be used if it is NULL.
Summary

done_ccb pointer can be used if it is NULL. To prevent this, move check for done_ccb == NULL to before done_ccb is used in the mprsas_stop_unit_done().

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

slm updated this revision to Diff 15601.Apr 25 2016, 10:41 PM
slm retitled this revision from to Move check for (done_ccb == NULL).
slm updated this object.
slm edited the test plan for this revision. (Show Details)
slm added reviewers: ken, scottl, ambrisko, asomers, mav, allanjude.
slm set the repository for this revision to rS FreeBSD src repository - subversion.
Herald added a subscriber: imp. · View Herald TranscriptApr 25 2016, 10:41 PM
asomers accepted this revision.Apr 25 2016, 11:03 PM
asomers edited edge metadata.
This revision is now accepted and ready to land.Apr 25 2016, 11:03 PM
ken accepted this revision.Apr 26 2016, 5:55 PM
ken edited edge metadata.

Looks good.

Closed by commit rS299272: done_ccb pointer can be used if it is NULL. (authored by slm). · Explain WhyMay 9 2016, 4:34 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
mpr/
6 lines

Diff 16070

View Options

head/sys/dev/mpr/mpr_sas_lsi.c

Loading...