Ros-diffs

ros-diffs@reactos.org

81390 discussions

[pschweitzer] 67911: [FASTFAT] [NTFS] Remove useless cast and use appropriate macro. Spotted by Thomas
by pschweitzer＠svn.reactos.org 25 May '15

25 May '15

Author: pschweitzer Date: Mon May 25 17:30:01 2015 New Revision: 67911 URL: http://svn.reactos.org/svn/reactos?rev=67911&view=rev Log: [FASTFAT] [NTFS] Remove useless cast and use appropriate macro. Spotted by Thomas Modified: trunk/reactos/drivers/filesystems/fastfat/cleanup.c trunk/reactos/drivers/filesystems/fastfat/dir.c trunk/reactos/drivers/filesystems/fastfat/finfo.c trunk/reactos/drivers/filesystems/fastfat/rw.c trunk/reactos/drivers/filesystems/fastfat/volume.c trunk/reactos/drivers/filesystems/ntfs/finfo.c Modified: trunk/reactos/drivers/filesystems/fastfat/cleanup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/cleanup.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/cleanup.c [iso-8859-1] Mon May 25 17:30:01 2015 @@ -47,12 +47,12 @@ else { if(!ExAcquireResourceExclusiveLite(&pFcb->MainResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { return STATUS_PENDING; } if(!ExAcquireResourceExclusiveLite(&pFcb->PagingIoResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { ExReleaseResourceLite(&pFcb->MainResource); return STATUS_PENDING; @@ -157,7 +157,7 @@ } if (!ExAcquireResourceExclusiveLite(&IrpContext->DeviceExt->DirResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { return VfatMarkIrpContextForQueue(IrpContext); } Modified: trunk/reactos/drivers/filesystems/fastfat/dir.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/dir.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/dir.c [iso-8859-1] Mon May 25 17:30:01 2015 @@ -422,7 +422,7 @@ Buffer = VfatGetUserBuffer(IrpContext->Irp, FALSE); if (!ExAcquireResourceSharedLite(&pFcb->MainResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { Status = VfatLockUserBuffer(IrpContext->Irp, BufferLength, IoWriteAccess); if (NT_SUCCESS(Status)) @@ -505,7 +505,7 @@ DirContext.ShortNameU.MaximumLength = sizeof(ShortNameBuffer); if (!ExAcquireResourceExclusiveLite(&IrpContext->DeviceExt->DirResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { ExReleaseResourceLite(&pFcb->MainResource); return VfatMarkIrpContextForQueue(IrpContext); Modified: trunk/reactos/drivers/filesystems/fastfat/finfo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/finfo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/finfo.c [iso-8859-1] Mon May 25 17:30:01 2015 @@ -1377,7 +1377,7 @@ if (!(FCB->Flags & FCB_IS_PAGE_FILE)) { if (!ExAcquireResourceSharedLite(&FCB->MainResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { return VfatMarkIrpContextForQueue(IrpContext); } @@ -1514,7 +1514,7 @@ if (FileInformationClass == FileRenameInformation) { if (!ExAcquireResourceExclusiveLite(&((PDEVICE_EXTENSION)IrpContext->DeviceObject->DeviceExtension)->DirResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { return VfatMarkIrpContextForQueue(IrpContext); } @@ -1523,7 +1523,7 @@ if (!(FCB->Flags & FCB_IS_PAGE_FILE)) { if (!ExAcquireResourceExclusiveLite(&FCB->MainResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { if (FileInformationClass == FileRenameInformation) { Modified: trunk/reactos/drivers/filesystems/fastfat/rw.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/rw.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/rw.c [iso-8859-1] Mon May 25 17:30:01 2015 @@ -887,7 +887,7 @@ if (Fcb->Flags & FCB_IS_PAGE_FILE) { if (!ExAcquireResourceSharedLite(Resource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { Resource = NULL; Status = STATUS_PENDING; @@ -897,7 +897,7 @@ else { if (!ExAcquireResourceExclusiveLite(Resource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { Resource = NULL; Status = STATUS_PENDING; Modified: trunk/reactos/drivers/filesystems/fastfat/volume.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/volume.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/volume.c [iso-8859-1] Mon May 25 17:30:01 2015 @@ -361,7 +361,7 @@ DPRINT("VfatQueryVolumeInformation(IrpContext %p)\n", IrpContext); if (!ExAcquireResourceSharedLite(&((PDEVICE_EXTENSION)IrpContext->DeviceObject->DeviceExtension)->DirResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { DPRINT1("DirResource failed!\n"); return VfatMarkIrpContextForQueue(IrpContext); @@ -436,7 +436,7 @@ DPRINT("VfatSetVolumeInformation(IrpContext %p)\n", IrpContext); if (!ExAcquireResourceExclusiveLite(&((PDEVICE_EXTENSION)IrpContext->DeviceObject->DeviceExtension)->DirResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { return VfatMarkIrpContextForQueue(IrpContext); } Modified: trunk/reactos/drivers/filesystems/ntfs/finfo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/f… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/finfo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/finfo.c [iso-8859-1] Mon May 25 17:30:01 2015 @@ -247,7 +247,7 @@ BufferLength = Stack->Parameters.QueryFile.Length; if (!ExAcquireResourceSharedLite(&Fcb->MainResource, - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + BooleanFlagOn(IrpContext->Flags, IRPCONTEXT_CANWAIT))) { return NtfsMarkIrpContextForQueue(IrpContext); }

