From 00b08b94feb12ff15a487bbe373f5f39702cd414 Mon Sep 17 00:00:00 2001 From: Matt Jacob Date: Thu, 15 Mar 2007 06:42:54 +0000 Subject: [PATCH] A silly buglet found by Coverity- check the return value from cam_periph_acquire. --- sys/cam/cam_xpt.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/sys/cam/cam_xpt.c b/sys/cam/cam_xpt.c index ba6eefb8f9d..c123b5cfcd0 100644 --- a/sys/cam/cam_xpt.c +++ b/sys/cam/cam_xpt.c @@ -5653,6 +5653,7 @@ static cam_status proberegister(struct cam_periph *periph, void *arg) { union ccb *request_ccb; /* CCB representing the probe request */ + cam_status status; probe_softc *softc; request_ccb = (union ccb *)arg; @@ -5679,7 +5680,12 @@ proberegister(struct cam_periph *periph, void *arg) periph_links.tqe); softc->flags = 0; periph->softc = softc; - cam_periph_acquire(periph); + status = cam_periph_acquire(periph); + if (status != CAM_REQ_CMP) { + return (status); + } + + /* * Ensure we've waited at least a bus settle * delay before attempting to probe the device.