From d3deac34eade92a59c9f9262249091a21405ff12 Mon Sep 17 00:00:00 2001
From: cperciva <cperciva@FreeBSD.org>
Date: Sun, 22 Feb 2004 01:08:33 +0000
Subject: [PATCH] If we're going to assert that logData != NULL, do it before
 we try to dereference logData.

Reported by:	"Ted Unangst" <tedu@coverity.com>
Approved by:	rwatson (mentor), scottl
---
 sys/dev/raidframe/rf_paritylog.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/dev/raidframe/rf_paritylog.c b/sys/dev/raidframe/rf_paritylog.c
index 459b5686c772..87c33e6d71af 100644
--- a/sys/dev/raidframe/rf_paritylog.c
+++ b/sys/dev/raidframe/rf_paritylog.c
@@ -690,9 +690,9 @@ rf_ParityLogAppend(
 	 * 
 	 * NON-BLOCKING */
 
+	RF_ASSERT(logData != NULL);
 	raidPtr = logData->common->raidPtr;
 	/* lock the region for the first item in logData */
-	RF_ASSERT(logData != NULL);
 	regionID = logData->regionID;
 	RF_LOCK_MUTEX(raidPtr->regionInfo[regionID].mutex);
 	RF_ASSERT(raidPtr->regionInfo[regionID].loggingEnabled);