From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Sat, 17 Nov 2012 22:11:42 +0000 (-0500)
Subject: Windows: buf_SetDirty add assertion
X-Git-Tag: upstream/1.8.0_pre1^2~1801
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=f03247ddd7f5b4a9cf8ef132757d794546c884ef;p=packages%2Fo%2Fopenafs.git

Windows: buf_SetDirty add assertion

the userp parameter must never be NULL.

Change-Id: Id785ac417bc869f708990f0bfe53809e3e5c71e5
Reviewed-on: http://gerrit.openafs.org/8472
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/WINNT/afsd/cm_buf.c b/src/WINNT/afsd/cm_buf.c
index 96b5a70f1..1b8ccecc7 100644
--- a/src/WINNT/afsd/cm_buf.c
+++ b/src/WINNT/afsd/cm_buf.c
@@ -1685,6 +1685,7 @@ void buf_SetDirty(cm_buf_t *bp, cm_req_t *reqp, afs_uint32 offset, afs_uint32 le
 {
     osi_assertx(bp->magic == CM_BUF_MAGIC, "invalid cm_buf_t magic");
     osi_assertx(bp->refCount > 0, "cm_buf_t refcount 0");
+    osi_assertx(userp != NULL, "userp is NULL");
 
     if (length == 0)
         return;