Ros-diffs

ros-diffs@reactos.org

81390 discussions

[janderwald] 35700: - Add constants
by janderwald＠svn.reactos.org 27 Aug '08

27 Aug '08

Author: janderwald Date: Wed Aug 27 10:19:40 2008 New Revision: 35700 URL: http://svn.reactos.org/svn/reactos?rev=35700&view=rev Log: - Add constants Modified: trunk/reactos/include/psdk/ipifcons.h Modified: trunk/reactos/include/psdk/ipifcons.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/ipifcons.h?re… ============================================================================== --- trunk/reactos/include/psdk/ipifcons.h [iso-8859-1] (original) +++ trunk/reactos/include/psdk/ipifcons.h [iso-8859-1] Wed Aug 27 10:19:40 2008 @@ -30,4 +30,9 @@ #define MIB_IF_TYPE_PPP 23 #define MIB_IF_TYPE_LOOPBACK 24 #define MIB_IF_TYPE_SLIP 28 + +#define IF_TYPE_OTHER 1 +#define IF_TYPE_ETHERNET_CSMACD 6 +#define IF_TYPE_IEEE80211 71 + #endif /* _IPIFCONS_H */

1 0

[jmorlan] 35699: Fix StretchBlt ugliness in qemu. Patch by grschneider.
by jmorlan＠svn.reactos.org 27 Aug '08

27 Aug '08

Author: jmorlan Date: Wed Aug 27 09:47:49 2008 New Revision: 35699 URL: http://svn.reactos.org/svn/reactos?rev=35699&view=rev Log: Fix StretchBlt ugliness in qemu. Patch by grschneider. Modified: trunk/reactos/subsystems/win32/win32k/dib/dib16bpp.c Modified: trunk/reactos/subsystems/win32/win32k/dib/dib16bpp.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/di… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/dib/dib16bpp.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/dib/dib16bpp.c [iso-8859-1] Wed Aug 27 09:47:49 2008 @@ -466,18 +466,6 @@ /* 16-bit HiColor (565 format) */ __inline PIXEL average16(PIXEL a, PIXEL b) { -// This one doesn't work -/* - if (a == b) - { - return a; - } - else - { - unsigned short mask = ~ (((a | b) & 0x0410) << 1); - return ((a & mask) + (b & mask)) >> 1; - }*/ /* if */ - // This one should be correct, but it's too long /* unsigned char r1, g1, b1, r2, g2, b2, rr, gr, br; @@ -499,8 +487,11 @@ return res; */ - // This one is the short form of the correct one ;-) - return (((a ^ b) & 0xf7deU) >> 1) + (a & b); + // This one is the short form of the correct one, but does not work for QEMU (expects 555 format) + //return (((a ^ b) & 0xf7deU) >> 1) + (a & b); + + //hack until short version works properly + return a; } //NOTE: If you change something here, please do the same in other dibXXbpp.c files!

1 0

[arty] 35698: Replace my rtl/bitmap.c change. Completely replace my section.c adaptations. Some WIP in cache. Much better, not perfect yet.
by arty＠svn.reactos.org 27 Aug '08

27 Aug '08