1 0

[pschweitzer] 67910: [NTFS] Acquire FCB resource (shared) before trying to attempt its data
by pschweitzer＠svn.reactos.org 25 May '15

25 May '15

Author: pschweitzer Date: Mon May 25 17:06:00 2015 New Revision: 67910 URL: http://svn.reactos.org/svn/reactos?rev=67910&view=rev Log: [NTFS] Acquire FCB resource (shared) before trying to attempt its data Modified: trunk/reactos/drivers/filesystems/ntfs/finfo.c Modified: trunk/reactos/drivers/filesystems/ntfs/finfo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/f… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/finfo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/finfo.c [iso-8859-1] Mon May 25 17:06:00 2015 @@ -246,6 +246,12 @@ SystemBuffer = Irp->AssociatedIrp.SystemBuffer; BufferLength = Stack->Parameters.QueryFile.Length; + if (!ExAcquireResourceSharedLite(&Fcb->MainResource, + (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT))) + { + return NtfsMarkIrpContextForQueue(IrpContext); + } + switch (FileInformationClass) { case FileStandardInformation: @@ -301,6 +307,8 @@ Status = STATUS_INVALID_PARAMETER; } + ExReleaseResourceLite(&Fcb->MainResource); + if (NT_SUCCESS(Status)) Irp->IoStatus.Information = Stack->Parameters.QueryFile.Length - BufferLength;

1 0

[pschweitzer] 67909: [NTFS] If IRP can't wait, queue it
by pschweitzer＠svn.reactos.org 25 May '15

25 May '15

Author: pschweitzer Date: Mon May 25 16:52:15 2015 New Revision: 67909 URL: http://svn.reactos.org/svn/reactos?rev=67909&view=rev Log: [NTFS] If IRP can't wait, queue it Modified: trunk/reactos/drivers/filesystems/ntfs/create.c Modified: trunk/reactos/drivers/filesystems/ntfs/create.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/c… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/create.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/create.c [iso-8859-1] Mon May 25 16:52:15 2015 @@ -428,6 +428,11 @@ DeviceExt = DeviceObject->DeviceExtension; + if (!(IrpContext->Flags & IRPCONTEXT_CANWAIT)) + { + return NtfsMarkIrpContextForQueue(IrpContext); + } + ExAcquireResourceExclusiveLite(&DeviceExt->DirResource, TRUE); Status = NtfsCreateFile(DeviceObject,

1 0

[pschweitzer] 67908: [NTFS] Implement NtfsMarkIrpContextForQueue() which allows marking an IRP to queue it
by pschweitzer＠svn.reactos.org 25 May '15

25 May '15

