/*
* Copyright (c) 2008, 2009, 2010, 2011 Kernel Drivers, LLC.
- * Copyright (c) 2009, 2010, 2011 Your File System, Inc.
+ * Copyright (c) 2009, 2010, 2011, 2014 Your File System, Inc.
* All rights reserved.
*
* Redistribution and use in source and binary forms, with or without
if( pFcb == NULL)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSQueryFileInfo Attempted access (%p) when pFcb == NULL\n",
- Irp);
+ Irp));
try_return( ntStatus = STATUS_INVALID_DEVICE_REQUEST);
}
&stAuthGroup);
ntStatus = AFSVerifyEntry( &stAuthGroup,
- pCcb->DirectoryCB);
+ pCcb->DirectoryCB,
+ FALSE);
if ( NT_SUCCESS( ntStatus))
{
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFS_TRACE_LEVEL_VERBOSE,
+ "AFSQueryFileInfo FID %08lX-%08lX-%08lX-%08lX Clearing Verify Flag\n",
+ pFcb->ObjectInformation->FileId.Cell,
+ pFcb->ObjectInformation->FileId.Volume,
+ pFcb->ObjectInformation->FileId.Vnode,
+ pFcb->ObjectInformation->FileId.Unique));
+
ClearFlag( pFcb->ObjectInformation->Flags, AFS_OBJECT_FLAGS_VERIFY);
}
else
// Grab the main shared right off the bat
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSQueryFileInfo Acquiring Fcb lock %p SHARED %08lX\n",
&pFcb->NPFcb->Resource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireShared( &pFcb->NPFcb->Resource,
TRUE);
if( pFcb->Header.NodeTypeCode == AFS_SPECIAL_SHARE_FCB)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSQueryFileInfo Processing request against SpecialShare Fcb\n");
+ "AFSQueryFileInfo Processing request against SpecialShare Fcb\n"));
ntStatus = AFSProcessShareQueryInfo( Irp,
pFcb,
}
else if( pFcb->Header.NodeTypeCode == AFS_IOCTL_FCB)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSQueryFileInfo request against PIOCtl Fcb\n");
+ "AFSQueryFileInfo request against PIOCtl Fcb\n"));
ntStatus = AFSProcessPIOCtlQueryInfo( Irp,
pFcb,
else if( pFcb->Header.NodeTypeCode == AFS_INVALID_FCB)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSQueryFileInfo request against Invalid Fcb\n");
+ "AFSQueryFileInfo request against Invalid Fcb\n"));
try_return( ntStatus = STATUS_ACCESS_DENIED);
}
try_return( ntStatus);
}
- ntStatus = AFSQueryAccess( Irp,
- pFcb,
- &pAllInfo->AccessInformation,
- &lLength);
+ //
+ // We skip setting AccessInformation since this is set by the IO Mgr prior
+ // to sending this request to the file system
+ //
- if( !NT_SUCCESS( ntStatus))
- {
+ if( lLength < sizeof( FILE_ACCESS_INFORMATION))
+ {
+ try_return( ntStatus = STATUS_INFO_LENGTH_MISMATCH);
+ }
- try_return( ntStatus);
- }
+ lLength -= sizeof( FILE_ACCESS_INFORMATION);
ntStatus = AFSQueryPositionInfo( Irp,
pFcb,
try_return( ntStatus);
}
- ntStatus = AFSQueryMode( Irp,
- pFcb,
- &pAllInfo->ModeInformation,
- &lLength);
+ //
+ // We skip setting ModeInformation and AlignmentInformation since this is set by the IO Mgr prior
+ // to sending this request to the file system
+ //
- if( !NT_SUCCESS( ntStatus))
- {
+ if( lLength < sizeof( FILE_MODE_INFORMATION))
+ {
+ try_return( ntStatus = STATUS_INFO_LENGTH_MISMATCH);
+ }
- try_return( ntStatus);
- }
+ lLength -= sizeof( FILE_MODE_INFORMATION);
- ntStatus = AFSQueryAlignment( Irp,
- pFcb,
- &pAllInfo->AlignmentInformation,
- &lLength);
+ if( lLength < sizeof( FILE_ALIGNMENT_INFORMATION))
+ {
+ try_return( ntStatus = STATUS_INFO_LENGTH_MISMATCH);
+ }
- if( !NT_SUCCESS( ntStatus))
- {
+ lLength -= sizeof( FILE_ALIGNMENT_INFORMATION);
- try_return( ntStatus);
- }
+ //
+ // Populate the name information
+ //
ntStatus = AFSQueryNameInfo( Irp,
pCcb->DirectoryCB,
pCcb->DirectoryCB != NULL)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSQueryFileInfo Failed to process request for %wZ FID %08lX-%08lX-%08lX-%08lX Status %08lX\n",
&pCcb->DirectoryCB->NameInformation.FileName,
- pCcb->DirectoryCB->ObjectInformation->FileId.Cell,
- pCcb->DirectoryCB->ObjectInformation->FileId.Volume,
- pCcb->DirectoryCB->ObjectInformation->FileId.Vnode,
- pCcb->DirectoryCB->ObjectInformation->FileId.Unique,
- ntStatus);
+ pFcb->ObjectInformation->FileId.Cell,
+ pFcb->ObjectInformation->FileId.Volume,
+ pFcb->ObjectInformation->FileId.Vnode,
+ pFcb->ObjectInformation->FileId.Unique,
+ ntStatus));
}
}
}
__except( AFSExceptionFilter( __FUNCTION__, GetExceptionCode(), GetExceptionInformation()) )
{
- AFSDbgLogMsg( 0,
+ AFSDbgTrace(( 0,
0,
- "EXCEPTION - AFSQueryFileInfo\n");
+ "EXCEPTION - AFSQueryFileInfo\n"));
AFSDumpTraceFilesFnc();
if( pFcb == NULL)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetFileInfo Attempted access (%p) when pFcb == NULL\n",
- Irp);
+ Irp));
try_return( ntStatus = STATUS_INVALID_DEVICE_REQUEST);
}
// Grab the Fcb EXCL
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetFileInfo Acquiring Fcb lock %p EXCL %08lX\n",
&pFcb->NPFcb->Resource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireExcl( &pFcb->NPFcb->Resource,
TRUE);
if( pFcb->Header.NodeTypeCode == AFS_IOCTL_FCB)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
- "AFSSetFileInfo Failing request against PIOCtl Fcb\n");
+ "AFSSetFileInfo Failing request against PIOCtl Fcb\n"));
try_return( ntStatus = STATUS_INVALID_DEVICE_REQUEST);
}
else if( pFcb->Header.NodeTypeCode == AFS_SPECIAL_SHARE_FCB)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSSetFileInfo Processing request against SpecialShare Fcb\n");
+ "AFSSetFileInfo Processing request against SpecialShare Fcb\n"));
ntStatus = AFSProcessShareSetInfo( Irp,
pFcb,
try_return( ntStatus);
}
- if( BooleanFlagOn( pFcb->ObjectInformation->VolumeCB->VolumeInformation.Characteristics, FILE_READ_ONLY_DEVICE))
+ if( FileInformationClass != FilePositionInformation &&
+ BooleanFlagOn( pFcb->ObjectInformation->VolumeCB->VolumeInformation.FileSystemAttributes, FILE_READ_ONLY_VOLUME))
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetFileInfo Request failed due to read only volume\n",
- Irp);
+ Irp));
try_return( ntStatus = STATUS_MEDIA_WRITE_PROTECTED);
}
if( pFcb->Header.NodeTypeCode == AFS_INVALID_FCB &&
FileInformationClass != FileDispositionInformation)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSSetFileInfo request against Invalid Fcb\n");
+ "AFSSetFileInfo request against Invalid Fcb\n"));
try_return( ntStatus = STATUS_ACCESS_DENIED);
}
RtlZeroMemory( &stParentFileId,
sizeof( AFSFileID));
- if( pFcb->ObjectInformation->ParentObjectInformation != NULL)
+ if( BooleanFlagOn( pFcb->ObjectInformation->Flags, AFS_OBJECT_FLAGS_PARENT_FID))
{
- stParentFileId = pFcb->ObjectInformation->ParentObjectInformation->FileId;
+
+ stParentFileId = pFcb->ObjectInformation->ParentFileId;
}
//
case FileBasicInformation:
{
- bUpdateFileInfo = TRUE;
-
ntStatus = AFSSetBasicInfo( Irp,
- pCcb->DirectoryCB);
+ pCcb->DirectoryCB,
+ &bUpdateFileInfo);
break;
}
AFSUnwindFileInfo( pFcb,
pCcb);
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetFileInfo Failed to send file info update to service request for %wZ FID %08lX-%08lX-%08lX-%08lX Status %08lX\n",
&pCcb->DirectoryCB->NameInformation.FileName,
- pCcb->DirectoryCB->ObjectInformation->FileId.Cell,
- pCcb->DirectoryCB->ObjectInformation->FileId.Volume,
- pCcb->DirectoryCB->ObjectInformation->FileId.Vnode,
- pCcb->DirectoryCB->ObjectInformation->FileId.Unique,
- ntStatus);
+ pFcb->ObjectInformation->FileId.Cell,
+ pFcb->ObjectInformation->FileId.Volume,
+ pFcb->ObjectInformation->FileId.Vnode,
+ pFcb->ObjectInformation->FileId.Unique,
+ ntStatus));
AFSReleaseResource( &pFcb->NPFcb->Resource);
}
pCcb->DirectoryCB != NULL)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetFileInfo Failed to process request for %wZ FID %08lX-%08lX-%08lX-%08lX Status %08lX\n",
&pCcb->DirectoryCB->NameInformation.FileName,
- pCcb->DirectoryCB->ObjectInformation->FileId.Cell,
- pCcb->DirectoryCB->ObjectInformation->FileId.Volume,
- pCcb->DirectoryCB->ObjectInformation->FileId.Vnode,
- pCcb->DirectoryCB->ObjectInformation->FileId.Unique,
- ntStatus);
+ pFcb->ObjectInformation->FileId.Cell,
+ pFcb->ObjectInformation->FileId.Volume,
+ pFcb->ObjectInformation->FileId.Vnode,
+ pFcb->ObjectInformation->FileId.Unique,
+ ntStatus));
}
}
}
__except( AFSExceptionFilter( __FUNCTION__, GetExceptionCode(), GetExceptionInformation()) )
{
- AFSDbgLogMsg( 0,
+ AFSDbgTrace(( 0,
0,
- "EXCEPTION - AFSSetFileInfo\n");
+ "EXCEPTION - AFSSetFileInfo\n"));
AFSDumpTraceFilesFnc();
IN OUT PFILE_BASIC_INFORMATION Buffer,
IN OUT PLONG Length)
{
- NTSTATUS ntStatus = STATUS_SUCCESS;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
AFSDeviceExt *pDeviceExt = (AFSDeviceExt *)AFSRDRDeviceObject->DeviceExtension;
ULONG ulFileAttribs = 0;
AFSFcb *pFcb = NULL;
AFSReleaseResource( &pFcb->NPFcb->Resource);
+ //
+ // Its a reparse point regardless of whether the file attributes
+ // can be retrieved for the target.
+ //
+
+ if ( ulFileAttribs == FILE_ATTRIBUTE_NORMAL)
+ {
+
+ ulFileAttribs = FILE_ATTRIBUTE_REPARSE_POINT;
+ }
+ else
+ {
+
+ ulFileAttribs |= FILE_ATTRIBUTE_REPARSE_POINT;
+ }
+
if( NT_SUCCESS( AFSRetrieveFileAttributes( pParentDirectoryCB,
DirectoryCB,
&uniParentPath,
&stFileInfo)))
{
- if ( ulFileAttribs == FILE_ATTRIBUTE_NORMAL)
- {
-
- ulFileAttribs = FILE_ATTRIBUTE_REPARSE_POINT;
- }
- else
- {
-
- ulFileAttribs |= FILE_ATTRIBUTE_REPARSE_POINT;
- }
-
if ( stFileInfo.FileAttributes & FILE_ATTRIBUTE_DIRECTORY)
{
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE_2,
"AFSQueryBasicInfo %wZ Type 0x%x Attrib 0x%x -> 0x%x\n",
&pCcb->DirectoryCB->NameInformation.FileName,
- pCcb->DirectoryCB->ObjectInformation->FileType,
- pCcb->DirectoryCB->ObjectInformation->FileAttributes,
- ulFileAttribs);
+ pFcb->ObjectInformation->FileType,
+ pFcb->ObjectInformation->FileAttributes,
+ ulFileAttribs));
Buffer->CreationTime = DirectoryCB->ObjectInformation->CreationTime;
Buffer->LastAccessTime = DirectoryCB->ObjectInformation->LastAccessTime;
}
*Length -= sizeof( FILE_BASIC_INFORMATION);
- }
- else
- {
- ntStatus = STATUS_BUFFER_TOO_SMALL;
+ ntStatus = STATUS_SUCCESS;
}
return ntStatus;
IN OUT PLONG Length)
{
- NTSTATUS ntStatus = STATUS_SUCCESS;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
AFSFcb *pFcb = NULL;
AFSCcb *pCcb = NULL;
PIO_STACK_LOCATION pIrpSp = IoGetCurrentIrpStackLocation( Irp);
RtlZeroMemory( Buffer,
*Length);
- Buffer->NumberOfLinks = 1;
+ Buffer->NumberOfLinks = DirectoryCB->ObjectInformation->Links;
Buffer->DeletePending = BooleanFlagOn( pCcb->DirectoryCB->Flags, AFS_DIR_ENTRY_PENDING_DELETE);
- Buffer->AllocationSize.QuadPart = (ULONGLONG)((DirectoryCB->ObjectInformation->AllocationSize.QuadPart/PAGE_SIZE) + 1) * PAGE_SIZE;
+ Buffer->AllocationSize.QuadPart = DirectoryCB->ObjectInformation->AllocationSize.QuadPart;
Buffer->EndOfFile = DirectoryCB->ObjectInformation->EndOfFile;
AFSReleaseResource( &pFcb->NPFcb->Resource);
+ //
+ // Its a reparse point regardless of whether or not the
+ // file attributes can be retrieved.
+ //
+
+ if ( ulFileAttribs == FILE_ATTRIBUTE_NORMAL)
+ {
+
+ ulFileAttribs = FILE_ATTRIBUTE_REPARSE_POINT;
+ }
+ else
+ {
+
+ ulFileAttribs |= FILE_ATTRIBUTE_REPARSE_POINT;
+ }
+
if( NT_SUCCESS( AFSRetrieveFileAttributes( pParentDirectoryCB,
DirectoryCB,
&uniParentPath,
&stFileInfo)))
{
- if ( ulFileAttribs == FILE_ATTRIBUTE_NORMAL)
- {
-
- ulFileAttribs = FILE_ATTRIBUTE_REPARSE_POINT;
- }
- else
- {
-
- ulFileAttribs |= FILE_ATTRIBUTE_REPARSE_POINT;
- }
-
if ( stFileInfo.FileAttributes & FILE_ATTRIBUTE_DIRECTORY)
{
TRUE);
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE_2,
"AFSQueryStandardInfo %wZ Type 0x%x Attrib 0x%x -> 0x%x\n",
&pCcb->DirectoryCB->NameInformation.FileName,
- pCcb->DirectoryCB->ObjectInformation->FileType,
- pCcb->DirectoryCB->ObjectInformation->FileAttributes,
- ulFileAttribs);
+ pFcb->ObjectInformation->FileType,
+ pFcb->ObjectInformation->FileAttributes,
+ ulFileAttribs));
Buffer->Directory = BooleanFlagOn( ulFileAttribs, FILE_ATTRIBUTE_DIRECTORY);
*Length -= sizeof( FILE_STANDARD_INFORMATION);
- }
- else
- {
- ntStatus = STATUS_BUFFER_TOO_SMALL;
+ ntStatus = STATUS_SUCCESS;
}
return ntStatus;
{
UNREFERENCED_PARAMETER(Irp);
- NTSTATUS ntStatus = STATUS_SUCCESS;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
if( *Length >= sizeof( FILE_INTERNAL_INFORMATION))
{
- Buffer->IndexNumber.HighPart = Fcb->ObjectInformation->FileId.Volume;
+ Buffer->IndexNumber.HighPart = Fcb->ObjectInformation->FileId.Vnode;
- Buffer->IndexNumber.LowPart = Fcb->ObjectInformation->FileId.Vnode;
+ Buffer->IndexNumber.LowPart = Fcb->ObjectInformation->FileId.Unique;
*Length -= sizeof( FILE_INTERNAL_INFORMATION);
- }
- else
- {
- ntStatus = STATUS_BUFFER_TOO_SMALL;
+ ntStatus = STATUS_SUCCESS;
}
return ntStatus;
UNREFERENCED_PARAMETER(Irp);
UNREFERENCED_PARAMETER(DirectoryCB);
- NTSTATUS ntStatus = STATUS_SUCCESS;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
RtlZeroMemory( Buffer,
*Length);
Buffer->EaSize = 0;
*Length -= sizeof( FILE_EA_INFORMATION);
- }
- else
- {
- ntStatus = STATUS_BUFFER_TOO_SMALL;
+ ntStatus = STATUS_SUCCESS;
}
return ntStatus;
{
UNREFERENCED_PARAMETER(Fcb);
- NTSTATUS ntStatus = STATUS_SUCCESS;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
PIO_STACK_LOCATION pIrpSp = IoGetCurrentIrpStackLocation( Irp);
if( *Length >= sizeof( FILE_POSITION_INFORMATION))
Buffer->CurrentByteOffset.QuadPart = pIrpSp->FileObject->CurrentByteOffset.QuadPart;
*Length -= sizeof( FILE_POSITION_INFORMATION);
- }
- else
- {
-
- ntStatus = STATUS_BUFFER_TOO_SMALL;
- }
-
- return ntStatus;
-}
-
-NTSTATUS
-AFSQueryAccess( IN PIRP Irp,
- IN AFSFcb *Fcb,
- IN OUT PFILE_ACCESS_INFORMATION Buffer,
- IN OUT PLONG Length)
-{
-
- UNREFERENCED_PARAMETER(Irp);
- UNREFERENCED_PARAMETER(Fcb);
- NTSTATUS ntStatus = STATUS_SUCCESS;
-
- if( *Length >= sizeof( FILE_ACCESS_INFORMATION))
- {
-
- RtlZeroMemory( Buffer,
- *Length);
-
- Buffer->AccessFlags = 0;
-
- *Length -= sizeof( FILE_ACCESS_INFORMATION);
- }
- else
- {
-
- ntStatus = STATUS_BUFFER_TOO_SMALL;
- }
-
- return ntStatus;
-}
-
-NTSTATUS
-AFSQueryMode( IN PIRP Irp,
- IN AFSFcb *Fcb,
- IN OUT PFILE_MODE_INFORMATION Buffer,
- IN OUT PLONG Length)
-{
-
- UNREFERENCED_PARAMETER(Irp);
- UNREFERENCED_PARAMETER(Fcb);
- NTSTATUS ntStatus = STATUS_SUCCESS;
-
- if( *Length >= sizeof( FILE_MODE_INFORMATION))
- {
-
- RtlZeroMemory( Buffer,
- *Length);
-
- Buffer->Mode = 0;
-
- *Length -= sizeof( FILE_MODE_INFORMATION);
- }
- else
- {
-
- ntStatus = STATUS_BUFFER_TOO_SMALL;
- }
-
- return ntStatus;
-}
-
-NTSTATUS
-AFSQueryAlignment( IN PIRP Irp,
- IN AFSFcb *Fcb,
- IN OUT PFILE_ALIGNMENT_INFORMATION Buffer,
- IN OUT PLONG Length)
-{
-
- UNREFERENCED_PARAMETER(Irp);
- UNREFERENCED_PARAMETER(Fcb);
- NTSTATUS ntStatus = STATUS_SUCCESS;
-
- if( *Length >= sizeof( FILE_ALIGNMENT_INFORMATION))
- {
-
- RtlZeroMemory( Buffer,
- *Length);
-
- Buffer->AlignmentRequirement = 1;
-
- *Length -= sizeof( FILE_ALIGNMENT_INFORMATION);
- }
- else
- {
- ntStatus = STATUS_BUFFER_TOO_SMALL;
+ ntStatus = STATUS_SUCCESS;
}
return ntStatus;
{
UNREFERENCED_PARAMETER(DirectoryCB);
- NTSTATUS ntStatus = STATUS_SUCCESS;
- ULONG ulCopyLength = 0;
- ULONG cchCopied = 0;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
AFSFcb *pFcb = NULL;
AFSCcb *pCcb = NULL;
IO_STACK_LOCATION *pIrpSp = IoGetCurrentIrpStackLocation( Irp);
- BOOLEAN bAddLeadingSlash = FALSE;
- BOOLEAN bAddTrailingSlash = FALSE;
- USHORT usFullNameLength = 0;
pFcb = (AFSFcb *)pIrpSp->FileObject->FsContext;
if( *Length >= FIELD_OFFSET( FILE_NAME_INFORMATION, FileName))
{
- RtlZeroMemory( Buffer,
- *Length);
-
- if( pCcb->FullFileName.Length == 0 ||
- pCcb->FullFileName.Buffer[ 0] != L'\\')
- {
- bAddLeadingSlash = TRUE;
- }
-
- if( pFcb->ObjectInformation->FileType == AFS_FILE_TYPE_DIRECTORY &&
- pCcb->FullFileName.Length > 0 &&
- pCcb->FullFileName.Buffer[ (pCcb->FullFileName.Length/sizeof( WCHAR)) - 1] != L'\\')
- {
- bAddTrailingSlash = TRUE;
- }
-
- usFullNameLength = sizeof( WCHAR) +
- AFSServerName.Length +
- pCcb->FullFileName.Length;
-
- if( bAddLeadingSlash)
- {
- usFullNameLength += sizeof( WCHAR);
- }
-
- if( bAddTrailingSlash)
- {
- usFullNameLength += sizeof( WCHAR);
- }
-
- if( *Length >= (LONG)(FIELD_OFFSET( FILE_NAME_INFORMATION, FileName) + (LONG)usFullNameLength))
- {
-
- ulCopyLength = (LONG)usFullNameLength;
- }
- else
- {
-
- ulCopyLength = *Length - FIELD_OFFSET( FILE_NAME_INFORMATION, FileName);
-
- ntStatus = STATUS_BUFFER_OVERFLOW;
- }
-
- Buffer->FileNameLength = (ULONG)usFullNameLength;
-
- *Length -= FIELD_OFFSET( FILE_NAME_INFORMATION, FileName);
+ RtlZeroMemory( Buffer,
+ *Length);
- if( ulCopyLength > 0)
- {
-
- Buffer->FileName[ 0] = L'\\';
- ulCopyLength -= sizeof( WCHAR);
-
- *Length -= sizeof( WCHAR);
- cchCopied += 1;
-
- if( ulCopyLength >= AFSServerName.Length)
- {
-
- RtlCopyMemory( &Buffer->FileName[ 1],
- AFSServerName.Buffer,
- AFSServerName.Length);
+ *Length -= FIELD_OFFSET( FILE_NAME_INFORMATION, FileName);
- ulCopyLength -= AFSServerName.Length;
- *Length -= AFSServerName.Length;
- cchCopied += AFSServerName.Length/sizeof( WCHAR);
-
- if ( ulCopyLength > 0 &&
- bAddLeadingSlash)
- {
-
- Buffer->FileName[ cchCopied] = L'\\';
-
- ulCopyLength -= sizeof( WCHAR);
- *Length -= sizeof( WCHAR);
- cchCopied++;
- }
-
- if( ulCopyLength >= pCcb->FullFileName.Length)
- {
-
- RtlCopyMemory( &Buffer->FileName[ cchCopied],
- pCcb->FullFileName.Buffer,
- pCcb->FullFileName.Length);
-
- ulCopyLength -= pCcb->FullFileName.Length;
- *Length -= pCcb->FullFileName.Length;
- cchCopied += pCcb->FullFileName.Length/sizeof( WCHAR);
-
- if( ulCopyLength > 0 &&
- bAddTrailingSlash)
- {
- Buffer->FileName[ cchCopied] = L'\\';
-
- *Length -= sizeof( WCHAR);
- }
- }
- else
- {
-
- RtlCopyMemory( &Buffer->FileName[ cchCopied],
- pCcb->FullFileName.Buffer,
- ulCopyLength);
-
- *Length -= ulCopyLength;
- }
- }
- }
- }
- else
- {
-
- ntStatus = STATUS_BUFFER_TOO_SMALL;
+ ntStatus = AFSGetFullFileName( pFcb,
+ pCcb,
+ &Buffer->FileNameLength,
+ Buffer->FileName,
+ Length);
}
return ntStatus;
{
UNREFERENCED_PARAMETER(Irp);
- NTSTATUS ntStatus = STATUS_BUFFER_TOO_SMALL;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
ULONG ulCopyLength = 0;
RtlZeroMemory( Buffer,
IN OUT PLONG Length)
{
- NTSTATUS ntStatus = STATUS_SUCCESS;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
AFSDeviceExt *pDeviceExt = (AFSDeviceExt *)AFSRDRDeviceObject->DeviceExtension;
AFSFcb *pFcb = NULL;
AFSCcb *pCcb = NULL;
AFSReleaseResource( &pFcb->NPFcb->Resource);
+ //
+ // Its a reparse point regardless of whether the file attributes
+ // can be retrieved for the target.
+ //
+
+ if ( ulFileAttribs == FILE_ATTRIBUTE_NORMAL)
+ {
+
+ ulFileAttribs = FILE_ATTRIBUTE_REPARSE_POINT;
+ }
+ else
+ {
+
+ ulFileAttribs |= FILE_ATTRIBUTE_REPARSE_POINT;
+ }
+
if( NT_SUCCESS( AFSRetrieveFileAttributes( pParentDirectoryCB,
DirectoryCB,
&uniParentPath,
&stFileInfo)))
{
- if ( ulFileAttribs == FILE_ATTRIBUTE_NORMAL)
- {
-
- ulFileAttribs = FILE_ATTRIBUTE_REPARSE_POINT;
- }
- else
- {
-
- ulFileAttribs |= FILE_ATTRIBUTE_REPARSE_POINT;
- }
-
if ( stFileInfo.FileAttributes & FILE_ATTRIBUTE_DIRECTORY)
{
TRUE);
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE_2,
"AFSQueryNetworkInfo %wZ Type 0x%x Attrib 0x%x -> 0x%x\n",
&pCcb->DirectoryCB->NameInformation.FileName,
- pCcb->DirectoryCB->ObjectInformation->FileType,
- pCcb->DirectoryCB->ObjectInformation->FileAttributes,
- ulFileAttribs);
+ pFcb->ObjectInformation->FileType,
+ pFcb->ObjectInformation->FileAttributes,
+ ulFileAttribs));
Buffer->CreationTime.QuadPart = DirectoryCB->ObjectInformation->CreationTime.QuadPart;
Buffer->LastAccessTime.QuadPart = DirectoryCB->ObjectInformation->LastAccessTime.QuadPart;
}
*Length -= sizeof( FILE_NETWORK_OPEN_INFORMATION);
- }
- else
- {
- ntStatus = STATUS_BUFFER_TOO_SMALL;
+ ntStatus = STATUS_SUCCESS;
}
return ntStatus;
{
UNREFERENCED_PARAMETER(Irp);
- NTSTATUS ntStatus = STATUS_BUFFER_TOO_SMALL;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
ULONG ulCopyLength = 0;
if( *Length >= FIELD_OFFSET( FILE_STREAM_INFORMATION, StreamName))
IN OUT PLONG Length)
{
- NTSTATUS ntStatus = STATUS_BUFFER_TOO_SMALL;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
AFSDeviceExt *pDeviceExt = (AFSDeviceExt *)AFSRDRDeviceObject->DeviceExtension;
AFSFcb *pFcb = NULL;
AFSCcb *pCcb = NULL;
AFSReleaseResource( &pFcb->NPFcb->Resource);
+ //
+ // Its a reparse point regardless of whether the file attributes
+ // can be retrieved for the target.
+ //
+
+ if ( ulFileAttribs == FILE_ATTRIBUTE_NORMAL)
+ {
+
+ ulFileAttribs = FILE_ATTRIBUTE_REPARSE_POINT;
+ }
+ else
+ {
+
+ ulFileAttribs |= FILE_ATTRIBUTE_REPARSE_POINT;
+ }
+
if( NT_SUCCESS( AFSRetrieveFileAttributes( pParentDirectoryCB,
DirectoryCB,
&uniParentPath,
&stFileInfo)))
{
- if ( ulFileAttribs == FILE_ATTRIBUTE_NORMAL)
- {
-
- ulFileAttribs = FILE_ATTRIBUTE_REPARSE_POINT;
- }
- else
- {
-
- ulFileAttribs |= FILE_ATTRIBUTE_REPARSE_POINT;
- }
-
if ( stFileInfo.FileAttributes & FILE_ATTRIBUTE_DIRECTORY)
{
TRUE);
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE_2,
"AFSAttribTagInfo %wZ Type 0x%x Attrib 0x%x -> 0x%x\n",
&pCcb->DirectoryCB->NameInformation.FileName,
- pCcb->DirectoryCB->ObjectInformation->FileType,
- pCcb->DirectoryCB->ObjectInformation->FileAttributes,
- ulFileAttribs);
+ pFcb->ObjectInformation->FileType,
+ pFcb->ObjectInformation->FileAttributes,
+ ulFileAttribs));
Buffer->FileAttributes = ulFileAttribs;
}
}
- if( BooleanFlagOn( DirectoryCB->ObjectInformation->FileAttributes, FILE_ATTRIBUTE_REPARSE_POINT))
+ if ( DirectoryCB->ObjectInformation->FileType == AFS_FILE_TYPE_MOUNTPOINT)
{
+
Buffer->ReparseTag = IO_REPARSE_TAG_SURROGATE|IO_REPARSE_TAG_OPENAFS_DFS;
}
+ else if( BooleanFlagOn( DirectoryCB->ObjectInformation->FileAttributes, FILE_ATTRIBUTE_REPARSE_POINT))
+ {
+
+ Buffer->ReparseTag = IO_REPARSE_TAG_SYMLINK;
+ }
*Length -= sizeof( FILE_ATTRIBUTE_TAG_INFORMATION);
UNREFERENCED_PARAMETER(Irp);
UNREFERENCED_PARAMETER(DirectoryCB);
- NTSTATUS ntStatus = STATUS_BUFFER_TOO_SMALL;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
if( *Length >= sizeof( FILE_REMOTE_PROTOCOL_INFORMATION))
{
{
UNREFERENCED_PARAMETER(DirectoryCB);
- NTSTATUS ntStatus = STATUS_SUCCESS;
- ULONG ulCopyLength = 0;
- ULONG cchCopied = 0;
+ NTSTATUS ntStatus = STATUS_INFO_LENGTH_MISMATCH;
AFSFcb *pFcb = NULL;
AFSCcb *pCcb = NULL;
IO_STACK_LOCATION *pIrpSp = IoGetCurrentIrpStackLocation( Irp);
- BOOLEAN bAddLeadingSlash = FALSE;
- USHORT usFullNameLength = 0;
pFcb = (AFSFcb *)pIrpSp->FileObject->FsContext;
if( *Length >= FIELD_OFFSET( FILE_NETWORK_PHYSICAL_NAME_INFORMATION, FileName))
{
- RtlZeroMemory( Buffer,
- *Length);
-
- if( pCcb->FullFileName.Length == 0 ||
- pCcb->FullFileName.Buffer[ 0] != L'\\')
- {
- bAddLeadingSlash = TRUE;
- }
-
- usFullNameLength = pCcb->FullFileName.Length;
-
- if( bAddLeadingSlash)
- {
- usFullNameLength += sizeof( WCHAR);
- }
-
- if( *Length >= (LONG)(FIELD_OFFSET( FILE_NETWORK_PHYSICAL_NAME_INFORMATION, FileName) + (LONG)usFullNameLength))
- {
- ulCopyLength = (LONG)usFullNameLength;
- }
- else
- {
-
- ulCopyLength = *Length - FIELD_OFFSET( FILE_NETWORK_PHYSICAL_NAME_INFORMATION, FileName);
-
- ntStatus = STATUS_BUFFER_OVERFLOW;
- }
-
- Buffer->FileNameLength = (ULONG)usFullNameLength;
-
- *Length -= FIELD_OFFSET( FILE_NETWORK_PHYSICAL_NAME_INFORMATION, FileName);
-
- if( ulCopyLength > 0)
- {
-
- if( bAddLeadingSlash)
- {
-
- Buffer->FileName[ cchCopied] = L'\\';
-
- ulCopyLength -= sizeof( WCHAR);
- *Length -= sizeof( WCHAR);
- cchCopied++;
- }
-
- if( ulCopyLength >= pCcb->FullFileName.Length)
- {
-
- RtlCopyMemory( &Buffer->FileName[ cchCopied],
- pCcb->FullFileName.Buffer,
- pCcb->FullFileName.Length);
-
- ulCopyLength -= pCcb->FullFileName.Length;
- *Length -= pCcb->FullFileName.Length;
- cchCopied += pCcb->FullFileName.Length/sizeof( WCHAR);
- }
- else
- {
-
- RtlCopyMemory( &Buffer->FileName[ cchCopied],
- pCcb->FullFileName.Buffer,
- ulCopyLength);
+ RtlZeroMemory( Buffer,
+ *Length);
- *Length -= ulCopyLength;
- }
- }
- }
- else
- {
+ *Length -= FIELD_OFFSET( FILE_NETWORK_PHYSICAL_NAME_INFORMATION, FileName);
- ntStatus = STATUS_BUFFER_TOO_SMALL;
+ ntStatus = AFSGetFullFileName( pFcb,
+ pCcb,
+ &Buffer->FileNameLength,
+ Buffer->FileName,
+ Length);
}
return ntStatus;
NTSTATUS
AFSSetBasicInfo( IN PIRP Irp,
- IN AFSDirectoryCB *DirectoryCB)
+ IN AFSDirectoryCB *DirectoryCB,
+ OUT BOOLEAN *bUpdateFileInfo)
{
NTSTATUS ntStatus = STATUS_SUCCESS;
PFILE_BASIC_INFORMATION pBuffer;
if( pBuffer->FileAttributes != (ULONGLONG)0)
{
- if( DirectoryCB->ObjectInformation->Fcb->Header.NodeTypeCode == AFS_FILE_FCB &&
- BooleanFlagOn( pBuffer->FileAttributes, FILE_ATTRIBUTE_DIRECTORY))
- {
+ //
+ // Make sure that the reparse point attribute is not modified.
+ // Fail if the RP attribute is requested but it is not
+ // already a RP. Otherwise, ignore it.
+ //
- try_return( ntStatus = STATUS_INVALID_PARAMETER);
- }
+ if ( !BooleanFlagOn( DirectoryCB->ObjectInformation->FileAttributes,
+ FILE_ATTRIBUTE_REPARSE_POINT) &&
+ BooleanFlagOn( pBuffer->FileAttributes,
+ FILE_ATTRIBUTE_REPARSE_POINT))
+ {
- if( DirectoryCB->ObjectInformation->Fcb->Header.NodeTypeCode == AFS_DIRECTORY_FCB)
- {
+ try_return( ntStatus = STATUS_INVALID_PARAMETER);
+ }
- pBuffer->FileAttributes |= FILE_ATTRIBUTE_DIRECTORY;
- }
+ //
+ // Make sure that the directory attribute is not modified.
+ // Fail if the D attribute is requested but it is not
+ // already a directory. Otherwise, ignore it.
+ //
+
+ if ( !BooleanFlagOn( DirectoryCB->ObjectInformation->FileAttributes,
+ FILE_ATTRIBUTE_DIRECTORY) &&
+ BooleanFlagOn( pBuffer->FileAttributes,
+ FILE_ATTRIBUTE_DIRECTORY))
+ {
+
+ try_return( ntStatus = STATUS_INVALID_PARAMETER);
+ }
- pCcb->FileUnwindInfo.FileAttributes = DirectoryCB->ObjectInformation->FileAttributes;
+ //
+ // Save the original value
+ //
- DirectoryCB->ObjectInformation->FileAttributes = pBuffer->FileAttributes;
+ pCcb->FileUnwindInfo.FileAttributes = DirectoryCB->ObjectInformation->FileAttributes;
- ulNotifyFilter |= FILE_NOTIFY_CHANGE_ATTRIBUTES;
+ if( BooleanFlagOn( pBuffer->FileAttributes, FILE_ATTRIBUTE_READONLY))
+ {
- SetFlag( DirectoryCB->ObjectInformation->Fcb->Flags, AFS_FCB_FLAG_FILE_MODIFIED);
+ //
+ // Set the readonly flag.
+ //
+
+ if ( !BooleanFlagOn( DirectoryCB->ObjectInformation->FileAttributes,
+ FILE_ATTRIBUTE_READONLY))
+ {
+
+ if ( DirectoryCB->ObjectInformation->FileAttributes == FILE_ATTRIBUTE_NORMAL)
+ {
+
+ DirectoryCB->ObjectInformation->FileAttributes = FILE_ATTRIBUTE_READONLY;
+ }
+ else
+ {
+
+ DirectoryCB->ObjectInformation->FileAttributes |= FILE_ATTRIBUTE_READONLY;
+ }
+
+ ulNotifyFilter |= FILE_NOTIFY_CHANGE_ATTRIBUTES;
+
+ SetFlag( DirectoryCB->ObjectInformation->Fcb->Flags, AFS_FCB_FLAG_FILE_MODIFIED);
+ }
+ }
+ else
+ {
+ //
+ // Reset the readonly flag.
+ //
+
+ if ( BooleanFlagOn( DirectoryCB->ObjectInformation->FileAttributes,
+ FILE_ATTRIBUTE_READONLY))
+ {
+
+ DirectoryCB->ObjectInformation->FileAttributes &= ~FILE_ATTRIBUTE_READONLY;
+
+ if ( DirectoryCB->ObjectInformation->FileAttributes == 0)
+ {
+
+ DirectoryCB->ObjectInformation->FileAttributes = FILE_ATTRIBUTE_NORMAL;
+ }
+
+ ulNotifyFilter |= FILE_NOTIFY_CHANGE_ATTRIBUTES;
+
+ SetFlag( DirectoryCB->ObjectInformation->Fcb->Flags, AFS_FCB_FLAG_FILE_MODIFIED);
+ }
+ }
}
pCcb->FileUnwindInfo.CreationTime.QuadPart = (ULONGLONG)-1;
ulNotifyFilter |= FILE_NOTIFY_CHANGE_LAST_WRITE;
SetFlag( DirectoryCB->ObjectInformation->Fcb->Flags, AFS_FCB_FLAG_FILE_MODIFIED | AFS_FCB_FLAG_UPDATE_LAST_WRITE_TIME);
+
+ SetFlag( pCcb->Flags, CCB_FLAG_LAST_WRITE_TIME_SET);
+
+ } else if ( pBuffer->LastWriteTime.QuadPart == (ULONGLONG)-1) {
+
+ SetFlag( pCcb->Flags, CCB_FLAG_LAST_WRITE_TIME_SET);
}
pCcb->FileUnwindInfo.ChangeTime.QuadPart = (ULONGLONG)-1;
if( ulNotifyFilter > 0)
{
- if( DirectoryCB->ObjectInformation->ParentObjectInformation != NULL)
+ *bUpdateFileInfo = TRUE;
+
+ if( BooleanFlagOn( DirectoryCB->ObjectInformation->Flags, AFS_OBJECT_FLAGS_PARENT_FID))
{
- AFSFsRtlNotifyFullReportChange( DirectoryCB->ObjectInformation->ParentObjectInformation,
- pCcb,
- (ULONG)ulNotifyFilter,
- (ULONG)FILE_ACTION_MODIFIED);
+ AFSObjectInfoCB * pParentObjectInfo = AFSFindObjectInfo( DirectoryCB->ObjectInformation->VolumeCB,
+ &DirectoryCB->ObjectInformation->ParentFileId,
+ TRUE);
+
+ if ( pParentObjectInfo != NULL)
+ {
+ AFSFsRtlNotifyFullReportChange( pParentObjectInfo,
+ pCcb,
+ (ULONG)ulNotifyFilter,
+ (ULONG)FILE_ACTION_MODIFIED);
+
+ AFSReleaseObjectInfo( &pParentObjectInfo);
+ }
}
}
if( pFcb->Header.NodeTypeCode == AFS_ROOT_FCB)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
- "AFSSetDispositionInfo Attempt to delete root entry\n");
+ "AFSSetDispositionInfo Attempt to delete root entry\n"));
try_return( ntStatus = STATUS_CANNOT_DELETE);
}
if( BooleanFlagOn( DirectoryCB->ObjectInformation->FileAttributes, FILE_ATTRIBUTE_READONLY))
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetDispositionInfo Attempt to delete read only entry %wZ\n",
- &DirectoryCB->NameInformation.FileName);
+ &DirectoryCB->NameInformation.FileName));
try_return( ntStatus = STATUS_CANNOT_DELETE);
}
if( !NT_SUCCESS( ntStatus))
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetDispositionInfo Cannot delete entry %wZ Status %08lX\n",
&DirectoryCB->NameInformation.FileName,
- ntStatus);
+ ntStatus));
try_return( ntStatus);
}
if( pFcb->ObjectInformation->Specific.Directory.ChildOpenHandleCount > 0)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetDispositionInfo Attempt to delete directory %wZ with open %u handles\n",
&DirectoryCB->NameInformation.FileName,
- pFcb->ObjectInformation->Specific.Directory.ChildOpenHandleCount);
+ pFcb->ObjectInformation->Specific.Directory.ChildOpenHandleCount));
try_return( ntStatus = STATUS_DIRECTORY_NOT_EMPTY);
}
+ //
+ // Make sure the directory is enumerated before checking to see if it is empty.
+ //
+
+ if( !BooleanFlagOn( pFcb->ObjectInformation->Flags, AFS_OBJECT_FLAGS_DIRECTORY_ENUMERATED))
+ {
+
+ AFSAcquireExcl( pFcb->ObjectInformation->Specific.Directory.DirectoryNodeHdr.TreeLock,
+ TRUE);
+
+ if( !BooleanFlagOn( pFcb->ObjectInformation->Flags, AFS_OBJECT_FLAGS_DIRECTORY_ENUMERATED))
+ {
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFS_TRACE_LEVEL_VERBOSE,
+ "AFSSetDispositionInfo Enumerating parent FID %08lX-%08lX-%08lX-%08lX\n",
+ pFcb->ObjectInformation->FileId.Cell,
+ pFcb->ObjectInformation->FileId.Volume,
+ pFcb->ObjectInformation->FileId.Vnode,
+ pFcb->ObjectInformation->FileId.Unique));
+
+ ntStatus = AFSEnumerateDirectory( &pCcb->AuthGroup,
+ pFcb->ObjectInformation,
+ TRUE);
+
+ if( !NT_SUCCESS( ntStatus))
+ {
+
+ AFSReleaseResource( pFcb->ObjectInformation->Specific.Directory.DirectoryNodeHdr.TreeLock);
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFS_TRACE_LEVEL_ERROR,
+ "AFSSetDispositionInfo Failed to enumerate parent FID %08lX-%08lX-%08lX-%08lX Status %08lX\n",
+ pFcb->ObjectInformation->FileId.Cell,
+ pFcb->ObjectInformation->FileId.Volume,
+ pFcb->ObjectInformation->FileId.Vnode,
+ pFcb->ObjectInformation->FileId.Unique,
+ ntStatus));
+
+ try_return( ntStatus);
+ }
+ }
+
+ AFSReleaseResource( pFcb->ObjectInformation->Specific.Directory.DirectoryNodeHdr.TreeLock);
+ }
+
if( !AFSIsDirectoryEmptyForDelete( pFcb))
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetDispositionInfo Attempt to delete non-empty directory %wZ\n",
- &DirectoryCB->NameInformation.FileName);
+ &DirectoryCB->NameInformation.FileName));
try_return( ntStatus = STATUS_DIRECTORY_NOT_EMPTY);
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetDispositionInfo Setting PENDING_DELETE on DirEntry %p Name %wZ\n",
DirectoryCB,
- &DirectoryCB->NameInformation.FileName);
+ &DirectoryCB->NameInformation.FileName));
SetFlag( pCcb->DirectoryCB->Flags, AFS_DIR_ENTRY_PENDING_DELETE);
}
{
BOOLEAN bMmFlushed;
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFS_TRACE_LEVEL_VERBOSE,
+ "AFSSetDispositionInfo Acquiring Fcb lock %p EXCL %08lX\n",
+ &pFcb->NPFcb->Resource,
+ PsGetCurrentThread()));
+
+ AFSAcquireExcl( &pFcb->NPFcb->Resource,
+ TRUE);
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING|AFS_SUBSYSTEM_SECTION_OBJECT,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetDispositionInfo Acquiring Fcb SectionObject lock %p EXCL %08lX\n",
&pFcb->NPFcb->SectionObjectResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireExcl( &pFcb->NPFcb->SectionObjectResource,
TRUE);
- //
- // Attempt to flush any outstanding data
- //
-
- bMmFlushed = MmFlushImageSection( &pFcb->NPFcb->SectionObjectPointers,
- MmFlushForDelete);
-
- if ( bMmFlushed)
+ __try
{
//
- // Set PENDING_DELETE before CcPurgeCacheSection to avoid a
- // deadlock with Trend Micro's Enterprise anti-virus product
- // which attempts to open the file which is being deleted.
+ // Attempt to flush any outstanding data
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
- AFS_TRACE_LEVEL_VERBOSE,
- "AFSSetDispositionInfo Setting PENDING_DELETE on DirEntry %p Name %wZ\n",
- DirectoryCB,
- &DirectoryCB->NameInformation.FileName);
+ bMmFlushed = MmFlushImageSection( &pFcb->NPFcb->SectionObjectPointers,
+ MmFlushForDelete);
- SetFlag( pCcb->DirectoryCB->Flags, AFS_DIR_ENTRY_PENDING_DELETE);
+ if ( bMmFlushed)
+ {
- //
- // Purge the cache as well
- //
+ //
+ // Set PENDING_DELETE before CcPurgeCacheSection to avoid a
+ // deadlock with Trend Micro's Enterprise anti-virus product
+ // which attempts to open the file which is being deleted.
+ //
- if( pFcb->NPFcb->SectionObjectPointers.DataSectionObject != NULL)
- {
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFS_TRACE_LEVEL_VERBOSE,
+ "AFSSetDispositionInfo Setting PENDING_DELETE on DirEntry %p Name %wZ\n",
+ DirectoryCB,
+ &DirectoryCB->NameInformation.FileName));
- if ( !CcPurgeCacheSection( &pFcb->NPFcb->SectionObjectPointers,
- NULL,
- 0,
- TRUE))
+ SetFlag( pCcb->DirectoryCB->Flags, AFS_DIR_ENTRY_PENDING_DELETE);
+
+ //
+ // Purge the cache as well
+ //
+
+ if( pFcb->NPFcb->SectionObjectPointers.DataSectionObject != NULL)
{
- SetFlag( pFcb->Flags, AFS_FCB_FLAG_PURGE_ON_CLOSE);
+ if ( !CcPurgeCacheSection( &pFcb->NPFcb->SectionObjectPointers,
+ NULL,
+ 0,
+ TRUE))
+ {
+
+ SetFlag( pFcb->Flags, AFS_FCB_FLAG_PURGE_ON_CLOSE);
+ }
}
}
}
+ __except( EXCEPTION_EXECUTE_HANDLER)
+ {
+
+ bMmFlushed = FALSE;
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ ntStatus = GetExceptionCode();
+
+ AFSDbgTrace(( 0,
+ 0,
+ "EXCEPTION - AFSSetDispositionInfo MmFlushImageSection failed FID %08lX-%08lX-%08lX-%08lX Status 0x%08lX\n",
+ pFcb->ObjectInformation->FileId.Cell,
+ pFcb->ObjectInformation->FileId.Volume,
+ pFcb->ObjectInformation->FileId.Vnode,
+ pFcb->ObjectInformation->FileId.Unique,
+ ntStatus));
+ }
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING|AFS_SUBSYSTEM_SECTION_OBJECT,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetDispositionInfo Releasing Fcb SectionObject lock %p EXCL %08lX\n",
&pFcb->NPFcb->SectionObjectResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSReleaseResource( &pFcb->NPFcb->SectionObjectResource);
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFS_TRACE_LEVEL_VERBOSE,
+ "AFSSetDispositionInfo Releasing Fcb lock %p EXCL %08lX\n",
+ &pFcb->NPFcb->Resource,
+ PsGetCurrentThread()));
+
+ AFSReleaseResource( &pFcb->NPFcb->Resource);
+
if ( !bMmFlushed)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetDispositionInfo Failed to flush image section for delete Entry %wZ\n",
- &DirectoryCB->NameInformation.FileName);
+ &DirectoryCB->NameInformation.FileName));
try_return( ntStatus = STATUS_CANNOT_DELETE);
}
}
+ else if( pFcb->Header.NodeTypeCode == AFS_SYMBOLIC_LINK_FCB ||
+ pFcb->Header.NodeTypeCode == AFS_MOUNT_POINT_FCB ||
+ pFcb->Header.NodeTypeCode == AFS_DFS_LINK_FCB ||
+ pFcb->Header.NodeTypeCode == AFS_INVALID_FCB)
+ {
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFS_TRACE_LEVEL_VERBOSE,
+ "AFSSetDispositionInfo Setting PENDING_DELETE on DirEntry %p Name %wZ\n",
+ DirectoryCB,
+ &DirectoryCB->NameInformation.FileName));
+
+ SetFlag( pCcb->DirectoryCB->Flags, AFS_DIR_ENTRY_PENDING_DELETE);
+ }
}
else
{
pSrcCcb = (AFSCcb *)pSrcFileObj->FsContext2;
pSrcObject = pSrcFcb->ObjectInformation;
- pSrcParentObject = pSrcFcb->ObjectInformation->ParentObjectInformation;
+
+ if ( BooleanFlagOn( pSrcObject->Flags, AFS_OBJECT_FLAGS_PARENT_FID))
+ {
+
+ pSrcParentObject = AFSFindObjectInfo( pSrcObject->VolumeCB,
+ &pSrcObject->ParentFileId,
+ TRUE);
+ }
+
+ if( pSrcParentObject == NULL)
+ {
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFS_TRACE_LEVEL_ERROR,
+ "AFSSetFileLinkInfo Unable to resolve SrcParentObject (INVALID_PARAMETER)\n"));
+
+ ASSERT( FALSE);
+
+ try_return( ntStatus = STATUS_INVALID_PARAMETER);
+ }
pFileLinkInfo = (PFILE_LINK_INFORMATION)Irp->AssociatedIrp.SystemBuffer;
if( pSrcFcb->Header.NodeTypeCode != AFS_FILE_FCB)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
- "AFSSetFileLinkInfo Attempt to non-file (INVALID_PARAMETER)\n");
+ "AFSSetFileLinkInfo Attempt to non-file (INVALID_PARAMETER)\n"));
try_return( ntStatus = STATUS_INVALID_PARAMETER);
}
// error.
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetFileLinkInfo Attempt to link %wZ to alternate directory by handle INVALID_PARAMETER\n",
- &pSrcCcb->DirectoryCB->NameInformation.FileName);
+ &pSrcCcb->DirectoryCB->NameInformation.FileName));
try_return( ntStatus = STATUS_INVALID_PARAMETER);
}
// will be moved to. Must obtain the TargetParentObject.
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetFileLinkInfo Attempt to link %wZ to alternate directory %wZ (NOT_SAME_DEVICE)\n",
&pSrcCcb->DirectoryCB->NameInformation.FileName,
- &uniFullTargetName);
+ &uniFullTargetName));
try_return( ntStatus = STATUS_NOT_SAME_DEVICE);
}
if( pTargetParentObject->VolumeCB != pSrcObject->VolumeCB)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetFileLinkInfo Attempt to link to different volume %wZ\n",
- &pSrcCcb->DirectoryCB->NameInformation.FileName);
+ &pSrcCcb->DirectoryCB->NameInformation.FileName));
try_return( ntStatus = STATUS_NOT_SAME_DEVICE);
}
if( pTargetDirEntry != NULL)
{
- ASSERT( pTargetParentObject == pTargetDirEntry->ObjectInformation->ParentObjectInformation);
+ ASSERT( BooleanFlagOn( pTargetDirEntry->ObjectInformation->Flags, AFS_OBJECT_FLAGS_PARENT_FID) &&
+ AFSIsEqualFID( &pTargetParentObject->FileId, &pTargetDirEntry->ObjectInformation->ParentFileId));
lCount = InterlockedIncrement( &pTargetDirEntry->DirOpenReferenceCount);
- AFSDbgLogMsg( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetFileLinkInfo Increment count on %wZ DE %p Ccb %p Cnt %d\n",
&pTargetDirEntry->NameInformation.FileName,
pTargetDirEntry,
pSrcCcb,
- lCount);
+ lCount));
ASSERT( lCount >= 0);
if( !pFileLinkInfo->ReplaceIfExists)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetFileLinkInfo Attempt to link with target collision %wZ Target %wZ\n",
&pSrcCcb->DirectoryCB->NameInformation.FileName,
- &pTargetDirEntry->NameInformation.FileName);
+ &pTargetDirEntry->NameInformation.FileName));
try_return( ntStatus = STATUS_OBJECT_NAME_COLLISION);
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING | AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING | AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetFileLinkInfo Target %wZ exists DE %p Count %d, performing delete of target\n",
&pTargetDirEntry->NameInformation.FileName,
pTargetDirEntry,
- lCount);
+ lCount));
//
// Pull the directory entry from the parent
}
else
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSSetFileLinkInfo Target does NOT exist, normal linking\n");
+ "AFSSetFileLinkInfo Target does NOT exist, normal linking\n"));
}
//
// request to the service.
//
- ntStatus = AFSNotifyHardLink( pSrcFcb->ObjectInformation,
+ ntStatus = AFSNotifyHardLink( pSrcObject,
&pSrcCcb->AuthGroup,
- pSrcFcb->ObjectInformation->ParentObjectInformation,
+ pSrcParentObject,
pTargetDcb->ObjectInformation,
pSrcCcb->DirectoryCB,
&uniTargetName,
!NT_SUCCESS( ntStatus))
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetFileLinkInfo Failed link of %wZ to target %wZ Status %08lX\n",
&pSrcCcb->DirectoryCB->NameInformation.FileName,
&uniTargetName,
- ntStatus);
+ ntStatus));
try_return( ntStatus);
}
ulNotificationAction = FILE_ACTION_ADDED;
}
- AFSFsRtlNotifyFullReportChange( pTargetParentObject->ParentObjectInformation,
+ AFSFsRtlNotifyFullReportChange( pTargetParentObject,
pSrcCcb,
(ULONG)ulNotifyFilter,
(ULONG)ulNotificationAction);
if( bTargetEntryExists)
{
- AFSInsertDirectoryNode( pTargetDirEntry->ObjectInformation->ParentObjectInformation,
+ AFSInsertDirectoryNode( pTargetParentObject,
pTargetDirEntry,
FALSE);
}
lCount = InterlockedDecrement( &pTargetDirEntry->DirOpenReferenceCount);
- AFSDbgLogMsg( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetFileLinkInfo Decrement count on %wZ DE %p Ccb %p Cnt %d\n",
&pTargetDirEntry->NameInformation.FileName,
pTargetDirEntry,
pSrcCcb,
- lCount);
+ lCount));
ASSERT( lCount >= 0);
}
lCount = InterlockedDecrement( &pNewTargetDirEntry->DirOpenReferenceCount);
- AFSDbgLogMsg( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetFileLinkInfo Decrement count on %wZ DE %p Ccb %p Cnt %d\n",
&pNewTargetDirEntry->NameInformation.FileName,
pNewTargetDirEntry,
pSrcCcb,
- lCount);
+ lCount));
ASSERT( lCount >= 0);
}
AFSReleaseResource( pTargetParentObject->Specific.Directory.DirectoryNodeHdr.TreeLock);
}
+
+ if ( pSrcParentObject != NULL)
+ {
+
+ AFSReleaseObjectInfo( &pSrcParentObject);
+ }
+
+ //
+ // No need to release pTargetParentObject as it is either a copy of pSrcParentObject
+ // or (AFSFcb *)pTargetFileObj->FsContext->ObjectInformation
+ //
+
+ pTargetParentObject = NULL;
}
return ntStatus;
pSrcCcb = (AFSCcb *)pSrcFileObj->FsContext2;
pSrcObject = pSrcFcb->ObjectInformation;
- pSrcParentObject = pSrcFcb->ObjectInformation->ParentObjectInformation;
+
+ if ( BooleanFlagOn( pSrcObject->Flags, AFS_OBJECT_FLAGS_PARENT_FID))
+ {
+
+ pSrcParentObject = AFSFindObjectInfo( pSrcObject->VolumeCB,
+ &pSrcObject->ParentFileId,
+ TRUE);
+ }
+
+ if( pSrcParentObject == NULL)
+ {
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFS_TRACE_LEVEL_ERROR,
+ "AFSSetRenameInfo Unable to resolve SrcParentObject (INVALID_PARAMETER)\n"));
+
+ ASSERT( FALSE);
+
+ try_return( ntStatus = STATUS_INVALID_PARAMETER);
+ }
//
// Perform some basic checks to ensure FS integrity
// Can't rename the root directory
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
- "AFSSetRenameInfo Attempt to rename root entry\n");
+ "AFSSetRenameInfo Attempt to rename root entry\n"));
try_return( ntStatus = STATUS_INVALID_PARAMETER);
}
// If there are any open children then fail the rename
//
- if( pSrcFcb->ObjectInformation->Specific.Directory.ChildOpenHandleCount > 0)
+ if( pSrcObject->Specific.Directory.ChildOpenHandleCount > 0)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetRenameInfo Attempt to rename directory with open children %wZ\n",
- &pSrcCcb->DirectoryCB->NameInformation.FileName);
+ &pSrcCcb->DirectoryCB->NameInformation.FileName));
try_return( ntStatus = STATUS_ACCESS_DENIED);
}
if ( pRenameInfo->RootDirectory)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
- "AFSSetRenameInfo Handle provided but no FileObject ntStatus INVALID_PARAMETER\n");
+ "AFSSetRenameInfo Handle provided but no FileObject ntStatus INVALID_PARAMETER\n"));
try_return( ntStatus = STATUS_INVALID_PARAMETER);
}
// will be moved to. Must obtain the TargetParentObject.
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetRenameInfo Attempt to move %wZ to %wZ -- not yet supported (NOT_SAME_DEVICE)\n",
&pSrcCcb->DirectoryCB->NameInformation.FileName,
- &uniFullTargetName);
+ &uniFullTargetName));
try_return( ntStatus = STATUS_NOT_SAME_DEVICE);
}
if( pTargetParentObject->VolumeCB != pSrcObject->VolumeCB)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetRenameInfo Attempt to rename directory to different volume %wZ\n",
- &pSrcCcb->DirectoryCB->NameInformation.FileName);
+ &pSrcCcb->DirectoryCB->NameInformation.FileName));
try_return( ntStatus = STATUS_NOT_SAME_DEVICE);
}
if( pTargetDirEntry != NULL)
{
- ASSERT( pTargetParentObject == pTargetDirEntry->ObjectInformation->ParentObjectInformation);
+ ASSERT( BooleanFlagOn( pTargetDirEntry->ObjectInformation->Flags, AFS_OBJECT_FLAGS_PARENT_FID) &&
+ AFSIsEqualFID( &pTargetParentObject->FileId, &pTargetDirEntry->ObjectInformation->ParentFileId));
lCount = InterlockedIncrement( &pTargetDirEntry->DirOpenReferenceCount);
-
- AFSDbgLogMsg( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetRenameInfo Increment count on %wZ DE %p Ccb %p Cnt %d\n",
&pTargetDirEntry->NameInformation.FileName,
pTargetDirEntry,
pSrcCcb,
- lCount);
+ lCount));
ASSERT( lCount >= 0);
+ //
+ // Need to acquire the ObjectInfoLock while checking for the Fcb OpenRefCount so we don't race with
+ // a tear down worker thread on the Fcb
+ //
+
+ AFSAcquireShared( &pTargetDirEntry->ObjectInformation->NonPagedInfo->ObjectInfoLock,
+ TRUE);
+
+ //
+ // Check there are no current opens on the target
+ //
+
+ if( BooleanFlagOn( pTargetDirEntry->ObjectInformation->FileAttributes, FILE_ATTRIBUTE_READONLY) ||
+ ( pTargetDirEntry->ObjectInformation->Fcb != NULL &&
+ pTargetDirEntry->ObjectInformation->Fcb->OpenReferenceCount != 0))
+ {
+
+ //
+ // Return the correct error code
+ //
+
+ if( BooleanFlagOn( pTargetDirEntry->Flags, AFS_DIR_ENTRY_PENDING_DELETE))
+ {
+ ntStatus = STATUS_DELETE_PENDING;
+ }
+ else
+ {
+ ntStatus = STATUS_ACCESS_DENIED;
+ }
+
+ AFSReleaseResource( &pTargetDirEntry->ObjectInformation->NonPagedInfo->ObjectInfoLock);
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFS_TRACE_LEVEL_ERROR,
+ "AFSSetRenameInfo Attempt to rename %s Target %wZ Status %08lX\n",
+ BooleanFlagOn( pTargetDirEntry->ObjectInformation->FileAttributes, FILE_ATTRIBUTE_READONLY)?"ReadOnly":"Open",
+ &pTargetDirEntry->NameInformation.FileName,
+ ntStatus));
+
+ try_return( ntStatus);
+ }
+
+ AFSReleaseResource( &pTargetDirEntry->ObjectInformation->NonPagedInfo->ObjectInfoLock);
+
if( !bReplaceIfExists)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetRenameInfo Attempt to rename directory with target collision %wZ Target %wZ\n",
&pSrcCcb->DirectoryCB->NameInformation.FileName,
- &pTargetDirEntry->NameInformation.FileName);
+ &pTargetDirEntry->NameInformation.FileName));
try_return( ntStatus = STATUS_OBJECT_NAME_COLLISION);
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING | AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING | AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetRenameInfo Target %wZ exists DE %p Count %d, performing delete of target\n",
&pTargetDirEntry->NameInformation.FileName,
pTargetDirEntry,
- pTargetDirEntry->DirOpenReferenceCount);
+ lCount));
//
// Pull the directory entry from the parent
}
else
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSSetRenameInfo Target does NOT exist, normal rename\n");
+ "AFSSetRenameInfo Target does NOT exist, normal rename\n"));
}
//
// same parent we do not pull the node from the enumeration list
//
- AFSRemoveDirNodeFromParent( pSrcFcb->ObjectInformation->ParentObjectInformation,
+ AFSRemoveDirNodeFromParent( pSrcParentObject,
pSrcCcb->DirectoryCB,
!bCommonParent);
// request to the service.
//
- ntStatus = AFSNotifyRename( pSrcFcb->ObjectInformation,
+ ntStatus = AFSNotifyRename( pSrcObject,
&pSrcCcb->AuthGroup,
- pSrcFcb->ObjectInformation->ParentObjectInformation,
+ pSrcParentObject,
pTargetDcb->ObjectInformation,
pSrcCcb->DirectoryCB,
&uniTargetName,
// Attempt to re-insert the directory entry
//
- AFSInsertDirectoryNode( pSrcFcb->ObjectInformation->ParentObjectInformation,
+ AFSInsertDirectoryNode( pSrcParentObject,
pSrcCcb->DirectoryCB,
!bCommonParent);
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetRenameInfo Failed rename of %wZ to target %wZ Status %08lX\n",
&pSrcCcb->DirectoryCB->NameInformation.FileName,
&uniTargetName,
- ntStatus);
+ ntStatus));
try_return( ntStatus);
}
// Set the notification up for the source file
//
- if( pSrcCcb->DirectoryCB->ObjectInformation->ParentObjectInformation == pTargetParentObject &&
+ if( pSrcParentObject == pTargetParentObject &&
!bTargetEntryExists)
{
ulNotificationAction = FILE_ACTION_REMOVED;
}
- if( pSrcCcb->DirectoryCB->ObjectInformation->FileType == AFS_FILE_TYPE_DIRECTORY)
+ if( pSrcObject->FileType == AFS_FILE_TYPE_DIRECTORY)
{
ulNotifyFilter = FILE_NOTIFY_CHANGE_DIR_NAME;
ulNotifyFilter = FILE_NOTIFY_CHANGE_FILE_NAME;
}
- AFSFsRtlNotifyFullReportChange( pSrcCcb->DirectoryCB->ObjectInformation->ParentObjectInformation,
+ AFSFsRtlNotifyFullReportChange( pSrcParentObject,
pSrcCcb,
(ULONG)ulNotifyFilter,
(ULONG)ulNotificationAction);
// Attempt to re-insert the directory entry
//
- AFSInsertDirectoryNode( pSrcFcb->ObjectInformation->ParentObjectInformation,
+ AFSInsertDirectoryNode( pSrcParentObject,
pSrcCcb->DirectoryCB,
!bCommonParent);
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSSetRenameInfo Failed update of dir entry %wZ to target %wZ Status %08lX\n",
&pSrcCcb->DirectoryCB->NameInformation.FileName,
&uniTargetName,
- ntStatus);
+ ntStatus));
try_return( ntStatus);
}
pSrcCcb->DirectoryCB->Type.Data.ShortNameTreeEntry.HashIndex = AFSGenerateCRC( &uniShortName,
TRUE);
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetRenameInfo Initialized short name hash for %wZ longname %wZ\n",
&uniShortName,
- &pSrcCcb->DirectoryCB->NameInformation.FileName);
+ &pSrcCcb->DirectoryCB->NameInformation.FileName));
}
else
{
// Update the parent pointer in the source object if they are different
//
- if( pSrcCcb->DirectoryCB->ObjectInformation->ParentObjectInformation != pTargetParentObject)
+ if( pSrcParentObject != pTargetParentObject)
{
- lCount = InterlockedDecrement( &pSrcCcb->DirectoryCB->ObjectInformation->ParentObjectInformation->Specific.Directory.ChildOpenHandleCount);
+ lCount = InterlockedDecrement( &pSrcParentObject->Specific.Directory.ChildOpenHandleCount);
- lCount = InterlockedDecrement( &pSrcCcb->DirectoryCB->ObjectInformation->ParentObjectInformation->Specific.Directory.ChildOpenReferenceCount);
+ lCount = InterlockedDecrement( &pSrcParentObject->Specific.Directory.ChildOpenReferenceCount);
lCount = InterlockedIncrement( &pTargetParentObject->Specific.Directory.ChildOpenHandleCount);
lCount = InterlockedIncrement( &pTargetParentObject->Specific.Directory.ChildOpenReferenceCount);
- pSrcCcb->DirectoryCB->ObjectInformation->ParentObjectInformation = pTargetParentObject;
+
+ //
+ // Guaranteed to be in the same volume
+ //
+
+ AFSAcquireExcl( pSrcParentObject->VolumeCB->ObjectInfoTree.TreeLock,
+ TRUE);
+
+ lCount = AFSObjectInfoIncrement( pTargetParentObject,
+ AFS_OBJECT_REFERENCE_CHILD);
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_OBJECT_REF_COUNTING,
+ AFS_TRACE_LEVEL_VERBOSE,
+ "AFSSetRenameInfo Increment count on parent object %p Cnt %d\n",
+ pTargetParentObject,
+ lCount));
+
+ lCount = AFSObjectInfoDecrement( pSrcParentObject,
+ AFS_OBJECT_REFERENCE_CHILD);
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_OBJECT_REF_COUNTING,
+ AFS_TRACE_LEVEL_VERBOSE,
+ "AFSSetRenameInfo Decrement count on parent object %p Cnt %d\n",
+ pSrcParentObject,
+ lCount));
+
+ pSrcObject->ParentFileId = pTargetParentObject->FileId;
+
+ SetFlag( pSrcObject->Flags, AFS_OBJECT_FLAGS_PARENT_FID);
+
+ AFSReleaseResource( pSrcParentObject->VolumeCB->ObjectInfoTree.TreeLock);
ulNotificationAction = FILE_ACTION_ADDED;
}
// Now update the notification for the target file
//
- AFSFsRtlNotifyFullReportChange( pTargetParentObject->ParentObjectInformation,
+ AFSFsRtlNotifyFullReportChange( pTargetParentObject,
pSrcCcb,
(ULONG)ulNotifyFilter,
(ULONG)ulNotificationAction);
if( bTargetEntryExists)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetRenameInfo Setting DELETE flag in dir entry %p name %wZ\n",
pTargetDirEntry,
- &pTargetDirEntry->NameInformation.FileName);
+ &pTargetDirEntry->NameInformation.FileName));
SetFlag( pTargetDirEntry->Flags, AFS_DIR_ENTRY_DELETED);
lCount = InterlockedDecrement( &pTargetDirEntry->DirOpenReferenceCount); // The count we added above
- AFSDbgLogMsg( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetRenameInfo Decrement count on %wZ DE %p Ccb %p Cnt %d\n",
&pTargetDirEntry->NameInformation.FileName,
pTargetDirEntry,
pSrcCcb,
- lCount);
+ lCount));
ASSERT( lCount >= 0);
- if( lCount == 0)
+ if( lCount == 0 &&
+ pTargetDirEntry->NameArrayReferenceCount <= 0)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetRenameInfo Deleting dir entry %p name %wZ\n",
pTargetDirEntry,
- &pTargetDirEntry->NameInformation.FileName);
+ &pTargetDirEntry->NameInformation.FileName));
AFSDeleteDirEntry( pTargetParentObject,
- pTargetDirEntry);
+ &pTargetDirEntry);
}
pTargetDirEntry = NULL;
// permit the locks to be obtained out of order risking a deadlock.
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING|AFS_SUBSYSTEM_SECTION_OBJECT,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetRenameInfo Acquiring Fcb lock %p EXCL %08lX\n",
&pTargetFcb->NPFcb->Resource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireExcl( &pTargetFcb->NPFcb->Resource,
TRUE);
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING|AFS_SUBSYSTEM_SECTION_OBJECT,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetRenameInfo Acquiring Fcb SectionObject lock %p EXCL %08lX\n",
&pTargetFcb->NPFcb->SectionObjectResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireExcl( &pTargetFcb->NPFcb->SectionObjectResource,
TRUE);
- //
- // Close the section in the event it was mapped
- //
+ __try
+ {
+
+ //
+ // Close the section in the event it was mapped
+ //
- if( !MmForceSectionClosed( &pTargetFcb->NPFcb->SectionObjectPointers,
- TRUE))
+ if( !MmForceSectionClosed( &pTargetFcb->NPFcb->SectionObjectPointers,
+ TRUE))
+ {
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_FILE_PROCESSING,
+ AFS_TRACE_LEVEL_ERROR,
+ "AFSSetRenameInfo Failed to delete section for target file %wZ\n",
+ &uniTargetName));
+ }
+ }
+ __except( AFSExceptionFilter( __FUNCTION__, GetExceptionCode(), GetExceptionInformation()))
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_FILE_PROCESSING,
- AFS_TRACE_LEVEL_ERROR,
- "AFSSetRenameInfo Failed to delete section for target file %wZ\n",
- &uniTargetName);
+ ntStatus = GetExceptionCode();
+
+ AFSDbgTrace(( 0,
+ 0,
+ "EXCEPTION - AFSSetRenameInfo MmForceSectionClosed failed FID %08lX-%08lX-%08lX-%08lX Status 0x%08lX\n",
+ pTargetFcb->ObjectInformation->FileId.Cell,
+ pTargetFcb->ObjectInformation->FileId.Volume,
+ pTargetFcb->ObjectInformation->FileId.Vnode,
+ pTargetFcb->ObjectInformation->FileId.Unique,
+ ntStatus));
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING|AFS_SUBSYSTEM_SECTION_OBJECT,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetRenameInfo Releasing Fcb SectionObject lock %p EXCL %08lX\n",
&pTargetFcb->NPFcb->SectionObjectResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSReleaseResource( &pTargetFcb->NPFcb->SectionObjectResource);
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetRenameInfo Releasing Fcb lock %p EXCL %08lX\n",
&pTargetFcb->NPFcb->Resource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSReleaseResource( &pTargetFcb->NPFcb->Resource);
}
if( bTargetEntryExists)
{
- AFSInsertDirectoryNode( pTargetDirEntry->ObjectInformation->ParentObjectInformation,
+
+ ASSERT( pTargetParentObject != NULL);
+
+ AFSInsertDirectoryNode( pTargetParentObject,
pTargetDirEntry,
FALSE);
}
lCount = InterlockedDecrement( &pTargetDirEntry->DirOpenReferenceCount);
- AFSDbgLogMsg( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_DIRENTRY_REF_COUNTING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetRenameInfo Decrement2 count on %wZ DE %p Ccb %p Cnt %d\n",
&pTargetDirEntry->NameInformation.FileName,
pTargetDirEntry,
pSrcCcb,
- lCount);
+ lCount));
ASSERT( lCount >= 0);
}
AFSReleaseResource( pSourceDirLock);
}
- }
- if ( bDereferenceTargetParentObject)
- {
+ if ( bDereferenceTargetParentObject)
+ {
+
+ ObDereferenceObject( pTargetParentFileObj);
+ }
+
+ if ( pSrcParentObject != NULL)
+ {
+
+ AFSReleaseObjectInfo( &pSrcParentObject);
+ }
+
+ //
+ // No need to release pTargetParentObject as it is either a copy of pSrcParentObject
+ // or (AFSFcb *)pTargetFileObj->FsContext->ObjectInformation
+ //
- ObDereferenceObject( pTargetParentFileObj);
+ pTargetParentObject = NULL;
}
return ntStatus;
if( pFcb->Header.AllocationSize.QuadPart > pBuffer->AllocationSize.QuadPart)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING|AFS_SUBSYSTEM_SECTION_OBJECT,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetAllocationInfo Acquiring Fcb SectionObject lock %p EXCL %08lX\n",
&pFcb->NPFcb->SectionObjectResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireExcl( &pFcb->NPFcb->SectionObjectResource,
TRUE);
- bUserMapped = !MmCanFileBeTruncated( pFileObject->SectionObjectPointer,
- &pBuffer->AllocationSize);
+ __try
+ {
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ bUserMapped = !MmCanFileBeTruncated( pFileObject->SectionObjectPointer,
+ &pBuffer->AllocationSize);
+ }
+ __except( AFSExceptionFilter( __FUNCTION__, GetExceptionCode(), GetExceptionInformation()))
+ {
+
+ bUserMapped = FALSE;
+
+ ntStatus = GetExceptionCode();
+
+ AFSDbgTrace(( 0,
+ 0,
+ "EXCEPTION - AFSSetAllocationInfo MmCanFileBeTruncated failed FID %08lX-%08lX-%08lX-%08lX Status 0x%08lX\n",
+ pFcb->ObjectInformation->FileId.Cell,
+ pFcb->ObjectInformation->FileId.Volume,
+ pFcb->ObjectInformation->FileId.Vnode,
+ pFcb->ObjectInformation->FileId.Unique,
+ ntStatus));
+ }
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING|AFS_SUBSYSTEM_SECTION_OBJECT,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetAllocationInfo Releasing Fcb SectionObject lock %p EXCL %08lX\n",
&pFcb->NPFcb->SectionObjectResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSReleaseResource( &pFcb->NPFcb->SectionObjectResource);
// If this is a truncation we need to grab the paging IO resource.
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetAllocationInfo Acquiring Fcb PagingIo lock %p EXCL %08lX\n",
&pFcb->NPFcb->PagingResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireExcl( &pFcb->NPFcb->PagingResource,
TRUE);
// Tell Cc if allocation is increased.
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetAllocationInfo Acquiring Fcb PagingIo lock %p EXCL %08lX\n",
&pFcb->NPFcb->PagingResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireExcl( &pFcb->NPFcb->PagingResource,
TRUE);
//
if (bTellService)
{
- ntStatus = AFSUpdateFileInformation( &pFcb->ObjectInformation->ParentObjectInformation->FileId,
+
+ ASSERT( BooleanFlagOn( pFcb->ObjectInformation->Flags, AFS_OBJECT_FLAGS_PARENT_FID));
+
+ ntStatus = AFSUpdateFileInformation( &pFcb->ObjectInformation->ParentFileId,
pFcb->ObjectInformation,
&pCcb->AuthGroup);
}
CcSetFileSizes( pFileObject,
(PCC_FILE_SIZES)&pFcb->Header.AllocationSize);
}
+
+ //
+ // Mark the file as modified so as to reflect the change into the last write on close.
+ //
+ SetFlag( pFileObject->Flags, FO_FILE_MODIFIED);
}
else
{
if( pBuffer->EndOfFile.QuadPart < pFcb->Header.FileSize.QuadPart)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING|AFS_SUBSYSTEM_SECTION_OBJECT,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetEndOfFileInfo Acquiring Fcb SectionObject lock %p EXCL %08lX\n",
&pFcb->NPFcb->SectionObjectResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireExcl( &pFcb->NPFcb->SectionObjectResource,
TRUE);
- bUserMapped = !MmCanFileBeTruncated( pFileObject->SectionObjectPointer,
- &pBuffer->EndOfFile);
+ __try
+ {
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ bUserMapped = !MmCanFileBeTruncated( pFileObject->SectionObjectPointer,
+ &pBuffer->EndOfFile);
+ }
+ __except( AFSExceptionFilter( __FUNCTION__, GetExceptionCode(), GetExceptionInformation()))
+ {
+
+ bUserMapped = FALSE;
+
+ ntStatus = GetExceptionCode();
+
+ AFSDbgTrace(( 0,
+ 0,
+ "EXCEPTION - AFSSetEndOfFileInfo MmCanFileBeTruncated failed FID %08lX-%08lX-%08lX-%08lX Status 0x%08lX\n",
+ pFcb->ObjectInformation->FileId.Cell,
+ pFcb->ObjectInformation->FileId.Volume,
+ pFcb->ObjectInformation->FileId.Vnode,
+ pFcb->ObjectInformation->FileId.Unique,
+ ntStatus));
+ }
+
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING|AFS_SUBSYSTEM_SECTION_OBJECT,
AFS_TRACE_LEVEL_VERBOSE,
"AFSSetEndOfFileInfo Releasing Fcb SectionObject lock %p EXCL %08lX\n",
&pFcb->NPFcb->SectionObjectResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSReleaseResource( &pFcb->NPFcb->SectionObjectResource);
// If this is a truncation we need to grab the paging
// IO resource.
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSSetAllocationInfo Acquiring Fcb PagingIo lock %p EXCL %08lX\n",
+ "AFSSetEndOfFileInfo Acquiring Fcb PagingIo lock %p EXCL %08lX\n",
&pFcb->NPFcb->PagingResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireExcl( &pFcb->NPFcb->PagingResource,
TRUE);
// If this is a truncation we need to grab the paging
// IO resource.
//
- AFSDbgLogMsg( AFS_SUBSYSTEM_LOCK_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_LOCK_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSSetAllocationInfo Acquiring Fcb PagingIo lock %p EXCL %08lX\n",
+ "AFSSetEndOfFileInfo Acquiring Fcb PagingIo lock %p EXCL %08lX\n",
&pFcb->NPFcb->PagingResource,
- PsGetCurrentThread());
+ PsGetCurrentThread()));
AFSAcquireExcl( &pFcb->NPFcb->PagingResource,
TRUE);
// Tell the server
//
- ntStatus = AFSUpdateFileInformation( &pFcb->ObjectInformation->ParentObjectInformation->FileId,
+ ASSERT( BooleanFlagOn( pFcb->ObjectInformation->Flags, AFS_OBJECT_FLAGS_PARENT_FID));
+
+ ntStatus = AFSUpdateFileInformation( &pFcb->ObjectInformation->ParentFileId,
pFcb->ObjectInformation,
&pCcb->AuthGroup);
AFSTrimExtents( pFcb,
&pFcb->Header.FileSize);
}
+
+ //
+ // Mark the file as modified so as to reflect the change into the last write on close.
+ //
+ SetFlag( pFileObject->Flags, FO_FILE_MODIFIED);
}
else
{
{
ulFileInformationClass = pIrpSp->Parameters.SetFile.FileInformationClass;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIPE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIPE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSProcessShareSetInfo On pipe %wZ Class %08lX\n",
&Ccb->DirectoryCB->NameInformation.FileName,
- ulFileInformationClass);
+ ulFileInformationClass));
pPipeInfo = AFSLockSystemBuffer( Irp,
pIrpSp->Parameters.SetFile.Length);
if( pPipeInfo == NULL)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIPE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIPE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSProcessShareSetInfo Failed to lock buffer on pipe %wZ\n",
- &Ccb->DirectoryCB->NameInformation.FileName);
+ &Ccb->DirectoryCB->NameInformation.FileName));
try_return( ntStatus = STATUS_INSUFFICIENT_RESOURCES);
}
if( !NT_SUCCESS( ntStatus))
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIPE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIPE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSProcessShareSetInfo Failed to send request to service on pipe %wZ Status %08lX\n",
&Ccb->DirectoryCB->NameInformation.FileName,
- ntStatus);
+ ntStatus));
try_return( ntStatus);
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIPE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIPE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSProcessShareSetInfo Completed request on pipe %wZ Class %08lX\n",
&Ccb->DirectoryCB->NameInformation.FileName,
- ulFileInformationClass);
+ ulFileInformationClass));
try_exit:
ulFileInformationClass = pIrpSp->Parameters.QueryFile.FileInformationClass;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIPE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIPE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSProcessShareQueryInfo On pipe %wZ Class %08lX\n",
&Ccb->DirectoryCB->NameInformation.FileName,
- ulFileInformationClass);
+ ulFileInformationClass));
pPipeInfo = AFSLockSystemBuffer( Irp,
pIrpSp->Parameters.QueryFile.Length);
if( pPipeInfo == NULL)
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIPE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIPE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSProcessShareQueryInfo Failed to lock buffer on pipe %wZ\n",
- &Ccb->DirectoryCB->NameInformation.FileName);
+ &Ccb->DirectoryCB->NameInformation.FileName));
try_return( ntStatus = STATUS_INSUFFICIENT_RESOURCES);
}
if( !NT_SUCCESS( ntStatus))
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIPE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIPE_PROCESSING,
AFS_TRACE_LEVEL_ERROR,
"AFSProcessShareQueryInfo Failed to send request to service on pipe %wZ Status %08lX\n",
&Ccb->DirectoryCB->NameInformation.FileName,
- ntStatus);
+ ntStatus));
try_return( ntStatus);
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIPE_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIPE_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSProcessShareQueryInfo Completed request on pipe %wZ Class %08lX\n",
&Ccb->DirectoryCB->NameInformation.FileName,
- ulFileInformationClass);
+ ulFileInformationClass));
try_exit:
case FileBasicInformation:
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSProcessPIOCtlQueryInfo (FileBasicInformation)\n");
+ "AFSProcessPIOCtlQueryInfo (FileBasicInformation)\n"));
if ( *Length >= sizeof( FILE_BASIC_INFORMATION))
{
case FileStandardInformation:
{
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSProcessPIOCtlQueryInfo (FileStandardInformation)\n");
+ "AFSProcessPIOCtlQueryInfo (FileStandardInformation)\n"));
if ( *Length >= sizeof( FILE_STANDARD_INFORMATION))
{
PFILE_NAME_INFORMATION pNameInfo = (PFILE_NAME_INFORMATION)Irp->AssociatedIrp.SystemBuffer;
UNICODE_STRING uniName;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSProcessPIOCtlQueryInfo (FileNameInformation)\n");
+ "AFSProcessPIOCtlQueryInfo (FileNameInformation)\n"));
pFcb = (AFSFcb *)pIrpSp->FileObject->FsContext;
pCcb = (AFSCcb *)pIrpSp->FileObject->FsContext2;
uniName.Buffer = pNameInfo->FileName;
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSProcessPIOCtlQueryInfo (FileNameInformation) Returning %wZ\n",
- &uniName);
+ &uniName));
}
}
PFILE_INTERNAL_INFORMATION pInternalInfo = (PFILE_INTERNAL_INFORMATION)Irp->AssociatedIrp.SystemBuffer;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
- "AFSProcessPIOCtlQueryInfo (FileInternalInformation)\n");
+ "AFSProcessPIOCtlQueryInfo (FileInternalInformation)\n"));
if( *Length >= sizeof( FILE_INTERNAL_INFORMATION))
{
{
ntStatus = STATUS_INVALID_PARAMETER;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_WARNING,
- "AFSProcessPIOCtlQueryInfo (FileAllInformation) Not Implemented\n");
+ "AFSProcessPIOCtlQueryInfo (FileAllInformation) Not Implemented\n"));
break;
}
{
ntStatus = STATUS_INVALID_PARAMETER;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_WARNING,
- "AFSProcessPIOCtlQueryInfo (FileEaInformation) Not Implemented\n");
+ "AFSProcessPIOCtlQueryInfo (FileEaInformation) Not Implemented\n"));
break;
}
{
ntStatus = STATUS_INVALID_PARAMETER;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_WARNING,
- "AFSProcessPIOCtlQueryInfo (FilePositionInformation) Not Implemented\n");
+ "AFSProcessPIOCtlQueryInfo (FilePositionInformation) Not Implemented\n"));
break;
}
{
ntStatus = STATUS_INVALID_PARAMETER;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_WARNING,
- "AFSProcessPIOCtlQueryInfo (FileAlternateNameInformation) Not Implemented\n");
+ "AFSProcessPIOCtlQueryInfo (FileAlternateNameInformation) Not Implemented\n"));
break;
}
{
ntStatus = STATUS_INVALID_PARAMETER;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_WARNING,
- "AFSProcessPIOCtlQueryInfo (FileNetworkOpenInformation) Not Implemented\n");
+ "AFSProcessPIOCtlQueryInfo (FileNetworkOpenInformation) Not Implemented\n"));
break;
}
{
ntStatus = STATUS_INVALID_PARAMETER;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_WARNING,
- "AFSProcessPIOCtlQueryInfo (FileStreamInformation) Not Implemented\n");
+ "AFSProcessPIOCtlQueryInfo (FileStreamInformation) Not Implemented\n"));
break;
}
{
ntStatus = STATUS_INVALID_PARAMETER;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_WARNING,
- "AFSProcessPIOCtlQueryInfo (FileAttributeTagInformation) Not Implemented\n");
+ "AFSProcessPIOCtlQueryInfo (FileAttributeTagInformation) Not Implemented\n"));
break;
}
{
ntStatus = STATUS_INVALID_PARAMETER;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_WARNING,
- "AFSProcessPIOCtlQueryInfo (FileRemoteProtocolInformation) Not Implemented\n");
+ "AFSProcessPIOCtlQueryInfo (FileRemoteProtocolInformation) Not Implemented\n"));
break;
}
{
ntStatus = STATUS_INVALID_PARAMETER;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_WARNING,
- "AFSProcessPIOCtlQueryInfo (FileNetworkPhysicalNameInformation) Not Implemented\n");
+ "AFSProcessPIOCtlQueryInfo (FileNetworkPhysicalNameInformation) Not Implemented\n"));
break;
}
{
ntStatus = STATUS_INVALID_PARAMETER;
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_WARNING,
"AFSProcessPIOCtlQueryInfo Not handling request %08lX\n",
- ulFileInformationClass);
+ ulFileInformationClass));
break;
}
}
}
- AFSDbgLogMsg( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
+ AFSDbgTrace(( AFS_SUBSYSTEM_PIOCTL_PROCESSING,
AFS_TRACE_LEVEL_VERBOSE,
"AFSProcessPIOCtlQueryInfo ntStatus %08lX\n",
- ntStatus);
+ ntStatus));
return ntStatus;
}
git://git.openafs.org / openafs.git / blobdiff