From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Wed, 8 Jul 2015 23:47:26 +0000 (-0400)
Subject: Windows: cm_Lookup return ambiguous filename to caller
X-Git-Tag: upstream/1.8.0_pre1^2~218
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=7e3289891d9032288f32b550ba6376f59f7e9a08;p=packages%2Fo%2Fopenafs.git

Windows: cm_Lookup return ambiguous filename to caller

cm_Lookup() must not mask a CM_ERROR_AMBIGUOUS_FILENAME error by
converting it to CM_ERROR_BPLUS_NOMATCH.  Doing so results in the
redirector believing that the object does not exist instead of
there being a STATUS_OBJECT_NAME_COLLISION.

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

diff --git a/src/WINNT/afsd/cm_vnodeops.c b/src/WINNT/afsd/cm_vnodeops.c
index 104db3356..1c3edd6df 100644
--- a/src/WINNT/afsd/cm_vnodeops.c
+++ b/src/WINNT/afsd/cm_vnodeops.c
@@ -1131,7 +1131,10 @@ retry_lookup:
                 goto haveFid;
             }
 
-            code = CM_ERROR_BPLUS_NOMATCH;
+	    if (code == CM_ERROR_AMBIGUOUS_FILENAME)
+		goto done;
+
+	    code = CM_ERROR_BPLUS_NOMATCH;
             goto notfound;
         }
 #endif