Author: pschweitzer Date: Mon May 25 16:52:02 2015 New Revision: 67908 URL: http://svn.reactos.org/svn/reactos?rev=67908&view=rev Log: [NTFS] Implement NtfsMarkIrpContextForQueue() which allows marking an IRP to queue it Modified: trunk/reactos/drivers/filesystems/ntfs/ntfs.h Modified: trunk/reactos/drivers/filesystems/ntfs/ntfs.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/n… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/ntfs.h [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/ntfs.h [iso-8859-1] Mon May 25 16:52:02 2015 @@ -461,9 +461,17 @@ extern PNTFS_GLOBAL_DATA NtfsGlobalData; -//int CdfsStrcmpi( wchar_t *str1, wchar_t *str2 ); -//void CdfsWstrcpy( wchar_t *str1, wchar_t *str2, int max ); - +FORCEINLINE +NTSTATUS +NtfsMarkIrpContextForQueue(PNTFS_IRP_CONTEXT IrpContext) +{ + PULONG Flags = &IrpContext->Flags; + + *Flags &= ~IRPCONTEXT_COMPLETE; + *Flags |= IRPCONTEXT_QUEUE; + + return STATUS_PENDING; +} /* attrib.c */

1 0

[tfaber] 67907: [WIN32K:NTUSER] - Use UserRefObjectCo in IntNotifyWinEvent to avoid a reference leak in case the call-out does not return - Sanitize list walk
by tfaber＠svn.reactos.org 25 May '15

25 May '15

Author: tfaber Date: Mon May 25 15:28:06 2015 New Revision: 67907 URL: http://svn.reactos.org/svn/reactos?rev=67907&view=rev Log: [WIN32K:NTUSER] - Use UserRefObjectCo in IntNotifyWinEvent to avoid a reference leak in case the call-out does not return - Sanitize list walk Modified: trunk/reactos/win32ss/user/ntuser/event.c Modified: trunk/reactos/win32ss/user/ntuser/event.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/event.… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/event.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/event.c [iso-8859-1] Mon May 25 15:28:06 2015 @@ -184,8 +184,9 @@ DWORD flags) { PEVENTHOOK pEH; - PLIST_ENTRY pLE; + PLIST_ENTRY ListEntry; PTHREADINFO pti, ptiCurrent; + USER_REFERENCE_ENTRY Ref; TRACE("IntNotifyWinEvent GlobalEvents = %p pWnd %p\n", GlobalEvents, pWnd); @@ -200,12 +201,13 @@ else pti = ptiCurrent; - pLE = GlobalEvents->Events.Flink; - pEH = CONTAINING_RECORD(pLE, EVENTHOOK, Chain); - do - { - if (!pEH) break; - UserReferenceObject(pEH); + ListEntry = GlobalEvents->Events.Flink; + ASSERT(ListEntry != &GlobalEvents->Events); + while (ListEntry != &GlobalEvents->Events) + { + pEH = CONTAINING_RECORD(ListEntry, EVENTHOOK, Chain); + ListEntry = ListEntry->Flink; + // Must be inside the event window. if ( Event >= pEH->eventMin && Event <= pEH->eventMax ) { @@ -217,6 +219,7 @@ (pEH->Flags & WINEVENT_SKIPOWNTHREAD && pEH->head.pti == pti) || pEH->head.pti->rpdesk != ptiCurrent->rpdesk ) ) // Same as hooks. { + UserRefObjectCo(pEH, &Ref); if (pEH->Flags & WINEVENT_INCONTEXT) { TRACE("In Event 0x%x, idObject %d hwnd %p\n", Event, idObject, pWnd ? UserHMGetHandle(pWnd) : NULL); @@ -241,12 +244,10 @@ idChild, PtrToUint(NtCurrentTeb()->ClientId.UniqueThread)); } + UserDerefObjectCo(pEH); } } - UserDereferenceObject(pEH); - pLE = pEH->Chain.Flink; - pEH = CONTAINING_RECORD(pLE, EVENTHOOK, Chain); - } while (pLE != &GlobalEvents->Events); + } } VOID

1 0

[tfaber] 67906: [RTL][NTOSKRNL] - Don't cast CONTAINING_RECORD
by tfaber＠svn.reactos.org 25 May '15

25 May '15

