From 1e6ec171caced9b7b3dd78c4931d7047bad07e1f Mon Sep 17 00:00:00 2001 From: Jeffrey Altman Date: Sat, 29 Dec 2012 16:10:14 -0500 Subject: [PATCH] Windows: RDR FILE Processing Tracing Additional/improved trace messaging for name not found and path not found errors. Change-Id: Ieedbee072c0f2c7050a33734fe16b4dfc0504cea Reviewed-on: http://gerrit.openafs.org/8861 Tested-by: BuildBot Reviewed-by: Jeffrey Altman Tested-by: Jeffrey Altman --- src/WINNT/afsrdr/kernel/lib/AFSCreate.cpp | 2 +- src/WINNT/afsrdr/kernel/lib/AFSGeneric.cpp | 16 +++++++++-- .../afsrdr/kernel/lib/AFSNameSupport.cpp | 28 +++++++++++++++++++ 3 files changed, 43 insertions(+), 3 deletions(-) diff --git a/src/WINNT/afsrdr/kernel/lib/AFSCreate.cpp b/src/WINNT/afsrdr/kernel/lib/AFSCreate.cpp index d42220d15..cf6b6f5c3 100644 --- a/src/WINNT/afsrdr/kernel/lib/AFSCreate.cpp +++ b/src/WINNT/afsrdr/kernel/lib/AFSCreate.cpp @@ -887,7 +887,7 @@ AFSCommonCreate( IN PDEVICE_OBJECT DeviceObject, AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING, AFS_TRACE_LEVEL_ERROR, - "AFSCommonCreate Failing access to %wZ\n", + "AFSCommonCreate Failing access to %wZ Name not found\n", &uniFileName); try_return( ntStatus = STATUS_OBJECT_NAME_NOT_FOUND); diff --git a/src/WINNT/afsrdr/kernel/lib/AFSGeneric.cpp b/src/WINNT/afsrdr/kernel/lib/AFSGeneric.cpp index 6289b3a21..201ca8dfc 100644 --- a/src/WINNT/afsrdr/kernel/lib/AFSGeneric.cpp +++ b/src/WINNT/afsrdr/kernel/lib/AFSGeneric.cpp @@ -1440,6 +1440,11 @@ AFSValidateSymLink( IN GUID *AuthGroup, pDirEntry->FileType == AFS_FILE_TYPE_INVALID) { + AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING, + AFS_TRACE_LEVEL_VERBOSE_2, + "AFSValidateSymLink Invalid type Status %08lX\n", + STATUS_OBJECT_NAME_NOT_FOUND); + try_return( ntStatus = STATUS_OBJECT_NAME_NOT_FOUND); } @@ -2252,6 +2257,12 @@ AFSSubstituteSysName( IN UNICODE_STRING *ComponentName, if( pSysName == NULL) { + AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING, + AFS_TRACE_LEVEL_VERBOSE_2, + "AFSSubstituteSysName No sysname %wZ Status %08lX\n", + &ComponentName, + STATUS_OBJECT_NAME_NOT_FOUND); + try_return( ntStatus = STATUS_OBJECT_NAME_NOT_FOUND); } @@ -8578,8 +8589,9 @@ AFSCheckSymlinkAccess( IN AFSDirectoryCB *ParentDirectoryCB, AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING, AFS_TRACE_LEVEL_VERBOSE_2, - "AFSCheckSymlinkAccess Failed to locate entry %wZ\n", - ComponentName); + "AFSCheckSymlinkAccess Failed to locate entry %wZ ntStatus %08X\n", + ComponentName, + STATUS_OBJECT_NAME_NOT_FOUND); try_return( ntStatus = STATUS_OBJECT_NAME_NOT_FOUND); } diff --git a/src/WINNT/afsrdr/kernel/lib/AFSNameSupport.cpp b/src/WINNT/afsrdr/kernel/lib/AFSNameSupport.cpp index 776fa33d6..b17c2d9b9 100644 --- a/src/WINNT/afsrdr/kernel/lib/AFSNameSupport.cpp +++ b/src/WINNT/afsrdr/kernel/lib/AFSNameSupport.cpp @@ -1503,6 +1503,16 @@ AFSLocateNameEntry( IN GUID *AuthGroup, { ntStatus = STATUS_OBJECT_PATH_NOT_FOUND; + + AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING, + AFS_TRACE_LEVEL_VERBOSE, + "AFSLocateNameEntry (FO: %08lX) Returning path not found for %wZ FID %08lX-%08lX-%08lX-%08lX\n", + FileObject, + &uniSearchName, + pCurrentObject->FileId.Cell, + pCurrentObject->FileId.Volume, + pCurrentObject->FileId.Vnode, + pCurrentObject->FileId.Unique); } else { @@ -1778,7 +1788,18 @@ AFSLocateNameEntry( IN GUID *AuthGroup, if( uniRemainingPath.Length > 0) { + ntStatus = STATUS_OBJECT_PATH_NOT_FOUND; + + AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING, + AFS_TRACE_LEVEL_VERBOSE, + "AFSLocateNameEntry (FO: %08lX) Returning path not found(2) for %wZ FID %08lX-%08lX-%08lX-%08lX\n", + FileObject, + &uniComponentName, + pCurrentObject->FileId.Cell, + pCurrentObject->FileId.Volume, + pCurrentObject->FileId.Vnode, + pCurrentObject->FileId.Unique); } else { @@ -3594,6 +3615,13 @@ AFSParseName( IN PIRP Irp, ntStatus == STATUS_OBJECT_PATH_NOT_FOUND) { + AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING, + AFS_TRACE_LEVEL_VERBOSE, + "AFSParseName (%08lX) AFSCheckCellName %wZ returned path not found; ntStatus %08X\n", + Irp, + &uniComponentName, + STATUS_OBJECT_NAME_NOT_FOUND); + ntStatus = STATUS_OBJECT_NAME_NOT_FOUND; } -- 2.39.5