From 5e305bbdf60acfba36ef67eb3dcbd54a794ecaca Mon Sep 17 00:00:00 2001 From: Aleksey Bragin Date: Sun, 14 Dec 2008 11:59:21 +0000 Subject: [PATCH] - Undo one "fix" from 38075. Coverity warned you about possible double check of the same pointer (though in || you probably can't go without it), but one check should definately stay! SpiGetLunExtension can return NULL! svn path=/trunk/; revision=38076 --- reactos/drivers/storage/scsiport/scsiport.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/reactos/drivers/storage/scsiport/scsiport.c b/reactos/drivers/storage/scsiport/scsiport.c index fa3d1fb0ec7..905773c169e 100644 --- a/reactos/drivers/storage/scsiport/scsiport.c +++ b/reactos/drivers/storage/scsiport/scsiport.c @@ -1957,6 +1957,9 @@ ScsiPortNotification(IN SCSI_NOTIFICATION_TYPE NotificationType, TargetId, Lun); + /* If returned LunExtension is NULL, break out */ + if (!LunExtension) break; + /* This request should not be processed if */ if ((LunExtension->ReadyLun) || (LunExtension->SrbInfo.Srb))