Author: tfaber Date: Mon May 25 15:04:09 2015 New Revision: 67906 URL: http://svn.reactos.org/svn/reactos?rev=67906&view=rev Log: [RTL][NTOSKRNL] - Don't cast CONTAINING_RECORD Modified: trunk/reactos/lib/rtl/generictable.c trunk/reactos/ntoskrnl/ex/sysinfo.c trunk/reactos/ntoskrnl/fsrtl/tunnel.c Modified: trunk/reactos/lib/rtl/generictable.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/generictable.c?rev… ============================================================================== --- trunk/reactos/lib/rtl/generictable.c [iso-8859-1] (original) +++ trunk/reactos/lib/rtl/generictable.c [iso-8859-1] Mon May 25 15:04:09 2015 @@ -438,9 +438,9 @@ if (NextI == OrderedElement) { /* Return it */ - return &((PTABLE_ENTRY_HEADER)CONTAINING_RECORD(OrderedNode, - TABLE_ENTRY_HEADER, - ListEntry))->UserData; + return &CONTAINING_RECORD(OrderedNode, + TABLE_ENTRY_HEADER, + ListEntry)->UserData; } /* Now check if we're farther behind */ @@ -505,9 +505,9 @@ Table->WhichOrderedElement = NextI; /* Return the element */ - return &((PTABLE_ENTRY_HEADER)CONTAINING_RECORD(OrderedNode, - TABLE_ENTRY_HEADER, - ListEntry))->UserData; + return &CONTAINING_RECORD(OrderedNode, + TABLE_ENTRY_HEADER, + ListEntry)->UserData; } /* EOF */ Modified: trunk/reactos/ntoskrnl/ex/sysinfo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ex/sysinfo.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/ex/sysinfo.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/ex/sysinfo.c [iso-8859-1] Mon May 25 15:04:09 2015 @@ -918,8 +918,7 @@ CurrentEntry = Process->Pcb.ThreadListHead.Flink; while (CurrentEntry != &Process->Pcb.ThreadListHead) { - CurrentThread = (PETHREAD)CONTAINING_RECORD(CurrentEntry, KTHREAD, - ThreadListEntry); + CurrentThread = CONTAINING_RECORD(CurrentEntry, ETHREAD, Tcb.ThreadListEntry); ThreadInfo->KernelTime.QuadPart = UInt32x32To64(CurrentThread->Tcb.KernelTime, KeMaximumIncrement); ThreadInfo->UserTime.QuadPart = UInt32x32To64(CurrentThread->Tcb.UserTime, KeMaximumIncrement); Modified: trunk/reactos/ntoskrnl/fsrtl/tunnel.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/fsrtl/tunnel.c?re… ============================================================================== --- trunk/reactos/ntoskrnl/fsrtl/tunnel.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/fsrtl/tunnel.c [iso-8859-1] Mon May 25 15:04:09 2015 @@ -105,7 +105,7 @@ while(Entry != &Cache->TimerQueue) { /* get node entry */ - CurEntry = (PTUNNEL_NODE_ENTRY)CONTAINING_RECORD(Entry, TUNNEL_NODE_ENTRY, TimerQueueEntry); + CurEntry = CONTAINING_RECORD(Entry, TUNNEL_NODE_ENTRY, TimerQueueEntry); /* get next entry */ NextEntry = Entry->Flink; @@ -124,7 +124,7 @@ /* If we have too many entries */ while (Cache->NumEntries > TunnelMaxEntries) { - CurEntry = (PTUNNEL_NODE_ENTRY)CONTAINING_RECORD(Entry, TUNNEL_NODE_ENTRY, TimerQueueEntry); + CurEntry = CONTAINING_RECORD(Entry, TUNNEL_NODE_ENTRY, TimerQueueEntry); FsRtlRemoveNodeFromTunnel(Cache, CurEntry, PoolList, &Rebalance); } } @@ -295,7 +295,7 @@ { /* and free them, one by one */ CurEntry = RemoveHeadList(PoolList); - CurNode = (PTUNNEL_NODE_ENTRY)CONTAINING_RECORD(CurEntry, TUNNEL_NODE_ENTRY, TimerQueueEntry); + CurNode = CONTAINING_RECORD(CurEntry, TUNNEL_NODE_ENTRY, TimerQueueEntry); FsRtlFreeTunnelNode(CurNode, 0); } } @@ -609,7 +609,7 @@ CurEntry = Cache->Cache; while (CurEntry) { - CurNode = (PTUNNEL_NODE_ENTRY)CONTAINING_RECORD(CurEntry, TUNNEL_NODE_ENTRY, SplayInfo); + CurNode = CONTAINING_RECORD(CurEntry, TUNNEL_NODE_ENTRY, SplayInfo); if (CurNode->DirectoryKey > DirectoryKey) { @@ -647,7 +647,7 @@ /* delete any matching key */ do { - CurNode = (PTUNNEL_NODE_ENTRY)CONTAINING_RECORD(LastEntry, TUNNEL_NODE_ENTRY, SplayInfo); + CurNode = CONTAINING_RECORD(LastEntry, TUNNEL_NODE_ENTRY, SplayInfo); Successors = RtlRealSuccessor(LastEntry); if (CurNode->DirectoryKey != DirectoryKey) @@ -704,7 +704,7 @@ while(Entry != &Cache->TimerQueue) { /* get node entry */ - CurEntry = (PTUNNEL_NODE_ENTRY)CONTAINING_RECORD(Entry, TUNNEL_NODE_ENTRY, TimerQueueEntry); + CurEntry = CONTAINING_RECORD(Entry, TUNNEL_NODE_ENTRY, TimerQueueEntry); /* get next entry */ NextEntry = Entry->Flink;