Author: arty Date: Wed Aug 27 09:25:09 2008 New Revision: 35698 URL: http://svn.reactos.org/svn/reactos?rev=35698&view=rev Log: Replace my rtl/bitmap.c change. Completely replace my section.c adaptations. Some WIP in cache. Much better, not perfect yet. Modified: branches/arty-newcc/lib/rtl/bitmap.c branches/arty-newcc/ntoskrnl/cache/cachesub.c branches/arty-newcc/ntoskrnl/cache/fssup.c branches/arty-newcc/ntoskrnl/cache/pinsup.c branches/arty-newcc/ntoskrnl/include/internal/cc.h branches/arty-newcc/ntoskrnl/mm/section.c Modified: branches/arty-newcc/lib/rtl/bitmap.c URL: http://svn.reactos.org/svn/reactos/branches/arty-newcc/lib/rtl/bitmap.c?rev… ============================================================================== --- branches/arty-newcc/lib/rtl/bitmap.c [iso-8859-1] (original) +++ branches/arty-newcc/lib/rtl/bitmap.c [iso-8859-1] Wed Aug 27 09:25:09 2008 @@ -867,14 +867,14 @@ RtlTestBit(PRTL_BITMAP BitMapHeader, ULONG BitNumber) { - PULONG Ptr; + PUCHAR Ptr; if (BitNumber >= BitMapHeader->SizeOfBitMap) return FALSE; - Ptr = (PULONG)BitMapHeader->Buffer + (BitNumber / 32); - - return (BOOLEAN)(*Ptr & (1 << (BitNumber % 32))); + Ptr = (PUCHAR)BitMapHeader->Buffer + (BitNumber / 8); + + return (BOOLEAN)(*Ptr & (1 << (BitNumber % 8))); } /* Modified: branches/arty-newcc/ntoskrnl/cache/cachesub.c URL: http://svn.reactos.org/svn/reactos/branches/arty-newcc/ntoskrnl/cache/cache… ============================================================================== --- branches/arty-newcc/ntoskrnl/cache/cachesub.c [iso-8859-1] (original) +++ branches/arty-newcc/ntoskrnl/cache/cachesub.c [iso-8859-1] Wed Aug 27 09:25:09 2008 @@ -185,6 +185,16 @@ LARGE_INTEGER ToWrite = *FileOffset; IO_STATUS_BLOCK IOSB; + if (!SectionObjectPointer->SharedCacheMap) + { + if (IoStatus) + { + IoStatus->Status = STATUS_SUCCESS; + IoStatus->Information = 0; + } + return; + } + BOOLEAN Result = CcpMapData ((PNOCC_CACHE_MAP)SectionObjectPointer->SharedCacheMap, FileOffset, @@ -227,6 +237,48 @@ IoStatus->Status = IOSB.Status; IoStatus->Information = 0; } +} + +BOOLEAN +NTAPI +CcFlushImageSection +(PSECTION_OBJECT_POINTERS SectionObjectPointer, + MMFLUSH_TYPE FlushType) +{ + PNOCC_CACHE_MAP Map = (PNOCC_CACHE_MAP)SectionObjectPointer->SharedCacheMap; + PNOCC_BCB Bcb; + PLIST_ENTRY Entry; + IO_STATUS_BLOCK IOSB; + BOOLEAN Result = TRUE; + + for (Entry = Map->AssociatedBcb.Flink; + Entry != &Map->AssociatedBcb; + Entry = Entry->Flink) + { + Bcb = CONTAINING_RECORD(Entry, NOCC_BCB, ThisFileList); + + if (!Bcb->Dirty) continue; + + switch (FlushType) + { + case MmFlushForDelete: + CcPurgeCacheSection + (Bcb->FileObject->SectionObjectPointer, + &Bcb->FileOffset, + Bcb->Length, + FALSE); + break; + case MmFlushForWrite: + CcFlushCache + (Bcb->FileObject->SectionObjectPointer, + &Bcb->FileOffset, + Bcb->Length, + &IOSB); + break; + } + } + + return Result; } // Always succeeds for us Modified: branches/arty-newcc/ntoskrnl/cache/fssup.c URL: http://svn.reactos.org/svn/reactos/branches/arty-newcc/ntoskrnl/cache/fssup… ============================================================================== --- branches/arty-newcc/ntoskrnl/cache/fssup.c [iso-8859-1] (original) +++ branches/arty-newcc/ntoskrnl/cache/fssup.c [iso-8859-1] Wed Aug 27 09:25:09 2008 @@ -108,8 +108,9 @@ FileObject->SectionObjectPointer->SharedCacheMap = Map; Map->FileObject = FileObject; Map->NumberOfMaps = 0; + Map->FileSizes = *FileSizes; + DPRINT("FileSizes->ValidDataLength %x\n", FileSizes->ValidDataLength.LowPart); InitializeListHead(&Map->AssociatedBcb); - return; } BOOLEAN @@ -174,7 +175,21 @@ CcSetFileSizes(IN PFILE_OBJECT FileObject, IN PCC_FILE_SIZES FileSizes) { - /* Nothing to do */ + PNOCC_CACHE_MAP Map = (PNOCC_CACHE_MAP)FileObject->SectionObjectPointer->SharedCacheMap; + if (!Map) return; + Map->FileSizes = *FileSizes; + DPRINT("FileSizes->ValidDataLength %x\n", FileSizes->ValidDataLength.LowPart); +} + +BOOLEAN +NTAPI +CcGetFileSizes(IN PFILE_OBJECT FileObject, + IN PCC_FILE_SIZES FileSizes) +{ + PNOCC_CACHE_MAP Map = (PNOCC_CACHE_MAP)FileObject->SectionObjectPointer->SharedCacheMap; + if (!Map) return FALSE; + *FileSizes = Map->FileSizes; + return TRUE; } BOOLEAN Modified: branches/arty-newcc/ntoskrnl/cache/pinsup.c URL: http://svn.reactos.org/svn/reactos/branches/arty-newcc/ntoskrnl/cache/pinsu… ============================================================================== --- branches/arty-newcc/ntoskrnl/cache/pinsup.c [iso-8859-1] (original) +++ branches/arty-newcc/ntoskrnl/cache/pinsup.c [iso-8859-1] Wed Aug 27 09:25:09 2008 @@ -454,9 +454,13 @@ if (!SectionObject) { + PNOCC_CACHE_MAP Map = (PNOCC_CACHE_MAP)FileObject->SectionObjectPointer->SharedCacheMap; + ULONG SectionSize = min(CACHE_STRIPE, Map->FileSizes.ValidDataLength.QuadPart - Target.QuadPart); + DPRINT("Allocating a cache stripe at %x:%d\n", + Target.LowPart, SectionSize); Status = CcpAllocateSection (FileObject, - CACHE_STRIPE, + SectionSize, PAGE_READWRITE, &SectionObject); @@ -587,7 +591,17 @@ FALSE, FALSE, NULL); - MmProbeAndLockPages(TheBcb->Pinned, KernelMode, IoReadAccess); + _SEH_TRY + { + MmProbeAndLockPages(TheBcb->Pinned, KernelMode, IoReadAccess); + } + _SEH_HANDLE + { + IoFreeMdl(TheBcb->Pinned); + TheBcb->Pinned = NULL; + Result = FALSE; + } + _SEH_END; } return TRUE; @@ -607,6 +621,58 @@ if (Result) { PNOCC_BCB TheBcb = (PNOCC_BCB)*Bcb; + if (!TheBcb->Pinned) + { + TheBcb->Pinned = IoAllocateMdl + (TheBcb->BaseAddress, + TheBcb->Length, + FALSE, + FALSE, + NULL); + _SEH_TRY + { + MmProbeAndLockPages(TheBcb->Pinned, KernelMode, IoReadAccess); + } + _SEH_HANDLE + { + IoFreeMdl(TheBcb->Pinned); + TheBcb->Pinned = NULL; + Result = FALSE; + } + _SEH_END; + } + } + + return Result; +} + +BOOLEAN +NTAPI +CcPreparePinWrite(IN PFILE_OBJECT FileObject, + IN PLARGE_INTEGER FileOffset, + IN ULONG Length, + IN BOOLEAN Zero, + IN ULONG Flags, + OUT PVOID *Bcb, + OUT PVOID *Buffer) +{ + BOOLEAN GotIt = CcPinMappedData + (FileObject, + FileOffset, + Length, + Flags, + Bcb); + + PNOCC_BCB TheBcb = (PNOCC_BCB)*Bcb; + ULONG Start = TheBcb - CcCacheSections; + + DPRINT("CcPreparePinWrite(#%x)\n", Start); + + if (GotIt) + { + CcCacheSections[Start].Dirty = TRUE; + *Buffer = (PVOID)((PCHAR)TheBcb->BaseAddress + (FileOffset->QuadPart - TheBcb->FileOffset.QuadPart)); + DPRINT("Returning Buffer: %x\n", *Buffer); if (!TheBcb->Pinned) { TheBcb->Pinned = IoAllocateMdl @@ -619,48 +685,6 @@ } } - return Result; -} - -BOOLEAN -NTAPI -CcPreparePinWrite(IN PFILE_OBJECT FileObject, - IN PLARGE_INTEGER FileOffset, - IN ULONG Length, - IN BOOLEAN Zero, - IN ULONG Flags, - OUT PVOID *Bcb, - OUT PVOID *Buffer) -{ - BOOLEAN GotIt = CcPinMappedData - (FileObject, - FileOffset, - Length, - Flags, - Bcb); - - PNOCC_BCB TheBcb = (PNOCC_BCB)*Bcb; - ULONG Start = TheBcb - CcCacheSections; - - DPRINT("CcPreparePinWrite(#%x)\n", Start); - - if (GotIt) - { - CcCacheSections[Start].Dirty = TRUE; - *Buffer = (PVOID)((PCHAR)TheBcb->BaseAddress + (FileOffset->QuadPart - TheBcb->FileOffset.QuadPart)); - DPRINT("Returning Buffer: %x\n", *Buffer); - if (!TheBcb->Pinned) - { - TheBcb->Pinned = IoAllocateMdl - (TheBcb->BaseAddress, - TheBcb->Length, - FALSE, - FALSE, - NULL); - MmProbeAndLockPages(TheBcb->Pinned, KernelMode, IoReadAccess); - } - } - DPRINT("Done\n"); return GotIt; } Modified: branches/arty-newcc/ntoskrnl/include/internal/cc.h URL: http://svn.reactos.org/svn/reactos/branches/arty-newcc/ntoskrnl/include/int… ============================================================================== --- branches/arty-newcc/ntoskrnl/include/internal/cc.h [iso-8859-1] (original) +++ branches/arty-newcc/ntoskrnl/include/internal/cc.h [iso-8859-1] Wed Aug 27 09:25:09 2008 @@ -102,27 +102,6 @@ LONG ActivePrefetches; } PFSN_PREFETCHER_GLOBALS, *PPFSN_PREFETCHER_GLOBALS; -#if 0 -typedef struct _BCB -{ - LIST_ENTRY BcbSegmentListHead; - LIST_ENTRY BcbRemoveListEntry; - BOOLEAN RemoveOnClose; - ULONG TimeStamp; - PFILE_OBJECT FileObject; - ULONG CacheSegmentSize; - LARGE_INTEGER AllocationSize; - LARGE_INTEGER FileSize; - PCACHE_MANAGER_CALLBACKS Callbacks; - PVOID LazyWriteContext; - KSPIN_LOCK BcbLock; - ULONG RefCount; -#if defined(DBG) || defined(KDBG) - BOOLEAN Trace; /* enable extra trace output for this BCB and it's cache segments */ -#endif -} BCB, *PBCB; -#endif - typedef struct _NOCC_BCB { /* Public part */ @@ -150,9 +129,10 @@ typedef struct _NOCC_CACHE_MAP { - LIST_ENTRY AssociatedBcb; - PFILE_OBJECT FileObject; - ULONG NumberOfMaps; + LIST_ENTRY AssociatedBcb; + PFILE_OBJECT FileObject; + ULONG NumberOfMaps; + CC_FILE_SIZES FileSizes; } NOCC_CACHE_MAP, *PNOCC_CACHE_MAP; VOID @@ -188,28 +168,14 @@ NTAPI CcInitCacheZeroPage(VOID); -NTSTATUS -NTAPI -CcRosFlushDirtyPages( - ULONG Target, - PULONG Count -); - -VOID -NTAPI -CcRosDereferenceCache(PFILE_OBJECT FileObject); - -VOID -NTAPI -CcRosReferenceCache(PFILE_OBJECT FileObject); - -VOID -NTAPI -CcRosSetRemoveOnClose(PSECTION_OBJECT_POINTERS SectionObjectPointer); - -NTSTATUS -NTAPI -CcTryToInitializeFileCache(PFILE_OBJECT FileObject); +/* Called by section.c */ +BOOLEAN +NTAPI +CcFlushImageSection(PSECTION_OBJECT_POINTERS SectionObjectPointer, MMFLUSH_TYPE FlushType); + +BOOLEAN +NTAPI +CcGetFileSizes(PFILE_OBJECT FileObject, PCC_FILE_SIZES FileSizes); /* * Macro for generic cache manage bugchecking. Note that this macro assumes Modified: branches/arty-newcc/ntoskrnl/mm/section.c URL: http://svn.reactos.org/svn/reactos/branches/arty-newcc/ntoskrnl/mm/section.… ============================================================================== --- branches/arty-newcc/ntoskrnl/mm/section.c [iso-8859-1] (original) +++ branches/arty-newcc/ntoskrnl/mm/section.c [iso-8859-1] Wed Aug 27 09:25:09 2008 @@ -45,7 +45,7 @@ /* INCLUDES *****************************************************************/ #include <ntoskrnl.h> -#define NDEBUG +//#define NDEBUG #include <internal/debug.h> #include <reactos/exeformat.h> @@ -155,66 +155,79 @@ ULONG Length, PIO_STATUS_BLOCK ReadStatus) { - NTSTATUS Status; - PIRP Irp = NULL; - KEVENT ReadWait; - PDEVICE_OBJECT DeviceObject = MmGetDeviceObjectForFile(FileObject); - PIO_STACK_LOCATION IrpSp; - - DPRINT1 - ("PAGING READ File %wZ Offset %x Length %d\n", - &FileObject->FileName, - FileOffset->u.LowPart, - Length); - - KeInitializeEvent(&ReadWait, NotificationEvent, FALSE); - - Irp = IoBuildAsynchronousFsdRequest - (IRP_MJ_READ, - DeviceObject, - Buffer, - PAGE_SIZE, - FileOffset, - ReadStatus); - - if (!Irp) + NTSTATUS Status; + PIRP Irp = NULL; + KEVENT ReadWait; + PDEVICE_OBJECT DeviceObject = MmGetDeviceObjectForFile(FileObject); + PCHAR BufferWalk; + PHYSICAL_ADDRESS Page; + PIO_STACK_LOCATION IrpSp; + + DPRINT1 + ("PAGING READ File %wZ Offset %x Length %d\n", + &FileObject->FileName, + FileOffset->u.LowPart, + Length); + + KeInitializeEvent(&ReadWait, NotificationEvent, FALSE); + + Irp = IoBuildAsynchronousFsdRequest + (IRP_MJ_READ, + DeviceObject, + Buffer, + Length, + FileOffset, + ReadStatus); + + if (!Irp) + { + return STATUS_NO_MEMORY; + } + + Irp->Flags |= IRP_PAGING_IO | IRP_SYNCHRONOUS_PAGING_IO | IRP_NOCACHE; + + ObReferenceObject(FileObject); + + Irp->UserEvent = &ReadWait; + Irp->Tail.Overlay.OriginalFileObject = FileObject; + Irp->Tail.Overlay.Thread = PsGetCurrentThread(); + IrpSp = IoGetNextIrpStackLocation(Irp); + IrpSp->FileObject = FileObject; + + Status = IoCallDriver(DeviceObject, Irp); + if (Status == STATUS_PENDING) + { + if (!NT_SUCCESS + (KeWaitForSingleObject + (&ReadWait, + Suspended, + KernelMode, + FALSE, + NULL))) { - return STATUS_NO_MEMORY; + DPRINT1("Warning: Failed to wait for synchronous IRP\n"); + ASSERT(FALSE); + ObDereferenceObject(FileObject); + return Status; } - - Irp->Flags |= IRP_PAGING_IO | IRP_SYNCHRONOUS_PAGING_IO | IRP_NOCACHE; - - ObReferenceObject(FileObject); - - Irp->UserEvent = &ReadWait; - Irp->Tail.Overlay.OriginalFileObject = FileObject; - Irp->Tail.Overlay.Thread = PsGetCurrentThread(); - IrpSp = IoGetNextIrpStackLocation(Irp); - IrpSp->FileObject = FileObject; - - Status = IoCallDriver(DeviceObject, Irp); - if (Status == STATUS_PENDING) - { - if (!NT_SUCCESS - (KeWaitForSingleObject - (&ReadWait, - Suspended, - KernelMode, - FALSE, - NULL))) - { - DPRINT1("Warning: Failed to wait for synchronous IRP\n"); - ASSERT(FALSE); - ObDereferenceObject(FileObject); - return Status; - } - } - - ObDereferenceObject(FileObject); - - DPRINT("Paging IO Done: %08x\n", ReadStatus->Status); - - return ReadStatus->Status; + } + + ObDereferenceObject(FileObject); + + DPRINT("Paging IO Done: %08x [%02x %02x %02x %02x ...]\n", ReadStatus->Status, + ((PCHAR)Buffer)[0] & 0xff, + ((PCHAR)Buffer)[1] & 0xff, + ((PCHAR)Buffer)[2] & 0xff, + ((PCHAR)Buffer)[3] & 0xff); + + for (BufferWalk = (PCHAR)Buffer; BufferWalk < (PCHAR)Buffer + Length; BufferWalk += PAGE_SIZE) + { + Page = MmGetPhysicalAddress((PVOID)BufferWalk); + DPRINT("Setting page clean: %x\n", Page.u.LowPart); + //MmSetCleanAllRmaps((PFN_TYPE)(Page.QuadPart >> PAGE_SHIFT)); + } + + return ReadStatus->Status; } NTSTATUS @@ -226,75 +239,82 @@ ULONG Length, PIO_STATUS_BLOCK ReadStatus) { - NTSTATUS Status; - PIRP Irp = NULL; - KEVENT ReadWait; - PDEVICE_OBJECT DeviceObject; - PIO_STACK_LOCATION IrpSp; - - ASSERT(FileObject); - ASSERT(FileOffset); - ASSERT(Buffer); - ASSERT(ReadStatus); - - DeviceObject = MmGetDeviceObjectForFile(FileObject); - - ASSERT(DeviceObject); - - DPRINT1 - ("PAGING WRITE: FileObject %x Offset %x Length %d\n", - &FileObject, - FileOffset->LowPart, - Length); - - KeInitializeEvent(&ReadWait, NotificationEvent, FALSE); - - Irp = IoBuildAsynchronousFsdRequest - (IRP_MJ_WRITE, - DeviceObject, - Buffer, - Length, - FileOffset, - ReadStatus); - - if (!Irp) + NTSTATUS Status; + PIRP Irp = NULL; + KEVENT ReadWait; + PDEVICE_OBJECT DeviceObject; + PCHAR BufferWalk; + PHYSICAL_ADDRESS Page; + PIO_STACK_LOCATION IrpSp; + + ASSERT(FileObject); + ASSERT(FileOffset); + ASSERT(Buffer); + ASSERT(ReadStatus); + + DeviceObject = MmGetDeviceObjectForFile(FileObject); + + ASSERT(DeviceObject); + + DPRINT1 + ("PAGING WRITE: FileObject %x Offset %x Length %d\n", + &FileObject, + FileOffset->LowPart, + Length); + + KeInitializeEvent(&ReadWait, NotificationEvent, FALSE); + + Irp = IoBuildAsynchronousFsdRequest + (IRP_MJ_WRITE, + DeviceObject, + Buffer, + Length, + FileOffset, + ReadStatus); + + if (!Irp) + { + return STATUS_NO_MEMORY; + } + + Irp->Flags |= IRP_PAGING_IO | IRP_SYNCHRONOUS_PAGING_IO | IRP_NOCACHE; + + ObReferenceObject(FileObject); + + Irp->UserEvent = &ReadWait; + Irp->Tail.Overlay.OriginalFileObject = FileObject; + Irp->Tail.Overlay.Thread = PsGetCurrentThread(); + IrpSp = IoGetNextIrpStackLocation(Irp); + IrpSp->FileObject = FileObject; + + Status = IoCallDriver(DeviceObject, Irp); + if (Status == STATUS_PENDING) + { + if (!NT_SUCCESS + (KeWaitForSingleObject + (&ReadWait, + Suspended, + KernelMode, + FALSE, + NULL))) { - return STATUS_NO_MEMORY; + DPRINT1("Warning: Failed to wait for synchronous IRP\n"); + ASSERT(FALSE); + ObDereferenceObject(FileObject); + return Status; } - - Irp->Flags |= IRP_PAGING_IO | IRP_SYNCHRONOUS_PAGING_IO | IRP_NOCACHE; - - ObReferenceObject(FileObject); - - Irp->UserEvent = &ReadWait; - Irp->Tail.Overlay.OriginalFileObject = FileObject; - Irp->Tail.Overlay.Thread = PsGetCurrentThread(); - IrpSp = IoGetNextIrpStackLocation(Irp); - IrpSp->FileObject = FileObject; - - Status = IoCallDriver(DeviceObject, Irp); - if (Status == STATUS_PENDING) - { - if (!NT_SUCCESS - (KeWaitForSingleObject - (&ReadWait, - Suspended, - KernelMode, - FALSE, - NULL))) - { - DPRINT1("Warning: Failed to wait for synchronous IRP\n"); - ASSERT(FALSE); - ObDereferenceObject(FileObject); - return Status; - } - } - - ObDereferenceObject(FileObject); - - DPRINT("Paging IO Done: %08x\n", ReadStatus->Status); - - return ReadStatus->Status; + } + + ObDereferenceObject(FileObject); + + DPRINT("Paging IO Done: %08x\n", ReadStatus->Status); + for (BufferWalk = (PCHAR)Buffer; BufferWalk < (PCHAR)Buffer + Length; BufferWalk += PAGE_SIZE) + { + Page = MmGetPhysicalAddress((PVOID)BufferWalk); + //MmSetCleanAllRmaps((PFN_TYPE)(Page.QuadPart >> PAGE_SHIFT)); + } + + return ReadStatus->Status; } NTSTATUS @@ -457,7 +477,7 @@ { DPRINT1("Image segment %d still referenced (was %d)\n", i, SectionSegments[i].ReferenceCount); - KEBUGCHECK(0); + ASSERT(FALSE); } MmFreePageTablesSectionSegment(&SectionSegments[i]); } @@ -475,7 +495,7 @@ if (Segment->ReferenceCount != 0) { DPRINT1("Data segment still referenced\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } MmFreePageTablesSectionSegment(Segment); ExFreePool(Segment); @@ -523,7 +543,7 @@ TAG_SECTION_PAGE_TABLE); if (Table == NULL) { - KEBUGCHECK(0); + ASSERT(FALSE); } memset(Table, 0, sizeof(SECTION_PAGE_TABLE)); DPRINT("Table %x\n", Table); @@ -576,16 +596,16 @@ if (Entry == 0) { DPRINT1("Entry == 0 for MmSharePageEntrySectionSegment\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } if (SHARE_COUNT_FROM_SSE(Entry) == MAX_SHARE_COUNT) { DPRINT1("Maximum share count reached\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } if (IS_SWAP_FROM_SSE(Entry)) { - KEBUGCHECK(0); + ASSERT(FALSE); } Entry = MAKE_SSE(PAGE_FROM_SSE(Entry), SHARE_COUNT_FROM_SSE(Entry) + 1); MmSetPageEntrySectionSegment(Segment, Offset, Entry); @@ -600,21 +620,22 @@ BOOLEAN PageOut) { ULONG Entry; + BOOLEAN IsDirectMapped = FALSE; Entry = MmGetPageEntrySectionSegment(Segment, Offset); if (Entry == 0) { DPRINT1("Entry == 0 for MmUnsharePageEntrySectionSegment\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } if (SHARE_COUNT_FROM_SSE(Entry) == 0) { DPRINT1("Zero share count for unshare\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } if (IS_SWAP_FROM_SSE(Entry)) { - KEBUGCHECK(0); + ASSERT(FALSE); } Entry = MAKE_SSE(PAGE_FROM_SSE(Entry), SHARE_COUNT_FROM_SSE(Entry) - 1); /* @@ -629,12 +650,33 @@ BOOLEAN IsImageSection; LARGE_INTEGER FileOffset; - FileOffset.QuadPart = Segment->FileOffset.QuadPart + Offset; + FileOffset.QuadPart = Offset + Segment->FileOffset.QuadPart; IsImageSection = Section->AllocationAttributes & SEC_IMAGE ? TRUE : FALSE; Page = PFN_FROM_SSE(Entry); FileObject = Section->FileObject; + if (FileObject != NULL && + !(Segment->Characteristics & IMAGE_SCN_MEM_SHARED)) + { + + if ((FileOffset.QuadPart % PAGE_SIZE) == 0 && + (Offset + PAGE_SIZE <= Segment->RawLength || !IsImageSection)) + { + IO_STATUS_BLOCK IOSB; + IsDirectMapped = TRUE; + CcFlushCache + (FileObject->SectionObjectPointer, + &FileOffset, + PAGE_SIZE, + &IOSB); + if (!NT_SUCCESS(IOSB.Status)) + { + DPRINT1("CcRosUnmapCacheSegment failed, status = %x\n", IOSB.Status); + ASSERT(FALSE); + } + } + } SavedSwapEntry = MmGetSavedSwapEntryPage(Page); if (SavedSwapEntry == 0) @@ -655,6 +697,10 @@ else { MmSetPageEntrySectionSegment(Segment, Offset, 0); + if (!IsDirectMapped) + { + MmReleasePageMemoryConsumer(MC_USER, Page); + } } } else @@ -676,7 +722,7 @@ if (!NT_SUCCESS(Status)) { DPRINT1("MM: Failed to write to swap page (Status was 0x%.8X)\n", Status); - KEBUGCHECK(0); + ASSERT(FALSE); } } MmSetPageEntrySectionSegment(Segment, Offset, MAKE_SWAP_SSE(SavedSwapEntry)); @@ -687,7 +733,7 @@ else { DPRINT1("Found a swapentry for a non private page in an image or data file sgment\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } } } @@ -696,88 +742,6 @@ MmSetPageEntrySectionSegment(Segment, Offset, Entry); } return(SHARE_COUNT_FROM_SSE(Entry) > 0); -} - -NTSTATUS -NTAPI -MiReadPage(PMEMORY_AREA MemoryArea, - PLARGE_INTEGER FileOffset, - PPFN_TYPE Page) -/* - * FUNCTION: Read a page for a section backed memory area. - * PARAMETERS: - * MemoryArea - Memory area to read the page for. - * Offset - Offset of the page to read. - * Page - Variable that receives a page contains the read data. - */ -{ - PFILE_OBJECT FileObject; - NTSTATUS Status = STATUS_SUCCESS; - BOOLEAN IsImageSection; - PVOID Buffer = NULL; - IO_STATUS_BLOCK ReadStatus; - FILE_STANDARD_INFORMATION FileInfo; - PMM_SECTION_SEGMENT Segment; - PROS_SECTION_OBJECT Section; - - *Page = 0; - - Segment = MemoryArea->Data.SectionData.Segment; - Section = MemoryArea->Data.SectionData.Section; - FileObject = Section->FileObject; - ASSERT(FileObject); - IsImageSection = Section->AllocationAttributes & SEC_IMAGE ? TRUE : FALSE; - - DPRINT("MiReadPage: FileObject %x, Offset %x\n", FileObject, FileOffset->LowPart); - - Status = MmRequestPageMemoryConsumer(MC_USER, FALSE, Page); - if (!NT_SUCCESS(Status)) - { - return Status; - } - - Buffer = MmCreateHyperspaceMapping(*Page); - if (!Buffer) - { - Status = STATUS_NO_MEMORY; - goto cleanup; - } - - DPRINT("Read File Name:[%wZ] At:%x\n", &FileObject->FileName, FileOffset->LowPart); - - /* Delayed section info, if needed */ - if (Section->MaximumSize.QuadPart == ~0ll) - { - Status = IoQueryFileInformation - (FileObject, - FileStandardInformation, - sizeof(FILE_STANDARD_INFORMATION), - &FileInfo, - &ReadStatus.Information); - - if (NT_SUCCESS(Status)) - { - Section->MaximumSize = FileInfo.EndOfFile; - Segment->RawLength = FileInfo.EndOfFile.QuadPart - Segment->FileOffset.QuadPart; - Segment->Length = PAGE_ROUND_UP(Segment->RawLength); - } - } - - /* - * If the cache segment isn't up to date then call the file - * system to read in the data. - */ - - Status = MiSimpleRead(FileObject, FileOffset, Buffer, max(PAGE_SIZE, Segment->FileOffset.QuadPart + Segment->RawLength - FileOffset->QuadPart), &ReadStatus); - DPRINT("MiSimpleRead: Status %08x\n", Status); - -cleanup: - if (Buffer) MmDeleteHyperspaceMapping(Buffer); - if (!NT_SUCCESS(Status) && *Page) MmReleasePageMemoryConsumer(MC_USER, *Page); - - DPRINT("Finished MiReadPage: Status %x\n", Status); - - return Status; } NTSTATUS @@ -850,7 +814,7 @@ if (PageOp == NULL) { DPRINT1("MmGetPageOp failed\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } /* @@ -868,12 +832,12 @@ if (Status != STATUS_SUCCESS) { DPRINT1("Failed to wait for page op, status = %x\n", Status); - KEBUGCHECK(0); + ASSERT(FALSE); } if (PageOp->Status == STATUS_PENDING) { DPRINT1("Woke for page op before completion\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } MmLockAddressSpace(AddressSpace); /* @@ -931,7 +895,7 @@ if (!NT_SUCCESS(Status)) { DPRINT1("Unable to create virtual mapping\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } MmInsertRmap(Page, Process, (PVOID)PAddress); } @@ -945,6 +909,16 @@ DPRINT("Address 0x%.8X\n", Address); return(STATUS_SUCCESS); } + else + { + Status = MmRequestPageMemoryConsumer(MC_USER, TRUE, &Page); + if (!NT_SUCCESS(Status)) + { + DPRINT("Didn't get a page\n"); + ASSERT(FALSE); + return Status; + } + } HasSwapEntry = MmIsPageSwapEntry(Process, (PVOID)PAddress); if (HasSwapEntry) @@ -960,7 +934,7 @@ if (Segment->Flags & MM_PAGEFILE_SEGMENT) { DPRINT1("Found a swaped out private page in a pagefile section.\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } MmUnlockSectionSegment(Segment); @@ -970,14 +944,14 @@ Status = MmRequestPageMemoryConsumer(MC_USER, TRUE, &Page); if (!NT_SUCCESS(Status)) { - KEBUGCHECK(0); + ASSERT(FALSE); } Status = MmReadFromSwapPage(SwapEntry, Page); if (!NT_SUCCESS(Status)) { DPRINT1("MmReadFromSwapPage failed, status = %x\n", Status); - KEBUGCHECK(0); + ASSERT(FALSE); } MmLockAddressSpace(AddressSpace); Status = MmCreateVirtualMapping(Process, @@ -988,7 +962,7 @@ if (!NT_SUCCESS(Status)) { DPRINT("MmCreateVirtualMapping failed, not out of memory\n"); - KEBUGCHECK(0); + ASSERT(FALSE); return(Status); } @@ -1033,7 +1007,7 @@ if (!NT_SUCCESS(Status)) { DPRINT("MmCreateVirtualMappingUnsafe failed, not out of memory\n"); - KEBUGCHECK(0); + ASSERT(FALSE); return(Status); } /* @@ -1069,7 +1043,7 @@ } if (!NT_SUCCESS(Status)) { - KEBUGCHECK(0); + ASSERT(FALSE); } Status = MmCreateVirtualMapping(Process, Address, @@ -1079,7 +1053,7 @@ if (!NT_SUCCESS(Status)) { DPRINT("MmCreateVirtualMapping failed, not out of memory\n"); - KEBUGCHECK(0); + ASSERT(FALSE); return(Status); } MmInsertRmap(Page, Process, (PVOID)PAddress); @@ -1115,7 +1089,7 @@ MmUnlockSectionSegment(Segment); MmUnlockAddressSpace(AddressSpace); - if ((Segment->Flags & MM_PAGEFILE_SEGMENT) || + if ((Segment->Flags & MM_PAGEFILE_SEGMENT) || (Offset >= PAGE_ROUND_UP(Segment->RawLength) && Section->AllocationAttributes & SEC_IMAGE)) { Status = MmRequestPageMemoryConsumer(MC_USER, TRUE, &Page); @@ -1126,17 +1100,37 @@ } else { - LARGE_INTEGER ReadOffset; - ReadOffset.QuadPart = - Segment->FileOffset.QuadPart + - Offset; - Status = MiReadPage(MemoryArea, &ReadOffset, &Page); + PVOID Buffer = MmCreateHyperspaceMapping(Page); + ULONG RemainingLength; + LARGE_INTEGER TotalOffset; + IO_STATUS_BLOCK IOSB; + + TotalOffset.QuadPart = Segment->FileOffset.QuadPart + Offset; + RemainingLength = Segment->RawLength - Offset; + + if ((RemainingLength > PAGE_SIZE) || (Segment->Length == 0)) + RemainingLength = PAGE_SIZE; + + DPRINT("Segment->RawLength %d\n", Segment->RawLength); + DPRINT("Segment->Length %d\n", Segment->Length); + DPRINT("Reading at offset %x (relative %x)\n", TotalOffset.LowPart, Offset); + DPRINT("RemainingLength %x\n", RemainingLength); + + Status = MiSimpleRead + (Section->FileObject, + &TotalOffset, + Buffer, + RemainingLength, + &IOSB); + if (!NT_SUCCESS(Status)) { DPRINT1("MiReadPage failed (Status %x)\n", Status); } - DPRINT("PAGING IO DONE: %08x\n", Status); - } + + MmDeleteHyperspaceMapping(Buffer); + } + if (!NT_SUCCESS(Status)) { /* @@ -1165,7 +1159,7 @@ if (Entry != Entry1) { DPRINT1("Someone changed ppte entry while we slept\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } /* @@ -1184,7 +1178,7 @@ if (!NT_SUCCESS(Status)) { DPRINT1("Unable to create virtual mapping\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } MmInsertRmap(Page, Process, (PVOID)PAddress); @@ -1213,13 +1207,13 @@ Status = MmRequestPageMemoryConsumer(MC_USER, TRUE, &Page); if (!NT_SUCCESS(Status)) { - KEBUGCHECK(0); + ASSERT(FALSE); } Status = MmReadFromSwapPage(SwapEntry, Page); if (!NT_SUCCESS(Status)) { - KEBUGCHECK(0); + ASSERT(FALSE); } /* @@ -1236,7 +1230,7 @@ if (Entry != Entry1) { DPRINT1("Someone changed ppte entry while we slept\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } /* @@ -1259,7 +1253,7 @@ if (!NT_SUCCESS(Status)) { DPRINT1("Unable to create virtual mapping\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } MmInsertRmap(Page, Process, (PVOID)PAddress); if (Locked) @@ -1291,7 +1285,7 @@ if (!NT_SUCCESS(Status)) { DPRINT1("Unable to create virtual mapping\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } MmInsertRmap(Page, Process, (PVOID)PAddress); if (Locked) @@ -1385,7 +1379,7 @@ if (PageOp == NULL) { DPRINT1("MmGetPageOp failed\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } /* @@ -1401,12 +1395,12 @@ if (Status == STATUS_TIMEOUT) { DPRINT1("Failed to wait for page op, status = %x\n", Status); - KEBUGCHECK(0); + ASSERT(FALSE); } if (PageOp->Status == STATUS_PENDING) { DPRINT1("Woke for page op before completion\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } /* * Restart the operation @@ -1428,7 +1422,7 @@ Status = MmRequestPageMemoryConsumer(MC_USER, TRUE, &NewPage); if (!NT_SUCCESS(Status)) { - KEBUGCHECK(0); + ASSERT(FALSE); } /* @@ -1453,13 +1447,13 @@ if (!NT_SUCCESS(Status)) { DPRINT("MmCreateVirtualMapping failed, not out of memory\n"); - KEBUGCHECK(0); + ASSERT(FALSE); return(Status); } if (!NT_SUCCESS(Status)) { DPRINT1("Unable to create virtual mapping\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } if (Locked) { @@ -1541,6 +1535,7 @@ LARGE_INTEGER FileOffset; NTSTATUS Status; PFILE_OBJECT FileObject; + BOOLEAN DirectMapped; BOOLEAN IsImageSection; PEPROCESS Process = MmGetAddressSpaceOwner(AddressSpace); @@ -1552,13 +1547,27 @@ Context.Segment = MemoryArea->Data.SectionData.Segment; Context.Section = MemoryArea->Data.SectionData.Section; - Context.Offset = (ULONG_PTR)Address - (ULONG_PTR)MemoryArea->StartingAddress - + MemoryArea->Data.SectionData.ViewOffset; - FileOffset.QuadPart = Context.Segment->FileOffset.QuadPart + Context.Offset; + Context.Offset = (ULONG_PTR)Address - (ULONG_PTR)MemoryArea->StartingAddress; + FileOffset.QuadPart = Context.Offset + Context.Segment->FileOffset.QuadPart; IsImageSection = Context.Section->AllocationAttributes & SEC_IMAGE ? TRUE : FALSE; FileObject = Context.Section->FileObject; + if (FileObject != NULL && + !(Context.Segment->Characteristics & IMAGE_SCN_MEM_SHARED)) + { + /* + * If the file system is letting us go directly to the cache and the + * memory area was mapped at an offset in the file which is page aligned + * then note this is a direct mapped page. + */ + if ((FileOffset.QuadPart % PAGE_SIZE) == 0 && + (Context.Offset + PAGE_SIZE <= Context.Segment->RawLength || !IsImageSection)) + { + DirectMapped = TRUE; + } + } + /* * This should never happen since mappings of physical memory are never @@ -1569,7 +1578,7 @@ DPRINT1("Trying to page out from physical memory section address 0x%X " "process %d\n", Address, Process ? Process->UniqueProcessId : 0); - KEBUGCHECK(0); + ASSERT(FALSE); } /* @@ -1580,7 +1589,7 @@ { DPRINT1("Trying to page out not-present page at (%d,0x%.8X).\n", Process ? Process->UniqueProcessId : 0, Address); - KEBUGCHECK(0); + ASSERT(FALSE); } Page = MmGetPfnForProcess(Process, Address); SwapEntry = MmGetSavedSwapEntryPage(Page); @@ -1600,6 +1609,7 @@ Context.Private = FALSE; } + MmReferencePage(Page); MmDeleteAllRmaps(Page, (PVOID)&Context, MmPageOutDeleteMapping); /* @@ -1611,7 +1621,7 @@ if (!(Context.Segment->Flags & MM_PAGEFILE_SEGMENT) && !(Context.Segment->Characteristics & IMAGE_SCN_MEM_SHARED)) { - KEBUGCHECK(0); + ASSERT(FALSE); } } @@ -1629,7 +1639,7 @@ { DPRINT1("Found a %s private page (address %x) in a pagefile segment.\n", Context.WasDirty ? "dirty" : "clean", Address); - KEBUGCHECK(0); + ASSERT(FALSE); } if (!Context.WasDirty && SwapEntry != 0) { @@ -1647,7 +1657,7 @@ { DPRINT1("Found a %s private page (address %x) in a shared section segment.\n", Context.WasDirty ? "dirty" : "clean", Address); - KEBUGCHECK(0); + ASSERT(FALSE); } if (!Context.WasDirty || SwapEntry != 0) { @@ -1662,6 +1672,45 @@ return(STATUS_SUCCESS); } } + else if (!Context.Private && DirectMapped) + { + IO_STATUS_BLOCK IOSB; + + if (SwapEntry != 0) + { + DPRINT1("Found a swapentry for a non private and direct mapped page (address %x)\n", + Address); + ASSERT(FALSE); + } + CcFlushCache + (FileObject->SectionObjectPointer, + &FileOffset, + PAGE_SIZE, + &IOSB); + Status = IOSB.Status; + if (!NT_SUCCESS(Status)) + { + DPRINT1("CCRosUnmapCacheSegment failed, status = %x\n", Status); + ASSERT(FALSE); + } + MmReleasePageMemoryConsumer(MC_CACHE, Page); + PageOp->Status = STATUS_SUCCESS; + MmspCompleteAndReleasePageOp(PageOp); + return(STATUS_SUCCESS); + } + else if (!Context.WasDirty && !DirectMapped && !Context.Private) + { + if (SwapEntry != 0) + { + DPRINT1("Found a swap entry for a non dirty, non private and not direct mapped page (address %x)\n", + Address); + ASSERT(FALSE); + } + MmReleasePageMemoryConsumer(MC_USER, Page); + PageOp->Status = STATUS_SUCCESS; + MmspCompleteAndReleasePageOp(PageOp); + return(STATUS_SUCCESS); + } else if (!Context.WasDirty && Context.Private && SwapEntry != 0) { MmSetSavedSwapEntryPage(Page, 0); @@ -1672,7 +1721,7 @@ MmUnlockAddressSpace(AddressSpace); if (!NT_SUCCESS(Status)) { - KEBUGCHECK(0); + ASSERT(FALSE); } MmReleasePageMemoryConsumer(MC_USER, Page); PageOp->Status = STATUS_SUCCESS; @@ -1800,7 +1849,7 @@ MmUnlockAddressSpace(AddressSpace); if (!NT_SUCCESS(Status)) { - KEBUGCHECK(0); + ASSERT(FALSE); } } else @@ -1822,6 +1871,7 @@ PMM_PAGEOP PageOp) { ULONG Offset; + LARGE_INTEGER TotalOffset; PROS_SECTION_OBJECT Section; PMM_SECTION_SEGMENT Segment; PFN_TYPE Page; @@ -1830,22 +1880,38 @@ BOOLEAN Private; NTSTATUS Status; PFILE_OBJECT FileObject; + BOOLEAN DirectMapped; BOOLEAN IsImageSection; PEPROCESS Process = MmGetAddressSpaceOwner(AddressSpace); Address = (PVOID)PAGE_ROUND_DOWN(Address); - Offset = (ULONG_PTR)Address - (ULONG_PTR)MemoryArea->StartingAddress - + MemoryArea->Data.SectionData.ViewOffset; + Offset = (ULONG_PTR)Address - (ULONG_PTR)MemoryArea->StartingAddress; /* * Get the segment and section. */ Segment = MemoryArea->Data.SectionData.Segment; + TotalOffset.QuadPart = Segment->FileOffset.QuadPart + Offset; Section = MemoryArea->Data.SectionData.Section; IsImageSection = Section->AllocationAttributes & SEC_IMAGE ? TRUE : FALSE; FileObject = Section->FileObject; + DirectMapped = FALSE; + if (FileObject != NULL && + !(Segment->Characteristics & IMAGE_SCN_MEM_SHARED)) + { + /* + * If the file system is letting us go directly to the cache and the + * memory area was mapped at an offset in the file which is page aligned + * then note this is a direct mapped page. + */ + if (((Offset + Segment->FileOffset.QuadPart) % PAGE_SIZE) == 0 && + (Offset + PAGE_SIZE <= Segment->RawLength || !IsImageSection)) + { + DirectMapped = TRUE; + } + } /* * This should never happen since mappings of physical memory are never @@ -1856,7 +1922,7 @@ DPRINT1("Trying to write back page from physical memory mapped at %X " "process %d\n", Address, Process ? Process->UniqueProcessId : 0); - KEBUGCHECK(0); + ASSERT(FALSE); } /* @@ -1867,7 +1933,7 @@ { DPRINT1("Trying to page out not-present page at (%d,0x%.8X).\n", Process ? Process->UniqueProcessId : 0, Address); - KEBUGCHECK(0); + ASSERT(FALSE); } Page = MmGetPfnForProcess(Process, Address); SwapEntry = MmGetSavedSwapEntryPage(Page); @@ -1890,6 +1956,23 @@ * Speculatively set all mappings of the page to clean. */ MmSetCleanAllRmaps(Page); + + /* + * If this page was direct mapped from the cache then the cache manager + * will take care of writing it back to disk. + */ + if (DirectMapped && !Private) + { + ASSERT(SwapEntry == 0); + CcFlushCache + (FileObject->SectionObjectPointer, + &TotalOffset, + PAGE_SIZE, + NULL); + PageOp->Status = STATUS_SUCCESS; + MmspCompleteAndReleasePageOp(PageOp); + return(STATUS_SUCCESS); + } /* * If necessary, allocate an entry in the paging file for this page @@ -1972,7 +2055,7 @@ PFN_TYPE Page; Offset = (ULONG_PTR)Address - (ULONG_PTR)MemoryArea->StartingAddress - + MemoryArea->Data.SectionData.ViewOffset; + + MemoryArea->Data.SectionData.ViewOffset; Entry = MmGetPageEntrySectionSegment(Segment, Offset); Page = MmGetPfnForProcess(Process, Address); @@ -2017,7 +2100,6 @@ if ((MemoryArea->Flags & SEC_NO_CHANGE) && Region->Protect != Protect) { - CHECKPOINT1; return STATUS_INVALID_PAGE_PROTECTION; } @@ -2222,7 +2304,7 @@ if (!NT_SUCCESS(Status)) { DPRINT1("Failed to create PhysicalMemory section\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } Status = ObInsertObject(PhysSection, NULL, @@ -2353,12 +2435,14 @@ */ { PROS_SECTION_OBJECT Section; + NTSTATUS Status; LARGE_INTEGER MaximumSize; PMM_SECTION_SEGMENT Segment; - NTSTATUS Status; ULONG FileAccess; - - DPRINT("MmCreateDataFileSection [%wZ]\n", &FileObject->FileName); + IO_STATUS_BLOCK Iosb; + CC_FILE_SIZES FileSizes; + FILE_STANDARD_INFORMATION FileInfo; + KIRQL OldIrql; /* * Create the section @@ -2374,7 +2458,6 @@ (PVOID*)(PVOID)&Section); if (!NT_SUCCESS(Status)) { - DPRINT("Failed: %08x\n", Status); return(Status); } /* @@ -2398,6 +2481,46 @@ } /* + * Reference the file handle + */ + ObReferenceObject(FileObject); + + /* A hack: If we're cached, we can overcome deadlocking with the upper + * layer filesystem call by retriving the object sizes from the cache + * which is made to keep track. If I had to guess, they were figuring + * out a similar problem. + */ + if (!CcGetFileSizes(FileObject, &FileSizes)) + { + /* This means the file isn't cached (yet) ... consequently, this + * call is not recursive from the cache. */ + /* + * FIXME: This is propably not entirely correct. We can't look into + * the standard FCB header because it might not be initialized yet + * (as in case of the EXT2FS driver by Manoj Paul Joseph where the + * standard file information is filled on first request). + */ + DPRINT("IoQueryFileInformation\n"); + Status = IoQueryFileInformation + (FileObject, + FileStandardInformation, + sizeof(FILE_STANDARD_INFORMATION), + &FileInfo, + &Iosb.Information); + DPRINT("IoQueryFileInformation -> %08x\n", Status); + + if (!NT_SUCCESS(Status)) + { + ObDereferenceObject(Section); + ObDereferenceObject(FileObject); + return Status; + } + + FileSizes.ValidDataLength = FileInfo.EndOfFile; + FileSizes.FileSize = FileInfo.EndOfFile; + } + + /* * FIXME: Revise this once a locking order for file size changes is * decided */ @@ -2407,52 +2530,68 @@ } else { - MaximumSize.QuadPart = ~0ll; - } - - /* - * Lock the file - */ - Status = MmspWaitForFileLock(FileObject); - if (Status != STATUS_SUCCESS) - { - ObDereferenceObject(Section); - ObDereferenceObject(FileObject); - DPRINT("Failed: %08x\n", Status); - return(Status); - } - + MaximumSize = FileSizes.ValidDataLength; + /* Mapping zero-sized files isn't allowed. */ + if (MaximumSize.QuadPart == 0) + { + ObDereferenceObject(Section); + ObDereferenceObject(FileObject); + return STATUS_FILE_INVALID; + } + } + + if (MaximumSize.QuadPart > FileSizes.ValidDataLength.QuadPart) + { + DPRINT("IoSetInformation expanding from %x to %x\n", + MaximumSize.LowPart, FileSizes.ValidDataLength.LowPart); + Status = IoSetInformation(FileObject, + FileAllocationInformation, + sizeof(LARGE_INTEGER), + &MaximumSize); + DPRINT("IoSetInformation -> %08x\n", Status); + + if (!NT_SUCCESS(Status)) + { + ObDereferenceObject(Section); + ObDereferenceObject(FileObject); + return(STATUS_SECTION_NOT_EXTENDED); + } + } + + Segment = ExAllocatePoolWithTag(NonPagedPool, sizeof(MM_SECTION_SEGMENT), + TAG_MM_SECTION_SEGMENT); + if (Segment == NULL) + { + ObDereferenceObject(Section); + ObDereferenceObject(FileObject); + return(STATUS_NO_MEMORY); + } + + ExInitializeFastMutex(&Segment->Lock); + Segment->ReferenceCount = 1; + Section->Segment = Segment; + + KeAcquireSpinLock(&FileObject->IrpListLock, &OldIrql); /* * If this file hasn't been mapped as a data file before then allocate a * section segment to describe the data file mapping */ if (FileObject->SectionObjectPointer->DataSectionObject == NULL) { - Segment = ExAllocatePoolWithTag(NonPagedPool, sizeof(MM_SECTION_SEGMENT), - TAG_MM_SECTION_SEGMENT); - if (Segment == NULL) - { - //KeSetEvent((PVOID)&FileObject->Lock, IO_NO_INCREMENT, FALSE); - ObDereferenceObject(Section); - ObDereferenceObject(FileObject); - DPRINT("Failed: STATUS_NO_MEMORY\n"); - return(STATUS_NO_MEMORY); - } - Section->Segment = Segment; - Segment->ReferenceCount = 1; - ExInitializeFastMutex(&Segment->Lock); + FileObject->SectionObjectPointer->DataSectionObject = (PVOID)Segment; + KeReleaseSpinLock(&FileObject->IrpListLock, OldIrql); + /* * Set the lock before assigning the segment to the file object */ ExAcquireFastMutex(&Segment->Lock); - FileObject->SectionObjectPointer->DataSectionObject = (PVOID)Segment; - + + DPRINT("Filling out Segment info (No previous data section)\n"); Segment->FileOffset.QuadPart = 0; Segment->Protection = SectionPageProtection; Segment->Flags = MM_DATAFILE_SEGMENT; Segment->Characteristics = 0; Segment->WriteCopy = FALSE; - if (AllocationAttributes & SEC_RESERVE) { Segment->Length = Segment->RawLength = 0; @@ -2460,13 +2599,18 @@ else { Segment->RawLength = MaximumSize.u.LowPart; - Segment->Length = MaximumSize.u.LowPart; + Segment->Length = PAGE_ROUND_UP(Segment->RawLength); } Segment->VirtualAddress = 0; RtlZeroMemory(&Segment->PageDirectory, sizeof(SECTION_PAGE_DIRECTORY)); } else { + KeReleaseSpinLock(&FileObject->IrpListLock, OldIrql); + ExFreePool(Segment); + + DPRINT("Filling out Segment info (previous data section)\n"); + /* * If the file is already mapped as a data file then we may need * to extend it @@ -2476,6 +2620,7 @@ DataSectionObject; Section->Segment = Segment; (void)InterlockedIncrementUL(&Segment->ReferenceCount); + MmLockSectionSegment(Segment); if (MaximumSize.u.LowPart > Segment->RawLength && @@ -2485,11 +2630,15 @@ Segment->Length = PAGE_ROUND_UP(Segment->RawLength); } } + MmUnlockSectionSegment(Segment); + Section->FileObject = FileObject; Section->MaximumSize = MaximumSize; + + //KeSetEvent((PVOID)&FileObject->Lock, IO_NO_INCREMENT, FALSE); + *SectionObject = Section; - DPRINT("Success\n"); return(STATUS_SUCCESS); } @@ -2564,20 +2713,19 @@ OUT PULONG ReadSize) { NTSTATUS Status; - PFILE_OBJECT FileObject = (PFILE_OBJECT)File; LARGE_INTEGER FileOffset; ULONG AdjustOffset; ULONG OffsetAdjustment; ULONG BufferSize; ULONG UsedSize; PVOID Buffer; - IO_STATUS_BLOCK ReadStatus; - + IO_STATUS_BLOCK Iosb; + ASSERT_IRQL_LESS(DISPATCH_LEVEL); if(Length == 0) { - KEBUGCHECK(STATUS_INVALID_PARAMETER_4); + ASSERT(FALSE); } FileOffset = *Offset; @@ -2585,7 +2733,7 @@ /* Negative/special offset: it cannot be used in this context */ if(FileOffset.u.HighPart < 0) { - KEBUGCHECK(STATUS_INVALID_PARAMETER_5); + ASSERT(FALSE); } AdjustOffset = PAGE_ROUND_DOWN(FileOffset.u.LowPart); @@ -2607,26 +2755,21 @@ UsedSize = 0; - Status = MiSimpleRead(FileObject, &FileOffset, Buffer, Length, &ReadStatus); - DPRINT("MiSimpleRead: Status %08x\n", Status); - - UsedSize = ReadStatus.Information; - - DPRINT("ExeFmtpReadFile -> %x:%x\n", Status, UsedSize); + Status = MiSimpleRead(File, &FileOffset, Buffer, BufferSize, &Iosb); if(NT_SUCCESS(Status) && UsedSize < OffsetAdjustment) { Status = STATUS_IN_PAGE_ERROR; ASSERT(!NT_SUCCESS(Status)); } + + UsedSize = Iosb.Information; if(NT_SUCCESS(Status)) { *Data = (PVOID)((ULONG_PTR)Buffer + OffsetAdjustment); *AllocBase = Buffer; *ReadSize = UsedSize - OffsetAdjustment; - DPRINT("Data @ %x Buffer @ %x Read Size %d\n", *Data, *AllocBase, *ReadSize); - DPRINT("First couple of bytes: %02x%02x\n", ((PCHAR)Data)[0] & 0xff, ((PCHAR)Data)[1] & 0xff); } else { @@ -2914,8 +3057,8 @@ /* Unaligned segments must be contiguous within the file */ if (Segment->FileOffset.QuadPart != - (EffectiveSegment->FileOffset.QuadPart + - EffectiveSegment->RawLength)) + (EffectiveSegment->FileOffset.QuadPart + + EffectiveSegment->RawLength)) { return FALSE; } @@ -3001,6 +3144,8 @@ &FileHeaderBuffer, &FileHeaderSize); + DPRINT("ExeFmtpReadFile %x\n", Status); + if (!NT_SUCCESS(Status)) return Status; @@ -3068,10 +3213,10 @@ if(ImageSectionObject->ImageBase == 0) { - if(ImageSectionObject->ImageCharacteristics & IMAGE_FILE_DLL) - ImageSectionObject->ImageBase = 0x10000000; - else - ImageSectionObject->ImageBase = 0x00400000; + if(ImageSectionObject->ImageCharacteristics & IMAGE_FILE_DLL) + ImageSectionObject->ImageBase = 0x10000000; + else + ImageSectionObject->ImageBase = 0x00400000; } /* @@ -3116,7 +3261,7 @@ { ExInitializeFastMutex(&ImageSectionObject->Segments[i].Lock); ImageSectionObject->Segments[i].ReferenceCount = 1; - + RtlZeroMemory(&ImageSectionObject->Segments[i].PageDirectory, sizeof(ImageSectionObject->Segments[i].PageDirectory)); } @@ -3165,12 +3310,7 @@ /* * Reference the file handle */ - Status = ObReferenceObject(FileObject); - - if (!NT_SUCCESS(Status)) - { - return Status; - } + ObReferenceObject(FileObject); /* * Create the section @@ -3196,7 +3336,7 @@ Section->SectionPageProtection = SectionPageProtection; Section->AllocationAttributes = AllocationAttributes; - if (!NT_SUCCESS(Status) || FileObject->SectionObjectPointer->ImageSectionObject == NULL) + if (FileObject->SectionObjectPointer->ImageSectionObject == NULL) { NTSTATUS StatusExeFmt; @@ -3214,6 +3354,7 @@ if (!NT_SUCCESS(StatusExeFmt)) { + DPRINT("StatusExeFmt %08x\n", StatusExeFmt); if(ImageSectionObject->Segments != NULL) ExFreePool(ImageSectionObject->Segments); @@ -3287,6 +3428,7 @@ Status = STATUS_SUCCESS; } Section->FileObject = FileObject; + //KeSetEvent((PVOID)&FileObject->Lock, IO_NO_INCREMENT, FALSE); *SectionObject = Section; return(Status); } @@ -3348,7 +3490,6 @@ SectionHandle); } - DPRINT("NtCreateSection -> %08x\n", Status); return Status; } @@ -3544,7 +3685,6 @@ */ if (Protect & ~PAGE_FLAGS_VALID_FROM_USER_MODE) { - CHECKPOINT1; return STATUS_INVALID_PARAMETER_10; } @@ -3558,7 +3698,6 @@ tmpProtect != PAGE_EXECUTE_READWRITE && tmpProtect != PAGE_EXECUTE_WRITECOPY) { - CHECKPOINT1; return STATUS_INVALID_PAGE_PROTECTION; } @@ -3685,11 +3824,12 @@ return(Status); } -VOID +VOID static MmFreeSectionPage(PVOID Context, MEMORY_AREA* MemoryArea, PVOID Address, PFN_TYPE Page, SWAPENTRY SwapEntry, BOOLEAN Dirty) { ULONG Entry; + PFILE_OBJECT FileObject; ULONG Offset; SWAPENTRY SavedSwapEntry; PMM_PAGEOP PageOp; @@ -3698,7 +3838,6 @@ PMM_SECTION_SEGMENT Segment; PMM_AVL_TABLE AddressSpace; PEPROCESS Process; - AddressSpace = (PMM_AVL_TABLE)Context; Process = MmGetAddressSpaceOwner(AddressSpace); @@ -3706,7 +3845,7 @@ Address = (PVOID)PAGE_ROUND_DOWN(Address); Offset = ((ULONG_PTR)Address - (ULONG_PTR)MemoryArea->StartingAddress) + - MemoryArea->Data.SectionData.ViewOffset; + MemoryArea->Data.SectionData.ViewOffset; Section = MemoryArea->Data.SectionData.Section; Segment = MemoryArea->Data.SectionData.Segment; @@ -3722,7 +3861,7 @@ if (Status != STATUS_SUCCESS) { DPRINT1("Failed to wait for page op, status = %x\n", Status); - KEBUGCHECK(0); + ASSERT(FALSE); } MmLockAddressSpace(AddressSpace); @@ -3739,39 +3878,17 @@ */ if (Segment->Flags & MM_DATAFILE_SEGMENT) { - if (Page == PFN_FROM_SSE(Entry) && Dirty) - { - LARGE_INTEGER FileOffset; - IO_STATUS_BLOCK ReadStatus; - ULONG RemainingLength; - PVOID Buffer = MmCreateHyperspaceMapping(Page); - NTSTATUS Status; - - FileOffset.QuadPart = Offset; - - MmUnlockSectionSegment(Segment); - MmUnlockAddressSpace(AddressSpace); - - RemainingLength = min(PAGE_SIZE, Segment->FileOffset.QuadPart + Segment->Length - FileOffset.QuadPart); - DPRINT("Writing no more than %d bytes\n", RemainingLength); - - if (RemainingLength > 0) - { - Status = MiSimpleWrite - (Section->FileObject, - &FileOffset, - Buffer, - min(RemainingLength, PAGE_SIZE), - &ReadStatus); - } - else - Status = STATUS_SUCCESS; - - MmLockAddressSpace(AddressSpace); - MmLockSectionSegment(Segment); - - MmDeleteHyperspaceMapping(Buffer); - } + if (Page == PFN_FROM_SSE(Entry) && Dirty) + { + IO_STATUS_BLOCK IOSB; + FileObject = MemoryArea->Data.SectionData.Section->FileObject; + CcFlushCache + (FileObject->SectionObjectPointer, + &Segment->FileOffset, + Segment->Length, + &IOSB); + ASSERT(SwapEntry == 0); + } } if (SwapEntry != 0) @@ -3782,7 +3899,7 @@ if (Segment->Flags & MM_PAGEFILE_SEGMENT) { DPRINT1("Found a swap entry for a page in a pagefile section.\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } MmFreeSwapPage(SwapEntry); } @@ -3797,7 +3914,7 @@ if (Segment->Flags & MM_PAGEFILE_SEGMENT) { DPRINT1("Found a private page in a pagefile section.\n"); - KEBUGCHECK(0); + ASSERT(FALSE); } /* * Just dereference private pages @@ -3923,7 +4040,7 @@ if (Status != STATUS_SUCCESS) { DPRINT1("Failed to wait for page op, status = %x\n", Status); - KEBUGCHECK(0); + ASSERT(FALSE); } MmLockAddressSpace(AddressSpace); MemoryArea = MmLocateMemoryAreaByAddress(AddressSpace, @@ -3969,7 +4086,7 @@ } if (i >= NrSegments) { - KEBUGCHECK(0); + ASSERT(FALSE); } for (i = 0; i < NrSegments; i++) @@ -4391,7 +4508,6 @@ Protect != PAGE_EXECUTE_READWRITE && Protect != PAGE_EXECUTE_WRITECOPY) { - CHECKPOINT1; return STATUS_INVALID_PAGE_PROTECTION; } @@ -4488,21 +4604,18 @@ if ((Protect & (PAGE_READWRITE|PAGE_EXECUTE_READWRITE)) && !(Section->SectionPageProtection & (PAGE_READWRITE|PAGE_EXECUTE_READWRITE))) { - CHECKPOINT1; return STATUS_SECTION_PROTECTION; } /* check for read access */ if ((Protect & (PAGE_READONLY|PAGE_WRITECOPY|PAGE_EXECUTE_READ|PAGE_EXECUTE_WRITECOPY)) && !(Section->SectionPageProtection & (PAGE_READONLY|PAGE_READWRITE|PAGE_WRITECOPY|PAGE_EXECUTE_READ|PAGE_EXECUTE_READWRITE|PAGE_EXECUTE_WRITECOPY))) { - CHECKPOINT1; return STATUS_SECTION_PROTECTION; } /* check for execute access */ if ((Protect & (PAGE_EXECUTE|PAGE_EXECUTE_READ|PAGE_EXECUTE_READWRITE|PAGE_EXECUTE_WRITECOPY)) && !(Section->SectionPageProtection & (PAGE_EXECUTE|PAGE_EXECUTE_READ|PAGE_EXECUTE_READWRITE|PAGE_EXECUTE_WRITECOPY))) { - CHECKPOINT1; return STATUS_SECTION_PROTECTION; } @@ -4510,7 +4623,7 @@ { /* Following this pointer would lead to us to the dark side */ /* What to do? Bugcheck? Return status? Do the mambo? */ - KEBUGCHECK(MEMORY_MANAGEMENT); + KeBugCheck(MEMORY_MANAGEMENT); } if (SectionOffset == NULL) @@ -4588,30 +4701,7 @@ MmFlushImageSection (IN PSECTION_OBJECT_POINTERS SectionObjectPointer, IN MMFLUSH_TYPE FlushType) { - - switch(FlushType) - { - case MmFlushForDelete: - if (SectionObjectPointer->ImageSectionObject || - SectionObjectPointer->DataSectionObject) - { - return FALSE; - } - break; - - case MmFlushForWrite: - break; - } - -#if 0 - CcFlushCache - (SectionObjectPointer, - NULL, - 0, - NULL); -#endif - - return FALSE; + return CcFlushImageSection(SectionObjectPointer, FlushType); } /* @@ -4627,50 +4717,6 @@ } -NTSTATUS STDCALL -MmMapViewInSystemSpaceAtOffset -(IN PVOID SectionObject, - OUT PVOID *MappedBase, - PLARGE_INTEGER FileOffset, - IN OUT PULONG ViewSize) -{ - PROS_SECTION_OBJECT Section; - PMM_AVL_TABLE AddressSpace; - NTSTATUS Status; - - DPRINT("MmMapViewInSystemSpaceAtOffset() called offset %x\n", FileOffset->LowPart); - - Section = (PROS_SECTION_OBJECT)SectionObject; - AddressSpace = MmGetKernelAddressSpace(); - - MmLockAddressSpace(AddressSpace); - - if ((*ViewSize) == 0) - { - (*ViewSize) = Section->MaximumSize.u.LowPart; - } - else if ((*ViewSize) > Section->MaximumSize.u.LowPart) - { - (*ViewSize) = Section->MaximumSize.u.LowPart; - } - - MmLockSectionSegment(Section->Segment); - - Status = MmMapViewOfSegment(AddressSpace, - Section, - Section->Segment, - MappedBase, - *ViewSize, - PAGE_READWRITE, - FileOffset->LowPart, - 0); - - MmUnlockSectionSegment(Section->Segment); - MmUnlockAddressSpace(AddressSpace); - - return Status; -} - /* * @implemented */ @@ -4690,6 +4736,7 @@ MmLockAddressSpace(AddressSpace); + if ((*ViewSize) == 0) { (*ViewSize) = Section->MaximumSize.u.LowPart; @@ -4700,6 +4747,7 @@ } MmLockSectionSegment(Section->Segment); + Status = MmMapViewOfSegment(AddressSpace, Section, @@ -4707,13 +4755,57 @@ MappedBase, *ViewSize, PAGE_READWRITE, - 0, + 0, 0); MmUnlockSectionSegment(Section->Segment); MmUnlockAddressSpace(AddressSpace); return Status; +} + +NTSTATUS STDCALL +MmMapViewInSystemSpaceAtOffset +(IN PVOID SectionObject, + OUT PVOID *MappedBase, + PLARGE_INTEGER FileOffset, + IN OUT PULONG ViewSize) +{ + PROS_SECTION_OBJECT Section; + PMM_AVL_TABLE AddressSpace; + NTSTATUS Status; + + DPRINT("MmMapViewInSystemSpaceAtOffset() called offset %x\n", FileOffset->LowPart); + + Section = (PROS_SECTION_OBJECT)SectionObject; + AddressSpace = MmGetKernelAddressSpace(); + + MmLockAddressSpace(AddressSpace); + + if ((*ViewSize) == 0) + { + (*ViewSize) = Section->MaximumSize.u.LowPart; + } + else if ((*ViewSize) > Section->MaximumSize.u.LowPart) + { + (*ViewSize) = Section->MaximumSize.u.LowPart; + } + + MmLockSectionSegment(Section->Segment); + + Status = MmMapViewOfSegment(AddressSpace, + Section, + Section->Segment, + MappedBase, + *ViewSize, + PAGE_READWRITE, + FileOffset->LowPart, + 0); + + MmUnlockSectionSegment(Section->Segment); + MmUnlockAddressSpace(AddressSpace); + + return Status; } /* @@ -4745,11 +4837,7 @@ AddressSpace = MmGetKernelAddressSpace(); - MmLockAddressSpace(AddressSpace); - Status = MmUnmapViewOfSegment(AddressSpace, MappedBase); - - MmUnlockAddressSpace(AddressSpace); return Status; } @@ -4830,9 +4918,8 @@ * Handle to a file to create a section mapped to a file * instead of a memory backed section; * - * FileObject - * Specifies a FILE_OBJECT. If this isn't NULL, then FileHandle - * is ignored. + * File + * Unknown. * * RETURN VALUE * Status. @@ -4846,100 +4933,104 @@ IN PLARGE_INTEGER MaximumSize, IN ULONG SectionPageProtection, IN ULONG AllocationAttributes, - IN HANDLE FileHandle OPTIONAL, - IN PFILE_OBJECT FileObject OPTIONAL) -{ - PFILE_OBJECT OriginalFileObject = FileObject; - ULONG Protection; - NTSTATUS Status; - ULONG FileAccess = 0; - BOOLEAN ReferencedFile = FALSE; - PROS_SECTION_OBJECT *SectionObject = (PROS_SECTION_OBJECT *)Section; - - /* - * Check the protection - */ - Protection = SectionPageProtection & ~(PAGE_GUARD|PAGE_NOCACHE); - if (Protection != PAGE_NOACCESS && - Protection != PAGE_READONLY && - Protection != PAGE_READWRITE && - Protection != PAGE_WRITECOPY && - Protection != PAGE_EXECUTE && - Protection != PAGE_EXECUTE_READ && - Protection != PAGE_EXECUTE_READWRITE && - Protection != PAGE_EXECUTE_WRITECOPY) - { - CHECKPOINT1; - return STATUS_INVALID_PAGE_PROTECTION; - } - - if ((DesiredAccess == SECTION_ALL_ACCESS || - (DesiredAccess & SECTION_MAP_WRITE)) && - (Protection == PAGE_READWRITE || - Protection == PAGE_EXECUTE_READWRITE)) - FileAccess = FILE_GENERIC_WRITE; - else - FileAccess = FILE_GENERIC_READ; - - /* - * Reference the file handle - */ - if (!FileObject && FileHandle) - { - Status = ObReferenceObjectByHandle(FileHandle, - FileAccess, - IoFileObjectType, - ExGetPreviousMode(), - (PVOID*)(PVOID)&FileObject, - NULL); - if (!NT_SUCCESS(Status)) - { - DPRINT("Failed: %x\n", Status); - return Status; - } - else - ReferencedFile = TRUE; - } - - if (AllocationAttributes & SEC_IMAGE) - { - DPRINT("Creating an image section\n"); - Status = MmCreateImageSection(SectionObject, - DesiredAccess, - ObjectAttributes, - MaximumSize, - SectionPageProtection, - AllocationAttributes, - FileObject); - if (!NT_SUCCESS(Status)) - DPRINT("Failed: %x\n", Status); - } - - else if (FileHandle != NULL || OriginalFileObject != NULL) - { - Status = MmCreateDataFileSection(SectionObject, - DesiredAccess, - ObjectAttributes, - MaximumSize, - SectionPageProtection, - AllocationAttributes, - FileObject); - if (!NT_SUCCESS(Status)) - DPRINT("Failed: %x\n", Status); - } - else - { - Status = MmCreatePageFileSection(SectionObject, - DesiredAccess, - ObjectAttributes, - MaximumSize, - SectionPageProtection, - AllocationAttributes); - if (!NT_SUCCESS(Status)) - DPRINT("Failed: %x\n", Status); - } - - return(Status); + IN HANDLE FileHandle OPTIONAL, + IN PFILE_OBJECT FileObject OPTIONAL) +{ + PFILE_OBJECT OriginalFileObject = FileObject; + ULONG Protection; + NTSTATUS Status; + ULONG FileAccess = 0; + BOOLEAN ReferencedFile = FALSE; + PROS_SECTION_OBJECT *SectionObject = (PROS_SECTION_OBJECT *)Section; + + /* + * Check the protection + */ + Protection = SectionPageProtection & ~(PAGE_GUARD|PAGE_NOCACHE); + if (Protection != PAGE_NOACCESS && + Protection != PAGE_READONLY && + Protection != PAGE_READWRITE && + Protection != PAGE_WRITECOPY && + Protection != PAGE_EXECUTE && + Protection != PAGE_EXECUTE_READ && + Protection != PAGE_EXECUTE_READWRITE && + Protection != PAGE_EXECUTE_WRITECOPY) + { + return STATUS_INVALID_PAGE_PROTECTION; + } + + if ((DesiredAccess == SECTION_ALL_ACCESS || + (DesiredAccess & SECTION_MAP_WRITE)) && + (Protection == PAGE_READWRITE || + Protection == PAGE_EXECUTE_READWRITE)) + FileAccess = FILE_GENERIC_WRITE; + else + FileAccess = FILE_GENERIC_READ; + + /* + * Reference the file handle + */ + if (!FileObject && FileHandle) + { + Status = ObReferenceObjectByHandle + (FileHandle, + FileAccess, + IoFileObjectType, + ExGetPreviousMode(), + (PVOID*)(PVOID)&FileObject, + NULL); + if (!NT_SUCCESS(Status)) + { + DPRINT("Failed: %x\n", Status); + return Status; + } + else + ReferencedFile = TRUE; + } + + if (AllocationAttributes & SEC_IMAGE) + { + DPRINT("Creating an image section\n"); + Status = MmCreateImageSection + (SectionObject, + DesiredAccess, + ObjectAttributes, + MaximumSize, + SectionPageProtection, + AllocationAttributes, + FileObject); + if (!NT_SUCCESS(Status)) + DPRINT("Failed: %x\n", Status); + } + else if (FileHandle != NULL || OriginalFileObject != NULL) + { + DPRINT("Creating a data section\n"); + Status = MmCreateDataFileSection + (SectionObject, + DesiredAccess, + ObjectAttributes, + MaximumSize, + SectionPageProtection, + AllocationAttributes, + FileObject); + if (!NT_SUCCESS(Status)) + DPRINT("Failed: %x\n", Status); + } + else + { + DPRINT("Creating a page file section\n"); + Status = MmCreatePageFileSection + (SectionObject, + DesiredAccess, + ObjectAttributes, + MaximumSize, + SectionPageProtection, + AllocationAttributes); + if (!NT_SUCCESS(Status)) + DPRINT("Failed: %x\n", Status); + } + + return(Status); } NTSTATUS

