d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove a stray else. It isn't needed (due to the return at the end of

Browse Source 
the if statement it pairs with). While not an error today, a careless
edit in the future could cause problems (though given the nature of
this specific code, the problems quite likely would be some variation
of "most direct access SCSI storage devices won't attach," which is
unlikely to go unnoticed).

PVS-Studio: V705
This commit is contained in:
Warner Losh 2016-02-18 15:12:52 +00:00
parent 2a9aa1a646
commit 787db28adf
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
sys/cam/scsi/scsi_da.c
View File

@ -3228,7 +3228,8 @@ dadone(struct cam_periph *periph, union ccb *done_ccb)
softc->state = DA_STATE_PROBE_RC; softc->state = DA_STATE_PROBE_RC;
xpt_schedule(periph, priority); xpt_schedule(periph, priority);
return; return;
} else }
/* /*
* Attach to anything that claims to be a * Attach to anything that claims to be a
* direct access or optical disk device, * direct access or optical disk device,