1 0

[pschweitzer] 67905: [NTFS] Handle IRP_MJ_FILE_SYSTEM_CONTROL with the dispatch routine
by pschweitzer＠svn.reactos.org 25 May '15

25 May '15

Author: pschweitzer Date: Mon May 25 12:55:51 2015 New Revision: 67905 URL: http://svn.reactos.org/svn/reactos?rev=67905&view=rev Log: [NTFS] Handle IRP_MJ_FILE_SYSTEM_CONTROL with the dispatch routine Modified: trunk/reactos/drivers/filesystems/ntfs/dispatch.c trunk/reactos/drivers/filesystems/ntfs/fsctl.c trunk/reactos/drivers/filesystems/ntfs/ntfs.c trunk/reactos/drivers/filesystems/ntfs/ntfs.h Modified: trunk/reactos/drivers/filesystems/ntfs/dispatch.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/d… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/dispatch.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/dispatch.c [iso-8859-1] Mon May 25 12:55:51 2015 @@ -104,6 +104,10 @@ case IRP_MJ_CREATE: Status = NtfsCreate(IrpContext); break; + + case IRP_MJ_FILE_SYSTEM_CONTROL: + Status = NtfsFileSystemControl(IrpContext); + break; } ASSERT((!(IrpContext->Flags & IRPCONTEXT_COMPLETE) && !(IrpContext->Flags & IRPCONTEXT_QUEUE)) || Modified: trunk/reactos/drivers/filesystems/ntfs/fsctl.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/f… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/fsctl.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/fsctl.c [iso-8859-1] Mon May 25 12:55:51 2015 @@ -866,20 +866,19 @@ NTSTATUS -NTAPI -NtfsFsdFileSystemControl(PDEVICE_OBJECT DeviceObject, - PIRP Irp) +NtfsFileSystemControl(PNTFS_IRP_CONTEXT IrpContext) { - PIO_STACK_LOCATION Stack; NTSTATUS Status; + PIRP Irp; + PDEVICE_OBJECT DeviceObject; DPRINT1("NtfsFileSystemControl() called\n"); - Stack = IoGetCurrentIrpStackLocation(Irp); - + DeviceObject = IrpContext->DeviceObject; + Irp = IrpContext->Irp; Irp->IoStatus.Information = 0; - switch (Stack->MinorFunction) + switch (IrpContext->MinorFunction) { case IRP_MN_KERNEL_CALL: DPRINT1("NTFS: IRP_MN_USER_FS_REQUEST\n"); @@ -901,14 +900,10 @@ break; default: - DPRINT1("NTFS FSC: MinorFunction %d\n", Stack->MinorFunction); + DPRINT1("NTFS FSC: MinorFunction %d\n", IrpContext->MinorFunction); Status = STATUS_INVALID_DEVICE_REQUEST; break; } - - Irp->IoStatus.Status = Status; - - IoCompleteRequest(Irp, IO_NO_INCREMENT); return Status; } Modified: trunk/reactos/drivers/filesystems/ntfs/ntfs.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/n… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/ntfs.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/ntfs.c [iso-8859-1] Mon May 25 12:55:51 2015 @@ -131,7 +131,7 @@ DriverObject->MajorFunction[IRP_MJ_QUERY_VOLUME_INFORMATION] = NtfsFsdDispatch; DriverObject->MajorFunction[IRP_MJ_SET_VOLUME_INFORMATION] = NtfsFsdDispatch; DriverObject->MajorFunction[IRP_MJ_DIRECTORY_CONTROL] = NtfsFsdDispatch; - DriverObject->MajorFunction[IRP_MJ_FILE_SYSTEM_CONTROL] = NtfsFsdFileSystemControl; + DriverObject->MajorFunction[IRP_MJ_FILE_SYSTEM_CONTROL] = NtfsFsdDispatch; DriverObject->MajorFunction[IRP_MJ_DEVICE_CONTROL] = NtfsFsdDispatch; return; Modified: trunk/reactos/drivers/filesystems/ntfs/ntfs.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/n… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/ntfs.h [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/ntfs.h [iso-8859-1] Mon May 25 12:55:51 2015 @@ -644,10 +644,8 @@ /* fsctl.c */ -DRIVER_DISPATCH NtfsFsdFileSystemControl; -NTSTATUS NTAPI -NtfsFsdFileSystemControl(PDEVICE_OBJECT DeviceObject, - PIRP Irp); +NTSTATUS +NtfsFileSystemControl(PNTFS_IRP_CONTEXT IrpContext); /* mft.c */