1 0

[sginsberg] 35697: - Get gdi32 and user32 to compile again - Update some system call prototypes and definitions
by sginsberg＠svn.reactos.org 27 Aug '08

27 Aug '08

Author: sginsberg Date: Wed Aug 27 09:22:38 2008 New Revision: 35697 URL: http://svn.reactos.org/svn/reactos?rev=35697&view=rev Log: - Get gdi32 and user32 to compile again - Update some system call prototypes and definitions Modified: branches/nwin32/dll/win32/gdi32/misc/hacks.c branches/nwin32/dll/win32/gdi32/objects/bitmap.c branches/nwin32/dll/win32/gdi32/objects/coord.c branches/nwin32/dll/win32/gdi32/objects/font.c branches/nwin32/dll/win32/user32/controls/scrollbar.c branches/nwin32/dll/win32/user32/misc/display.c branches/nwin32/dll/win32/user32/windows/bitmap.c branches/nwin32/dll/win32/user32/windows/class.c branches/nwin32/dll/win32/user32/windows/cursor.c branches/nwin32/dll/win32/user32/windows/icon.c branches/nwin32/dll/win32/user32/windows/input.c branches/nwin32/dll/win32/user32/windows/menu.c branches/nwin32/dll/win32/user32/windows/message.c branches/nwin32/dll/win32/user32/windows/nonclient.c branches/nwin32/dll/win32/user32/windows/window.c branches/nwin32/dll/win32/user32/windows/winpos.c branches/nwin32/include/reactos/win32k/ntuser.h branches/nwin32/subsystems/win32/win32k/ntuser/cursor.c branches/nwin32/subsystems/win32/win32k/ntuser/display.c Modified: branches/nwin32/dll/win32/gdi32/misc/hacks.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/gdi32/misc/hac… ============================================================================== --- branches/nwin32/dll/win32/gdi32/misc/hacks.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/gdi32/misc/hacks.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -21,7 +21,8 @@ int nYOffset, LPPOINT lpPoint) { - return NtGdiOffsetViewportOrgEx(hdc, nXOffset, nYOffset, lpPoint); + /* Unimplemented for now */ + return FALSE; } /* @@ -35,7 +36,8 @@ int nYOffset, LPPOINT lpPoint) { - return NtGdiOffsetWindowOrgEx(hdc, nXOffset, nYOffset, lpPoint); + /* Unimplemented for now */ + return FALSE; } Modified: branches/nwin32/dll/win32/gdi32/objects/bitmap.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/gdi32/objects/… ============================================================================== --- branches/nwin32/dll/win32/gdi32/objects/bitmap.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/gdi32/objects/bitmap.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -315,7 +315,6 @@ 0); } -#if 0 /* * @implemented */ @@ -406,20 +405,6 @@ RestoreDC(SavehDC, -1); return LinesCopied; -} -#endif - -INT -STDCALL -SetDIBits(HDC hdc, - HBITMAP hbmp, - UINT uStartScan, - UINT cScanLines, - CONST VOID *lpvBits, - CONST BITMAPINFO *lpbmi, - UINT fuColorUse) -{ - return NtGdiSetDIBits(hdc, hbmp, uStartScan, cScanLines, lpvBits, lpbmi, fuColorUse); } /* Modified: branches/nwin32/dll/win32/gdi32/objects/coord.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/gdi32/objects/… ============================================================================== --- branches/nwin32/dll/win32/gdi32/objects/coord.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/gdi32/objects/coord.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -377,7 +377,8 @@ } return TRUE; #endif - return NtGdiSetViewportExtEx(hdc, nXExtent, nYExtent, lpSize); + /* Unimplemented for now */ + return FALSE; } /* @@ -439,7 +440,8 @@ Dc_Attr->flXform |= (PAGE_XLATE_CHANGED|DEVICE_TO_WORLD_INVALID); return TRUE; #endif - return NtGdiSetWindowOrgEx(hdc,X,Y,lpPoint); + /* Unimplemented for now */ + return FALSE; } /* @@ -510,7 +512,8 @@ } return TRUE; // Return TRUE. #endif - return NtGdiSetWindowExtEx(hdc, nXExtent, nYExtent, lpSize); + /* Unimplemented for now */ + return FALSE; } /* @@ -559,7 +562,8 @@ Dc_Attr->ptlViewportOrg.y = Y; return TRUE; #endif - return NtGdiSetViewportOrgEx(hdc,X,Y,lpPoint); + /* Unimplemented for now */ + return FALSE; } /* Modified: branches/nwin32/dll/win32/gdi32/objects/font.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/gdi32/objects/… ============================================================================== --- branches/nwin32/dll/win32/gdi32/objects/font.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/gdi32/objects/font.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -283,74 +283,8 @@ IntEnumFontFamilies(HDC Dc, LPLOGFONTW LogFont, PVOID EnumProc, LPARAM lParam, BOOL Unicode) { - int FontFamilyCount; - int FontFamilySize; - PFONTFAMILYINFO Info; - int Ret = 0; - int i; - ENUMLOGFONTEXA EnumLogFontExA; - NEWTEXTMETRICEXA NewTextMetricExA; - - Info = RtlAllocateHeap(GetProcessHeap(), 0, - INITIAL_FAMILY_COUNT * sizeof(FONTFAMILYINFO)); - if (NULL == Info) - { - return 0; - } - FontFamilyCount = NtGdiGetFontFamilyInfo(Dc, LogFont, Info, INITIAL_FAMILY_COUNT); - if (FontFamilyCount < 0) - { - RtlFreeHeap(GetProcessHeap(), 0, Info); - return 0; - } - if (INITIAL_FAMILY_COUNT < FontFamilyCount) - { - FontFamilySize = FontFamilyCount; - RtlFreeHeap(GetProcessHeap(), 0, Info); - Info = RtlAllocateHeap(GetProcessHeap(), 0, - FontFamilyCount * sizeof(FONTFAMILYINFO)); - if (NULL == Info) - { - return 0; - } - FontFamilyCount = NtGdiGetFontFamilyInfo(Dc, LogFont, Info, FontFamilySize); - if (FontFamilyCount < 0 || FontFamilySize < FontFamilyCount) - { - RtlFreeHeap(GetProcessHeap(), 0, Info); - return 0; - } - } - - for (i = 0; i < FontFamilyCount; i++) - { - if (Unicode) - { - Ret = ((FONTENUMPROCW) EnumProc)( - &Info[i].EnumLogFontEx, - &Info[i].NewTextMetricEx, - Info[i].FontType, lParam); - } - else - { // Could use EnumLogFontExW2A here? - LogFontW2A(&EnumLogFontExA.elfLogFont, &Info[i].EnumLogFontEx.elfLogFont); - WideCharToMultiByte(CP_THREAD_ACP, 0, Info[i].EnumLogFontEx.elfFullName, -1, - (LPSTR)EnumLogFontExA.elfFullName, LF_FULLFACESIZE, NULL, NULL); - WideCharToMultiByte(CP_THREAD_ACP, 0, Info[i].EnumLogFontEx.elfStyle, -1, - (LPSTR)EnumLogFontExA.elfStyle, LF_FACESIZE, NULL, NULL); - WideCharToMultiByte(CP_THREAD_ACP, 0, Info[i].EnumLogFontEx.elfScript, -1, - (LPSTR)EnumLogFontExA.elfScript, LF_FACESIZE, NULL, NULL); - NewTextMetricExW2A(&NewTextMetricExA, - &Info[i].NewTextMetricEx); - Ret = ((FONTENUMPROCA) EnumProc)( - &EnumLogFontExA, - &NewTextMetricExA, - Info[i].FontType, lParam); - } - } - - RtlFreeHeap(GetProcessHeap(), 0, Info); - - return Ret; + /* Unimplemented for now, should use NewEnumFontFamiliesExW */ + return 0; } /* Modified: branches/nwin32/dll/win32/user32/controls/scrollbar.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/control… ============================================================================== --- branches/nwin32/dll/win32/user32/controls/scrollbar.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/controls/scrollbar.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -58,14 +58,14 @@ #define SCROLL_MIN_THUMB 6 /* Thumb-tracking info */ -static HWND ScrollTrackingWin = 0; -static INT ScrollTrackingBar = 0; -static INT ScrollTrackingPos = 0; -static INT ScrollTrackingVal = 0; -static BOOL ScrollMovingThumb = FALSE; - -static DWORD ScrollTrackHitTest = SCROLL_NOWHERE; -static BOOL ScrollTrackVertical; +//static HWND ScrollTrackingWin = 0; +//static INT ScrollTrackingBar = 0; +//static INT ScrollTrackingPos = 0; +//static INT ScrollTrackingVal = 0; +//static BOOL ScrollMovingThumb = FALSE; + +//static DWORD ScrollTrackHitTest = SCROLL_NOWHERE; +//static BOOL ScrollTrackVertical; HBRUSH DefWndControlColor(HDC hDC, UINT ctlType); @@ -254,6 +254,7 @@ DrawFrameControl(hDC, &RectRB, DFC_SCROLL, ScrollDirFlagRB); } +#if 0 static VOID FASTCALL IntScrollDrawMovingThumb(HDC Dc, PSCROLLBARINFO ScrollBarInfo, BOOL Vertical) { @@ -306,13 +307,13 @@ return OBJID_CLIENT; } +#endif static BOOL FASTCALL IntGetScrollBarInfo(HWND Wnd, INT Bar, PSCROLLBARINFO ScrollBarInfo) { - ScrollBarInfo->cbSize = sizeof(SCROLLBARINFO); - - return NtUserGetScrollBarInfo(Wnd, IntScrollGetObjectId(Bar), ScrollBarInfo); + UNIMPLEMENTED; + return FALSE; } void @@ -383,6 +384,7 @@ } } +#if 0 static BOOL FASTCALL IntScrollPtInRectEx(LPRECT Rect, POINT Pt, BOOL Vertical) { @@ -467,6 +469,7 @@ return SCROLL_THUMB; } +#endif /*********************************************************************** @@ -485,6 +488,9 @@ INT *ArrowSize, INT *ThumbSize, INT *ThumbPos) { + UNIMPLEMENTED; + return FALSE; +#if 0 INT Pixels; BOOL Vertical; RECT ClientRect; @@ -622,6 +628,7 @@ } return Vertical; +#endif } /*********************************************************************** @@ -630,6 +637,7 @@ * Compute the current scroll position based on the thumb position in pixels * from the top of the scroll-bar. */ +#if 0 static UINT FASTCALL IntScrollGetThumbVal(HWND Wnd, INT SBType, PSCROLLBARINFO ScrollBarInfo, BOOL Vertical, INT Pos) @@ -699,6 +707,7 @@ return Pt; } +#endif /*********************************************************************** * IntScrollDrawSizeGrip @@ -782,6 +791,9 @@ static VOID FASTCALL IntScrollHandleScrollEvent(HWND Wnd, INT SBType, UINT Msg, POINT Pt) { + UNIMPLEMENTED; + return; +#if 0 static POINT PrevPt; /* Previous mouse position for timer events */ static UINT TrackThumbPos; /* Thumb position when tracking started. */ static INT LastClickPos; /* Position in the scroll-bar of the last @@ -1057,6 +1069,7 @@ } ReleaseDC(Wnd, Dc); +#endif } @@ -1140,16 +1153,8 @@ static INT FASTCALL IntScrollGetScrollPos(HWND Wnd, INT Bar) { - SCROLLINFO ScrollInfo; - - ScrollInfo.cbSize = sizeof(SCROLLINFO); - ScrollInfo.fMask = SIF_POS; - if (! NtUserGetScrollInfo(Wnd, Bar, &ScrollInfo)) - { - return 0; - } - - return ScrollInfo.nPos; + UNIMPLEMENTED; + return 0; } static BOOL FASTCALL @@ -1166,7 +1171,7 @@ ScrollInfo.cbSize = sizeof(SCROLLINFO); ScrollInfo.fMask = SIF_RANGE; - Result = NtUserGetScrollInfo(Wnd, Bar, &ScrollInfo); + Result = FALSE; if (Result) { *MinPos = ScrollInfo.nMin; @@ -1436,7 +1441,7 @@ return NtUserSetScrollInfo(Wnd, SB_CTL, (SCROLLINFO *) lParam, wParam); case SBM_GETSCROLLINFO: - return NtUserGetScrollInfo(Wnd, SB_CTL, (SCROLLINFO *) lParam); + return FALSE; case 0x00e5: case 0x00e7: @@ -1491,7 +1496,8 @@ BOOL STDCALL GetScrollBarInfo(HWND hWnd, LONG idObject, PSCROLLBARINFO psbi) { - return NtUserGetScrollBarInfo(hWnd, idObject, psbi); + UNIMPLEMENTED; + return FALSE; } /* @@ -1506,7 +1512,8 @@ } else { - return NtUserGetScrollInfo(Wnd, SBType, Info); + UNIMPLEMENTED; + return FALSE; } } @@ -1580,7 +1587,7 @@ * Call NtUserGetScrollInfo() to get the previous position that * we will later return. */ - if (NtUserGetScrollInfo(hWnd, nBar, &ScrollInfo)) + if (FALSE) { Result = ScrollInfo.nPos; if (Result != nPos) Modified: branches/nwin32/dll/win32/user32/misc/display.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/misc/di… ============================================================================== --- branches/nwin32/dll/win32/user32/misc/display.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/misc/display.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -134,66 +134,8 @@ MONITORENUMPROC lpfnEnum, LPARAM dwData) { - INT iCount, i; - HMONITOR *hMonitorList; - LPRECT pRectList; - HANDLE hHeap; - - /* get list of monitors/rects */ - iCount = NtUserEnumDisplayMonitors(hdc, lprcClip, NULL, NULL, 0); - if (iCount < 0) - { - /* FIXME: SetLastError() */ - return FALSE; - } - if (iCount == 0) - { - return TRUE; - } - - hHeap = GetProcessHeap(); - hMonitorList = HeapAlloc(hHeap, 0, sizeof (HMONITOR) * iCount); - if (hMonitorList == NULL) - { - SetLastError(ERROR_NOT_ENOUGH_MEMORY); - return FALSE; - } - pRectList = HeapAlloc(hHeap, 0, sizeof (RECT) * iCount); - if (pRectList == NULL) - { - HeapFree(hHeap, 0, hMonitorList); - SetLastError(ERROR_NOT_ENOUGH_MEMORY); - return FALSE; - } - - iCount = NtUserEnumDisplayMonitors(hdc, lprcClip, hMonitorList, pRectList, iCount); - if (iCount <= 0) - { - /* FIXME: SetLastError() */ - HeapFree(hHeap, 0, hMonitorList); - HeapFree(hHeap, 0, pRectList); - return FALSE; - } - - /* enumerate list */ - for (i = 0; i < iCount; i++) - { - HMONITOR hMonitor = hMonitorList[i]; - LPRECT pMonitorRect = pRectList + i; - HDC hMonitorDC = NULL; - - if (hdc != NULL) - { - /* make monitor DC */ - hMonitorDC = hdc; - } - - if (!lpfnEnum(hMonitor, hMonitorDC, pMonitorRect, dwData)) - break; - } - HeapFree(hHeap, 0, hMonitorList); - HeapFree(hHeap, 0, pRectList); - return TRUE; + UNIMPLEMENTED; + return FALSE; } @@ -361,7 +303,7 @@ { if (lpmi->cbSize == sizeof (MONITORINFO)) { - return NtUserGetMonitorInfo(hMonitor, lpmi); + return FALSE; } else if (lpmi->cbSize != sizeof (MONITORINFOEXA)) { @@ -374,7 +316,7 @@ INT res; miExW.cbSize = sizeof (MONITORINFOEXW); - if (!NtUserGetMonitorInfo(hMonitor, (LPMONITORINFO)&miExW)) + if (FALSE) { return FALSE; } @@ -402,7 +344,8 @@ HMONITOR hMonitor, LPMONITORINFO lpmi) { - return NtUserGetMonitorInfo(hMonitor, lpmi); + UNIMPLEMENTED; + return FALSE; } @@ -415,7 +358,8 @@ IN POINT ptPoint, IN DWORD dwFlags ) { - return NtUserMonitorFromPoint(ptPoint, dwFlags); + UNIMPLEMENTED; + return NULL; } @@ -428,7 +372,8 @@ IN LPCRECT lpcRect, IN DWORD dwFlags ) { - return NtUserMonitorFromRect(lpcRect, dwFlags); + UNIMPLEMENTED; + return NULL; } @@ -441,7 +386,8 @@ IN HWND hWnd, IN DWORD dwFlags ) { - return NtUserMonitorFromWindow(hWnd, dwFlags); + UNIMPLEMENTED; + return NULL; } @@ -478,7 +424,7 @@ else pDevModeW = NULL; - rc = NtUserChangeDisplaySettings ( pDeviceName, pDevModeW, hwnd, dwflags, lParam ); + rc = NtUserChangeDisplaySettings ( pDeviceName, pDevModeW, dwflags, lParam ); if (lpszDeviceName != NULL) RtlFreeUnicodeString ( &DeviceName ); @@ -521,7 +467,7 @@ else pDeviceName = NULL; - rc = NtUserChangeDisplaySettings ( pDeviceName, lpDevMode, hwnd, dwflags, lParam ); + rc = NtUserChangeDisplaySettings ( pDeviceName, lpDevMode, dwflags, lParam ); return rc; } Modified: branches/nwin32/dll/win32/user32/windows/bitmap.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/windows… ============================================================================== --- branches/nwin32/dll/win32/user32/windows/bitmap.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/windows/bitmap.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -242,7 +242,8 @@ * Let's see if we already loaded it */ if (fuLoad & LR_SHARED) { - hIcon = NtUserFindExistingCursorIcon(hinst, hResInfo, 0, 0); + PPOINT Point = NULL; + hIcon = NtUserFindExistingCursorIcon(hinst, hResInfo, Point); if (hIcon) { return hIcon; @@ -270,7 +271,7 @@ if (hIcon && 0 != (fuLoad & LR_SHARED)) { -#if 1 +#if 0 NtUserSetCursorIconData((HICON)hIcon, NULL, NULL, hinst, hResInfo, (HRSRC)NULL); #else Modified: branches/nwin32/dll/win32/user32/windows/class.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/windows… ============================================================================== --- branches/nwin32/dll/win32/user32/windows/class.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/windows/class.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -23,6 +23,9 @@ LPCSTR lpszClass, LPWNDCLASSEXA lpwcx) { + UNIMPLEMENTED; + return FALSE; +#if 0 UNICODE_STRING ClassName = {0}; BOOL Ret; @@ -71,6 +74,7 @@ } return Ret; +#endif } @@ -84,6 +88,9 @@ LPCWSTR lpszClass, LPWNDCLASSEXW lpwcx) { + UNIMPLEMENTED; + return FALSE; +#if 0 UNICODE_STRING ClassName = {0}; TRACE("%p class/atom: %S/%04x %p\n", hInstance, @@ -120,6 +127,7 @@ lpwcx, NULL, FALSE); +#endif } @@ -292,7 +300,7 @@ _SEH_END; if (Wnd == NULL) - Ret = NtUserGetClassLong(hWnd, nIndex, TRUE); + Ret = 0; return Ret; } @@ -405,7 +413,7 @@ _SEH_END; if (Wnd == NULL) - Ret = NtUserGetClassLong(hWnd, nIndex, FALSE); + Ret = 0; return Ret; } @@ -426,9 +434,10 @@ ClassName.MaximumLength = nMaxCount; ClassName.Buffer = lpClassName; + /* FIXME: Conver to unicode!!! */ Result = NtUserGetClassName(hWnd, - (PUNICODE_STRING)&ClassName, - TRUE); + FALSE, + (PUNICODE_STRING)&ClassName); TRACE("%p class/atom: %s/%04x %x\n", hWnd, IS_ATOM(lpClassName) ? NULL : lpClassName, @@ -456,8 +465,8 @@ ClassName.Buffer = lpClassName; Result = NtUserGetClassName(hWnd, - &ClassName, - FALSE); + FALSE, + &ClassName); TRACE("%p class/atom: %S/%04x %x\n", hWnd, IS_ATOM(lpClassName) ? NULL : lpClassName, @@ -485,7 +494,7 @@ if ((nIndex < 0) && (nIndex != GCW_ATOM)) return 0; - return (WORD) NtUserGetClassLong ( hWnd, nIndex, TRUE ); + return 0; } @@ -536,7 +545,7 @@ case GWL_WNDPROC: /* Call win32k for this as a callproc handle may need to be created */ - return NtUserGetWindowLong(hWnd, nIndex, TRUE); + return 0; default: SetLastError(ERROR_INVALID_PARAMETER); @@ -593,7 +602,7 @@ case GWL_WNDPROC: /* Call win32k for this as a callproc handle may need to be created */ - return NtUserGetWindowLong(hWnd, nIndex, FALSE); + return 0; default: SetLastError(ERROR_INVALID_PARAMETER); @@ -654,6 +663,7 @@ /* * Create a small icon based on a standard icon */ +#if 0 static HICON CreateSmallIcon(HICON StdIcon) { @@ -814,6 +824,7 @@ return SmallIcon; } +#endif ATOM STDCALL @@ -832,6 +843,9 @@ ATOM STDCALL RegisterClassExA(CONST WNDCLASSEXA *lpwcx) { + UNIMPLEMENTED; + return 0; +#if 0 RTL_ATOM Atom; WNDCLASSEXA WndClass; UNICODE_STRING ClassName; @@ -917,6 +931,7 @@ RtlFreeUnicodeString(&ClassName); return (ATOM)Atom; +#endif } /* @@ -925,6 +940,9 @@ ATOM STDCALL RegisterClassExW(CONST WNDCLASSEXW *lpwcx) { + UNIMPLEMENTED; + return 0; +#if 0 ATOM Atom; WNDCLASSEXW WndClass; UNICODE_STRING ClassName; @@ -1005,6 +1023,7 @@ lpwcx->style, lpwcx->cbClsExtra, lpwcx->cbWndExtra, WndClass); return Atom; +#endif } /* Modified: branches/nwin32/dll/win32/user32/windows/cursor.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/windows… ============================================================================== --- branches/nwin32/dll/win32/user32/windows/cursor.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/windows/cursor.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -76,7 +76,7 @@ if(GetIconInfo((HANDLE)pcur, &IconInfo)) { - return (HCURSOR)NtUserCreateCursorIconHandle(&IconInfo, FALSE); + return 0; } return (HCURSOR)0; } @@ -145,7 +145,7 @@ IconInfo.yHotspot = yHotSpot; IconInfo.hbmColor = 0; - return (HCURSOR)NtUserCreateCursorIconHandle(&IconInfo, FALSE); + return NULL; } Modified: branches/nwin32/dll/win32/user32/windows/icon.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/windows… ============================================================================== --- branches/nwin32/dll/win32/user32/windows/icon.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/windows/icon.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -88,7 +88,8 @@ ImageData, bwBIH, DIB_RGB_COLORS); /* Create the icon based on everything we have so far */ - return NtUserCreateCursorIconHandle(&IconInfo, FALSE); + UNIMPLEMENTED; + return NULL; } HICON @@ -139,7 +140,8 @@ IconInfo.hbmColor = (HBITMAP)0; /* Create the icon based on everything we have so far */ - return NtUserCreateCursorIconHandle(&IconInfo, FALSE); + UNIMPLEMENTED; + return NULL; } @@ -195,7 +197,8 @@ return (HICON)0; } - return NtUserCreateCursorIconHandle(&IconInfo, FALSE); + UNIMPLEMENTED; + return NULL; } @@ -343,7 +346,8 @@ return (HICON)0; }*/ - return (HICON)NtUserCreateCursorIconHandle(IconInfo, TRUE); + UNIMPLEMENTED; + return NULL; } Modified: branches/nwin32/dll/win32/user32/windows/input.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/windows… ============================================================================== --- branches/nwin32/dll/win32/user32/windows/input.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/windows/input.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -289,7 +289,8 @@ BOOL STDCALL GetLastInputInfo(PLASTINPUTINFO plii) { - return NtUserGetLastInputInfo(plii); + UNIMPLEMENTED; + return FALSE; } Modified: branches/nwin32/dll/win32/user32/windows/menu.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/windows… ============================================================================== --- branches/nwin32/dll/win32/user32/windows/menu.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/windows/menu.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -131,7 +131,8 @@ MenuInfo->cbSize = sizeof(ROSMENUINFO); MenuInfo->fMask = MIM_BACKGROUND | MIM_HELPID | MIM_MAXHEIGHT | MIM_MENUDATA | MIM_STYLE; - return NtUserMenuInfo(Menu, MenuInfo, FALSE); + UNIMPLEMENTED; + return FALSE; } /*********************************************************************** @@ -145,7 +146,8 @@ MenuInfo->cbSize = sizeof(ROSMENUINFO); MenuInfo->fMask = MIM_BACKGROUND | MIM_HELPID | MIM_MAXHEIGHT | MIM_MENUDATA | MIM_STYLE; - return NtUserMenuInfo(MenuInfo->Self, MenuInfo, TRUE); + UNIMPLEMENTED; + return FALSE; } /*********************************************************************** @@ -180,9 +182,10 @@ | MIIM_ID | MIIM_STATE | MIIM_STRING | MIIM_SUBMENU | MIIM_TYPE; ItemInfo->dwTypeData = NULL; - if (! NtUserMenuItemInfo(Menu, Index, TRUE, ItemInfo, FALSE)) + if (TRUE) { ItemInfo->fType = 0; + UNIMPLEMENTED; return FALSE; } @@ -196,9 +199,10 @@ return FALSE; } - if (! NtUserMenuItemInfo(Menu, Index, TRUE, ItemInfo, FALSE)) + if (TRUE) { ItemInfo->fType = 0; + UNIMPLEMENTED; return FALSE; } ItemInfo->dwTypeData[ItemInfo->cch - 1] = UNICODE_NULL; @@ -222,7 +226,8 @@ { ItemInfo->cch = strlenW(ItemInfo->dwTypeData); } - Ret = NtUserMenuItemInfo(Menu, Index, TRUE, ItemInfo, TRUE); + UNIMPLEMENTED; + Ret = FALSE; return Ret; } @@ -250,6 +255,9 @@ static INT FASTCALL MenuGetAllRosMenuItemInfo(HMENU Menu, PROSMENUITEMINFO *ItemInfo) { + UNIMPLEMENTED; + return -1; +#if 0 DWORD BufSize; BufSize = NtUserBuildMenuItemList(Menu, (VOID *) 1, 0, 0); @@ -264,6 +272,7 @@ } return NtUserBuildMenuItemList(Menu, *ItemInfo, BufSize, 0); +#endif } /*********************************************************************** @@ -4128,7 +4137,8 @@ UINT fByPos, UINT gmdiFlags) { - return NtUserGetMenuDefaultItem(hMenu, fByPos, gmdiFlags); + UNIMPLEMENTED; + return -1; } @@ -4149,7 +4159,8 @@ mi.cbSize = sizeof(MENUINFO); mi.fMask = lpcmi->fMask; - res = NtUserMenuInfo(hmenu, &mi, FALSE); + UNIMPLEMENTED; + res = FALSE; memcpy(lpcmi, &mi, sizeof(MENUINFO)); return res; @@ -4180,7 +4191,7 @@ mii.cbSize = sizeof(MENUITEMINFOW); mii.fMask = MIIM_ID | MIIM_SUBMENU; - if (! NtUserMenuItemInfo(hMenu, nPos, MF_BYPOSITION, &mii, FALSE)) + if (TRUE) { return -1; } @@ -4222,7 +4233,8 @@ if(!(mii->fMask & (MIIM_TYPE | MIIM_STRING))) { /* No text requested, just pass on */ - return NtUserMenuItemInfo(Menu, Item, ByPosition, (PROSMENUITEMINFO) mii, FALSE); + UNIMPLEMENTED; + return FALSE; } AnsiBuffer = mii->dwTypeData; @@ -4238,7 +4250,7 @@ miiW.dwTypeData[0] = 0; } - if (!NtUserMenuItemInfo(Menu, Item, ByPosition, (PROSMENUITEMINFO)&miiW, FALSE)) + if (TRUE) { if (miiW.dwTypeData) RtlFreeHeap(GetProcessHeap(), 0, miiW.dwTypeData); return FALSE; @@ -4305,7 +4317,8 @@ if(!(mii->fMask & (MIIM_TYPE | MIIM_STRING))) { /* No text requested, just pass on */ - return NtUserMenuItemInfo(Menu, Item, ByPosition, (PROSMENUITEMINFO) mii, FALSE); + UNIMPLEMENTED; + return FALSE; } String = mii->dwTypeData; @@ -4321,7 +4334,7 @@ miiW.dwTypeData[0] = 0; } - if (!NtUserMenuItemInfo(Menu, Item, ByPosition, (PROSMENUITEMINFO) &miiW, FALSE)) + if (TRUE) { if (miiW.dwTypeData) RtlFreeHeap(GetProcessHeap(), 0, miiW.dwTypeData); return FALSE; @@ -4379,7 +4392,7 @@ mii.fMask = MIIM_STATE | MIIM_FTYPE | MIIM_SUBMENU; SetLastError(0); - if(NtUserMenuItemInfo(hMenu, uId, uFlags, &mii, FALSE)) + if(FALSE) { UINT nSubItems = 0; if(mii.hSubMenu) @@ -4473,7 +4486,7 @@ mi.cbSize = sizeof(MENUITEMINFOW); mi.fMask = MIIM_SUBMENU; - if (NtUserMenuItemInfo(hMenu, (UINT)nPos, MF_BYPOSITION, &mi, FALSE)) + if (FALSE) { return IsMenu(mi.hSubMenu) ? mi.hSubMenu : NULL; } @@ -4925,7 +4938,9 @@ return res; memcpy(&mi, lpcmi, sizeof(MENUINFO)); - return NtUserMenuInfo(hmenu, &mi, TRUE); + + UNIMPLEMENTED; + return FALSE; } @@ -4945,8 +4960,7 @@ memset ( &uItem, 0, sizeof(uItem) ); uItem.fMask = MIIM_STATE | MIIM_BITMAP; - if(!(NtUserMenuItemInfo(hMenu, uPosition, - (BOOL)(MF_BYPOSITION & uFlags), &uItem, FALSE))) return FALSE; + if(TRUE) return FALSE; if (!hBitmapChecked && !hBitmapUnchecked) { @@ -4958,8 +4972,7 @@ uItem.hbmpUnchecked = hBitmapUnchecked; uItem.fState |= MF_USECHECKBITMAPS; } - return NtUserMenuItemInfo(hMenu, uPosition, - (BOOL)(MF_BYPOSITION & uFlags), &uItem, TRUE); + return FALSE; } @@ -5013,8 +5026,8 @@ UnicodeString.Buffer = NULL; } - Result = NtUserMenuItemInfo(hMenu, uItem, fByPosition, - (PROSMENUITEMINFO)&MenuItemInfoW, TRUE); + UNIMPLEMENTED; + Result = FALSE; if (UnicodeString.Buffer != NULL) { @@ -5054,8 +5067,8 @@ { MenuItemInfoW.cch = strlenW(MenuItemInfoW.dwTypeData); } - Result = NtUserMenuItemInfo(hMenu, uItem, fByPosition, - (PROSMENUITEMINFO)&MenuItemInfoW, TRUE); + UNIMPLEMENTED; + Result = FALSE; return Result; } @@ -5183,10 +5196,7 @@ mi.cbSize = sizeof(ROSMENUINFO); mi.fMask = MIM_HELPID; - if(NtUserMenuInfo(hmenu, &mi, FALSE)) - { - return mi.dwContextHelpID; - } + UNIMPLEMENTED; return 0; } Modified: branches/nwin32/dll/win32/user32/windows/message.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/windows… ============================================================================== --- branches/nwin32/dll/win32/user32/windows/message.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/windows/message.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -265,6 +265,7 @@ return; } +#if 0 static BOOL FASTCALL MsgiUMToKMReply(PMSG UMMsg, PMSG KMMsg, LRESULT *Result) { @@ -272,6 +273,7 @@ return TRUE; } +#endif static BOOL FASTCALL MsgiKMToUMMessage(PMSG KMMsg, PMSG UMMsg) @@ -1702,6 +1704,9 @@ WPARAM wParam, LPARAM lParam) { + UNIMPLEMENTED; + return 0; +#if 0 MSG UMMsg, KMMsg; NTUSERSENDMESSAGEINFO Info; LRESULT Result; @@ -1749,6 +1754,7 @@ } return Result; +#endif } @@ -1758,6 +1764,9 @@ LRESULT STDCALL SendMessageA(HWND Wnd, UINT Msg, WPARAM wParam, LPARAM lParam) { + UNIMPLEMENTED; + return 0; +#if 0 MSG AnsiMsg, UcMsg; MSG KMMsg; LRESULT Result; @@ -1831,6 +1840,7 @@ } return Result; +#endif } /* @@ -1891,6 +1901,9 @@ UINT uTimeout, PDWORD_PTR lpdwResult) { + UNIMPLEMENTED; + return 0; +#if 0 MSG AnsiMsg; MSG UcMsg; LRESULT Result; @@ -1955,6 +1968,7 @@ SPY_ExitMessage(SPY_RESULT_OK, hWnd, Msg, Result, wParam, lParam); return Result; +#endif } @@ -1972,6 +1986,9 @@ UINT uTimeout, PDWORD_PTR lpdwResult) { + UNIMPLEMENTED; + return 0; +#if 0 NTUSERSENDMESSAGEINFO Info; LRESULT Result; @@ -1993,6 +2010,7 @@ SPY_ExitMessage(SPY_RESULT_OK, hWnd, Msg, Result, wParam, lParam); return Result; +#endif } @@ -2007,6 +2025,9 @@ WPARAM wParam, LPARAM lParam) { + UNIMPLEMENTED; + return FALSE; +#if 0 MSG AnsiMsg, UcMsg; MSG KMMsg; LRESULT Result; @@ -2031,6 +2052,7 @@ MsgiAnsiToUnicodeCleanup(&UcMsg, &AnsiMsg); return Result; +#endif } @@ -2045,6 +2067,9 @@ WPARAM wParam, LPARAM lParam) { + UNIMPLEMENTED; + return FALSE; +#if 0 MSG UMMsg, KMMsg; LRESULT Result; @@ -2061,6 +2086,7 @@ MsgiUMToKMCleanup(&UMMsg, &KMMsg); return Result; +#endif } Modified: branches/nwin32/dll/win32/user32/windows/nonclient.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/windows… ============================================================================== --- branches/nwin32/dll/win32/user32/windows/nonclient.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/windows/nonclient.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -583,7 +583,7 @@ ssbi.rgstate[0] |= STATE_SYSTEM_OFFSCREEN; else ssbi.rgstate[0] &= ~STATE_SYSTEM_OFFSCREEN; - NtUserSetScrollBarInfo(hWnd, OBJID_VSCROLL, &ssbi); + //NtUserSetScrollBarInfo(hWnd, OBJID_VSCROLL, &ssbi); if(ssbi.rgstate[0] & STATE_SYSTEM_OFFSCREEN) Style &= ~WS_VSCROLL; } @@ -602,7 +602,7 @@ ssbi.rgstate[0] |= STATE_SYSTEM_OFFSCREEN; else ssbi.rgstate[0] &= ~STATE_SYSTEM_OFFSCREEN; - NtUserSetScrollBarInfo(hWnd, OBJID_HSCROLL, &ssbi); + //NtUserSetScrollBarInfo(hWnd, OBJID_HSCROLL, &ssbi); if(ssbi.rgstate[0] & STATE_SYSTEM_OFFSCREEN) Style &= ~WS_HSCROLL; } Modified: branches/nwin32/dll/win32/user32/windows/window.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/windows… ============================================================================== --- branches/nwin32/dll/win32/user32/windows/window.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/windows/window.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -1043,7 +1043,8 @@ _SEH_END; if (!Wnd) /* Fall back to win32k... */ - Ret = NtUserGetWindow(hWnd, uCmd); + UNIMPLEMENTED; + Ret = NULL; return Ret; } Modified: branches/nwin32/dll/win32/user32/windows/winpos.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/dll/win32/user32/windows… ============================================================================== --- branches/nwin32/dll/win32/user32/windows/winpos.c [iso-8859-1] (original) +++ branches/nwin32/dll/win32/user32/windows/winpos.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -75,6 +75,9 @@ WinPosGetMinMaxInfo(HWND hWnd, POINT* MaxSize, POINT* MaxPos, POINT* MinTrack, POINT* MaxTrack) { + UNIMPLEMENTED; + return 0; +#if o MINMAXINFO MinMax; if(NtUserGetMinMaxInfo(hWnd, &MinMax, TRUE)) @@ -90,6 +93,7 @@ if (MaxTrack) *MaxTrack = MinMax.ptMaxTrackSize; } return 0; //FIXME: what does it return? +#endif } Modified: branches/nwin32/include/reactos/win32k/ntuser.h URL: http://svn.reactos.org/svn/reactos/branches/nwin32/include/reactos/win32k/n… ============================================================================== --- branches/nwin32/include/reactos/win32k/ntuser.h [iso-8859-1] (original) +++ branches/nwin32/include/reactos/win32k/ntuser.h [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -882,7 +882,6 @@ NtUserChangeDisplaySettings( PUNICODE_STRING lpszDeviceName, LPDEVMODEW lpDevMode, - HWND hwnd, DWORD dwflags, LPVOID lParam); @@ -1204,8 +1203,7 @@ NtUserFindExistingCursorIcon( HMODULE hModule, HRSRC hRsrc, - LONG cx, - LONG cy); + PPOINT Point); // FIXME: Verify! HWND NTAPI @@ -2127,11 +2125,9 @@ NTAPI NtUserSetCursorIconData( HANDLE Handle, - PBOOL fIcon, - POINT *Hotspot, HMODULE hModule, - HRSRC hRsrc, - HRSRC hGroupRsrc); + PUNICODE_STRING pstrResName, + PICONINFO pIconInfo); DWORD NTAPI Modified: branches/nwin32/subsystems/win32/win32k/ntuser/cursor.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/subsystems/win32/win32k/… ============================================================================== --- branches/nwin32/subsystems/win32/win32k/ntuser/cursor.c [iso-8859-1] (original) +++ branches/nwin32/subsystems/win32/win32k/ntuser/cursor.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -35,8 +35,7 @@ APIENTRY NtUserFindExistingCursorIcon(HMODULE hModule, HRSRC hRsrc, - LONG cx, - LONG cy) + PPOINT Point) { UNIMPLEMENTED; return NULL; @@ -86,15 +85,12 @@ return FALSE; } - BOOL -APIENTRY +NTAPI NtUserSetCursorIconData(HANDLE Handle, - PBOOL fIcon, - POINT *Hotspot, HMODULE hModule, - HRSRC hRsrc, - HRSRC hGroupRsrc) + PUNICODE_STRING pstrResName, + PICONINFO pIconInfo) { UNIMPLEMENTED; return FALSE; Modified: branches/nwin32/subsystems/win32/win32k/ntuser/display.c URL: http://svn.reactos.org/svn/reactos/branches/nwin32/subsystems/win32/win32k/… ============================================================================== --- branches/nwin32/subsystems/win32/win32k/ntuser/display.c [iso-8859-1] (original) +++ branches/nwin32/subsystems/win32/win32k/ntuser/display.c [iso-8859-1] Wed Aug 27 09:22:38 2008 @@ -18,7 +18,6 @@ APIENTRY NtUserChangeDisplaySettings(PUNICODE_STRING lpszDeviceName, LPDEVMODEW lpDevMode, - HWND hWnd, DWORD dwflags, LPVOID lParam) {

