From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Wed, 17 Oct 2012 00:26:43 +0000 (-0400)
Subject: Windows: OpenTargetDirectory AFSInitFcb Reparse Test
X-Git-Tag: upstream/1.8.0_pre1^2~1911
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=7fd98901620cfb2f44f53b1d717834e39959c538;p=packages%2Fo%2Fopenafs.git

Windows: OpenTargetDirectory AFSInitFcb Reparse Test

In AFSOpenTargetDirectory the test to determine if AFSInitFcb
allocated a FCB or returned an existing one (STATUS_REPARSE)
was reversed.  If AFSInitFcb was called and AFSOpenTargetDirectory
eventually failed, an in use FCB would be freed.

Change-Id: Ie5974fee5202685575f286fe4fef43dfdc0698cb
Reviewed-on: http://gerrit.openafs.org/8244
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Rod Widdowson <rdw@steadingsoftware.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/WINNT/afsrdr/kernel/lib/AFSCreate.cpp b/src/WINNT/afsrdr/kernel/lib/AFSCreate.cpp
index cfaae808c..8f8d9949b 100644
--- a/src/WINNT/afsrdr/kernel/lib/AFSCreate.cpp
+++ b/src/WINNT/afsrdr/kernel/lib/AFSCreate.cpp
@@ -2308,7 +2308,7 @@ AFSOpenTargetDirectory( IN PIRP Irp,
 
                 AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
                               AFS_TRACE_LEVEL_ERROR,
-                              "AFSProcessCreate (%08lX) Failed to initialize fcb %wZ Status %08lX\n",
+                              "AFSOpenTargetDirectory (%08lX) Failed to initialize fcb %wZ Status %08lX\n",
                               Irp,
                               &ParentDirectoryCB->NameInformation.FileName,
                               ntStatus);
@@ -2316,7 +2316,7 @@ AFSOpenTargetDirectory( IN PIRP Irp,
                 try_return( ntStatus);
             }
 
-            if ( ntStatus == STATUS_REPARSE)
+            if ( ntStatus != STATUS_REPARSE)
             {
 
                 bAllocatedFcb = TRUE;