1 0

[ekohl] 67904: [SYSDM] - Implement changing the hardware profile order. - Keep button states up-to-date.
by ekohl＠svn.reactos.org 25 May '15

25 May '15

Author: ekohl Date: Mon May 25 12:23:12 2015 New Revision: 67904 URL: http://svn.reactos.org/svn/reactos?rev=67904&view=rev Log: [SYSDM] - Implement changing the hardware profile order. - Keep button states up-to-date. Modified: trunk/reactos/dll/cpl/sysdm/hardprof.c Modified: trunk/reactos/dll/cpl/sysdm/hardprof.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/cpl/sysdm/hardprof.c?r… ============================================================================== --- trunk/reactos/dll/cpl/sysdm/hardprof.c [iso-8859-1] (original) +++ trunk/reactos/dll/cpl/sysdm/hardprof.c [iso-8859-1] Mon May 25 12:23:12 2015 @@ -36,6 +36,31 @@ WCHAR szDestinationName[PROFILE_NAME_LENGTH]; PPROFILEDATA pProfileData; } PROFILENAMES, *PPROFILENAMES; + + +static +VOID +UpdateButtons( + HWND hwndDlg, + PPROFILEDATA pProfileData) +{ + EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFCOPY), (pProfileData->dwSelectedProfileIndex != (DWORD)-1) ? TRUE : FALSE); + EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFRENAME), (pProfileData->dwSelectedProfileIndex != (DWORD)-1) ? TRUE : FALSE); + EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFDEL), (pProfileData->dwSelectedProfileIndex != (DWORD)-1) ? TRUE : FALSE); + + if (pProfileData->dwProfileCount < 2) + { + EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFUP), FALSE); + EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFDWN), FALSE); + } + else + { + EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFUP), + (pProfileData->dwSelectedProfileIndex > 0) ? TRUE : FALSE); + EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFDWN), + (pProfileData->dwSelectedProfileIndex < pProfileData->dwProfileCount - 1) ? TRUE : FALSE); + } +} static @@ -171,6 +196,8 @@ pNewProfile->dwPreferenceOrder = pNewProfile->dwProfileNumber; SendDlgItemMessageW(hwndDlg, IDC_HRDPROFLSTBOX, LB_ADDSTRING, 0, (LPARAM)pNewProfile->szFriendlyName); + + UpdateButtons(hwndDlg, pProfileData); } @@ -320,6 +347,39 @@ pProfileData->pProfiles = pProfiles; SendDlgItemMessageW(hwndDlg, IDC_HRDPROFLSTBOX, LB_SETCURSEL, pProfileData->dwSelectedProfileIndex, 0); + + UpdateButtons(hwndDlg, pProfileData); +} + + +static +VOID +MoveHardwareProfile( + HWND hwndDlg, + PPROFILEDATA pProfileData, + BOOL bMoveUp) +{ + PROFILE TempProfile; + PPROFILE pSrcProfile, pDstProfile; + DWORD dwSrcIndex, dwDstIndex; + + dwSrcIndex = pProfileData->dwSelectedProfileIndex; + dwDstIndex = bMoveUp ? (dwSrcIndex - 1) : (dwSrcIndex + 1); + + pSrcProfile = &pProfileData->pProfiles[dwSrcIndex]; + pDstProfile = &pProfileData->pProfiles[dwDstIndex]; + + CopyMemory(&TempProfile, pDstProfile, sizeof(PROFILE)); + CopyMemory(pDstProfile, pSrcProfile, sizeof(PROFILE)); + CopyMemory(pSrcProfile, &TempProfile, sizeof(PROFILE)); + + SendDlgItemMessageW(hwndDlg, IDC_HRDPROFLSTBOX, LB_DELETESTRING, dwSrcIndex, 0); + SendDlgItemMessageW(hwndDlg, IDC_HRDPROFLSTBOX, LB_INSERTSTRING, dwDstIndex, (LPARAM)pDstProfile->szFriendlyName); + + pProfileData->dwSelectedProfileIndex = dwDstIndex; + SendDlgItemMessageW(hwndDlg, IDC_HRDPROFLSTBOX, LB_SETCURSEL, pProfileData->dwSelectedProfileIndex, 0); + + UpdateButtons(hwndDlg, pProfileData); } @@ -634,6 +694,14 @@ DeleteHardwareProfile(hwndDlg, pProfileData); break; + case IDC_HRDPROFUP: + MoveHardwareProfile(hwndDlg, pProfileData, TRUE); + break; + + case IDC_HRDPROFDWN: + MoveHardwareProfile(hwndDlg, pProfileData, FALSE); + break; + case IDC_HRDPROFWAIT: EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFEDIT), FALSE); return TRUE; @@ -646,10 +714,7 @@ if (HIWORD(wParam) == LBN_SELCHANGE) { pProfileData->dwSelectedProfileIndex = (DWORD)SendDlgItemMessage(hwndDlg, IDC_HRDPROFLSTBOX, LB_GETCURSEL, 0, 0); - - EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFCOPY), (pProfileData->dwSelectedProfileIndex != (DWORD)-1) ? TRUE : FALSE); - EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFRENAME), (pProfileData->dwSelectedProfileIndex != (DWORD)-1) ? TRUE : FALSE); - EnableWindow(GetDlgItem(hwndDlg, IDC_HRDPROFDEL), (pProfileData->dwSelectedProfileIndex != (DWORD)-1) ? TRUE : FALSE); + UpdateButtons(hwndDlg, pProfileData); } return TRUE;