1 0

[tkreuzer] 35696: enhance macros
by tkreuzer＠svn.reactos.org 27 Aug '08

27 Aug '08

Author: tkreuzer Date: Wed Aug 27 09:16:04 2008 New Revision: 35696 URL: http://svn.reactos.org/svn/reactos?rev=35696&view=rev Log: enhance macros Modified: branches/ros-amd64-bringup/reactos/boot/freeldr/freeldr/include/arch/amd64/amd64.h Modified: branches/ros-amd64-bringup/reactos/boot/freeldr/freeldr/include/arch/amd64/amd64.h URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/boot/… ============================================================================== --- branches/ros-amd64-bringup/reactos/boot/freeldr/freeldr/include/arch/amd64/amd64.h [iso-8859-1] (original) +++ branches/ros-amd64-bringup/reactos/boot/freeldr/freeldr/include/arch/amd64/amd64.h [iso-8859-1] Wed Aug 27 09:16:04 2008 @@ -34,10 +34,10 @@ #define PtrToPfn(p) \ ((((ULONGLONG)p) >> PAGE_SHIFT) & 0xfffffffULL) -#define VAtoPXI(va) (((va) >> PXI_SHIFT) & 0x1FF) -#define VAtoPPI(va) (((va) >> PPI_SHIFT) & 0x1FF) -#define VAtoPDI(va) (((va) >> PDI_SHIFT) & 0x1FF) -#define VAtoPTI(va) (((va) >> PTI_SHIFT) & 0x1FF) +#define VAtoPXI(va) ((((ULONG64)va) >> PXI_SHIFT) & 0x1FF) +#define VAtoPPI(va) ((((ULONG64)va) >> PPI_SHIFT) & 0x1FF) +#define VAtoPDI(va) ((((ULONG64)va) >> PDI_SHIFT) & 0x1FF) +#define VAtoPTI(va) ((((ULONG64)va) >> PTI_SHIFT) & 0x1FF) #define HYPERSPACE_BASE 0xfffff70000000000ULL #define HAL_BASE 0xffffffff80000000ULL

