[ros-diffs] [fireball] 38076: - 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!

Author: fireball
Date: Sun Dec 14 05:59:21 2008
New Revision: 38076

URL: http://svn.reactos.org/svn/reactos?rev=38076&view=rev
Log:
- 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!

Modified:
    trunk/reactos/drivers/storage/scsiport/scsiport.c

Modified: trunk/reactos/drivers/storage/scsiport/scsiport.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/storage/scsiport/sc...
==============================================================================
--- trunk/reactos/drivers/storage/scsiport/scsiport.c [iso-8859-1] (original)
+++ trunk/reactos/drivers/storage/scsiport/scsiport.c [iso-8859-1] Sun Dec 14 05:59:21 2008
@@ -1956,6 +1956,9 @@
                                               PathId,
                                               TargetId,
                                               Lun);
+
+            /* If returned LunExtension is NULL, break out */
+            if (!LunExtension) break;

/* This request should not be processed if */
             if ((LunExtension->ReadyLun) ||