1 0

[pschweitzer] 67903: [NTFS] Handle IRP_MJ_CREATTE with the dispatch routine
by pschweitzer＠svn.reactos.org 25 May '15

25 May '15

Author: pschweitzer Date: Mon May 25 11:58:23 2015 New Revision: 67903 URL: http://svn.reactos.org/svn/reactos?rev=67903&view=rev Log: [NTFS] Handle IRP_MJ_CREATTE with the dispatch routine Modified: trunk/reactos/drivers/filesystems/ntfs/create.c trunk/reactos/drivers/filesystems/ntfs/dispatch.c trunk/reactos/drivers/filesystems/ntfs/ntfs.c trunk/reactos/drivers/filesystems/ntfs/ntfs.h Modified: trunk/reactos/drivers/filesystems/ntfs/create.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/c… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/create.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/create.c [iso-8859-1] Mon May 25 11:58:23 2015 @@ -405,43 +405,34 @@ * fail immediately */ Irp->IoStatus.Information = (NT_SUCCESS(Status)) ? FILE_OPENED : 0; - Irp->IoStatus.Status = Status; return Status; } NTSTATUS -NTAPI -NtfsFsdCreate(PDEVICE_OBJECT DeviceObject, - PIRP Irp) +NtfsCreate(PNTFS_IRP_CONTEXT IrpContext) { PDEVICE_EXTENSION DeviceExt; NTSTATUS Status; - + PDEVICE_OBJECT DeviceObject; + + DeviceObject = IrpContext->DeviceObject; if (DeviceObject == NtfsGlobalData->DeviceObject) { /* DeviceObject represents FileSystem instead of logical volume */ DPRINT("Opening file system\n"); - Irp->IoStatus.Information = FILE_OPENED; - Status = STATUS_SUCCESS; - goto ByeBye; + IrpContext->Irp->IoStatus.Information = FILE_OPENED; + return STATUS_SUCCESS; } DeviceExt = DeviceObject->DeviceExtension; - FsRtlEnterFileSystem(); ExAcquireResourceExclusiveLite(&DeviceExt->DirResource, TRUE); Status = NtfsCreateFile(DeviceObject, - Irp); + IrpContext->Irp); ExReleaseResourceLite(&DeviceExt->DirResource); - FsRtlExitFileSystem(); - -ByeBye: - Irp->IoStatus.Status = Status; - IoCompleteRequest(Irp, - NT_SUCCESS(Status) ? IO_DISK_INCREMENT : IO_NO_INCREMENT); return Status; } Modified: trunk/reactos/drivers/filesystems/ntfs/dispatch.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/d… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/dispatch.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/dispatch.c [iso-8859-1] Mon May 25 11:58:23 2015 @@ -100,6 +100,10 @@ case IRP_MJ_CLOSE: Status = NtfsClose(IrpContext); break; + + case IRP_MJ_CREATE: + Status = NtfsCreate(IrpContext); + break; } ASSERT((!(IrpContext->Flags & IRPCONTEXT_COMPLETE) && !(IrpContext->Flags & IRPCONTEXT_QUEUE)) || Modified: trunk/reactos/drivers/filesystems/ntfs/ntfs.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/n… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/ntfs.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/ntfs.c [iso-8859-1] Mon May 25 11:58:23 2015 @@ -123,7 +123,7 @@ NTAPI NtfsInitializeFunctionPointers(PDRIVER_OBJECT DriverObject) { - DriverObject->MajorFunction[IRP_MJ_CREATE] = NtfsFsdCreate; + DriverObject->MajorFunction[IRP_MJ_CREATE] = NtfsFsdDispatch; DriverObject->MajorFunction[IRP_MJ_CLOSE] = NtfsFsdDispatch; DriverObject->MajorFunction[IRP_MJ_READ] = NtfsFsdDispatch; DriverObject->MajorFunction[IRP_MJ_WRITE] = NtfsFsdDispatch; Modified: trunk/reactos/drivers/filesystems/ntfs/ntfs.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/n… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/ntfs.h [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/ntfs.h [iso-8859-1] Mon May 25 11:58:23 2015 @@ -527,10 +527,8 @@ /* create.c */ -DRIVER_DISPATCH NtfsFsdCreate; -NTSTATUS NTAPI -NtfsFsdCreate(PDEVICE_OBJECT DeviceObject, - PIRP Irp); +NTSTATUS +NtfsCreate(PNTFS_IRP_CONTEXT IrpContext); /* devctl.c */