1 0

[tkreuzer] 35695: Merge 34717 from ros-amd64-bringup branch: - Fix prototypes of _findfirst, _findnext, _findclose, _findfirsti64, _findnexti64 _wfindfirst, _wfindnext, _wfindfirst64, _wfindnext64, strndup, - Fix a number of pointer to int casts
by tkreuzer＠svn.reactos.org 27 Aug '08

27 Aug '08

Author: tkreuzer Date: Tue Aug 26 22:59:24 2008 New Revision: 35695 URL: http://svn.reactos.org/svn/reactos?rev=35695&view=rev Log: Merge 34717 from ros-amd64-bringup branch: - Fix prototypes of _findfirst, _findnext, _findclose, _findfirsti64, _findnexti64 _wfindfirst, _wfindnext, _wfindfirst64, _wfindnext64, strndup, - Fix a number of pointer to int casts Modified: trunk/reactos/include/crt/io.h trunk/reactos/include/reactos/wine/library.h trunk/reactos/lib/3rdparty/mingw/gccmain.c trunk/reactos/lib/3rdparty/mingw/pseudo-reloc.c trunk/reactos/lib/sdk/crt/misc/getargs.c trunk/reactos/lib/sdk/crt/search/lfind.c trunk/reactos/lib/sdk/crt/search/lsearch.c trunk/reactos/lib/sdk/crt/stdio/find.c Modified: trunk/reactos/include/crt/io.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/crt/io.h?rev=35695… ============================================================================== --- trunk/reactos/include/crt/io.h [iso-8859-1] (original) +++ trunk/reactos/include/crt/io.h [iso-8859-1] Tue Aug 26 22:59:24 2008 @@ -129,10 +129,10 @@ * _findclose calls. _findnext also returns -1 if no match could be found, * and 0 if a match was found. Call _findclose when you are finished. */ -/* FIXME: Should these all use intptr_t, as per recent MSDN docs? */ -_CRTIMP long __cdecl __MINGW_NOTHROW _findfirst (const char*, struct _finddata_t*); -_CRTIMP int __cdecl __MINGW_NOTHROW _findnext (long, struct _finddata_t*); -_CRTIMP int __cdecl __MINGW_NOTHROW _findclose (long); + +_CRTIMP intptr_t __cdecl __MINGW_NOTHROW _findfirst (const char*, struct _finddata_t*); +_CRTIMP intptr_t __cdecl __MINGW_NOTHROW _findnext (intptr_t, struct _finddata_t*); +_CRTIMP int __cdecl __MINGW_NOTHROW _findclose (intptr_t); _CRTIMP int __cdecl __MINGW_NOTHROW _chdir (const char*); _CRTIMP char* __cdecl __MINGW_NOTHROW _getcwd (char*, int); @@ -143,8 +143,8 @@ #ifdef __MSVCRT__ _CRTIMP __int64 __cdecl __MINGW_NOTHROW _filelengthi64(int); -_CRTIMP long __cdecl __MINGW_NOTHROW _findfirsti64(const char*, struct _finddatai64_t*); -_CRTIMP int __cdecl __MINGW_NOTHROW _findnexti64(long, struct _finddatai64_t*); +_CRTIMP intptr_t __cdecl __MINGW_NOTHROW _findfirsti64(const char*, struct _finddatai64_t*); +_CRTIMP int __cdecl __MINGW_NOTHROW _findnexti64(intptr_t, struct _finddatai64_t*); _CRTIMP __int64 __cdecl __MINGW_NOTHROW _lseeki64(int, __int64, int); _CRTIMP __int64 __cdecl __MINGW_NOTHROW _telli64(int); /* These require newer versions of msvcrt.dll (6.1 or higher). */ @@ -259,14 +259,14 @@ _CRTIMP int __cdecl __MINGW_NOTHROW _waccess(const wchar_t*, int); _CRTIMP int __cdecl __MINGW_NOTHROW _wchmod(const wchar_t*, int); _CRTIMP int __cdecl __MINGW_NOTHROW _wcreat(const wchar_t*, int); -_CRTIMP long __cdecl __MINGW_NOTHROW _wfindfirst(const wchar_t*, struct _wfinddata_t*); -_CRTIMP int __cdecl __MINGW_NOTHROW _wfindnext(long, struct _wfinddata_t *); +_CRTIMP intptr_t __cdecl __MINGW_NOTHROW _wfindfirst(const wchar_t*, struct _wfinddata_t*); +_CRTIMP intptr_t __cdecl __MINGW_NOTHROW _wfindnext(intptr_t, struct _wfinddata_t *); _CRTIMP int __cdecl __MINGW_NOTHROW _wunlink(const wchar_t*); _CRTIMP int __cdecl __MINGW_NOTHROW _wopen(const wchar_t*, int, ...); _CRTIMP int __cdecl __MINGW_NOTHROW _wsopen(const wchar_t*, int, int, ...); _CRTIMP wchar_t * __cdecl __MINGW_NOTHROW _wmktemp(wchar_t*); -_CRTIMP long __cdecl __MINGW_NOTHROW _wfindfirsti64(const wchar_t*, struct _wfinddatai64_t*); -_CRTIMP int __cdecl __MINGW_NOTHROW _wfindnexti64(long, struct _wfinddatai64_t*); +_CRTIMP intptr_t __cdecl __MINGW_NOTHROW _wfindfirsti64(const wchar_t*, struct _wfinddatai64_t*); +_CRTIMP int __cdecl __MINGW_NOTHROW _wfindnexti64(intptr_t, struct _wfinddatai64_t*); #if __MSVCRT_VERSION__ >= 0x0601 _CRTIMP intptr_t __cdecl __MINGW_NOTHROW _wfindfirst64(const wchar_t*, struct __wfinddata64_t*); _CRTIMP intptr_t __cdecl __MINGW_NOTHROW _wfindnext64(intptr_t, struct __wfinddata64_t*); Modified: trunk/reactos/include/reactos/wine/library.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/reactos/wine/libra… ============================================================================== --- trunk/reactos/include/reactos/wine/library.h [iso-8859-1] (original) +++ trunk/reactos/include/reactos/wine/library.h [iso-8859-1] Tue Aug 26 22:59:24 2008 @@ -23,6 +23,7 @@ #include <stdarg.h> #include <sys/types.h> +#include <stdint.h> #include <windef.h> #include <winbase.h> @@ -131,9 +132,9 @@ /* helper functions to manipulate the LDT_ENTRY structure */ inline static void wine_ldt_set_base( LDT_ENTRY *ent, const void *base ) { - ent->BaseLow = (WORD)(unsigned long)base; - ent->HighWord.Bits.BaseMid = (BYTE)((unsigned long)base >> 16); - ent->HighWord.Bits.BaseHi = (BYTE)((unsigned long)base >> 24); + ent->BaseLow = (WORD)(intptr_t)base; + ent->HighWord.Bits.BaseMid = (BYTE)((intptr_t)base >> 16); + ent->HighWord.Bits.BaseHi = (BYTE)((intptr_t)base >> 24); } inline static void wine_ldt_set_limit( LDT_ENTRY *ent, unsigned int limit ) { @@ -144,8 +145,8 @@ inline static void *wine_ldt_get_base( const LDT_ENTRY *ent ) { return (void *)(ent->BaseLow | - (unsigned long)ent->HighWord.Bits.BaseMid << 16 | - (unsigned long)ent->HighWord.Bits.BaseHi << 24); + (intptr_t)ent->HighWord.Bits.BaseMid << 16 | + (intptr_t)ent->HighWord.Bits.BaseHi << 24); } inline static unsigned int wine_ldt_get_limit( const LDT_ENTRY *ent ) { Modified: trunk/reactos/lib/3rdparty/mingw/gccmain.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/3rdparty/mingw/gccmain… ============================================================================== --- trunk/reactos/lib/3rdparty/mingw/gccmain.c [iso-8859-1] (original) +++ trunk/reactos/lib/3rdparty/mingw/gccmain.c [iso-8859-1] Tue Aug 26 22:59:24 2008 @@ -13,6 +13,7 @@ /* Needed for the atexit prototype. */ #include <stdlib.h> +#include <stddef.h> typedef void (*func_ptr) (void); extern func_ptr __CTOR_LIST__[]; @@ -37,8 +38,8 @@ void __do_global_ctors (void) { - unsigned long nptrs = (unsigned long) __CTOR_LIST__[0]; - unsigned i; + unsigned long nptrs = (unsigned long) (ptrdiff_t) __CTOR_LIST__[0]; + unsigned long i; /* * If the first entry in the constructor list is -1 then the list @@ -47,8 +48,7 @@ */ if (nptrs == -1) { - for (nptrs = 0; __CTOR_LIST__[nptrs + 1] != 0; nptrs++) - ; + for (nptrs = 0; __CTOR_LIST__[nptrs + 1] != 0; nptrs++); } /* @@ -77,3 +77,4 @@ } } + Modified: trunk/reactos/lib/3rdparty/mingw/pseudo-reloc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/3rdparty/mingw/pseudo-… ============================================================================== --- trunk/reactos/lib/3rdparty/mingw/pseudo-reloc.c [iso-8859-1] (original) +++ trunk/reactos/lib/3rdparty/mingw/pseudo-reloc.c [iso-8859-1] Tue Aug 26 22:59:24 2008 @@ -28,12 +28,12 @@ static void do_pseudo_reloc (void* start, void* end, void* base) { - DWORD reloc_target; + ptrdiff_t reloc_target; runtime_pseudo_reloc* r; for (r = (runtime_pseudo_reloc*) start; r < (runtime_pseudo_reloc*) end; r++) { - reloc_target = (DWORD) base + r->target; - *((DWORD*) reloc_target) += r->addend; + reloc_target = (ptrdiff_t) base + r->target; + *((ptrdiff_t*) reloc_target) += r->addend; } } Modified: trunk/reactos/lib/sdk/crt/misc/getargs.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/sdk/crt/misc/getargs.c… ============================================================================== --- trunk/reactos/lib/sdk/crt/misc/getargs.c [iso-8859-1] (original) +++ trunk/reactos/lib/sdk/crt/misc/getargs.c [iso-8859-1] Tue Aug 26 22:59:24 2008 @@ -24,7 +24,7 @@ extern HANDLE hHeap; -char* strndup(char* name, int len) +char* strndup(char* name, size_t len) { char *s = malloc(len + 1); if (s != NULL) @@ -35,7 +35,7 @@ return s; } -wchar_t* wcsndup(wchar_t* name, int len) +wchar_t* wcsndup(wchar_t* name, size_t len) { wchar_t *s = malloc((len + 1) * sizeof(wchar_t)); if (s != NULL) @@ -362,3 +362,4 @@ return &__wargv; } + Modified: trunk/reactos/lib/sdk/crt/search/lfind.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/sdk/crt/search/lfind.c… ============================================================================== --- trunk/reactos/lib/sdk/crt/search/lfind.c [iso-8859-1] (original) +++ trunk/reactos/lib/sdk/crt/search/lfind.c [iso-8859-1] Tue Aug 26 22:59:24 2008 @@ -1,12 +1,11 @@ +#include <stdlib.h> #include <search.h> -#include <stdlib.h> - /* * @implemented */ -void *_lfind(const void *key, const void *base, size_t *nelp, - size_t width, int (*compar)(const void *, const void *)) +void *_lfind(const void *key, const void *base, unsigned int *nelp, + unsigned int width, int (*compar)(const void *, const void *)) { char* char_base = (char*)base; unsigned int i; @@ -19,3 +18,4 @@ return NULL; } + Modified: trunk/reactos/lib/sdk/crt/search/lsearch.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/sdk/crt/search/lsearch… ============================================================================== --- trunk/reactos/lib/sdk/crt/search/lsearch.c [iso-8859-1] (original) +++ trunk/reactos/lib/sdk/crt/search/lsearch.c [iso-8859-1] Tue Aug 26 22:59:24 2008 @@ -1,11 +1,11 @@ -#include <search.h> #include <stdlib.h> #include <string.h> +#include <search.h> /* * @implemented */ -void *_lsearch(const void *key, void *base, size_t *nelp, size_t width, +void *_lsearch(const void *key, void *base, unsigned int *nelp, unsigned int width, int (*compar)(const void *, const void *)) { void *ret_find = _lfind(key,base,nelp,width,compar); @@ -19,3 +19,4 @@ return base; } + Modified: trunk/reactos/lib/sdk/crt/stdio/find.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/sdk/crt/stdio/find.c?r… ============================================================================== --- trunk/reactos/lib/sdk/crt/stdio/find.c [iso-8859-1] (original) +++ trunk/reactos/lib/sdk/crt/stdio/find.c [iso-8859-1] Tue Aug 26 22:59:24 2008 @@ -4,17 +4,13 @@ /* * @implemented */ -#if defined(_UNICODE) || !(__MINGW32_MAJOR_VERSION < 3 || __MINGW32_MINOR_VERSION < 3) -long -#else -int -#endif +intptr_t _tfindfirst(const _TCHAR* _name, struct _tfinddata_t* result) { WIN32_FIND_DATA FindFileData; long hFindFile; - hFindFile = (long)FindFirstFile(_name, &FindFileData); + hFindFile = (intptr_t)FindFirstFile(_name, &FindFileData); if (hFindFile == -1) { _dosmaperr(GetLastError()); return -1; @@ -33,13 +29,7 @@ /* * @implemented */ -int _tfindnext( -#if defined(_UNICODE) || !(__MINGW32_MAJOR_VERSION < 3 || __MINGW32_MINOR_VERSION < 3) - long handle, -#else - int handle, -#endif - struct _tfinddata_t* result) +intptr_t _tfindnext(intptr_t handle, struct _tfinddata_t* result) { WIN32_FIND_DATA FindFileData; @@ -62,12 +52,12 @@ /* * @implemented */ -long _tfindfirsti64(const _TCHAR *_name, struct _tfinddatai64_t *result) +intptr_t _tfindfirsti64(const _TCHAR *_name, struct _tfinddatai64_t *result) { WIN32_FIND_DATA FindFileData; long hFindFile; - hFindFile = (long)FindFirstFile(_name, &FindFileData); + hFindFile = (intptr_t)FindFirstFile(_name, &FindFileData); if (hFindFile == -1) { _dosmaperr(GetLastError()); @@ -85,14 +75,10 @@ return hFindFile; } -//_CRTIMP long __cdecl _findfirsti64(const char*, struct _finddatai64_t*); -//_CRTIMP int __cdecl _findnexti64(long, struct _finddatai64_t*); - - /* * @implemented */ -int _tfindnexti64(long handle, struct _tfinddatai64_t *result) +int _tfindnexti64(intptr_t handle, struct _tfinddatai64_t *result) { WIN32_FIND_DATA FindFileData; @@ -119,15 +105,9 @@ /* * @implemented */ -int _findclose( -#if __MINGW32_MAJOR_VERSION < 3 || __MINGW32_MINOR_VERSION < 3 - int handle -#else - long handle -#endif - ) +int _findclose(intptr_t handle) { - if (!FindClose((void*)handle)) { + if (!FindClose((HANDLE)handle)) { _dosmaperr(GetLastError()); return -1; }

1 0

[tkreuzer] 35694: Add Cch functions to strsafe lib.
by tkreuzer＠svn.reactos.org 27 Aug '08

27 Aug '08

Author: tkreuzer Date: Tue Aug 26 22:03:39 2008 New Revision: 35694 URL: http://svn.reactos.org/svn/reactos?rev=35694&view=rev Log: Add Cch functions to strsafe lib. Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatExA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatExW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNExA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNExW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyExA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyExW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNExA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNExW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsExA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsExW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchLengthA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchLengthW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfExA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfExW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfExA.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfExW.c (with props) branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfW.c (with props) Modified: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/strsafe.rbuild Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,14 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCatA _StringCchCatA +#include <strsafe.h> + +#undef StringCchCatA +HRESULT __stdcall +StringCchCatA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPCSTR pszSrc) +{ + /* Use the inlined version */ + return _StringCchCatA(pszDest, cbDest, pszSrc); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatExA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatExA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatExA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,18 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCatExA _StringCchCatExA +#include <strsafe.h> + +#undef StringCchCatExA +HRESULT __stdcall +StringCchCatExA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPCSTR pszSrc, + STRSAFE_LPSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags) + +{ + /* Use the inlined version */ + return _StringCchCatExA(pszDest, cbDest, pszSrc, ppszDestEnd, pcbRemaining, dwFlags); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatExA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatExW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatExW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatExW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,18 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCatExW _StringCchCatExW +#include <strsafe.h> + +#undef StringCchCatExW +HRESULT __stdcall +StringCchCatExW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPCWSTR pszSrc, + STRSAFE_LPWSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags) + +{ + /* Use the inlined version */ + return _StringCchCatExW(pszDest, cbDest, pszSrc, ppszDestEnd, pcbRemaining, dwFlags); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatExW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,15 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCatNA _StringCchCatNA +#include <strsafe.h> + +#undef StringCchCatNA +HRESULT __stdcall +StringCbCatNA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPCSTR pszSrc, + size_t cbMaxAppend) +{ + /* Use the inlined version */ + return _StringCchCatNA(pszDest, cbDest, pszSrc, cbMaxAppend); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNExA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNExA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNExA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,19 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCatNExA _StringCchCatNExA +#include <strsafe.h> + +#undef StringCchCatNExA +HRESULT __stdcall +StringCchCatNExA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPCSTR pszSrc, + size_t cbMaxAppend, + STRSAFE_LPSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags) + +{ + /* Use the inlined version */ + return _StringCchCatNExA(pszDest, cbDest, pszSrc, cbMaxAppend, ppszDestEnd, pcbRemaining, dwFlags); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNExA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNExW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNExW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNExW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,19 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCatNExW _StringCchCatNExW +#include <strsafe.h> + +#undef StringCchCatNExW +HRESULT __stdcall +StringCchCatNExW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPCWSTR pszSrc, + size_t cbMaxAppend, + STRSAFE_LPWSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags) + +{ + /* Use the inlined version */ + return _StringCchCatNExW(pszDest, cbDest, pszSrc, cbMaxAppend, ppszDestEnd, pcbRemaining, dwFlags); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNExW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,15 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCatNW _StringCchCatNW +#include <strsafe.h> + +#undef StringCchCatNW +HRESULT __stdcall +StringCchCatNW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPCWSTR pszSrc, + size_t cbMaxAppend) +{ + /* Use the inlined version */ + return _StringCchCatNW(pszDest, cbDest, pszSrc, cbMaxAppend); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatNW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,14 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCatW _StringCchCatW +#include <strsafe.h> + +#undef StringCchCatW +HRESULT __stdcall +StringCchCatW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPCWSTR pszSrc) +{ + /* Use the inlined version */ + return _StringCchCatW(pszDest, cbDest, pszSrc); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCatW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,14 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCopyA _StringCchCopyA +#include <strsafe.h> + +#undef StringCchCopyA +HRESULT __stdcall +StringCbCopyA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPCSTR pszSrc) +{ + /* Use the inlined version */ + return _StringCchCopyA(pszDest, cbDest, pszSrc); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyExA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyExA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyExA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,17 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCopyExA _StringCchCopyExA +#include <strsafe.h> + +#undef StringCchCopyExA +HRESULT __stdcall +StringCchCopyExA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPCSTR pszSrc, + STRSAFE_LPSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags) +{ + /* Use the inlined version */ + return _StringCchCopyExA(pszDest, cbDest, pszSrc, ppszDestEnd, pcbRemaining, dwFlags); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyExA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyExW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyExW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyExW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,18 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCopyExW _StringCchCopyExW +#include <strsafe.h> + +#undef StringCchCopyExW +HRESULT __stdcall +StringCchCopyExW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPCWSTR pszSrc, + STRSAFE_LPWSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags) + +{ + /* Use the inlined version */ + return _StringCchCopyExW(pszDest, cbDest, pszSrc, ppszDestEnd, pcbRemaining, dwFlags); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyExW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,15 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCopyNA _StringCchCopyNA +#include <strsafe.h> + +#undef StringCchCopyNA +HRESULT __stdcall +StringCchCopyNA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPCSTR pszSrc, + size_t cbSrc) +{ + /* Use the inlined version */ + return _StringCchCopyNA(pszDest, cbDest, pszSrc, cbSrc); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNExA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNExA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNExA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,19 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCopyNExA _StringCchCopyNExA +#include <strsafe.h> + +#undef StringCchCopyNExA +HRESULT __stdcall +StringCchCopyNExA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPCSTR pszSrc, + size_t cbMaxAppend, + STRSAFE_LPSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags) + +{ + /* Use the inlined version */ + return _StringCchCopyNExA(pszDest, cbDest, pszSrc, cbMaxAppend, ppszDestEnd, pcbRemaining, dwFlags); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNExA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNExW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNExW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNExW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,19 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCopyNExW _StringCchCopyNExW +#include <strsafe.h> + +#undef StringCchCopyNExW +HRESULT __stdcall +StringCchCopyNExW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPCWSTR pszSrc, + size_t cbMaxAppend, + STRSAFE_LPWSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags) + +{ + /* Use the inlined version */ + return _StringCchCopyNExW(pszDest, cbDest, pszSrc, cbMaxAppend, ppszDestEnd, pcbRemaining, dwFlags); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNExW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,15 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCopyNW _StringCchCopyNW +#include <strsafe.h> + +#undef StringCchCopyNW +HRESULT __stdcall +StringCchCopyNW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPCWSTR pszSrc, + size_t cbSrc) +{ + /* Use the inlined version */ + return _StringCchCopyNW(pszDest, cbDest, pszSrc, cbSrc); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyNW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,14 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchCopyW _StringCchCopyW +#include <strsafe.h> + +#undef StringCchCopyW +HRESULT __stdcall +StringCchCopyW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPCWSTR pszSrc) +{ + /* Use the inlined version */ + return _StringCchCopyW(pszDest, cbDest, pszSrc); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchCopyW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,13 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchGetsA _StringCchGetsA +#include <strsafe.h> + +#undef StringCchGetsA +HRESULT __stdcall +StringCchGetsA( + STRSAFE_LPSTR pszDest, + size_t cbDest) +{ + /* Use the inlined version */ + return _StringCchGetsA(pszDest, cbDest); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsExA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsExA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsExA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,16 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchGetsExA _StringCchGetsExA +#include <strsafe.h> + +#undef StringCchGetsExA +HRESULT __stdcall +StringCchGetsExA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags) +{ + /* Use the inlined version */ + return _StringCchGetsExA(pszDest, cbDest, ppszDestEnd, pcbRemaining, dwFlags); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsExA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsExW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsExW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsExW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,16 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchGetsExW _StringCchGetsExW +#include <strsafe.h> + +#undef StringCchGetsExW +HRESULT __stdcall +StringCchGetsExW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPWSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags) +{ + /* Use the inlined version */ + return _StringCchGetsExW(pszDest, cbDest, ppszDestEnd, pcbRemaining, dwFlags); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsExW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,13 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchGetsW _StringCchGetsW +#include <strsafe.h> + +#undef StringCchGetsW +HRESULT __stdcall +StringCchGetsW( + STRSAFE_LPWSTR pszDest, + size_t cbDest) +{ + /* Use the inlined version */ + return _StringCchGetsW(pszDest, cbDest); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchGetsW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchLengthA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchLengthA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchLengthA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,14 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchLengthA _StringCchLengthA +#include <strsafe.h> + +#undef StringCchLengthA +HRESULT __stdcall +StringCchLengthA( + STRSAFE_LPCSTR psz, + size_t cbMax, + size_t *pcb) +{ + /* Use the inlined version */ + return _StringCchLengthA(psz, cbMax, pcb); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchLengthA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchLengthW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchLengthW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchLengthW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,14 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchLengthW _StringCchLengthW +#include <strsafe.h> + +#undef StringCchLengthW +HRESULT __stdcall +StringCchLengthW( + STRSAFE_LPCWSTR psz, + size_t cbMax, + size_t *pcb) +{ + /* Use the inlined version */ + return _StringCchLengthW(psz, cbMax, pcb); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchLengthW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,19 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchPrintfA _StringCchPrintfA +#include <strsafe.h> + +#undef StringCchPrintfA +HRESULT __stdcall +StringCchPrintfA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPCSTR pszFormat, + ...) +{ + HRESULT result; + va_list args; + va_start(args, pszFormat); + result = StringCchVPrintfA(pszDest, cbDest, pszFormat, args); + va_end(args); + return result; +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfExA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfExA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfExA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,22 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchPrintfExA _StringCchPrintfExA +#include <strsafe.h> + +#undef StringCchPrintfExA +HRESULT __stdcall +StringCchPrintfExA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags, + STRSAFE_LPCSTR pszFormat, + ...) +{ + HRESULT result; + va_list args; + va_start(args, pszFormat); + result = StringCchVPrintfExA(pszDest, cbDest, ppszDestEnd, pcbRemaining, dwFlags, pszFormat, args); + va_end(args); + return result; +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfExA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfExW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfExW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfExW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,22 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchPrintfExW _StringCchPrintfExW +#include <strsafe.h> + +#undef StringCchPrintfExW +HRESULT __stdcall +StringCchPrintfExW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPWSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags, + STRSAFE_LPCWSTR pszFormat, + ...) +{ + HRESULT result; + va_list args; + va_start(args, pszFormat); + result = StringCchVPrintfExW(pszDest, cbDest, ppszDestEnd, pcbRemaining, dwFlags, pszFormat, args); + va_end(args); + return result; +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfExW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,19 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchPrintfW _StringCchPrintfW +#include <strsafe.h> + +#undef StringCchPrintfW +HRESULT __stdcall +StringCchPrintfW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPCWSTR pszFormat, + ...) +{ + HRESULT result; + va_list args; + va_start(args, pszFormat); + result = StringCchVPrintfW(pszDest, cbDest, pszFormat, args); + va_end(args); + return result; +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchPrintfW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,15 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchVPrintfA _StringCchVPrintfA +#include <strsafe.h> + +#undef StringCchVPrintfA +HRESULT __stdcall +StringCchVPrintfA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPCSTR pszFormat, + va_list args) +{ + /* Use the inlined version */ + return _StringCchVPrintfA(pszDest, cbDest, pszFormat, args); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfExA.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfExA.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfExA.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,18 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchVPrintfExA _StringCchVPrintfExA +#include <strsafe.h> + +#undef StringCchVPrintfExA +HRESULT __stdcall +StringCchVPrintfExA( + STRSAFE_LPSTR pszDest, + size_t cbDest, + STRSAFE_LPSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags, + STRSAFE_LPCSTR pszFormat, + va_list args) +{ + /* Use the inlined version */ + return _StringCchVPrintfExA(pszDest, cbDest, ppszDestEnd, pcbRemaining, dwFlags, pszFormat, args); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfExA.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfExW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfExW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfExW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,18 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchVPrintfExW _StringCchVPrintfExW +#include <strsafe.h> + +#undef StringCchVPrintfExW +HRESULT __stdcall +StringCchVPrintfExW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPWSTR *ppszDestEnd, + size_t *pcbRemaining, + STRSAFE_DWORD dwFlags, + STRSAFE_LPCWSTR pszFormat, + va_list args) +{ + /* Use the inlined version */ + return _StringCchVPrintfExW(pszDest, cbDest, ppszDestEnd, pcbRemaining, dwFlags, pszFormat, args); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfExW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfW.c URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfW.c (added) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfW.c [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -1,0 +1,15 @@ +#define STRSAFE_NO_CB_FUNCTIONS +#define StringCchVPrintfW _StringCchVPrintfW +#include <strsafe.h> + +#undef StringCchVPrintfW +HRESULT __stdcall +StringCchVPrintfW( + STRSAFE_LPWSTR pszDest, + size_t cbDest, + STRSAFE_LPCWSTR pszFormat, + va_list args) +{ + /* Use the inlined version */ + return _StringCchVPrintfW(pszDest, cbDest, pszFormat, args); +} Propchange: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/StringCchVPrintfW.c ------------------------------------------------------------------------------ svn:eol-style = native Modified: branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/strsafe.rbuild URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/lib/s… ============================================================================== --- branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/strsafe.rbuild [iso-8859-1] (original) +++ branches/ros-amd64-bringup/reactos/lib/sdk/strsafe/strsafe.rbuild [iso-8859-1] Tue Aug 26 22:03:39 2008 @@ -32,4 +32,34 @@ <file>StringCbVPrintfW.c</file> <file>StringCbVPrintfExA.c</file> <file>StringCbVPrintfExW.c</file> + <file>StringCchCatA.c</file> + <file>StringCchCatW.c</file> + <file>StringCchCatExA.c</file> + <file>StringCchCatExW.c</file> + <file>StringCchCatNA.c</file> + <file>StringCchCatNW.c</file> + <file>StringCchCatNExA.c</file> + <file>StringCchCatNExW.c</file> + <file>StringCchCopyA.c</file> + <file>StringCchCopyW.c</file> + <file>StringCchCopyExA.c</file> + <file>StringCchCopyExW.c</file> + <file>StringCchCopyNA.c</file> + <file>StringCchCopyNW.c</file> + <file>StringCchCopyNExA.c</file> + <file>StringCchCopyNExW.c</file> + <file>StringCchGetsA.c</file> + <file>StringCchGetsW.c</file> + <file>StringCchGetsExA.c</file> + <file>StringCchGetsExW.c</file> + <file>StringCchLengthA.c</file> + <file>StringCchLengthW.c</file> + <file>StringCchPrintfA.c</file> + <file>StringCchPrintfW.c</file> + <file>StringCchPrintfExA.c</file> + <file>StringCchPrintfExW.c</file> + <file>StringCchVPrintfA.c</file> + <file>StringCchVPrintfW.c</file> + <file>StringCchVPrintfExA.c</file> + <file>StringCchVPrintfExW.c</file> </module>

1 0

[tkreuzer] 35693: More fixes in strsafe.h
by tkreuzer＠svn.reactos.org 27 Aug '08

27 Aug '08

Author: tkreuzer Date: Tue Aug 26 22:01:25 2008 New Revision: 35693 URL: http://svn.reactos.org/svn/reactos?rev=35693&view=rev Log: More fixes in strsafe.h Modified: branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h Modified: branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/inclu… ============================================================================== --- branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h [iso-8859-1] (original) +++ branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h [iso-8859-1] Tue Aug 26 22:01:25 2008 @@ -218,8 +218,6 @@ /* Create inlined versions */ #define STRSAFEAPI HRESULT static __inline__ -#ifndef STRSAFE_NO_CB_FUNCTIONS - STRSAFEAPI StringCxxCat(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc) { return 0; // FIXME @@ -325,12 +323,6 @@ va_end(args); return result; } - -#endif // !STRSAFE_NO_CB_FUNCTIONS - -#ifndef STRSAFE_NO_CCH_FUNCTIONS - -#endif // !STRSAFE_NO_CCH_FUNCTIONS #endif // !STRSAFE_LIB

1 0

[tkreuzer] 35692: Fix some strsafe.h definitions.
by tkreuzer＠svn.reactos.org 27 Aug '08

27 Aug '08

Author: tkreuzer Date: Tue Aug 26 21:08:24 2008 New Revision: 35692 URL: http://svn.reactos.org/svn/reactos?rev=35692&view=rev Log: Fix some strsafe.h definitions. Modified: branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h Modified: branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/inclu… ============================================================================== --- branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h [iso-8859-1] (original) +++ branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h [iso-8859-1] Tue Aug 26 21:08:24 2008 @@ -28,6 +28,7 @@ #define STRSAFE_PASS2 /* Implement Cb functions for ansi and unicode */ +#ifndef STRSAFE_NO_CB_FUNCTIONS #define STRSAFE_CB #define STRSAFE_UNICODE 0 # include <strsafe.h> @@ -36,14 +37,17 @@ # include <strsafe.h> #undef STRSAFE_UNICODE #undef STRSAFE_CB +#endif // !STRSAFE_NO_CB_FUNCTIONS /* Implement Cch functions for ansi and unicode */ +#ifndef STRSAFE_NO_CCH_FUNCTIONS #define STRSAFE_UNICODE 0 # include <strsafe.h> #undef STRSAFE_UNICODE #define STRSAFE_UNICODE 1 # include <strsafe.h> #undef STRSAFE_UNICODE +#endif // !STRSAFE_NO_CCH_FUNCTIONS #undef STRSAFE_PASS2 @@ -193,28 +197,21 @@ /* Normal function prototypes only */ #define STRSAFEAPI HRESULT __stdcall -#ifndef STRSAFE_NO_CB_FUNCTIONS -STRSAFEAPI StringCat(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc); -STRSAFEAPI StringCatEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags); -STRSAFEAPI StringCatN(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, size_t cbMaxAppend); -STRSAFEAPI StringCatNEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, size_t cbMaxAppend, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags); -STRSAFEAPI StringCopy(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc); -STRSAFEAPI StringCopyEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags); -STRSAFEAPI StringCopyN(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, size_t cbSrc); -STRSAFEAPI StringCopyNEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, size_t cbSrc, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags); -STRSAFEAPI StringGets(STRSAFE_LPTSTR pszDest, size_t cxDest); -STRSAFEAPI StringGetsEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags); -STRSAFEAPI StringLength(STRSAFE_LPCTSTR psz, size_t cxMax, size_t *pcb); -STRSAFEAPI StringPrintf(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszFormat, ...); -STRSAFEAPI StringPrintfEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags, STRSAFE_LPCTSTR pszFormat, ...); -STRSAFEAPI StringVPrintf(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszFormat, va_list args); -STRSAFEAPI StringVPrintfEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags, LPCTSTR pszFormat, va_list args); -#endif // !STRSAFE_NO_CB_FUNCTIONS - -#ifndef STRSAFE_NO_CCH_FUNCTIONS - -#endif // !STRSAFE_NO_CCH_FUNCTIONS - +STRSAFEAPI StringCxxCat(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc); +STRSAFEAPI StringCxxCatEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags); +STRSAFEAPI StringCxxCatN(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, size_t cbMaxAppend); +STRSAFEAPI StringCxxCatNEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, size_t cbMaxAppend, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags); +STRSAFEAPI StringCxxCopy(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc); +STRSAFEAPI StringCxxCopyEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags); +STRSAFEAPI StringCxxCopyN(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, size_t cbSrc); +STRSAFEAPI StringCxxCopyNEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszSrc, size_t cbSrc, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags); +STRSAFEAPI StringCxxGets(STRSAFE_LPTSTR pszDest, size_t cxDest); +STRSAFEAPI StringCxxGetsEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags); +STRSAFEAPI StringCxxLength(STRSAFE_LPCTSTR psz, size_t cxMax, size_t *pcb); +STRSAFEAPI StringCxxPrintf(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszFormat, ...); +STRSAFEAPI StringCxxPrintfEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags, STRSAFE_LPCTSTR pszFormat, ...); +STRSAFEAPI StringCxxVPrintf(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPCTSTR pszFormat, va_list args); +STRSAFEAPI StringCxxVPrintfEx(STRSAFE_LPTSTR pszDest, size_t cxDest, STRSAFE_LPTSTR *ppszDestEnd, size_t *pcbRemaining, STRSAFE_DWORD dwFlags, LPCTSTR pszFormat, va_list args); #else // !STRSAFE_LIB

1 0

[tkreuzer] 35691: Optimize and shorten StringCxxLength
by tkreuzer＠svn.reactos.org 27 Aug '08

27 Aug '08

Author: tkreuzer Date: Tue Aug 26 20:47:34 2008 New Revision: 35691 URL: http://svn.reactos.org/svn/reactos?rev=35691&view=rev Log: Optimize and shorten StringCxxLength Modified: branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h Modified: branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h URL: http://svn.reactos.org/svn/reactos/branches/ros-amd64-bringup/reactos/inclu… ============================================================================== --- branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h [iso-8859-1] (original) +++ branches/ros-amd64-bringup/reactos/include/psdk/strsafe.h [iso-8859-1] Tue Aug 26 20:47:34 2008 @@ -286,11 +286,7 @@ return STRSAFE_E_INVALID_PARAMETER; } - while (*psz != 0 && cch != 0) - { - cch--; - psz++; - } + for (--psz; *(++psz) != 0 && --cch > 0;); if (cch == 0) {

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