1 0

[pschweitzer] 67902: [NTOSKRNL] Fix a fixme: add a tag for tunnel cache allocations CORE-7272
by pschweitzer＠svn.reactos.org 25 May '15

25 May '15

Author: pschweitzer Date: Mon May 25 11:49:14 2015 New Revision: 67902 URL: http://svn.reactos.org/svn/reactos?rev=67902&view=rev Log: [NTOSKRNL] Fix a fixme: add a tag for tunnel cache allocations CORE-7272 Modified: trunk/reactos/ntoskrnl/fsrtl/tunnel.c Modified: trunk/reactos/ntoskrnl/fsrtl/tunnel.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/fsrtl/tunnel.c?re… ============================================================================== --- trunk/reactos/ntoskrnl/fsrtl/tunnel.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/fsrtl/tunnel.c [iso-8859-1] Mon May 25 11:49:14 2015 @@ -244,7 +244,7 @@ } /* initialize look aside list */ - ExInitializePagedLookasideList(&TunnelLookasideList, NULL, NULL, 0, DEFAULT_ENTRY_SIZE, 0 /*FIXME*/, TunnelEntries); + ExInitializePagedLookasideList(&TunnelLookasideList, NULL, NULL, 0, DEFAULT_ENTRY_SIZE, 'TunL', TunnelEntries); } LONG

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs