- Ros-dev - ReactOS.org

Re: [ros-dev] [ros-diffs] [jgardou] 64121: [KDGDB] - introduce KDGDB, a KDCOM-like DLL, wrapping the KD protocol and the GDB remote protocol together. It is not fully functional, but for now it permits source-level debuggin...
by Timo Kreuzer 13 Sep '14

13 Sep '14

Awesome stuff! Am 11.09.2014 22:55, schrieb jgardou(a)svn.reactos.org: > Author: jgardou > Date: Thu Sep 11 20:55:42 2014 > New Revision: 64121 > > URL: http://svn.reactos.org/svn/reactos?rev=64121&view=rev > Log: > [KDGDB] > - introduce KDGDB, a KDCOM-like DLL, wrapping the KD protocol and the GDB remote protocol together. > It is not fully functional, but for now it permits source-level debugging in some modules. More will be added as I feel the need and find the time to work a bit more on it. (That is, unless an angel comes and resume the work) > To use it, set GDB and _WINKD_ to TRUE in your CMakeCache.txt. Using separate debug symbols is also a good idea. > >

5 11

Re: [ros-dev] [ros-diffs] [jgardou] 64127: [KDGDB] - Add a callback mechanism permitting to "simulate" KD send <-> receive loop without having to actually communicate to GDB - Use that to update the program counter when con...
by Timo Kreuzer 12 Sep '14

12 Sep '14

while(1); is not really a good solution. It should be something that is visible in GDB. But I fear that a KeBugCheck() won't work here, since it will rely on a working KD connection. Maybe a KD_ASSERT() macro, that calls KdAssert() on failure or something, resets the GDB connection somehow, issues a debug print and then halts. Am 12.09.2014 22:23, schrieb jgardou(a)svn.reactos.org: > Author: jgardou > Date: Fri Sep 12 20:23:08 2014 > New Revision: 64127 > > URL: http://svn.reactos.org/svn/reactos?rev=64127&view=rev > Log: > [KDGDB] > - Add a callback mechanism permitting to "simulate" KD send <-> receive loop without having to actually communicate to GDB > - Use that to update the program counter when cont'ing a breakpoint > Now cont'ing an assertion failure is possible, since we actually get beyond the int 3 instruction > > Modified: > trunk/reactos/drivers/base/kdgdb/gdb_input.c > trunk/reactos/drivers/base/kdgdb/kdgdb.h > trunk/reactos/drivers/base/kdgdb/kdpacket.c > > +) > +{ > + DBGKD_MANIPULATE_STATE64* State = (DBGKD_MANIPULATE_STATE64*)MessageHeader->Buffer; > + > + /* We just confirm that all went well */ > + if ((PacketType != PACKET_TYPE_KD_STATE_MANIPULATE) > + || (State->ApiNumber != DbgKdSetContextApi) > + || (State->ReturnStatus != STATUS_SUCCESS)) > + { > + /* Should we bugcheck ? */ > + while (1); > + } > +} > + > +static > +KDSTATUS > +SetContext( > + _Out_ DBGKD_MANIPULATE_STATE64* State, > + _Out_ PSTRING MessageData, > + _Out_ PULONG MessageLength, > + _Inout_ PKD_CONTEXT KdContext, > + _In_opt_ KDP_MANIPULATESTATE_HANDLER ManipulateStateHandler > +) > +{ > + State->ApiNumber = DbgKdSetContextApi; > + State->Processor = CurrentStateChange.Processor; > + State->ReturnStatus = STATUS_SUCCESS; > + State->ProcessorLevel = CurrentStateChange.ProcessorLevel; > + MessageData->Length = sizeof(CurrentContext); > + > + if (MessageData->MaximumLength < sizeof(CurrentContext)) > + { > + while (1); > + } > + >

2 1

Re: [ros-dev] [ros-diffs] [tkreuzer] 64078: [PSEH] - On clang builds we need to do the return twice trick, since we don't have asm goto, which would allow us to specify labels in the code where we can branch to on an except...
by Thomas Faber 12 Sep '14

12 Sep '14

On 2014-09-07 23:40, tkreuzer(a)svn.reactos.org wrote: > --- trunk/reactos/lib/pseh/i386/pseh3_i386.S [iso-8859-1] (original) > +++ trunk/reactos/lib/pseh/i386/pseh3_i386.S [iso-8859-1] Sun Sep 7 21:40:07 2014 > @@ -30,6 +30,10 @@ > mov [eax + SEH3_REGISTRATION_FRAME_Esi], esi > mov [eax + SEH3_REGISTRATION_FRAME_Edi], edi > > + /* Safe the return address */ > + mov ebx, [esp] > + mov [eax + SEH3_REGISTRATION_FRAME_ReturnAddress], ebx > + > .global __SEH3$_RegisterFrameWithStackLayout > __SEH3$_RegisterFrameWithStackLayout: > I think we need to restore ebx before returning? I don't see us telling the compiler that it was clobbered anywhere.

1 0

Re: [ros-dev] [ros-diffs] [akhaldi] 64113: [GLU32] * Link with mingwex. * Allow multiple definitions at link time because longjmp exists in both our msvcrt and mingwex. CORE-8516
by Timo Kreuzer 11 Sep '14

11 Sep '14

Are you sure that we will link to the correct one? I doubt that the one from mingwex is compatible. Am 11.09.2014 12:13, schrieb akhaldi(a)svn.reactos.org: > Author: akhaldi > Date: Thu Sep 11 10:13:05 2014 > New Revision: 64113 > > URL: http://svn.reactos.org/svn/reactos?rev=64113&view=rev > Log: > [GLU32] > * Link with mingwex. > * Allow multiple definitions at link time because longjmp exists in both our msvcrt and mingwex. > CORE-8516 > > Modified: > trunk/reactos/dll/opengl/glu32/CMakeLists.txt > > Modified: trunk/reactos/dll/opengl/glu32/CMakeLists.txt > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/opengl/glu32/CMakeList… > ============================================================================== > --- trunk/reactos/dll/opengl/glu32/CMakeLists.txt [iso-8859-1] (original) > +++ trunk/reactos/dll/opengl/glu32/CMakeLists.txt [iso-8859-1] Thu Sep 11 10:13:05 2014 > @@ -114,6 +114,11 @@ > > if(NOT MSVC) > add_target_compile_flags(glu32 "-Wno-write-strings -Wno-unused-but-set-variable") > + if(CMAKE_C_COMPILER_ID STREQUAL "Clang") > + target_link_libraries(glu32 mingwex) > + #FIXME: longjmp exists in both our msvcrt and mingwex > + add_target_link_flags(glu32 "-Wl,--allow-multiple-definition") > + endif() > endif() > > add_importlibs(glu32 opengl32 gdi32 msvcrt kernel32 ntdll) > > >

1 0

Re: [ros-dev] [ros-diffs] [akhaldi] 64094: [CMAKE/CLANG] * Globally link mingwex for C modules. CORE-8516
by Jérôme Gardou 09 Sep '14

09 Sep '14

Hey Amine Is there any reason to not use our own mingwex build ? Le 09/09/2014 20:11, akhaldi(a)svn.reactos.org a écrit : > Author: akhaldi > Date: Tue Sep 9 18:11:10 2014 > New Revision: 64094 > > URL: http://svn.reactos.org/svn/reactos?rev=64094&view=rev > Log: > [CMAKE/CLANG] > * Globally link mingwex for C modules. > CORE-8516 > > Modified: > trunk/reactos/toolchain-clang.cmake > > Modified: trunk/reactos/toolchain-clang.cmake > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/toolchain-clang.cmake?rev=… > ============================================================================== > --- trunk/reactos/toolchain-clang.cmake [iso-8859-1] (original) > +++ trunk/reactos/toolchain-clang.cmake [iso-8859-1] Tue Sep 9 18:11:10 2014 > @@ -56,7 +56,7 @@ > set(CMAKE_ASM_CREATE_STATIC_LIBRARY ${CMAKE_C_CREATE_STATIC_LIBRARY}) > > # Don't link with anything by default unless we say so > -set(CMAKE_C_STANDARD_LIBRARIES "-lgcc" CACHE STRING "Standard C Libraries") > +set(CMAKE_C_STANDARD_LIBRARIES "-lmingwex -lgcc" CACHE STRING "Standard C Libraries") > > #MARK_AS_ADVANCED(CLEAR CMAKE_CXX_STANDARD_LIBRARIES) > set(CMAKE_CXX_STANDARD_LIBRARIES "-lgcc" CACHE STRING "Standard C++ Libraries") > >

1 0

Re: [ros-dev] [ros-diffs] [tkreuzer] 64078: [PSEH] - On clang builds we need to do the return twice trick, since we don't have asm goto, which would allow us to specify labels in the code where we can branch to on an except...
by Thomas Faber 08 Sep '14

08 Sep '14

Making the member conditional but writing to it unconditionally seems like it would overflow the stack variable on GCC? On 2014-09-07 23:40, tkreuzer(a)svn.reactos.org wrote: > +#ifdef __clang__ > + void *ReturnAddress; > #endif > + /* Safe the return address */ > + mov ebx, [esp] > + mov [eax + SEH3_REGISTRATION_FRAME_ReturnAddress], ebx

2 1

Re: [ros-dev] [ros-diffs] [jgardou] 64022: [SECUR32] - Update spec file [SCHANNEL] - Update stubs list Please blame binutils
by Timo Kreuzer 01 Sep '14

01 Sep '14

Am 01.09.2014 14:21, schrieb jgardou(a)svn.reactos.org: > Author: jgardou > Date: Mon Sep 1 12:21:16 2014 > New Revision: 64022 > > URL: http://svn.reactos.org/svn/reactos?rev=64022&view=rev > Log: > [SECUR32] > - Update spec file > [SCHANNEL] > - Update stubs list > Please blame binutils > > Modified: > trunk/reactos/dll/win32/schannel/schannel.spec > trunk/reactos/dll/win32/schannel/schannel_wine.c > trunk/reactos/dll/win32/schannel/stubs.c > trunk/reactos/dll/win32/secur32/secur32.spec > > Modified: trunk/reactos/dll/win32/schannel/schannel.spec > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/schannel/schanne… > ============================================================================== > --- trunk/reactos/dll/win32/schannel/schannel.spec [iso-8859-1] (original) > +++ trunk/reactos/dll/win32/schannel/schannel.spec [iso-8859-1] Mon Sep 1 12:21:16 2014 > @@ -15,14 +15,14 @@ > @ stdcall InitSecurityInterfaceW() schan_InitSecurityInterfaceW > @ stdcall InitializeSecurityContextA(ptr ptr str long long long ptr long ptr ptr ptr ptr) schan_InitializeSecurityContextA > @ stdcall InitializeSecurityContextW(ptr ptr wstr long long long ptr long ptr ptr ptr ptr) schan_InitializeSecurityContextW > -@ stdcall MakeSignature(ptr long ptr long) secur32.MakeSignature > +@ stdcall MakeSignature(ptr long ptr long) schan_MakeSignature > @ stub OpenSslPerformanceData > @ stdcall QueryContextAttributesA(ptr long ptr) schan_QueryContextAttributesA > @ stdcall QueryContextAttributesW(ptr long ptr) schan_QueryContextAttributesW > -@ stdcall QuerySecurityPackageInfoA(str ptr) secur32.QuerySecurityPackageInfoA > -@ stdcall QuerySecurityPackageInfoW(wstr ptr) secur32.QuerySecurityPackageInfoW > -@ stdcall RevertSecurityContext(ptr) secur32.RevertSecurityContext > -@ stdcall SealMessage(ptr long ptr long) secur32.SealMessage > +@ stdcall QuerySecurityPackageInfoA(str ptr) schan_QuerySecurityPackageInfoA > +@ stdcall QuerySecurityPackageInfoW(wstr ptr) schan_QuerySecurityPackageInfoW > +@ stdcall RevertSecurityContext(ptr) schan_RevertSecurityContext > +@ stdcall SealMessage(ptr long ptr long) schan_EncryptMessage > @ stdcall SpLsaModeInitialize(long ptr ptr ptr) > @ stdcall SpUserModeInitialize(long ptr ptr ptr) > @ stub SslCrackCertificate > @@ -33,5 +33,5 @@ > @ stub SslGenerateRandomBits > @ stub SslGetMaximumKeySize > @ stub SslLoadCertificate > -@ stdcall UnsealMessage(ptr ptr long ptr) secur32.UnsealMessage > -@ stdcall VerifySignature(ptr ptr long ptr) secur32.VerifySignature > +@ stdcall UnsealMessage(ptr ptr long ptr) schan_DecryptMessage > +@ stdcall VerifySignature(ptr ptr long ptr) schan_VerifySignature This change is not correct. The following functions from schannel are forwarded to secur32: MakeSignature, QueryContextAttributesA/W, QuerySecurityPackageInfoA/W, RevertSecurityContext, SealMessage, UnsealMessage, VerifySignature Timo

1 0

Re: [ros-dev] [ros-diffs] [hbelusca] 64017: [BOOTVID] - Synchronize correctly arm/bootdata.c with i386, as it was done previously. - Code formatting: whitespace fixes, add braces/brackets and spaces where needed; comments s...
by Timo Kreuzer 01 Sep '14

01 Sep '14

Am 01.09.2014 01:00, schrieb hbelusca(a)svn.reactos.org: > Author: hbelusca > Date: Sun Aug 31 23:00:29 2014 > New Revision: 64017 > > URL: http://svn.reactos.org/svn/reactos?rev=64017&view=rev > Log: > [BOOTVID] > - Synchronize correctly arm/bootdata.c with i386, as it was done previously. Why is that file duplicated at all, if the contents are identical?

2 1

Re: [ros-dev] [ros-diffs] [tfaber] 63953: [FASTFAT] - Do not use direct I/O since it limits read/write operations to 64 MB CORE-8410 #resolve
by cameron.gutman＠reactos.org 27 Aug '14

27 Aug '14

This seems strange. I understand that NT 5.x has a limitation of 64 MB MDLs on x86, but direct I/O was designed for exactly this case of large I/O operations. Has anyone verified that 2k3 doesn’t do MDL chaining for direct I/O when the transfer is larger than a single MDL can hold? Also aren’t there changes to the IRP dispatch routines required to cope with not having an Irp->MdlAddress pointer anymore? Cameron From: tfaber(a)svn.reactos.org Sent: ‎Tuesday‎, ‎August‎ ‎26‎, ‎2014 ‎6‎:‎41‎ ‎AM To: ros-diffs(a)reactos.org Author: tfaber Date: Tue Aug 26 13:41:57 2014 New Revision: 63953 URL: http://svn.reactos.org/svn/reactos?rev=63953&view=rev Log: [FASTFAT] - Do not use direct I/O since it limits read/write operations to 64 MB CORE-8410 #resolve Modified: trunk/reactos/drivers/filesystems/fastfat/fsctl.c Modified: trunk/reactos/drivers/filesystems/fastfat/fsctl.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/fsctl.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/fsctl.c [iso-8859-1] Tue Aug 26 13:41:57 2014 @@ -442,8 +442,7 @@ goto ByeBye; } - DeviceObject->Flags = DeviceObject->Flags | DO_DIRECT_IO; - DeviceExt = (PVOID) DeviceObject->DeviceExtension; + DeviceExt = DeviceObject->DeviceExtension; RtlZeroMemory(DeviceExt, ROUND_UP(sizeof(DEVICE_EXTENSION), sizeof(ULONG)) + sizeof(HASHENTRY*) * HashTableSize); DeviceExt->FcbHashTable = (HASHENTRY**)((ULONG_PTR)DeviceExt + ROUND_UP(sizeof(DEVICE_EXTENSION), sizeof(ULONG))); DeviceExt->HashTableSize = HashTableSize;

2 1

mplay translation
by Javier Agustìn Fernàndez Arroyo 26 Aug '14

26 Aug '14

hello folks, i have just updated the spanish translatiuon of mplay. Its here: https://jira.reactos.org/browse/CORE-8470 Enjoy! :)

1 0

Re: [ros-dev] [ros-diffs] [tfaber] 63928: [FASTFAT] - Properly handle errors in CcInitializeCacheMap, CcCopyRead and CcCopyWrite CORE-8410
by Jérôme Gardou 25 Aug '14

25 Aug '14

Hey Thomas I think that you want to use _SEH2_YIELD in a few places here, that is, unless we want to completely ditch pseh2 in favor of pseh3 ;-) Regards Jérôme Le 24/08/2014 05:28, tfaber(a)svn.reactos.org a écrit : > Author: tfaber > Date: Sun Aug 24 03:28:01 2014 > New Revision: 63928 > > URL: http://svn.reactos.org/svn/reactos?rev=63928&view=rev > Log: > [FASTFAT] > - Properly handle errors in CcInitializeCacheMap, CcCopyRead and CcCopyWrite > CORE-8410 > > Modified: > trunk/reactos/drivers/filesystems/fastfat/CMakeLists.txt > trunk/reactos/drivers/filesystems/fastfat/fcb.c > trunk/reactos/drivers/filesystems/fastfat/fsctl.c > trunk/reactos/drivers/filesystems/fastfat/rw.c > trunk/reactos/drivers/filesystems/fastfat/vfat.h > > Modified: trunk/reactos/drivers/filesystems/fastfat/CMakeLists.txt > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… > ============================================================================== > --- trunk/reactos/drivers/filesystems/fastfat/CMakeLists.txt [iso-8859-1] (original) > +++ trunk/reactos/drivers/filesystems/fastfat/CMakeLists.txt [iso-8859-1] Sun Aug 24 03:28:01 2014 > @@ -26,6 +26,7 @@ > add_library(fastfat SHARED ${SOURCE} vfatfs.rc) > > set_module_type(fastfat kernelmodedriver) > +target_link_libraries(fastfat ${PSEH_LIB}) > add_importlibs(fastfat ntoskrnl hal) > > add_pch(fastfat vfat.h SOURCE) > > Modified: trunk/reactos/drivers/filesystems/fastfat/fcb.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… > ============================================================================== > --- trunk/reactos/drivers/filesystems/fastfat/fcb.c [iso-8859-1] (original) > +++ trunk/reactos/drivers/filesystems/fastfat/fcb.c [iso-8859-1] Sun Aug 24 03:28:01 2014 > @@ -340,12 +340,14 @@ > { > PFILE_OBJECT fileObject; > PVFATCCB newCCB; > + NTSTATUS status; > > fileObject = IoCreateStreamFileObject (NULL, vcb->StorageDevice); > > newCCB = ExAllocateFromNPagedLookasideList(&VfatGlobalData->CcbLookasideList); > if (newCCB == NULL) > { > + ObDereferenceObject(fileObject); > return STATUS_INSUFFICIENT_RESOURCES; > } > RtlZeroMemory(newCCB, sizeof (VFATCCB)); > @@ -356,11 +358,24 @@ > fcb->FileObject = fileObject; > fcb->RefCount++; > > - CcInitializeCacheMap(fileObject, > - (PCC_FILE_SIZES)(&fcb->RFCB.AllocationSize), > - TRUE, > - &VfatGlobalData->CacheMgrCallbacks, > - fcb); > + _SEH2_TRY > + { > + CcInitializeCacheMap(fileObject, > + (PCC_FILE_SIZES)(&fcb->RFCB.AllocationSize), > + TRUE, > + &VfatGlobalData->CacheMgrCallbacks, > + fcb); > + } > + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) > + { > + status = _SEH2_GetExceptionCode(); > + fcb->RefCount--; > + fcb->FileObject = NULL; > + ExFreeToNPagedLookasideList(&VfatGlobalData->CcbLookasideList, newCCB); > + ObDereferenceObject(fileObject); > + return status; > + } > + _SEH2_END; > > fcb->Flags |= FCB_CACHE_INITIALIZED; > return STATUS_SUCCESS; > > Modified: trunk/reactos/drivers/filesystems/fastfat/fsctl.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… > ============================================================================== > --- trunk/reactos/drivers/filesystems/fastfat/fsctl.c [iso-8859-1] (original) > +++ trunk/reactos/drivers/filesystems/fastfat/fsctl.c [iso-8859-1] Sun Aug 24 03:28:01 2014 > @@ -550,11 +550,20 @@ > Fcb->RFCB.ValidDataLength = Fcb->RFCB.FileSize; > Fcb->RFCB.AllocationSize = Fcb->RFCB.FileSize; > > - CcInitializeCacheMap(DeviceExt->FATFileObject, > - (PCC_FILE_SIZES)(&Fcb->RFCB.AllocationSize), > - TRUE, > - &VfatGlobalData->CacheMgrCallbacks, > - Fcb); > + _SEH2_TRY > + { > + CcInitializeCacheMap(DeviceExt->FATFileObject, > + (PCC_FILE_SIZES)(&Fcb->RFCB.AllocationSize), > + TRUE, > + &VfatGlobalData->CacheMgrCallbacks, > + Fcb); > + } > + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) > + { > + Status = _SEH2_GetExceptionCode(); > + goto ByeBye; > + } > + _SEH2_END; > > DeviceExt->LastAvailableCluster = 2; > ExInitializeResourceLite(&DeviceExt->FatResource); > > Modified: trunk/reactos/drivers/filesystems/fastfat/rw.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… > ============================================================================== > --- trunk/reactos/drivers/filesystems/fastfat/rw.c [iso-8859-1] (original) > +++ trunk/reactos/drivers/filesystems/fastfat/rw.c [iso-8859-1] Sun Aug 24 03:28:01 2014 > @@ -673,22 +673,35 @@ > Status = /*STATUS_END_OF_FILE*/STATUS_SUCCESS; > } > > - if (IrpContext->FileObject->PrivateCacheMap == NULL) > - { > - CcInitializeCacheMap(IrpContext->FileObject, > - (PCC_FILE_SIZES)(&Fcb->RFCB.AllocationSize), > - FALSE, > - &(VfatGlobalData->CacheMgrCallbacks), > - Fcb); > - } > - > - if (!CcCopyRead(IrpContext->FileObject, &ByteOffset, Length, > - (BOOLEAN)(IrpContext->Flags & IRPCONTEXT_CANWAIT), Buffer, > - &IrpContext->Irp->IoStatus)) > - { > - Status = STATUS_PENDING; > - goto ByeBye; > - } > + _SEH2_TRY > + { > + if (IrpContext->FileObject->PrivateCacheMap == NULL) > + { > + CcInitializeCacheMap(IrpContext->FileObject, > + (PCC_FILE_SIZES)(&Fcb->RFCB.AllocationSize), > + FALSE, > + &(VfatGlobalData->CacheMgrCallbacks), > + Fcb); > + } > + > + if (!CcCopyRead(IrpContext->FileObject, > + &ByteOffset, > + Length, > + (IrpContext->Flags & IRPCONTEXT_CANWAIT) != 0, > + Buffer, > + &IrpContext->Irp->IoStatus)) > + { > + ASSERT((IrpContext->Flags & IRPCONTEXT_CANWAIT) == 0); > + Status = STATUS_PENDING; > + goto ByeBye; > + } > + } > + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) > + { > + Status = _SEH2_GetExceptionCode(); > + goto ByeBye; > + } > + _SEH2_END; > > if (!NT_SUCCESS(IrpContext->Irp->IoStatus.Status)) > { > @@ -947,30 +960,42 @@ > { > // cached write > > - if (IrpContext->FileObject->PrivateCacheMap == NULL) > - { > - CcInitializeCacheMap(IrpContext->FileObject, > - (PCC_FILE_SIZES)(&Fcb->RFCB.AllocationSize), > - FALSE, > - &VfatGlobalData->CacheMgrCallbacks, > - Fcb); > - } > - > - if (ByteOffset.QuadPart > OldFileSize.QuadPart) > - { > - CcZeroData(IrpContext->FileObject, &OldFileSize, &ByteOffset, TRUE); > - } > - > - if (CcCopyWrite(IrpContext->FileObject, &ByteOffset, Length, > - 1 /*IrpContext->Flags & IRPCONTEXT_CANWAIT*/, Buffer)) > - { > - IrpContext->Irp->IoStatus.Information = Length; > - Status = STATUS_SUCCESS; > - } > - else > - { > - Status = STATUS_UNSUCCESSFUL; > - } > + _SEH2_TRY > + { > + if (IrpContext->FileObject->PrivateCacheMap == NULL) > + { > + CcInitializeCacheMap(IrpContext->FileObject, > + (PCC_FILE_SIZES)(&Fcb->RFCB.AllocationSize), > + FALSE, > + &VfatGlobalData->CacheMgrCallbacks, > + Fcb); > + } > + > + if (ByteOffset.QuadPart > OldFileSize.QuadPart) > + { > + CcZeroData(IrpContext->FileObject, &OldFileSize, &ByteOffset, TRUE); > + } > + > + if (CcCopyWrite(IrpContext->FileObject, > + &ByteOffset, > + Length, > + TRUE /*(IrpContext->Flags & IRPCONTEXT_CANWAIT) != 0*/, > + Buffer)) > + { > + IrpContext->Irp->IoStatus.Information = Length; > + Status = STATUS_SUCCESS; > + } > + else > + { > + ASSERT(FALSE /*(IrpContext->Flags & IRPCONTEXT_CANWAIT) == 0*/); > + Status = STATUS_UNSUCCESSFUL; > + } > + } > + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) > + { > + Status = _SEH2_GetExceptionCode(); > + } > + _SEH2_END; > } > else > { > > Modified: trunk/reactos/drivers/filesystems/fastfat/vfat.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… > ============================================================================== > --- trunk/reactos/drivers/filesystems/fastfat/vfat.h [iso-8859-1] (original) > +++ trunk/reactos/drivers/filesystems/fastfat/vfat.h [iso-8859-1] Sun Aug 24 03:28:01 2014 > @@ -4,6 +4,7 @@ > #include <ntifs.h> > #include <ntdddisk.h> > #include <dos.h> > +#include <pseh/pseh2.h> > > #define USE_ROS_CC_AND_FS > > >

2 1

Re: [ros-dev] [ros-diffs] [jgardou] 63933: [WIN32K] - Enable commented freetype call: set font face size and coordinate transformation matrix Patch by Huw Campbell, reviewed by a gazillion of people. CORE-4657 #resolve #com...
by Thomas Faber 25 Aug '14

25 Aug '14

On 2014-08-24 15:28, jgardou(a)svn.reactos.org wrote: > -// FT_Set_Pixel_Sizes(ft_face, > -// TextObj->logfont.elfEnumLogfontEx.elfLogFont.lfWidth, > + FT_Set_Pixel_Sizes(ft_face, > + TextObj->logfont.elfEnumLogfontEx.elfLogFont.lfWidth, The abs() was added in the second version of the patch because this way it breaks tests. See https://jira.reactos.org/browse/CORE-4657?focusedCommentId=60379&page=com.a…

1 0

Re: [ros-dev] [ros-diffs] [hbelusca] 63940: [MPLAY32] Implement a single-window mode for videos. Patch by Ricardo Hanke. TO ALL TRANSLATORS: Update the translations! CORE-7910 #resolve #comment Committed, thanks!
by Thomas Faber 25 Aug '14

25 Aug '14

On 2014-08-24 17:40, hbelusca(a)svn.reactos.org wrote: > + mciSendCommand(wDeviceId, MCI_PUT, MCI_DGV_PUT_DESTINATION | MCI_DGV_RECT | MCI_WAIT, (DWORD)&mciPut); A DWORD cannot fit a pointer. This function takes DWORD_PTR.

1 0

Re: [ros-dev] [ros-diffs] [hbelusca] 63885: [KERNEL32] OpenConsoleW was implemented :P [CONSOLE.CPL][CONSRV] Create all the fonts needed at console startup, and then just select what's needed when fonts attributes need to b...
by Thomas Faber 14 Aug '14

14 Aug '14

On 2014-08-14 22:10, hbelusca(a)svn.reactos.org wrote: > + if (FaceName != GuiData->GuiInfo.FaceName) > + { > + SIZE_T Length = min(wcslen(FaceName) + 1, LF_FACESIZE); // wcsnlen > + wcsncpy(GuiData->GuiInfo.FaceName, FaceName, LF_FACESIZE); > + GuiData->GuiInfo.FaceName[Length] = L'\0'; // NULL-terminate > + } Buffer overflow. You mean [Length - 1]

1 0

On the server outage
by Colin Finck 12 Aug '14

12 Aug '14

Dear all, As you've certainly noticed, one of our main servers providing the www.reactos.org website along with JIRA and FishEye was down yesterday. Problems began yesterday around 13:00 UTC and lasted till some minutes ago. This has happened due to a severe hardware failure on the hypervisor machine serving the affected VMs. Our hosting provider needed two attempts to find and replace the failed hardware. Finally, the crash has caused one database table to get corrupted. This one was repaired just now. No data has been lost though. All services should be up again now. If you still notice anything not working, please report to me. We're sorry for the caused inconveniences! Best regards, -- Colin Finck System & Network Administrator ReactOS Deutschland e.V.

1 1

Introducing ReactOS development accounts
by Colin Finck 10 Aug '14

10 Aug '14

TL;DR ===== Your SVN account == "ReactOS development account", because it will be used for more services than just SVN. You can now change the password on your own at https://lam.reactos.org Hi all, I've been trying to centralize our user accounts for some time now and finally have something to show. Your known SVN credentials have been imported into an LDAP-based user directory now. This gives us the ability to easily plug various services for developers into the directory: SVN, BuildBot, the participant list for meetings, especially the upcoming services like Mumble, development VMs and file sharing. You can view your account details and change your password on your own now. Just visit https://lam.reactos.org to do so. The E-Mail address shown there will be used e.g. for sending meeting invitations. As we all want the migration to go flawlessy, I've set up a temporary SVN repository at svn://svn.reactos.org/sasltest This should work just like our "reactos" repository: If you have write access there, you have write access in "sasltest" as well. It already works well for me and my SVN client, but I want confirmations from others too. Please do so by committing some junk into sasltest :) We have some members, who don't have SVN write access but participate in the monthly meetings. They will shortly receive an E-Mail with their account details. That doesn't give you SVN write access though, it's just for participating in the meetings for now :P I'll keep you informed when we finally plug our main repositories into the LDAP database. Note that this doesn't change anything with the ReactOS Website accounts and IServ mail accounts. Former ones will always be in a separate database as everybody can sign up there. Cheers, Colin

2 1

Re: [ros-dev] [ros-diffs] [dgorbachev] 63823: [NTOSKRNL][PSEH3] - Add inline asm dependencies to fix undefined reference errors when building with link-time optimization. - Improve formatting. - No functional changes.
by Timo Kreuzer 07 Aug '14

07 Aug '14

Am 07.08.2014 01:10, schrieb dgorbachev(a)svn.reactos.org: > - : "m" (*(_TrylevelFrame)), "m" (*(_DataTable)), "c"(__builtin_alloca(0)) \ > + : "m" (*(_TrylevelFrame)), "m" (*(_DataTable)), "c" (__builtin_alloca(0)), "p" (_SEH3$_RegisterFrameWithNonVolatiles) \ What's this? Was that intentional? Timo

2 1

Re: [ros-dev] [ros-diffs] [hbelusca] 63803: [KERNEL32] Add a bunch of missing _SEH_YIELD in 'return' clues in _SEH_TRY clauses. [KERNEL32][CONSRV] Make kernel32 / winsrv console CSR structures Win2k3-compliant for Read/Writ...
by Thomas Faber 04 Aug '14

04 Aug '14

On 2014-08-04 18:25, hbelusca(a)svn.reactos.org wrote: > [KERNEL32] > Add a bunch of missing _SEH_YIELD in 'return' clues in _SEH_TRY clauses. That should be pointless with PSEH3. I believe Timo has in fact been removing them.

1 0

Status Meeting (July 2014)
by Aleksey Bragin 31 Jul '14

31 Jul '14

Hello, Let me invite you to the monthly status meeting taking place tomorrow, 31st of July, 19:00 UTC. If there are any proposals regarding moving the meeting date/time - please send them today. IRC service will only be started shortly before the meeting. Your participation passwords and server address will be emailed to you shortly before the meeting starts, and they are going to be different once again as they are not stored in any database. Hopefully it's not much of inconvenience. If someone still is not getting passwords sent before a meeting - please email Pierre before the meeting started to get one. Please send agenda proposals to me before the meeting, so that we can start with a proposed agenda. Also all developers, please, may I so kindly ask you to prepare a very brief one-two lines message containing two things: 1. What you did since the previous meeting 2. What you are going to do in the coming month This is needed to understand who is working on what, to plan things, etc. Prepare that in advance! Everyone will have max 1 minute! Regards, Aleksey Bragin

4 3

Fezile server down
by Colin Finck 31 Jul '14

31 Jul '14

Hi all, Fezile, one of our servers in Sweden, unexpectedly disappeared after a reboot today. Unfortunately, this is one of our older servers without an IPMI module, so we have to wait till it's rebooted manually. I'll keep you informed when this is done. The server outage affects the following services: * iso.reactos.org * doxygen.reactos.org * cppcheck.reactos.org * VMware Player Test slave * VMware Player Patchbot We apologize for the inconveniences! If you have any idea where we can still get a suitable IPMI module for these servers, we would be highly interested! The needed models are ASUS ASMB3 and Tyan M3291. Best regards, Colin Finck

4 7

Re: [ros-dev] [ros-diffs] [dreimer] 63785: [CARDS] Replace the old ugly 16 color cards with better looking ones. (From: http://www.united3dartists.com/forum/viewtopic.php?f=31&t=954) Remove 10 of 12 backgrounds and add some...
by Colin Finck 31 Jul '14

31 Jul '14

Daniel, do you have a written permission to put these third-party files under LGPL? Which is essentially what you're doing when adding the files to our cards.dll. Same goes for the Bavarian cards and decks. I doubt they have been put in the public domain.. We already needed to "fix" cards.dll once for not using GPLed cards, let's not repeat this again. - Colin

2 3

Re: [ros-dev] [ros-diffs] [jgardou] 63784: [NDK] - Make the MMWSLE structure multi-arch aware
by Timo Kreuzer 30 Jul '14

30 Jul '14

That name seems slightly misleading (PFN is usually associated with physical ranges, this is a virtual range). Maybe something like MM_VPN_BITS_SIZE would be better. If it is not used anywhere else, I'd just use an #ifdef _WIN64 ... (assuming it's the same for all 32 bit and the same for all 64 bit architectures) Am 30.07.2014 15:11, schrieb jgardou(a)svn.reactos.org: > +#define MM_PAGE_FRAME_NUMBER_SIZE 52

1 0

Re: [ros-dev] [ros-diffs] [jgardou] 63731: [WIN32K] - Correctly use the Clip, Meta, Rao, API and Vis Regions in DCs - Update the DC clipping object on demand each time a blt is performed - Reduce the number of region alloca...
by Timo Kreuzer 24 Jul '14

24 Jul '14

Great patch, but next time please seperate formatting from actual code changes, since it's barely possible to see what exactly has changed and what is only changed indentation. (Says the guy who often enough has mixed this himself *cough*) Am 23.07.2014 18:05, schrieb jgardou(a)svn.reactos.org: > Author: jgardou > Date: Wed Jul 23 16:05:47 2014 > New Revision: 63731 > > URL: http://svn.reactos.org/svn/reactos?rev=63731&view=rev > Log: > [WIN32K] > - Correctly use the Clip, Meta, Rao, API and Vis Regions in DCs > - Update the DC clipping object on demand each time a blt is performed > - Reduce the number of region allocated with handles when it's not needed > This commit fixes numerous bugs regarding clipping, most notably SetDIBitsToDevice overlapping foreground windows > CORE-8353 #resolve #comment There are still a few quirks in winetests which will get fixed in a next commit > CORE-7628 #comment should be fixed in rev 63731, please confirm. > > Modified: > trunk/reactos/win32ss/gdi/ntgdi/arc.c > trunk/reactos/win32ss/gdi/ntgdi/bitblt.c > trunk/reactos/win32ss/gdi/ntgdi/cliprgn.c > trunk/reactos/win32ss/gdi/ntgdi/cliprgn.h > trunk/reactos/win32ss/gdi/ntgdi/dc.h > trunk/reactos/win32ss/gdi/ntgdi/dclife.c > trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c > trunk/reactos/win32ss/gdi/ntgdi/dcstate.c > trunk/reactos/win32ss/gdi/ntgdi/dcutil.c > trunk/reactos/win32ss/gdi/ntgdi/dibobj.c > trunk/reactos/win32ss/gdi/ntgdi/fillshap.c > trunk/reactos/win32ss/gdi/ntgdi/freetype.c > trunk/reactos/win32ss/gdi/ntgdi/line.c > trunk/reactos/win32ss/gdi/ntgdi/path.c > trunk/reactos/win32ss/gdi/ntgdi/region.c > trunk/reactos/win32ss/gdi/ntgdi/region.h > trunk/reactos/win32ss/user/ntuser/cursoricon.c > trunk/reactos/win32ss/user/ntuser/cursoricon_new.c > trunk/reactos/win32ss/user/ntuser/dce.h > trunk/reactos/win32ss/user/ntuser/windc.c > > Modified: trunk/reactos/win32ss/gdi/ntgdi/arc.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/arc.c?re… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/arc.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/arc.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -335,8 +335,7 @@ > > worker.l = dwStartAngle; > worker1.l = dwSweepAngle; > - DC_vPrepareDCsForBlit(pDC, pDC->rosdc.CombinedClip->rclBounds, > - NULL, pDC->rosdc.CombinedClip->rclBounds); > + DC_vPrepareDCsForBlit(pDC, NULL, NULL, NULL); > if (pDC->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) > DC_vUpdateFillBrush(pDC); > if (pDC->pdcattr->ulDirty_ & (DIRTY_LINE | DC_PEN_DIRTY)) > @@ -381,8 +380,7 @@ > return TRUE; > } > > - DC_vPrepareDCsForBlit(dc, dc->rosdc.CombinedClip->rclBounds, > - NULL, dc->rosdc.CombinedClip->rclBounds); > + DC_vPrepareDCsForBlit(dc, NULL, NULL, NULL); > > if (dc->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) > DC_vUpdateFillBrush(dc); > > Modified: trunk/reactos/win32ss/gdi/ntgdi/bitblt.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/bitblt.c… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/bitblt.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/bitblt.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -107,7 +107,7 @@ > > /* Prepare DCs for blit */ > TRACE("Preparing DCs for blit\n"); > - DC_vPrepareDCsForBlit(DCDest, DestRect, DCSrc, SourceRect); > + DC_vPrepareDCsForBlit(DCDest, &DestRect, DCSrc, &SourceRect); > > /* Determine surfaces to be used in the bitblt */ > BitmapDest = DCDest->dclevel.pSurface; > @@ -268,7 +268,7 @@ > rcSrc.bottom += DCSrc->ptlDCOrig.y; > > /* Prepare for blit */ > - DC_vPrepareDCsForBlit(DCDest, rcDest, DCSrc, rcSrc); > + DC_vPrepareDCsForBlit(DCDest, &rcDest, DCSrc, &rcSrc); > > BitmapDest = DCDest->dclevel.pSurface; > if (!BitmapDest) > @@ -456,7 +456,7 @@ > } > > /* Prepare blit */ > - DC_vPrepareDCsForBlit(DCDest, DestRect, DCSrc, SourceRect); > + DC_vPrepareDCsForBlit(DCDest, &DestRect, DCSrc, &SourceRect); > > if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) > DC_vUpdateFillBrush(DCDest); > @@ -647,7 +647,7 @@ > BrushOrigin.y = 0; > > /* Only prepare Source and Dest, hdcMask represents a DIB */ > - DC_vPrepareDCsForBlit(DCDest, DestRect, DCSrc, SourceRect); > + DC_vPrepareDCsForBlit(DCDest, &DestRect, DCSrc, &SourceRect); > > if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) > DC_vUpdateFillBrush(DCDest); > @@ -826,7 +826,7 @@ > BrushOrigin.y = pbrush->ptOrigin.y + pdc->ptlDCOrig.y; > #endif > > - DC_vPrepareDCsForBlit(pdc, DestRect, NULL, DestRect); > + DC_vPrepareDCsForBlit(pdc, &DestRect, NULL, NULL); > > psurf = pdc->dclevel.pSurface; > > > Modified: trunk/reactos/win32ss/gdi/ntgdi/cliprgn.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/cliprgn.… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/cliprgn.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/cliprgn.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -11,164 +11,102 @@ > #define NDEBUG > #include <debug.h> > > -int FASTCALL > -CLIPPING_UpdateGCRegion(DC* Dc) > -{ > - PROSRGNDATA CombinedRegion; > - //HRGN hRgnVis; > - PREGION prgnClip, prgnGCClip; > - > - /* Would prefer this, but the rest of the code sucks... */ > - //ASSERT(Dc->rosdc.hGCClipRgn); > - //ASSERT(Dc->rosdc.hClipRgn); > - ASSERT(Dc->prgnVis); > - //hRgnVis = Dc->prgnVis->BaseObject.hHmgr; > - > - if (Dc->rosdc.hGCClipRgn == NULL) > - Dc->rosdc.hGCClipRgn = IntSysCreateRectRgn(0, 0, 0, 0); > - > - prgnGCClip = REGION_LockRgn(Dc->rosdc.hGCClipRgn); > - ASSERT(prgnGCClip); > - > - if (Dc->rosdc.hClipRgn == NULL) > - IntGdiCombineRgn(prgnGCClip, Dc->prgnVis, NULL, RGN_COPY); > - else > - { > - prgnClip = REGION_LockRgn(Dc->rosdc.hClipRgn); // FIXME: Locking order, ugh! > - IntGdiCombineRgn(prgnGCClip, Dc->prgnVis, prgnClip, RGN_AND); > - REGION_UnlockRgn(prgnClip); > - } > - REGION_UnlockRgn(prgnGCClip); > - > - NtGdiOffsetRgn(Dc->rosdc.hGCClipRgn, Dc->ptlDCOrig.x, Dc->ptlDCOrig.y); > - > - if((CombinedRegion = RGNOBJAPI_Lock(Dc->rosdc.hGCClipRgn, NULL))) > - { > - CLIPOBJ *CombinedClip; > - > - CombinedClip = IntEngCreateClipRegion(CombinedRegion->rdh.nCount, > - CombinedRegion->Buffer, > - &CombinedRegion->rdh.rcBound); > - > - RGNOBJAPI_Unlock(CombinedRegion); > - > - if ( !CombinedClip ) > - { > - DPRINT1("IntEngCreateClipRegion() failed\n"); > - return ERROR; > - } > - > - if(Dc->rosdc.CombinedClip != NULL) > - IntEngDeleteClipRegion(Dc->rosdc.CombinedClip); > - > - Dc->rosdc.CombinedClip = CombinedClip ; > - } > - > - return NtGdiOffsetRgn(Dc->rosdc.hGCClipRgn, -Dc->ptlDCOrig.x, -Dc->ptlDCOrig.y); > -} > - > -INT FASTCALL > -GdiSelectVisRgn(HDC hdc, HRGN hrgn) > -{ > - int retval; > - DC *dc; > - PREGION prgn; > - > - if (!hrgn) > - { > - EngSetLastError(ERROR_INVALID_PARAMETER); > - return ERROR; > - } > - if (!(dc = DC_LockDc(hdc))) > - { > - EngSetLastError(ERROR_INVALID_HANDLE); > - return ERROR; > - } > - > - dc->fs &= ~DC_FLAG_DIRTY_RAO; > - > - ASSERT (dc->prgnVis != NULL); > - > - prgn = RGNOBJAPI_Lock(hrgn, NULL); > - retval = prgn ? IntGdiCombineRgn(dc->prgnVis, prgn, NULL, RGN_COPY) : ERROR; > - RGNOBJAPI_Unlock(prgn); > - if ( retval != ERROR ) > - { > +VOID > +FASTCALL > +GdiSelectVisRgn( > + HDC hdc, > + PREGION prgn) > +{ > + DC *dc; > + > + if (!(dc = DC_LockDc(hdc))) > + { > + EngSetLastError(ERROR_INVALID_HANDLE); > + return; > + } > + > + dc->fs |= DC_FLAG_DIRTY_RAO; > + > + ASSERT(dc->prgnVis != NULL); > + ASSERT(prgn != NULL); > + > + IntGdiCombineRgn(dc->prgnVis, prgn, NULL, RGN_COPY); > IntGdiOffsetRgn(dc->prgnVis, -dc->ptlDCOrig.x, -dc->ptlDCOrig.y); > - CLIPPING_UpdateGCRegion(dc); > - } > - DC_UnlockDc(dc); > - > - return retval; > -} > - > - > -int FASTCALL GdiExtSelectClipRgn(PDC dc, > - HRGN hrgn, > - int fnMode) > -{ > - // dc->fs &= ~DC_FLAG_DIRTY_RAO; > - > - if (!hrgn) > - { > - if (fnMode == RGN_COPY) > - { > - if (dc->rosdc.hClipRgn != NULL) > - { > - GreDeleteObject(dc->rosdc.hClipRgn); > - dc->rosdc.hClipRgn = NULL; > - } > - } > - else > - { > - EngSetLastError(ERROR_INVALID_PARAMETER); > - return ERROR; > - } > - } > - else > - { > - if (!dc->rosdc.hClipRgn) > - { > - RECTL rect; > - if(dc->prgnVis) > - { > + > + DC_UnlockDc(dc); > +} > + > + > +int > +FASTCALL > +IntGdiExtSelectClipRgn( > + PDC dc, > + PREGION prgn, > + int fnMode) > +{ > + if (!prgn) > + { > + if (fnMode == RGN_COPY) > + { > + if (dc->dclevel.prgnClip != NULL) > + { > + REGION_Delete(dc->dclevel.prgnClip); > + dc->dclevel.prgnClip = NULL; > + dc->fs |= DC_FLAG_DIRTY_RAO; > + } > + return SIMPLEREGION; > + } > + else > + { > + EngSetLastError(ERROR_INVALID_PARAMETER); > + return ERROR; > + } > + } > + > + if (!dc->dclevel.prgnClip) > + { > + RECTL rect; > + > REGION_GetRgnBox(dc->prgnVis, &rect); > - dc->rosdc.hClipRgn = IntSysCreateRectRgnIndirect(&rect); > - } > - else > - { > - dc->rosdc.hClipRgn = IntSysCreateRectRgn(0, 0, 0, 0); > - } > - } > + dc->dclevel.prgnClip = IntSysCreateRectpRgnIndirect(&rect); > + } > + > + dc->fs |= DC_FLAG_DIRTY_RAO; > + > if(fnMode == RGN_COPY) > - { > - NtGdiCombineRgn(dc->rosdc.hClipRgn, hrgn, 0, fnMode); > - } > - else > - NtGdiCombineRgn(dc->rosdc.hClipRgn, dc->rosdc.hClipRgn, hrgn, fnMode); > - } > - > - return CLIPPING_UpdateGCRegion(dc); > -} > - > - > -int APIENTRY NtGdiExtSelectClipRgn(HDC hDC, > - HRGN hrgn, > - int fnMode) > -{ > - int retval; > - DC *dc; > - > - if (!(dc = DC_LockDc(hDC))) > - { > - EngSetLastError(ERROR_INVALID_HANDLE); > - return ERROR; > - } > - > - retval = GdiExtSelectClipRgn ( dc, hrgn, fnMode ); > - > - DC_UnlockDc(dc); > - return retval; > + return IntGdiCombineRgn(dc->dclevel.prgnClip, prgn, 0, fnMode); > + > + return IntGdiCombineRgn(dc->dclevel.prgnClip, dc->dclevel.prgnClip, prgn, fnMode); > +} > + > + > +int > +APIENTRY > +NtGdiExtSelectClipRgn( > + HDC hDC, > + HRGN hrgn, > + int fnMode) > +{ > + int retval; > + DC *dc; > + PREGION prgn; > + > + if (!(dc = DC_LockDc(hDC))) > + { > + EngSetLastError(ERROR_INVALID_HANDLE); > + return ERROR; > + } > + > + prgn = REGION_LockRgn(hrgn); > + > + /* IntGdiExtSelectClipRgn takes care of checking for NULL region */ > + retval = IntGdiExtSelectClipRgn(dc, prgn, fnMode); > + > + if (prgn) > + REGION_UnlockRgn(prgn); > + > + DC_UnlockDc(dc); > + return retval; > } > > INT FASTCALL > @@ -177,7 +115,6 @@ > INT retval; > PDC dc; > PROSRGNDATA pRgnNew, pRgn = NULL; > - BOOL Unlock = FALSE; // Small HACK > > if (!(dc = DC_LockDc(hDC))) > { > @@ -193,10 +130,9 @@ > { > pRgn = dc->dclevel.prgnMeta; > } > - else if (dc->rosdc.hClipRgn) > + else if (dc->dclevel.prgnClip) // CLIPRGN > { > - Unlock = TRUE ; > - pRgn = REGION_LockRgn(dc->rosdc.hClipRgn); // CLIPRGN > + pRgn = dc->dclevel.prgnClip; > } > > if (pRgn) > @@ -206,7 +142,6 @@ > if (!pRgnNew) > { > DC_UnlockDc(dc); > - if(Unlock) REGION_UnlockRgn(pRgn); > return ERROR; > } > > @@ -217,12 +152,11 @@ > REGION_Delete(pRgnNew); > > DC_UnlockDc(dc); > - if(Unlock) REGION_UnlockRgn(pRgn); > return retval; > } > > retval = REGION_GetRgnBox(dc->prgnVis, rc); > - IntDPtoLP(dc, (LPPOINT)rc, 2); > + > DC_UnlockDc(dc); > > return retval; > @@ -265,53 +199,49 @@ > int RightRect, > int BottomRect) > { > - INT Result; > - RECTL Rect; > - PREGION prgnNew, prgnClip; > - PDC dc = DC_LockDc(hDC); > - > - if (!dc) > - { > - EngSetLastError(ERROR_INVALID_HANDLE); > - return ERROR; > - } > - > - Rect.left = LeftRect; > - Rect.top = TopRect; > - Rect.right = RightRect; > - Rect.bottom = BottomRect; > - > - IntLPtoDP(dc, (LPPOINT)&Rect, 2); > - > - prgnNew = IntSysCreateRectpRgnIndirect(&Rect); > - if (!prgnNew) > - { > - Result = ERROR; > - } > - else > - { > - if (!dc->rosdc.hClipRgn) > - { > - dc->rosdc.hClipRgn = IntSysCreateRectRgn(0, 0, 0, 0); > - prgnClip = REGION_LockRgn(dc->rosdc.hClipRgn); > - IntGdiCombineRgn(prgnClip, dc->prgnVis, prgnNew, RGN_DIFF); > - REGION_UnlockRgn(prgnClip); > - Result = SIMPLEREGION; > - } > - else > - { > - prgnClip = REGION_LockRgn(dc->rosdc.hClipRgn); > - Result = IntGdiCombineRgn(prgnClip, prgnClip, prgnNew, RGN_DIFF); > - REGION_UnlockRgn(prgnClip); > - } > - REGION_Delete(prgnNew); > - } > - if (Result != ERROR) > - CLIPPING_UpdateGCRegion(dc); > - > - DC_UnlockDc(dc); > - > - return Result; > + INT Result; > + RECTL Rect; > + PREGION prgnNew; > + PDC dc = DC_LockDc(hDC); > + > + if (!dc) > + { > + EngSetLastError(ERROR_INVALID_HANDLE); > + return ERROR; > + } > + > + Rect.left = LeftRect; > + Rect.top = TopRect; > + Rect.right = RightRect; > + Rect.bottom = BottomRect; > + > + IntLPtoDP(dc, (LPPOINT)&Rect, 2); > + > + prgnNew = IntSysCreateRectpRgnIndirect(&Rect); > + if (!prgnNew) > + { > + Result = ERROR; > + } > + else > + { > + if (!dc->dclevel.prgnClip) > + { > + dc->dclevel.prgnClip = IntSysCreateRectpRgn(0, 0, 0, 0); > + IntGdiCombineRgn(dc->dclevel.prgnClip, dc->prgnVis, prgnNew, RGN_DIFF); > + Result = SIMPLEREGION; > + } > + else > + { > + Result = IntGdiCombineRgn(dc->dclevel.prgnClip, dc->dclevel.prgnClip, prgnNew, RGN_DIFF); > + } > + REGION_Delete(prgnNew); > + } > + if (Result != ERROR) > + dc->fs |= DC_FLAG_DIRTY_RAO; > + > + DC_UnlockDc(dc); > + > + return Result; > } > > int APIENTRY NtGdiIntersectClipRect(HDC hDC, > @@ -320,200 +250,191 @@ > int RightRect, > int BottomRect) > { > - INT Result; > - RECTL Rect; > - HRGN NewRgn; > - PDC dc = DC_LockDc(hDC); > - > - DPRINT("NtGdiIntersectClipRect(%p, %d,%d-%d,%d)\n", > - hDC, LeftRect, TopRect, RightRect, BottomRect); > - > - if (!dc) > - { > - EngSetLastError(ERROR_INVALID_HANDLE); > - return ERROR; > - } > - > - Rect.left = LeftRect; > - Rect.top = TopRect; > - Rect.right = RightRect; > - Rect.bottom = BottomRect; > - > - IntLPtoDP(dc, (LPPOINT)&Rect, 2); > - > - NewRgn = IntSysCreateRectRgnIndirect(&Rect); > - if (!NewRgn) > - { > - Result = ERROR; > - } > - else if (!dc->rosdc.hClipRgn) > - { > - dc->rosdc.hClipRgn = NewRgn; > - Result = SIMPLEREGION; > - } > - else > - { > - Result = NtGdiCombineRgn(dc->rosdc.hClipRgn, dc->rosdc.hClipRgn, NewRgn, RGN_AND); > - GreDeleteObject(NewRgn); > - } > - if (Result != ERROR) > - CLIPPING_UpdateGCRegion(dc); > - > - DC_UnlockDc(dc); > - > - return Result; > + INT Result; > + RECTL Rect; > + PREGION pNewRgn; > + PDC dc = DC_LockDc(hDC); > + > + DPRINT("NtGdiIntersectClipRect(%p, %d,%d-%d,%d)\n", > + hDC, LeftRect, TopRect, RightRect, BottomRect); > + > + if (!dc) > + { > + EngSetLastError(ERROR_INVALID_HANDLE); > + return ERROR; > + } > + > + Rect.left = LeftRect; > + Rect.top = TopRect; > + Rect.right = RightRect; > + Rect.bottom = BottomRect; > + > + IntLPtoDP(dc, (LPPOINT)&Rect, 2); > + > + pNewRgn = IntSysCreateRectpRgnIndirect(&Rect); > + if (!pNewRgn) > + { > + Result = ERROR; > + } > + else if (!dc->dclevel.prgnClip) > + { > + dc->dclevel.prgnClip = pNewRgn; > + Result = SIMPLEREGION; > + } > + else > + { > + Result = IntGdiCombineRgn(dc->dclevel.prgnClip, dc->dclevel.prgnClip, pNewRgn, RGN_AND); > + REGION_Delete(pNewRgn); > + } > + if (Result != ERROR) > + dc->fs |= DC_FLAG_DIRTY_RAO; > + > + DC_UnlockDc(dc); > + > + return Result; > } > > int APIENTRY NtGdiOffsetClipRgn(HDC hDC, > int XOffset, > int YOffset) > { > - INT Result; > - DC *dc; > - > - if(!(dc = DC_LockDc(hDC))) > - { > - EngSetLastError(ERROR_INVALID_HANDLE); > - return ERROR; > - } > - > - if(dc->rosdc.hClipRgn != NULL) > - { > - Result = NtGdiOffsetRgn(dc->rosdc.hClipRgn, > - XOffset, > - YOffset); > - CLIPPING_UpdateGCRegion(dc); > - } > - else > - { > - Result = NULLREGION; > - } > - > - DC_UnlockDc(dc); > - return Result; > + INT Result; > + DC *dc; > + > + if(!(dc = DC_LockDc(hDC))) > + { > + EngSetLastError(ERROR_INVALID_HANDLE); > + return ERROR; > + } > + > + if(dc->dclevel.prgnClip != NULL) > + { > + Result = IntGdiOffsetRgn(dc->dclevel.prgnClip, > + XOffset, > + YOffset); > + dc->fs |= DC_FLAG_DIRTY_RAO; > + } > + else > + { > + Result = NULLREGION; > + } > + > + DC_UnlockDc(dc); > + return Result; > } > > BOOL APIENTRY NtGdiPtVisible(HDC hDC, > int X, > int Y) > { > - HRGN rgn; > - DC *dc; > - > - if(!(dc = DC_LockDc(hDC))) > - { > - EngSetLastError(ERROR_INVALID_HANDLE); > - return FALSE; > - } > - > - rgn = dc->rosdc.hGCClipRgn; > - DC_UnlockDc(dc); > - > - return (rgn ? NtGdiPtInRegion(rgn, X, Y) : FALSE); > -} > - > -BOOL APIENTRY NtGdiRectVisible(HDC hDC, > - LPRECT UnsafeRect) > -{ > - NTSTATUS Status = STATUS_SUCCESS; > - PROSRGNDATA Rgn; > - PDC dc = DC_LockDc(hDC); > - BOOL Result = FALSE; > - RECTL Rect; > - > - if (!dc) > - { > - EngSetLastError(ERROR_INVALID_HANDLE); > - return FALSE; > - } > - > - _SEH2_TRY > - { > - ProbeForRead(UnsafeRect, > + BOOL ret = FALSE; > + PDC dc; > + > + if(!(dc = DC_LockDc(hDC))) > + { > + EngSetLastError(ERROR_INVALID_HANDLE); > + return FALSE; > + } > + > + if (dc->prgnRao) > + { > + POINT pt = {X, Y}; > + IntLPtoDP(dc, &pt, 1); > + ret = REGION_PtInRegion(dc->prgnRao, pt.x, pt.y); > + } > + > + DC_UnlockDc(dc); > + > + return ret; > +} > + > +BOOL > +APIENTRY > +NtGdiRectVisible( > + HDC hDC, > + LPRECT UnsafeRect) > +{ > + NTSTATUS Status = STATUS_SUCCESS; > + PDC dc = DC_LockDc(hDC); > + BOOL Result = FALSE; > + RECTL Rect; > + > + if (!dc) > + { > + EngSetLastError(ERROR_INVALID_HANDLE); > + return FALSE; > + } > + > + _SEH2_TRY > + { > + ProbeForRead(UnsafeRect, > sizeof(RECT), > 1); > - Rect = *UnsafeRect; > - } > - _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) > - { > - Status = _SEH2_GetExceptionCode(); > - } > - _SEH2_END; > - > - if(!NT_SUCCESS(Status)) > - { > - DC_UnlockDc(dc); > - SetLastNtError(Status); > - return FALSE; > - } > - > - if (dc->rosdc.hGCClipRgn) > - { > - if((Rgn = (PROSRGNDATA)RGNOBJAPI_Lock(dc->rosdc.hGCClipRgn, NULL))) > - { > + Rect = *UnsafeRect; > + } > + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) > + { > + Status = _SEH2_GetExceptionCode(); > + } > + _SEH2_END; > + > + if(!NT_SUCCESS(Status)) > + { > + DC_UnlockDc(dc); > + SetLastNtError(Status); > + return FALSE; > + } > + > + if (dc->fs & DC_FLAG_DIRTY_RAO) > + CLIPPING_UpdateGCRegion(dc); > + > + if (dc->prgnRao) > + { > IntLPtoDP(dc, (LPPOINT)&Rect, 2); > - Result = REGION_RectInRegion(Rgn, &Rect); > - RGNOBJAPI_Unlock(Rgn); > - } > - } > - DC_UnlockDc(dc); > - > - return Result; > + Result = REGION_RectInRegion(dc->prgnRao, &Rect); > + } > + DC_UnlockDc(dc); > + > + return Result; > } > > int > FASTCALL > IntGdiSetMetaRgn(PDC pDC) > { > - INT Ret = ERROR; > - PROSRGNDATA TempRgn; > - > - if ( pDC->dclevel.prgnMeta ) > - { > - if ( pDC->dclevel.prgnClip ) > - { > - TempRgn = IntSysCreateRectpRgn(0,0,0,0); > - if (TempRgn) > - { > - Ret = IntGdiCombineRgn( TempRgn, > - pDC->dclevel.prgnMeta, > - pDC->dclevel.prgnClip, > - RGN_AND); > - if ( Ret ) > - { > - GDIOBJ_vDereferenceObject(&pDC->dclevel.prgnMeta->BaseObject); > - if (!((PROSRGNDATA)pDC->dclevel.prgnMeta)->BaseObject.ulShareCount) > - REGION_Delete(pDC->dclevel.prgnMeta); > - > - pDC->dclevel.prgnMeta = TempRgn; > - > - GDIOBJ_vDereferenceObject(&pDC->dclevel.prgnClip->BaseObject); > - if (!((PROSRGNDATA)pDC->dclevel.prgnClip)->BaseObject.ulShareCount) > - REGION_Delete(pDC->dclevel.prgnClip); > - > - pDC->dclevel.prgnClip = NULL; > - > - IntGdiReleaseRaoRgn(pDC); > - } > - else > - REGION_Delete(TempRgn); > - } > - } > - else > - Ret = REGION_Complexity(pDC->dclevel.prgnMeta); > - } > - else > - { > - if ( pDC->dclevel.prgnClip ) > - { > - Ret = REGION_Complexity(pDC->dclevel.prgnClip); > - pDC->dclevel.prgnMeta = pDC->dclevel.prgnClip; > - pDC->dclevel.prgnClip = NULL; > - } > - else > - Ret = SIMPLEREGION; > - } > - return Ret; > + INT Ret = ERROR; > + > + if ( pDC->dclevel.prgnMeta ) > + { > + if ( pDC->dclevel.prgnClip ) > + { > + Ret = IntGdiCombineRgn(pDC->dclevel.prgnMeta, pDC->dclevel.prgnMeta, pDC->dclevel.prgnClip, RGN_AND); > + if (Ret != ERROR) > + { > + REGION_Delete(pDC->dclevel.prgnClip); > + pDC->dclevel.prgnClip = NULL; > + IntGdiReleaseRaoRgn(pDC); > + } > + } > + else > + Ret = REGION_Complexity(pDC->dclevel.prgnMeta); > + } > + else > + { > + if ( pDC->dclevel.prgnClip ) > + { > + Ret = REGION_Complexity(pDC->dclevel.prgnClip); > + pDC->dclevel.prgnMeta = pDC->dclevel.prgnClip; > + pDC->dclevel.prgnClip = NULL; > + } > + else > + Ret = SIMPLEREGION; > + } > + > + if (Ret != ERROR) > + pDC->fs |= DC_FLAG_DIRTY_RAO; > + > + return Ret; > } > > > @@ -533,88 +454,95 @@ > return Ret; > } > > -INT FASTCALL > -NEW_CLIPPING_UpdateGCRegion(PDC pDC) > -{ > - CLIPOBJ * co; > - > - /* Must have VisRgn set to a valid state! */ > - ASSERT (pDC->prgnVis); > - > -// FIXME: this seems to be broken! > - > - if (pDC->prgnAPI) > - { > - REGION_Delete(pDC->prgnAPI); > - pDC->prgnAPI = IntSysCreateRectpRgn(0,0,0,0); > - } > - > - if (pDC->prgnRao) > - { > - REGION_Delete(pDC->prgnRao); > - pDC->prgnRao = IntSysCreateRectpRgn(0,0,0,0); > - } > - > - if (!pDC->prgnRao) > - { > - return ERROR; > - } > - > - if (pDC->dclevel.prgnMeta && pDC->dclevel.prgnClip) > - { > - IntGdiCombineRgn( pDC->prgnAPI, > - pDC->dclevel.prgnClip, > - pDC->dclevel.prgnMeta, > - RGN_AND); > - } > - else > - { > - if (pDC->dclevel.prgnClip) > - { > - IntGdiCombineRgn( pDC->prgnAPI, > - pDC->dclevel.prgnClip, > - NULL, > - RGN_COPY); > - } > - else if (pDC->dclevel.prgnMeta) > - { > - IntGdiCombineRgn( pDC->prgnAPI, > - pDC->dclevel.prgnMeta, > - NULL, > - RGN_COPY); > - } > - } > - > - IntGdiCombineRgn( pDC->prgnRao, > - pDC->prgnVis, > - pDC->prgnAPI, > - RGN_AND); > - > - RtlCopyMemory(&pDC->erclClip, > +VOID > +FASTCALL > +CLIPPING_UpdateGCRegion(PDC pDC) > +{ > + CLIPOBJ * co; > + > + /* Must have VisRgn set to a valid state! */ > + ASSERT (pDC->prgnVis); > + > + if (pDC->prgnAPI) > + { > + REGION_Delete(pDC->prgnAPI); > + pDC->prgnAPI = NULL; > + } > + > + if (pDC->prgnRao) > + REGION_Delete(pDC->prgnRao); > + > + pDC->prgnRao = IntSysCreateRectpRgn(0,0,0,0); > + > + ASSERT(pDC->prgnRao); > + > + if (pDC->dclevel.prgnMeta || pDC->dclevel.prgnClip) > + { > + pDC->prgnAPI = IntSysCreateRectpRgn(0,0,0,0); > + if (!pDC->dclevel.prgnMeta) > + { > + IntGdiCombineRgn(pDC->prgnAPI, > + pDC->dclevel.prgnClip, > + NULL, > + RGN_COPY); > + } > + else if (!pDC->dclevel.prgnClip) > + { > + IntGdiCombineRgn(pDC->prgnAPI, > + pDC->dclevel.prgnMeta, > + NULL, > + RGN_COPY); > + } > + else > + { > + IntGdiCombineRgn(pDC->prgnAPI, > + pDC->dclevel.prgnClip, > + pDC->dclevel.prgnMeta, > + RGN_AND); > + } > + } > + > + if (pDC->prgnAPI) > + { > + IntGdiCombineRgn(pDC->prgnRao, > + pDC->prgnVis, > + pDC->prgnAPI, > + RGN_AND); > + } > + else > + { > + IntGdiCombineRgn(pDC->prgnRao, > + pDC->prgnVis, > + NULL, > + RGN_COPY); > + } > + > + > + IntGdiOffsetRgn(pDC->prgnRao, pDC->ptlDCOrig.x, pDC->ptlDCOrig.y); > + > + RtlCopyMemory(&pDC->erclClip, > &pDC->prgnRao->rdh.rcBound, > sizeof(RECTL)); > > - pDC->fs &= ~DC_FLAG_DIRTY_RAO; > - > - IntGdiOffsetRgn(pDC->prgnRao, pDC->ptlDCOrig.x, pDC->ptlDCOrig.y); > - > - // pDC->co should be used. Example, CLIPOBJ_cEnumStart uses XCLIPOBJ to build > - // the rects from region objects rects in pClipRgn->Buffer. > - // With pDC->co.pClipRgn->Buffer, > - // pDC->co.pClipRgn = pDC->prgnRao ? pDC->prgnRao : pDC->prgnVis; > - > - co = IntEngCreateClipRegion(pDC->prgnRao->rdh.nCount, > - pDC->prgnRao->Buffer, > - &pDC->erclClip); > - if (co) > - { > - if (pDC->rosdc.CombinedClip != NULL) > - IntEngDeleteClipRegion(pDC->rosdc.CombinedClip); > - > - pDC->rosdc.CombinedClip = co; > - } > - > - return IntGdiOffsetRgn(pDC->prgnRao, -pDC->ptlDCOrig.x, -pDC->ptlDCOrig.y); > + pDC->fs &= ~DC_FLAG_DIRTY_RAO; > + > + // pDC->co should be used. Example, CLIPOBJ_cEnumStart uses XCLIPOBJ to build > + // the rects from region objects rects in pClipRgn->Buffer. > + // With pDC->co.pClipRgn->Buffer, > + // pDC->co.pClipRgn = pDC->prgnRao ? pDC->prgnRao : pDC->prgnVis; > + > + co = IntEngCreateClipRegion(pDC->prgnRao->rdh.nCount, > + pDC->prgnRao->Buffer, > + &pDC->erclClip); > + if (co) > + { > + if (pDC->rosdc.CombinedClip != NULL) > + IntEngDeleteClipRegion(pDC->rosdc.CombinedClip); > + > + pDC->rosdc.CombinedClip = co; > + } > + > + IntGdiOffsetRgn(pDC->prgnRao, -pDC->ptlDCOrig.x, -pDC->ptlDCOrig.y); > } > > /* EOF */ > > Modified: trunk/reactos/win32ss/gdi/ntgdi/cliprgn.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/cliprgn.… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/cliprgn.h [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/cliprgn.h [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -1,6 +1,6 @@ > #pragma once > > INT FASTCALL GdiGetClipBox(HDC hDC, RECTL *rc); > -INT FASTCALL GdiSelectVisRgn(HDC hdc, HRGN hrgn); > -INT FASTCALL GdiExtSelectClipRgn (PDC dc, HRGN hrgn, int fnMode); > -int FASTCALL CLIPPING_UpdateGCRegion(DC* Dc); > +VOID FASTCALL GdiSelectVisRgn(HDC hdc, PREGION prgn); > +INT FASTCALL IntGdiExtSelectClipRgn (PDC dc, PREGION prgn, int fnMode); > +VOID FASTCALL CLIPPING_UpdateGCRegion(DC* Dc); > > Modified: trunk/reactos/win32ss/gdi/ntgdi/dc.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dc.h?rev… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/dc.h [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/dc.h [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -48,9 +48,6 @@ > > typedef struct _ROS_DC_INFO > { > - HRGN hClipRgn; /* Clip region (may be 0) */ > - HRGN hGCClipRgn; /* GC clip region (ClipRgn AND VisRgn) */ > - > CLIPOBJ *CombinedClip; > } ROS_DC_INFO; > > @@ -199,9 +196,9 @@ > > BOOL FASTCALL DC_InvertXform(const XFORM *xformSrc, XFORM *xformDest); > VOID FASTCALL DC_vUpdateViewportExt(PDC pdc); > -VOID FASTCALL DC_vCopyState(PDC pdcSrc, PDC pdcDst, BOOL to); > +VOID FASTCALL DC_vCopyState(PDC pdcSrc, PDC pdcDst, BOOL To); > VOID FASTCALL DC_vFinishBlit(PDC pdc1, PDC pdc2); > -VOID FASTCALL DC_vPrepareDCsForBlit(PDC pdc1, RECT rc1, PDC pdc2, RECT rc2); > +VOID FASTCALL DC_vPrepareDCsForBlit(PDC pdcDest, const RECT* rcDest, PDC pdcSrc, const RECT* rcSrc); > > VOID NTAPI DC_vRestoreDC(IN PDC pdc, INT iSaveLevel); > > > Modified: trunk/reactos/win32ss/gdi/ntgdi/dclife.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dclife.c… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/dclife.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/dclife.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -95,22 +95,11 @@ > void > DC_InitHack(PDC pdc) > { > - HRGN hVisRgn; > - > TextIntRealizeFont(pdc->pdcattr->hlfntNew,NULL); > pdc->pdcattr->iCS_CP = ftGdiGetTextCharsetInfo(pdc,NULL,0); > > /* This should never fail */ > ASSERT(pdc->dclevel.ppal); > - > - /* Select regions */ > - pdc->rosdc.hClipRgn = NULL; > - pdc->rosdc.hGCClipRgn = NULL; > - > - hVisRgn = IntSysCreateRectRgn(0, 0, 1, 1); > - ASSERT(hVisRgn); > - GdiSelectVisRgn(pdc->BaseObject.hHmgr, hVisRgn); > - GreDeleteObject(hVisRgn); > } > > VOID > @@ -170,7 +159,7 @@ > pdc->erclBoundsApp.right = 0x00007ffc; // FIXME > pdc->erclBoundsApp.bottom = 0x00000333; // FIXME > pdc->erclClip = pdc->erclBounds; > -// pdc->co > + pdc->co = gxcoTrivial; > > pdc->fs |= DC_SYNCHRONIZEACCESS | DC_ACCUM_APP | DC_PERMANANT | DC_DISPLAY; > } > @@ -185,7 +174,7 @@ > pdc->erclBounds.bottom = 0; > pdc->erclBoundsApp = pdc->erclBounds; > pdc->erclClip = pdc->erclWindow; > - //pdc->co = NULL > + pdc->co = gxcoTrivial; > } > > //pdc->dcattr.VisRectRegion: > @@ -237,6 +226,8 @@ > /* Setup regions */ > pdc->prgnAPI = NULL; > pdc->prgnRao = NULL; > + pdc->dclevel.prgnClip = NULL; > + pdc->dclevel.prgnMeta = NULL; > /* Allocate a Vis region */ > pdc->prgnVis = IntSysCreateRectpRgn(0, 0, pdc->dclevel.sizl.cx, pdc->dclevel.sizl.cy); > ASSERT(pdc->prgnVis); > @@ -372,17 +363,17 @@ > LFONT_ShareUnlockFont(pdc->dclevel.plfnt); > > /* Free regions */ > - if (pdc->rosdc.hClipRgn && GreIsHandleValid(pdc->rosdc.hClipRgn)) > - GreDeleteObject(pdc->rosdc.hClipRgn); > + if (pdc->dclevel.prgnClip) > + REGION_Delete(pdc->dclevel.prgnClip); > + if (pdc->dclevel.prgnMeta) > + REGION_Delete(pdc->dclevel.prgnMeta); > if (pdc->prgnVis) > - { > REGION_Delete(pdc->prgnVis); > - } > - if (pdc->rosdc.hGCClipRgn && GreIsHandleValid(pdc->rosdc.hGCClipRgn)) > - { > - GreDeleteObject(pdc->rosdc.hGCClipRgn); > - } > - if (NULL != pdc->rosdc.CombinedClip) > + if (pdc->prgnRao) > + REGION_Delete(pdc->prgnRao); > + if (pdc->prgnAPI) > + REGION_Delete(pdc->prgnAPI); > + if (pdc->rosdc.CombinedClip) > IntEngDeleteClipRegion(pdc->rosdc.CombinedClip); > > PATH_Delete(pdc->dclevel.hPath); > @@ -397,17 +388,6 @@ > NTAPI > DC_vSetOwner(PDC pdc, ULONG ulOwner) > { > - > - if (pdc->rosdc.hClipRgn) > - { > - IntGdiSetRegionOwner(pdc->rosdc.hClipRgn, ulOwner); > - } > - > - if (pdc->rosdc.hGCClipRgn) > - { > - IntGdiSetRegionOwner(pdc->rosdc.hGCClipRgn, ulOwner); > - } > - > if (pdc->dclevel.hPath) > { > GreSetObjectOwner(pdc->dclevel.hPath, ulOwner); > @@ -466,7 +446,7 @@ > void > DC_vUpdateDC(PDC pdc) > { > - HRGN hVisRgn ; > + // PREGION VisRgn ; > PPDEVOBJ ppdev = pdc->ppdev ; > > pdc->dhpdev = ppdev->dhpdev; > @@ -475,10 +455,12 @@ > pdc->dclevel.pSurface = PDEVOBJ_pSurface(ppdev); > > PDEVOBJ_sizl(pdc->ppdev, &pdc->dclevel.sizl); > - hVisRgn = NtGdiCreateRectRgn(0, 0, pdc->dclevel.sizl.cx, pdc->dclevel.sizl.cy); > - ASSERT(hVisRgn); > - GdiSelectVisRgn(pdc->BaseObject.hHmgr, hVisRgn); > - GreDeleteObject(hVisRgn); > +#if 0 > + VisRgn = IntSysCreateRectpRgn(0, 0, pdc->dclevel.sizl.cx, pdc->dclevel.sizl.cy); > + ASSERT(VisRgn); > + GdiSelectVisRgn(pdc->BaseObject.hHmgr, VisRgn); > + REGION_Delete(VisRgn); > +#endif > > pdc->flGraphicsCaps = ppdev->devinfo.flGraphicsCaps; > pdc->flGraphicsCaps2 = ppdev->devinfo.flGraphicsCaps2; > @@ -492,75 +474,106 @@ > * from where we take pixels. */ > VOID > FASTCALL > -DC_vPrepareDCsForBlit(PDC pdc1, > - RECT rc1, > - PDC pdc2, > - RECT rc2) > +DC_vPrepareDCsForBlit( > + PDC pdcDest, > + const RECT* rcDest, > + PDC pdcSrc, > + const RECT* rcSrc) > { > PDC pdcFirst, pdcSecond; > - PRECT prcFirst, prcSecond; > + const RECT *prcFirst, *prcSecond; > > /* Update brushes */ > - if (pdc1->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) > - DC_vUpdateFillBrush(pdc1); > - if (pdc1->pdcattr->ulDirty_ & (DIRTY_LINE | DC_PEN_DIRTY)) > - DC_vUpdateLineBrush(pdc1); > - if(pdc1->pdcattr->ulDirty_ & DIRTY_TEXT) > - DC_vUpdateTextBrush(pdc1); > + if (pdcDest->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) > + DC_vUpdateFillBrush(pdcDest); > + if (pdcDest->pdcattr->ulDirty_ & (DIRTY_LINE | DC_PEN_DIRTY)) > + DC_vUpdateLineBrush(pdcDest); > + if(pdcDest->pdcattr->ulDirty_ & DIRTY_TEXT) > + DC_vUpdateTextBrush(pdcDest); > > /* Lock them in good order */ > - if(pdc2) > - { > - if((ULONG_PTR)pdc1->ppdev->hsemDevLock >= (ULONG_PTR)pdc2->ppdev->hsemDevLock) > - { > - pdcFirst = pdc1; > - prcFirst = &rc1; > - pdcSecond = pdc2; > - prcSecond = &rc2; > + if(pdcSrc) > + { > + if((ULONG_PTR)pdcDest->ppdev->hsemDevLock >= (ULONG_PTR)pdcSrc->ppdev->hsemDevLock) > + { > + pdcFirst = pdcDest; > + prcFirst = rcDest; > + pdcSecond = pdcSrc; > + prcSecond = rcSrc; > } > else > { > - pdcFirst = pdc2; > - prcFirst = &rc2; > - pdcSecond = pdc1; > - prcSecond = &rc1; > + pdcFirst = pdcSrc; > + prcFirst = rcSrc; > + pdcSecond = pdcDest; > + prcSecond = rcDest; > } > } > else > { > - pdcFirst = pdc1 ; > - prcFirst = &rc1; > + pdcFirst = pdcDest ; > + prcFirst = rcDest; > pdcSecond = NULL; > prcSecond = NULL; > } > > - if(pdcFirst && pdcFirst->dctype == DCTYPE_DIRECT) > + /* Update clipping of dest DC if needed */ > + if (pdcDest->dctype == DCTYPE_DIRECT) > + { > + DCE* dce = DceGetDceFromDC(pdcDest->BaseObject.hHmgr); > + if (dce) > + DceUpdateVisRgn(dce, dce->pwndOrg, dce->DCXFlags); > + } > + > + if (pdcDest->fs & DC_FLAG_DIRTY_RAO) > + CLIPPING_UpdateGCRegion(pdcDest); > + > + /* Lock and update first DC */ > + if(pdcFirst->dctype == DCTYPE_DIRECT) > { > EngAcquireSemaphore(pdcFirst->ppdev->hsemDevLock); > - MouseSafetyOnDrawStart(pdcFirst->ppdev, > - prcFirst->left, > - prcFirst->top, > - prcFirst->right, > - prcFirst->bottom) ; > /* Update surface if needed */ > if(pdcFirst->ppdev->pSurface != pdcFirst->dclevel.pSurface) > { > DC_vUpdateDC(pdcFirst); > } > } > - if(pdcSecond && pdcSecond->dctype == DCTYPE_DIRECT) > + > + if(pdcFirst->dctype == DCTYPE_DIRECT) > + { > + if (!prcFirst) > + prcFirst = &pdcFirst->erclClip; > + > + MouseSafetyOnDrawStart(pdcFirst->ppdev, > + prcFirst->left, > + prcFirst->top, > + prcFirst->right, > + prcFirst->bottom) ; > + } > + > + if (!pdcSecond) > + return; > + > + /* Lock and update second DC */ > + if(pdcSecond->dctype == DCTYPE_DIRECT) > { > EngAcquireSemaphore(pdcSecond->ppdev->hsemDevLock); > - MouseSafetyOnDrawStart(pdcSecond->ppdev, > - prcSecond->left, > - prcSecond->top, > - prcSecond->right, > - prcSecond->bottom) ; > /* Update surface if needed */ > if(pdcSecond->ppdev->pSurface != pdcSecond->dclevel.pSurface) > { > DC_vUpdateDC(pdcSecond); > } > + } > + > + if(pdcSecond->dctype == DCTYPE_DIRECT) > + { > + if (!prcSecond) > + prcSecond = &pdcSecond->erclClip; > + MouseSafetyOnDrawStart(pdcSecond->ppdev, > + prcSecond->left, > + prcSecond->top, > + prcSecond->right, > + prcSecond->bottom) ; > } > } > > > Modified: trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -330,7 +330,7 @@ > PDC pdc; > HBITMAP hbmpOld; > PSURFACE psurfNew, psurfOld; > - HRGN hVisRgn; > + PREGION VisRgn; > HDC hdcOld; > ULONG cBitsPixel; > ASSERT_NOGDILOCKS(); > @@ -452,15 +452,15 @@ > pdc->pdcattr->ulDirty_ |= DIRTY_FILL | DIRTY_LINE; > > /* FIXME: Improve by using a region without a handle and selecting it */ > - hVisRgn = IntSysCreateRectRgn( 0, > + VisRgn = IntSysCreateRectpRgn( 0, > 0, > pdc->dclevel.sizl.cx, > pdc->dclevel.sizl.cy); > > - if (hVisRgn) > - { > - GdiSelectVisRgn(hdc, hVisRgn); > - GreDeleteObject(hVisRgn); > + if (VisRgn) > + { > + GdiSelectVisRgn(hdc, VisRgn); > + REGION_Delete(VisRgn); > } > > /* Unlock the DC */ > @@ -509,7 +509,10 @@ > /* Construct a region from the path */ > else if (PATH_PathToRegion(pPath, pdcattr->jFillMode, &hrgnPath)) > { > - success = GdiExtSelectClipRgn(pdc, hrgnPath, Mode) != ERROR; > + PREGION prgnPath = REGION_LockRgn(hrgnPath); > + ASSERT(prgnPath); > + success = IntGdiExtSelectClipRgn(pdc, prgnPath, Mode) != ERROR; > + REGION_UnlockRgn(prgnPath); > GreDeleteObject( hrgnPath ); > > /* Empty the path */ > @@ -693,9 +696,7 @@ > { > INT ret = 0; > PDC pdc; > - HRGN hrgnSrc = NULL; > PREGION prgnSrc = NULL; > - POINTL ptlOrg; > > pdc = DC_LockDc(hdc); > if (!pdc) > @@ -707,8 +708,7 @@ > switch (iCode) > { > case CLIPRGN: > - hrgnSrc = pdc->rosdc.hClipRgn; > -// if (pdc->dclevel.prgnClip) prgnSrc = pdc->dclevel.prgnClip; > + prgnSrc = pdc->dclevel.prgnClip; > break; > > case METARGN: > @@ -716,14 +716,15 @@ > break; > > case APIRGN: > + if (pdc->fs & DC_FLAG_DIRTY_RAO) > + CLIPPING_UpdateGCRegion(pdc); > if (pdc->prgnAPI) > { > prgnSrc = pdc->prgnAPI; > } > -// else if (pdc->dclevel.prgnClip) prgnSrc = pdc->dclevel.prgnClip; > - else if (pdc->rosdc.hClipRgn) > + else if (pdc->dclevel.prgnClip) > { > - hrgnSrc = pdc->rosdc.hClipRgn; > + prgnSrc = pdc->dclevel.prgnClip; > } > else if (pdc->dclevel.prgnMeta) > { > @@ -739,26 +740,18 @@ > break; > } > > - if (hrgnSrc) > - { > - ret = NtGdiCombineRgn(hrgnDest, hrgnSrc, 0, RGN_COPY) == ERROR ? -1 : 1; > - } > - else if (prgnSrc) > + if (prgnSrc) > { > PREGION prgnDest = REGION_LockRgn(hrgnDest); > if (prgnDest) > { > ret = IntGdiCombineRgn(prgnDest, prgnSrc, 0, RGN_COPY) == ERROR ? -1 : 1; > + if ((ret == 1) && (iCode == SYSRGN)) > + IntGdiOffsetRgn(prgnDest, pdc->ptlDCOrig.x, pdc->ptlDCOrig.y); > REGION_UnlockRgn(prgnDest); > } > else > ret = -1; > - } > - > - if (iCode == SYSRGN) > - { > - ptlOrg = pdc->ptlDCOrig; > - NtGdiOffsetRgn(hrgnDest, ptlOrg.x, ptlOrg.y ); > } > > DC_UnlockDc(pdc); > > Modified: trunk/reactos/win32ss/gdi/ntgdi/dcstate.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dcstate.… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/dcstate.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/dcstate.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -51,23 +51,26 @@ > pdcDst->dclevel.plfnt = pdcSrc->dclevel.plfnt; > > /* Get/SetDCState() don't change hVisRgn field ("Undoc. Windows" p.559). */ > - if (To) // Copy "To" SaveDC state. > - { > - if (pdcSrc->rosdc.hClipRgn) > - { > - pdcDst->rosdc.hClipRgn = IntSysCreateRectRgn(0, 0, 0, 0); > - NtGdiCombineRgn(pdcDst->rosdc.hClipRgn, pdcSrc->rosdc.hClipRgn, 0, RGN_COPY); > - } > - // FIXME: Handle prgnMeta! > - } > - else // Copy "!To" RestoreDC state. > - { /* The VisRectRegion field needs to be set to a valid state */ > - GdiExtSelectClipRgn(pdcDst, pdcSrc->rosdc.hClipRgn, RGN_COPY); > - } > -} > - > - > -BOOL FASTCALL > + if (!To) > + { > + IntGdiExtSelectClipRgn(pdcDst, pdcSrc->dclevel.prgnClip, RGN_COPY); > + if (pdcDst->dclevel.prgnMeta) > + { > + REGION_Delete(pdcDst->dclevel.prgnMeta); > + pdcDst->dclevel.prgnMeta = NULL; > + } > + if (pdcSrc->dclevel.prgnMeta) > + { > + pdcDst->dclevel.prgnMeta = IntSysCreateRectpRgn(0, 0, 0, 0); > + IntGdiCombineRgn(pdcDst->dclevel.prgnMeta, pdcSrc->dclevel.prgnMeta, NULL, RGN_COPY); > + } > + pdcDst->fs |= DC_FLAG_DIRTY_RAO; > + } > +} > + > + > +BOOL > +FASTCALL > IntGdiCleanDC(HDC hDC) > { > PDC dc; > @@ -87,6 +90,16 @@ > DC_vUpdateTextBrush(dc); > } > > + /* DC_vCopyState frees the Clip rgn and the Meta rgn. Take care of the other ones > + * There is no need to clear prgnVis, as UserGetDC updates it immediately. */ > + if (dc->prgnRao) > + REGION_Delete(dc->prgnRao); > + if (dc->prgnAPI) > + REGION_Delete(dc->prgnAPI); > + dc->prgnRao = dc->prgnAPI = NULL; > + > + dc->fs |= DC_FLAG_DIRTY_RAO; > + > DC_UnlockDc(dc); > > return TRUE; > @@ -272,7 +285,7 @@ > GDIOBJ_vSetObjectOwner(&pdcSave->BaseObject, GDI_OBJ_HMGR_PUBLIC); > > /* Copy the current state */ > - DC_vCopyState(pdc, pdcSave, TRUE); > + DC_vCopyState(pdc, pdcSave, FALSE); > > /* Only memory DC's change their surface */ > if (pdc->dctype == DCTYPE_MEMORY) > > Modified: trunk/reactos/win32ss/gdi/ntgdi/dcutil.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dcutil.c… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/dcutil.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/dcutil.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -367,7 +367,7 @@ > } > else if (Flags & DCHF_VALIDATEVISRGN || 0 == Flags) > { > - dc->fs &= ~DC_FLAG_DIRTY_RAO; > + //dc->fs &= ~DC_FLAG_DIRTY_RAO; > } > > DC_UnlockDc(dc); > > Modified: trunk/reactos/win32ss/gdi/ntgdi/dibobj.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dibobj.c… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/dibobj.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/dibobj.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -502,7 +502,7 @@ > } > > /* This is actually a blit */ > - DC_vPrepareDCsForBlit(pDC, rcDest, NULL, rcDest); > + DC_vPrepareDCsForBlit(pDC, &rcDest, NULL, NULL); > pSurf = pDC->dclevel.pSurface; > if (!pSurf) > { > @@ -1184,7 +1184,7 @@ > } > > /* Prepare DC for blit */ > - DC_vPrepareDCsForBlit(pdc, rcDst, NULL, rcSrc); > + DC_vPrepareDCsForBlit(pdc, &rcDst, NULL, NULL); > > psurfDst = pdc->dclevel.pSurface; > > > Modified: trunk/reactos/win32ss/gdi/ntgdi/fillshap.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/fillshap… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/fillshap.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/fillshap.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -307,7 +307,7 @@ > tmpFillBrushObj.ptOrigin.x += dc->ptlDCOrig.x; > tmpFillBrushObj.ptOrigin.y += dc->ptlDCOrig.y; > > - DC_vPrepareDCsForBlit(dc, RectBounds, NULL, RectBounds); > + DC_vPrepareDCsForBlit(dc, &RectBounds, NULL, NULL); > > ret = IntFillEllipse( dc, > CenterX - RadiusX, > @@ -480,8 +480,7 @@ > return TRUE; > } > > - DC_vPrepareDCsForBlit(dc, dc->rosdc.CombinedClip->rclBounds, > - NULL, dc->rosdc.CombinedClip->rclBounds); > + DC_vPrepareDCsForBlit(dc, NULL, NULL, NULL); > > if (dc->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) > DC_vUpdateFillBrush(dc); > @@ -567,7 +566,7 @@ > DestRect.bottom--; > } > > - DC_vPrepareDCsForBlit(dc, DestRect, NULL, DestRect); > + DC_vPrepareDCsForBlit(dc, &DestRect, NULL, NULL); > > if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) > DC_vUpdateFillBrush(dc); > @@ -796,7 +795,7 @@ > else > { > > - DC_vPrepareDCsForBlit(dc, RectBounds, NULL, RectBounds); > + DC_vPrepareDCsForBlit(dc, &RectBounds, NULL, NULL); > > RtlCopyMemory(&brushTemp, pbrFill, sizeof(brushTemp)); > brushTemp.ptOrigin.x += RectBounds.left - Left; > @@ -962,9 +961,9 @@ > > EXLATEOBJ_vInitialize(&exlo, &gpalRGB, psurf->ppal, 0, 0, 0); > > + DC_vPrepareDCsForBlit(pdc, &rclExtent, NULL, NULL); > + > ASSERT(pdc->rosdc.CombinedClip); > - > - DC_vPrepareDCsForBlit(pdc, rclExtent, NULL, rclExtent); > > bRet = IntEngGradientFill(&psurf->SurfObj, > pdc->rosdc.CombinedClip, > @@ -1091,6 +1090,13 @@ > return TRUE; > } > > + psurf = dc->dclevel.pSurface; > + if (!psurf) > + { > + Ret = FALSE; > + goto cleanup; > + } > + > pdcattr = dc->pdcattr; > > if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) > @@ -1103,20 +1109,18 @@ > Pt.y = YStart; > IntLPtoDP(dc, (LPPOINT)&Pt, 1); > > - Ret = NtGdiPtInRegion(dc->rosdc.hGCClipRgn, Pt.x, Pt.y); > - if (Ret) > - IntGdiGetRgnBox(dc->rosdc.hGCClipRgn,(LPRECT)&DestRect); > + if (dc->prgnRao) > + { > + Ret = REGION_PtInRegion(dc->prgnRao, Pt.x, Pt.y); > + if (Ret) > + REGION_GetRgnBox(dc->prgnRao ,(LPRECT)&DestRect); > + else > + goto cleanup; > + } > else > - goto cleanup; > - > - DC_vPrepareDCsForBlit(dc, DestRect, NULL, DestRect); > - > - psurf = dc->dclevel.pSurface; > - if (!psurf) > - { > - Ret = FALSE; > - goto cleanup; > - } > + RECTL_vSetRect(&DestRect, 0, psurf->SurfObj.sizlBitmap.cx, 0, psurf->SurfObj.sizlBitmap.cy); > + > + DC_vPrepareDCsForBlit(dc, &DestRect, NULL, NULL); > > EXLATEOBJ_vInitialize(&exlo, &gpalRGB, psurf->ppal, 0, 0xffffff, 0); > > @@ -1127,10 +1131,11 @@ > ConvColor = XLATEOBJ_iXlate(&exlo.xlo, Color); > Ret = DIB_XXBPP_FloodFillSolid(&psurf->SurfObj, &dc->eboFill.BrushObject, &DestRect, &Pt, ConvColor, FillType); > > + DC_vFinishBlit(dc, NULL); > + > EXLATEOBJ_vCleanup(&exlo); > > cleanup: > - DC_vFinishBlit(dc, NULL); > DC_UnlockDc(dc); > return Ret; > } > > Modified: trunk/reactos/win32ss/gdi/ntgdi/freetype.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/freetype… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/freetype.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/freetype.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -3212,7 +3212,7 @@ > LONGLONG TextLeft, RealXStart; > ULONG TextTop, previous, BackgroundLeft; > FT_Bool use_kerning; > - RECTL DestRect, MaskRect, DummyRect = {0, 0, 0, 0}; > + RECTL DestRect, MaskRect; > POINTL SourcePoint, BrushOrigin; > HBITMAP HSourceGlyph; > SURFOBJ *SourceGlyphSurf; > @@ -3308,7 +3308,7 @@ > DestRect.right += dc->ptlDCOrig.x; > DestRect.bottom += dc->ptlDCOrig.y; > > - DC_vPrepareDCsForBlit(dc, DestRect, NULL, DestRect); > + DC_vPrepareDCsForBlit(dc, &DestRect, NULL, NULL); > > if (pdcattr->ulDirty_ & DIRTY_BACKGROUND) > DC_vUpdateBackgroundBrush(dc); > @@ -3502,7 +3502,7 @@ > BackgroundLeft = (RealXStart + 32) >> 6; > > /* Lock blit with a dummy rect */ > - DC_vPrepareDCsForBlit(dc, DummyRect, NULL, DummyRect); > + DC_vPrepareDCsForBlit(dc, NULL, NULL, NULL); > > psurf = dc->dclevel.pSurface ; > if(!psurf) psurf = psurfDefaultBitmap; > > Modified: trunk/reactos/win32ss/gdi/ntgdi/line.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/line.c?r… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/line.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/line.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -257,8 +257,7 @@ > if (PATH_IsPathOpen(dc->dclevel)) > return PATH_Polyline(dc, pt, Count); > > - DC_vPrepareDCsForBlit(dc, dc->rosdc.CombinedClip->rclBounds, > - NULL, dc->rosdc.CombinedClip->rclBounds); > + DC_vPrepareDCsForBlit(dc, NULL, NULL, NULL); > > if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) > DC_vUpdateFillBrush(dc); > @@ -411,7 +410,7 @@ > rcLockRect.right += dc->ptlDCOrig.x; > rcLockRect.bottom += dc->ptlDCOrig.y; > > - DC_vPrepareDCsForBlit(dc, rcLockRect, NULL, rcLockRect); > + DC_vPrepareDCsForBlit(dc, &rcLockRect, NULL, NULL); > > if (dc->pdcattr->ulDirty_ & (DIRTY_LINE | DC_PEN_DIRTY)) > DC_vUpdateLineBrush(dc); > > Modified: trunk/reactos/win32ss/gdi/ntgdi/path.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/path.c?r… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/path.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/path.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -2511,8 +2511,7 @@ > return FALSE; > } > > - DC_vPrepareDCsForBlit(dc, dc->rosdc.CombinedClip->rclBounds, > - NULL, dc->rosdc.CombinedClip->rclBounds); > + DC_vPrepareDCsForBlit(dc, NULL, NULL, NULL); > > pdcattr = dc->pdcattr; > > @@ -2777,8 +2776,7 @@ > return FALSE; > } > > - DC_vPrepareDCsForBlit(pDc, pDc->rosdc.CombinedClip->rclBounds, > - NULL, pDc->rosdc.CombinedClip->rclBounds); > + DC_vPrepareDCsForBlit(pDc, NULL, NULL, NULL); > > pdcattr = pDc->pdcattr; > > @@ -2822,8 +2820,7 @@ > return FALSE; > } > > - DC_vPrepareDCsForBlit(pDc, pDc->rosdc.CombinedClip->rclBounds, > - NULL, pDc->rosdc.CombinedClip->rclBounds); > + DC_vPrepareDCsForBlit(pDc, NULL, NULL, NULL); > > pdcattr = pDc->pdcattr; > > > Modified: trunk/reactos/win32ss/gdi/ntgdi/region.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/region.c… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/region.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/region.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -2300,11 +2300,13 @@ > VOID FASTCALL > IntGdiReleaseRaoRgn(PDC pDC) > { > - INT Index = GDI_HANDLE_GET_INDEX(pDC->BaseObject.hHmgr); > - PGDI_TABLE_ENTRY Entry = &GdiHandleTable->Entries[Index]; > - pDC->fs |= DC_FLAG_DIRTY_RAO; > - Entry->Flags |= GDI_ENTRY_VALIDATE_VIS; > - RECTL_vSetEmptyRect(&pDC->erclClip); > + INT Index = GDI_HANDLE_GET_INDEX(pDC->BaseObject.hHmgr); > + PGDI_TABLE_ENTRY Entry = &GdiHandleTable->Entries[Index]; > + pDC->fs |= DC_FLAG_DIRTY_RAO; > + Entry->Flags |= GDI_ENTRY_VALIDATE_VIS; > + RECTL_vSetEmptyRect(&pDC->erclClip); > + REGION_Delete(pDC->prgnRao); > + pDC->prgnRao = NULL; > } > > VOID FASTCALL > @@ -2503,14 +2505,15 @@ > return FALSE; > } > > - NtGdiCombineRgn(tmpVisRgn, tmpVisRgn, dc->rosdc.hGCClipRgn, RGN_AND); > - > visrgn = RGNOBJAPI_Lock(tmpVisRgn, NULL); > if (visrgn == NULL) > { > GreDeleteObject(tmpVisRgn); > return FALSE; > } > + > + if (dc->prgnRao) > + IntGdiCombineRgn(visrgn, visrgn, dc->prgnRao, RGN_AND); > > ClipRegion = IntEngCreateClipRegion(visrgn->rdh.nCount, > visrgn->Buffer, > @@ -2533,6 +2536,29 @@ > > // Fill the region > return bRet; > +} > + > +BOOL > +FASTCALL > +REGION_PtInRegion( > + PREGION prgn, > + INT X, > + INT Y) > +{ > + ULONG i; > + PRECT r; > + > + if (prgn->rdh.nCount > 0 && INRECT(prgn->rdh.rcBound, X, Y)) > + { > + r = prgn->Buffer; > + for (i = 0; i < prgn->rdh.nCount; i++) > + { > + if (INRECT(r[i], X, Y)) > + return TRUE; > + } > + } > + > + return FALSE; > } > > BOOL > @@ -3861,27 +3887,19 @@ > INT Y > ) > { > - PROSRGNDATA rgn; > - ULONG i; > - PRECTL r; > - > - if (!(rgn = RGNOBJAPI_Lock(hRgn, NULL) ) ) > + PREGION prgn; > + BOOL ret; > + > + if (!(prgn = RGNOBJAPI_Lock(hRgn, NULL) ) ) > return FALSE; > > - if (rgn->rdh.nCount > 0 && INRECT(rgn->rdh.rcBound, X, Y)) > - { > - r = rgn->Buffer; > - for (i = 0; i < rgn->rdh.nCount; i++) > - { > - if (INRECT(*r, X, Y)) > - { > - RGNOBJAPI_Unlock(rgn); > - return TRUE; > - } > - r++; > - } > - } > - RGNOBJAPI_Unlock(rgn); > + ret = REGION_PtInRegion(prgn, X, Y); > + > + RGNOBJAPI_Unlock(prgn); > + return ret; > + > + > + RGNOBJAPI_Unlock(prgn); > return FALSE; > } > > > Modified: trunk/reactos/win32ss/gdi/ntgdi/region.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/region.h… > ============================================================================== > --- trunk/reactos/win32ss/gdi/ntgdi/region.h [iso-8859-1] (original) > +++ trunk/reactos/win32ss/gdi/ntgdi/region.h [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -27,6 +27,7 @@ > VOID FASTCALL REGION_UnionRectWithRgn(ROSRGNDATA *rgn, const RECTL *rect); > INT FASTCALL REGION_GetRgnBox(PROSRGNDATA Rgn, RECTL *pRect); > BOOL FASTCALL REGION_RectInRegion(PROSRGNDATA Rgn, const RECTL *rc); > +BOOL FASTCALL REGION_PtInRegion(PREGION, INT, INT); > BOOL FASTCALL REGION_CropAndOffsetRegion(PROSRGNDATA rgnDst, PROSRGNDATA rgnSrc, const RECTL *rect, const POINT *off); > VOID FASTCALL REGION_SetRectRgn(PROSRGNDATA pRgn, INT LeftRect, INT TopRect, INT RightRect, INT BottomRect); > VOID NTAPI REGION_vCleanup(PVOID ObjectBody); > > Modified: trunk/reactos/win32ss/user/ntuser/cursoricon.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/cursor… > ============================================================================== > --- trunk/reactos/win32ss/user/ntuser/cursoricon.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/user/ntuser/cursoricon.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -1287,7 +1287,7 @@ > RECTL_vOffsetRect(&rcDest, pdc->ptlDCOrig.x, pdc->ptlDCOrig.y); > > /* Prepare the underlying surface */ > - DC_vPrepareDCsForBlit(pdc, rcDest, NULL, rcDest); > + DC_vPrepareDCsForBlit(pdc, &rcDest, NULL, NULL); > > /* Get the clip object */ > pdcClipObj = pdc->rosdc.CombinedClip; > @@ -1472,7 +1472,7 @@ > RECTL_vOffsetRect(&rcDest, pdc->ptlDCOrig.x, pdc->ptlDCOrig.y); > > /* Prepare the underlying surface */ > - DC_vPrepareDCsForBlit(pdc, rcDest, NULL, rcDest); > + DC_vPrepareDCsForBlit(pdc, &rcDest, NULL, NULL); > > /* Get the clip object */ > pdcClipObj = pdc->rosdc.CombinedClip; > > Modified: trunk/reactos/win32ss/user/ntuser/cursoricon_new.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/cursor… > ============================================================================== > --- trunk/reactos/win32ss/user/ntuser/cursoricon_new.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/user/ntuser/cursoricon_new.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -1195,7 +1195,7 @@ > RECTL_vOffsetRect(&rcDest, pdc->ptlDCOrig.x, pdc->ptlDCOrig.y); > > /* Prepare the underlying surface */ > - DC_vPrepareDCsForBlit(pdc, rcDest, NULL, rcDest); > + DC_vPrepareDCsForBlit(pdc, &rcDest, NULL, NULL); > > /* We now have our destination surface and rectangle */ > psurfDest = pdc->dclevel.pSurface; > > Modified: trunk/reactos/win32ss/user/ntuser/dce.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/dce.h?… > ============================================================================== > --- trunk/reactos/win32ss/user/ntuser/dce.h [iso-8859-1] (original) > +++ trunk/reactos/win32ss/user/ntuser/dce.h [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -51,3 +51,5 @@ > HWND FASTCALL UserGethWnd(HDC,PWNDOBJ*); > void FASTCALL DceFreeWindowDCE(PWND); > void FASTCALL DceFreeThreadDCE(PTHREADINFO); > +VOID FASTCALL DceUpdateVisRgn(DCE *Dce, PWND Window, ULONG Flags); > +DCE* FASTCALL DceGetDceFromDC(HDC hdc); > > Modified: trunk/reactos/win32ss/user/ntuser/windc.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/windc.… > ============================================================================== > --- trunk/reactos/win32ss/user/ntuser/windc.c [iso-8859-1] (original) > +++ trunk/reactos/win32ss/user/ntuser/windc.c [iso-8859-1] Wed Jul 23 16:05:47 2014 > @@ -46,21 +46,47 @@ > return IntGdiCreateDC(&DriverName, NULL, NULL, NULL, FALSE); > } > > +/* Returns the DCE pointer from the HDC handle */ > +DCE* > +FASTCALL > +DceGetDceFromDC(HDC hdc) > +{ > + LIST_ENTRY* Entry = LEDce.Flink; > + DCE* dce; > + > + while (Entry != &LEDce) > + { > + dce = CONTAINING_RECORD(Entry, DCE, List); > + if (dce->hDC == hdc) > + return dce; > + Entry = Entry->Flink; > + } > + > + return NULL; > +} > + > static > -HRGN FASTCALL > +PREGION FASTCALL > DceGetVisRgn(PWND Window, ULONG Flags, HWND hWndChild, ULONG CFlags) > { > - HRGN VisRgn; > - > - VisRgn = VIS_ComputeVisibleRegion( Window, > - 0 == (Flags & DCX_WINDOW), > - 0 != (Flags & DCX_CLIPCHILDREN), > - 0 != (Flags & DCX_CLIPSIBLINGS)); > - > - if (VisRgn == NULL) > - VisRgn = IntSysCreateRectRgn(0, 0, 0, 0); > - > - return VisRgn; > + PREGION RetRgn; > + HRGN hVisRgn; > + hVisRgn = VIS_ComputeVisibleRegion( Window, > + 0 == (Flags & DCX_WINDOW), > + 0 != (Flags & DCX_CLIPCHILDREN), > + 0 != (Flags & DCX_CLIPSIBLINGS)); > + > + RetRgn = IntSysCreateRectpRgn(0, 0, 0, 0); > + > + if (hVisRgn != NULL) > + { > + PREGION VisRgn = REGION_LockRgn(hVisRgn); > + IntGdiCombineRgn(RetRgn, VisRgn, NULL, RGN_COPY); > + REGION_UnlockRgn(VisRgn); > + GreDeleteObject(hVisRgn); > + } > + > + return RetRgn; > } > > PDCE FASTCALL > @@ -150,6 +176,7 @@ > dc->ptlDCOrig.y = Window->rcClient.top; > } > } > + dc->fs |= DC_FLAG_DIRTY_RAO; > DC_UnlockDc(dc); > } > > @@ -175,10 +202,11 @@ > IntGdiSetHookFlags(Dce->hDC, DCHF_INVALIDATEVISRGN); > } > > -static VOID FASTCALL > +VOID > +FASTCALL > DceUpdateVisRgn(DCE *Dce, PWND Window, ULONG Flags) > { > - HANDLE hRgnVisible = NULL; > + PREGION RgnVisible = NULL; > ULONG DcxFlags; > PWND DesktopWindow; > > @@ -189,7 +217,7 @@ > Parent = Window->spwndParent; > if(!Parent) > { > - hRgnVisible = NULL; > + RgnVisible = NULL; > goto noparent; > } > > @@ -202,23 +230,23 @@ > { > DcxFlags = Flags & ~(DCX_CLIPSIBLINGS | DCX_CLIPCHILDREN | DCX_WINDOW); > } > - hRgnVisible = DceGetVisRgn(Parent, DcxFlags, Window->head.h, Flags); > + RgnVisible = DceGetVisRgn(Parent, DcxFlags, Window->head.h, Flags); > } > else if (Window == NULL) > { > DesktopWindow = UserGetWindowObject(IntGetDesktopWindow()); > if (NULL != DesktopWindow) > { > - hRgnVisible = IntSysCreateRectRgnIndirect(&DesktopWindow->rcWindow); > + RgnVisible = IntSysCreateRectpRgnIndirect(&DesktopWindow->rcWindow); > } > else > { > - hRgnVisible = NULL; > + RgnVisible = NULL; > } > } > else > { > - hRgnVisible = DceGetVisRgn(Window, Flags, 0, 0); > + RgnVisible = DceGetVisRgn(Window, Flags, 0, 0); > } > > noparent: > @@ -226,33 +254,37 @@ > { > if(Dce->hrgnClip != NULL) > { > - NtGdiCombineRgn(hRgnVisible, hRgnVisible, Dce->hrgnClip, RGN_AND); > + PREGION RgnClip = REGION_LockRgn(Dce->hrgnClip); > + IntGdiCombineRgn(RgnVisible, RgnVisible, RgnClip, RGN_AND); > + REGION_UnlockRgn(RgnClip); > } > else > { > - if(hRgnVisible != NULL) > - { > - GreDeleteObject(hRgnVisible); > - } > - hRgnVisible = IntSysCreateRectRgn(0, 0, 0, 0); > - } > - } > - else if (Flags & DCX_EXCLUDERGN && Dce->hrgnClip != NULL) > - { > - NtGdiCombineRgn(hRgnVisible, hRgnVisible, Dce->hrgnClip, RGN_DIFF); > + if(RgnVisible != NULL) > + { > + REGION_Delete(RgnVisible); > + } > + RgnVisible = IntSysCreateRectpRgn(0, 0, 0, 0); > + } > + } > + else if ((Flags & DCX_EXCLUDERGN) && Dce->hrgnClip != NULL) > + { > + PREGION RgnClip = REGION_LockRgn(Dce->hrgnClip); > + IntGdiCombineRgn(RgnVisible, RgnVisible, RgnClip, RGN_DIFF); > + REGION_UnlockRgn(RgnClip); > } > > Dce->DCXFlags &= ~DCX_DCEDIRTY; > - GdiSelectVisRgn(Dce->hDC, hRgnVisible); > + GdiSelectVisRgn(Dce->hDC, RgnVisible); > > if (VerifyWnd(Window)) // Window maybe dead by this time before finishing the DCE release. > { > IntEngWindowChanged(Window, WOC_RGN_CLIENT); > } > > - if (hRgnVisible != NULL) > - { > - GreDeleteObject(hRgnVisible); > + if (RgnVisible != NULL) > + { > + REGION_Delete(RgnVisible); > } > } > > @@ -889,10 +921,10 @@ > dc->ptlDCOrig.y = CurrentWindow->rcClient.top; > } > > - if (NULL != dc->rosdc.hClipRgn) > + if (NULL != dc->dclevel.prgnClip) > { > - NtGdiOffsetRgn(dc->rosdc.hClipRgn, DeltaX, DeltaY); > - CLIPPING_UpdateGCRegion(dc); > + IntGdiOffsetRgn(dc->dclevel.prgnClip, DeltaX, DeltaY); > + dc->fs |= DC_FLAG_DIRTY_RAO; > } > if (NULL != pDCE->hrgnClip) > { > > >

1 0

Re: [ros-dev] [ros-diffs] [ekohl] 63713: [BOOTDATA] Add / enable NLS settings for Arabic, Azeri, Kyrgyz, Latvian, Tatar and Uzbek. CORE-8348 #resolve
by Hermès BÉLUSCA - MAÏTO 13 Jul '14

13 Jul '14

Hello Eric, Are those changes (mentioned below) necessary for NLS settings ? Cheers, Hermès. ------------------------------------------------------------- Author: ekohl Date: Sun Jul 13 16:40:44 2014 New Revision: 63713 URL: http://svn.reactos.org/svn/reactos?rev=63713&view=rev Log: [BOOTDATA] Add / enable NLS settings for Arabic, Azeri, Kyrgyz, Latvian, Tatar and Uzbek. CORE-8348 #resolve Modified: trunk/reactos/boot/bootdata/hivesys.inf Modified: trunk/reactos/boot/bootdata/hivesys.inf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/bootdata/hivesys.inf?r ev=63713&r1=63712&r2=63713&view=diff ============================================================================ == --- trunk/reactos/boot/bootdata/hivesys.inf [iso-8859-1] (original) +++ trunk/reactos/boot/bootdata/hivesys.inf [iso-8859-1] Sun Jul 13 16:40:44 2014 [...] ; PNP Root device HKLM,"SYSTEM\CurrentControlSet\Enum\HTREE\ROOT\0","",0x00000000,"" + +HKLM,"SYSTEM\CurrentControlSet\Hardware Profiles",,0x00000012 +HKLM,"SYSTEM\CurrentControlSet\Hardware Profiles\0000",,0x00000012 +HKLM,"SYSTEM\CurrentControlSet\Hardware Profiles\0000\Software",,0x00000012 +HKLM,"SYSTEM\CurrentControlSet\Hardware Profiles\0000\System",,0x00000012 ; Afd driver HKLM,"SYSTEM\CurrentControlSet\Services\Afd","ErrorControl",0x00010001,0x000 00001 @@ -1963,7 +2003,7 @@ HKLM,"SYSTEM\Setup","SystemSetupInProgress",0x00010001,0x00000001 ; Debug channels -;HKLM,"SYSTEM\CurrentControlSet\Control\Session Manager\Environment","DEBUGCHANNEL",0x00020000,"+ole,+rpc" +HKLM,"SYSTEM\CurrentControlSet\Control\Session Manager\Environment","DEBUGCHANNEL",0x00020000,"+sam,+netapi32,+msports,+dev mgr" ; Winsrv configuration HKLM,"SYSTEM\CurrentControlSet\Control\PriorityControl",,0x00000012

2 1

Re: [ros-dev] [ros-diffs] [jgardou] 63700: [NTOSKRNL] - Use ZwSetValueKey instead of NtSetValueKey - Properly probe and copy parameters when NtSetValueKey is called from UMode CORE-7738 #resolve #comment fixed in r63700
by Alex Ionescu 10 Jul '14

10 Jul '14

Calling Nt in those functions was by design, and is how Windows does it as well. I can explain Nt vs Zw (for the 100th time) as needed, but for now, I'd kindly ask for a revert please. Second, the new code does not conform to formatting and style guidelines. Please fix the code before comitting. Best regards, Alex Ionescu On Tue, Jul 8, 2014 at 11:42 AM, <jgardou(a)svn.reactos.org> wrote: > Author: jgardou > Date: Tue Jul 8 18:42:20 2014 > New Revision: 63700 > > URL: http://svn.reactos.org/svn/reactos?rev=63700&view=rev > Log: > [NTOSKRNL] > - Use ZwSetValueKey instead of NtSetValueKey > - Properly probe and copy parameters when NtSetValueKey is called from > UMode > CORE-7738 #resolve #comment fixed in r63700 > > Modified: > trunk/reactos/ntoskrnl/config/cmconfig.c > trunk/reactos/ntoskrnl/config/cmsysini.c > trunk/reactos/ntoskrnl/config/i386/cmhardwr.c > trunk/reactos/ntoskrnl/config/ntapi.c > trunk/reactos/ntoskrnl/config/powerpc/cmhardwr.c > trunk/reactos/ntoskrnl/ex/init.c > trunk/reactos/ntoskrnl/io/iomgr/bootlog.c > > Modified: trunk/reactos/ntoskrnl/config/cmconfig.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/config/cmconfig.c… > > ============================================================================== > --- trunk/reactos/ntoskrnl/config/cmconfig.c [iso-8859-1] (original) > +++ trunk/reactos/ntoskrnl/config/cmconfig.c [iso-8859-1] Tue Jul 8 > 18:42:20 2014 > @@ -97,7 +97,7 @@ > > /* Setup the component information key */ > RtlInitUnicodeString(&ValueName, L"Component Information"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_BINARY, > @@ -124,7 +124,7 @@ > if (NT_SUCCESS(Status)) > { > /* Save the identifier in the registry */ > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_SZ, > @@ -178,7 +178,7 @@ > CmpConfigurationData->BusNumber = BusNumber; > > /* Save the actual data */ > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_FULL_RESOURCE_DESCRIPTOR, > > Modified: trunk/reactos/ntoskrnl/config/cmsysini.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/config/cmsysini.c… > > ============================================================================== > --- trunk/reactos/ntoskrnl/config/cmsysini.c [iso-8859-1] (original) > +++ trunk/reactos/ntoskrnl/config/cmsysini.c [iso-8859-1] Tue Jul 8 > 18:42:20 2014 > @@ -416,7 +416,7 @@ > > /* Key opened, now write to the key */ > RtlInitUnicodeString(&KeyName, L"SystemStartOptions"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &KeyName, > 0, > REG_SZ, > @@ -427,7 +427,7 @@ > /* Setup value name for system boot device in ARC format */ > RtlInitUnicodeString(&KeyName, L"SystemBootDevice"); > RtlCreateUnicodeStringFromAsciiz(&ValueName, > LoaderBlock->ArcBootDeviceName); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &KeyName, > 0, > REG_SZ, > @@ -474,7 +474,7 @@ > OBJ_CASE_INSENSITIVE, > NULL, > NULL); > - Status = NtOpenKey(&SelectHandle, KEY_READ, &ObjectAttributes); > + Status = ZwOpenKey(&SelectHandle, KEY_READ, &ObjectAttributes); > if (!NT_SUCCESS(Status)) > { > /* ReactOS Hack: Hard-code current to 001 for SetupLdr */ > @@ -488,16 +488,17 @@ > OBJ_CASE_INSENSITIVE, > NULL, > NULL); > - Status = NtCreateKey(&KeyHandle, > + Status = ZwCreateKey(&KeyHandle, > KEY_ALL_ACCESS, > &ObjectAttributes, > 0, > NULL, > 0, > &Disposition); > - if (!NT_SUCCESS(Status)) return Status; > - > - /* Don't need the handle */ > + if (!NT_SUCCESS(Status)) > + return Status; > + > + /* We don't need the handle */ > ZwClose(KeyHandle); > > /* Use hard-coded setting */ > @@ -533,7 +534,7 @@ > OBJ_CASE_INSENSITIVE, > NULL, > NULL); > - Status = NtCreateKey(&KeyHandle, > + Status = ZwCreateKey(&KeyHandle, > KEY_CREATE_LINK, > &ObjectAttributes, > 0, > @@ -557,7 +558,7 @@ > Status = RtlAnsiStringToUnicodeString(&KeyName, &TempString, FALSE); > > /* Set the value */ > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &CmSymbolicLinkValueName, > 0, > REG_LINK, > @@ -572,7 +573,7 @@ > KeyHandle, > NULL); > Status = NtOpenKey(&ConfigHandle, KEY_READ, &ObjectAttributes); > - NtClose(KeyHandle); > + ZwClose(KeyHandle); > > /* Check if we don't have one */ > if (!NT_SUCCESS(Status)) > @@ -689,7 +690,7 @@ > ASSERT(STATUS_SUCCESS == Status); > > /* Set it */ > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &CmSymbolicLinkValueName, > 0, > REG_LINK, > @@ -2180,7 +2181,7 @@ > RtlInitUnicodeString(&ValueData, > Buffer); > > - NtSetValueKey(CurrentVersionKeyHandle, > + ZwSetValueKey(CurrentVersionKeyHandle, > &ValueName, > 0, > REG_SZ, > > Modified: trunk/reactos/ntoskrnl/config/i386/cmhardwr.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/config/i386/cmhar… > > ============================================================================== > --- trunk/reactos/ntoskrnl/config/i386/cmhardwr.c [iso-8859-1] > (original) > +++ trunk/reactos/ntoskrnl/config/i386/cmhardwr.c [iso-8859-1] Tue > Jul 8 18:42:20 2014 > @@ -268,7 +268,7 @@ > > /* Set the value */ > RtlInitUnicodeString(&ValueName, L"PhysicalAddressExtension"); > - NtSetValueKey(KeyHandle, > + ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_DWORD, > @@ -463,7 +463,7 @@ > > /* Add it to the registry */ > RtlInitUnicodeString(&ValueName, > L"ProcessorNameString"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_SZ, > @@ -486,7 +486,7 @@ > > /* Add it to the registry */ > RtlInitUnicodeString(&ValueName, L"VendorIdentifier"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_SZ, > @@ -502,7 +502,7 @@ > { > /* Add them to the registry */ > RtlInitUnicodeString(&ValueName, L"FeatureSet"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_DWORD, > @@ -515,7 +515,7 @@ > { > /* Add it to the registry */ > RtlInitUnicodeString(&ValueName, L"~MHz"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_DWORD, > @@ -528,7 +528,7 @@ > { > /* Add it to the registry */ > RtlInitUnicodeString(&ValueName, L"Update Signature"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_BINARY, > @@ -621,7 +621,7 @@ > > /* Write the date into the registry */ > RtlInitUnicodeString(&ValueName, L"SystemBiosDate"); > - Status = NtSetValueKey(SystemHandle, > + Status = ZwSetValueKey(SystemHandle, > &ValueName, > 0, > REG_SZ, > @@ -643,7 +643,7 @@ > if (NT_SUCCESS(Status)) > { > /* Save it to the registry */ > - Status = NtSetValueKey(BiosHandle, > + Status = ZwSetValueKey(BiosHandle, > &ValueName, > 0, > REG_SZ, > @@ -703,7 +703,7 @@ > > /* Write the BIOS Version to the registry */ > RtlInitUnicodeString(&ValueName, L"SystemBiosVersion"); > - Status = NtSetValueKey(SystemHandle, > + Status = ZwSetValueKey(SystemHandle, > &ValueName, > 0, > REG_MULTI_SZ, > @@ -746,7 +746,7 @@ > > /* Write the date into the registry */ > RtlInitUnicodeString(&ValueName, L"VideoBiosDate"); > - Status = NtSetValueKey(SystemHandle, > + Status = ZwSetValueKey(SystemHandle, > &ValueName, > 0, > REG_SZ, > @@ -801,7 +801,7 @@ > > /* Write the BIOS Version to the registry */ > RtlInitUnicodeString(&ValueName, L"VideoBiosVersion"); > - Status = NtSetValueKey(SystemHandle, > + Status = ZwSetValueKey(SystemHandle, > &ValueName, > 0, > REG_MULTI_SZ, > > Modified: trunk/reactos/ntoskrnl/config/ntapi.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/config/ntapi.c?re… > > ============================================================================== > --- trunk/reactos/ntoskrnl/config/ntapi.c [iso-8859-1] (original) > +++ trunk/reactos/ntoskrnl/config/ntapi.c [iso-8859-1] Tue Jul 8 > 18:42:20 2014 > @@ -33,7 +33,10 @@ > CM_PARSE_CONTEXT ParseContext = {0}; > HANDLE Handle; > PAGED_CODE(); > - DPRINT("NtCreateKey(OB name %wZ)\n", ObjectAttributes->ObjectName); > + > + DPRINT("NtCreateKey(Path: %wZ, Root %x, Access: %x, CreateOptions > %x)\n", > + ObjectAttributes->ObjectName, ObjectAttributes->RootDirectory, > + DesiredAccess, CreateOptions); > > /* Check for user-mode caller */ > if (PreviousMode != KernelMode) > @@ -60,7 +63,8 @@ > sizeof(OBJECT_ATTRIBUTES), > sizeof(ULONG)); > > - if (Disposition) ProbeForWriteUlong(Disposition); > + if (Disposition) > + ProbeForWriteUlong(Disposition); > } > _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) > { > @@ -101,6 +105,8 @@ > } > _SEH2_END; > > + DPRINT("Returning handle %x, Status %x.\n", Handle, Status); > + > /* Return status */ > return Status; > } > @@ -116,7 +122,8 @@ > NTSTATUS Status; > KPROCESSOR_MODE PreviousMode = ExGetPreviousMode(); > PAGED_CODE(); > - DPRINT("NtOpenKey(OB 0x%wZ)\n", ObjectAttributes->ObjectName); > + DPRINT("NtOpenKey(Path: %wZ, Root %x, Access: %x)\n", > + ObjectAttributes->ObjectName, > ObjectAttributes->RootDirectory, DesiredAccess); > > /* Check for user-mode caller */ > if (PreviousMode != KernelMode) > @@ -166,6 +173,8 @@ > _SEH2_END; > } > > + DPRINT("Returning handle %x, Status %x.\n", Handle, Status); > + > /* Return status */ > return Status; > } > @@ -301,6 +310,7 @@ > > /* Dereference and return status */ > ObDereferenceObject(KeyObject); > + DPRINT("Returning status %x.\n", Status); > return Status; > } > > @@ -604,14 +614,52 @@ > IN PVOID Data, > IN ULONG DataSize) > { > - NTSTATUS Status; > - PCM_KEY_BODY KeyObject; > + NTSTATUS Status = STATUS_SUCCESS; > + PCM_KEY_BODY KeyObject = NULL; > REG_SET_VALUE_KEY_INFORMATION SetValueKeyInfo; > REG_POST_OPERATION_INFORMATION PostOperationInfo; > - UNICODE_STRING ValueNameCopy = *ValueName; > + UNICODE_STRING ValueNameCopy; > + KPROCESSOR_MODE PreviousMode; > + > PAGED_CODE(); > + > + PreviousMode = ExGetPreviousMode(); > + > + if (Data && !DataSize) > + return STATUS_INVALID_PARAMETER; > + > + /* Probe and copy the data */ > + if ((PreviousMode != KernelMode) && Data) > + { > + PVOID DataCopy = ExAllocatePoolWithTag(PagedPool, DataSize, > TAG_CM); > + if (!DataCopy) > + return STATUS_NO_MEMORY; > + _SEH2_TRY > + { > + ProbeForRead(Data, DataSize, 1); > + RtlCopyMemory(DataCopy, Data, DataSize); > + } > + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) > + { > + Status = _SEH2_GetExceptionCode(); > + } > + _SEH2_END; > + > + if (!NT_SUCCESS(Status)) > + { > + ExFreePoolWithTag(DataCopy, TAG_CM); > + return Status; > + } > + Data = DataCopy; > + } > + > + /* Capture the string */ > + Status = ProbeAndCaptureUnicodeString(&ValueNameCopy, PreviousMode, > ValueName); > + if (!NT_SUCCESS(Status)) > + goto end; > + > DPRINT("NtSetValueKey() KH 0x%p, VN '%wZ', TI %x, T %lu, DS %lu\n", > - KeyHandle, ValueName, TitleIndex, Type, DataSize); > + KeyHandle, &ValueNameCopy, TitleIndex, Type, DataSize); > > /* Verify that the handle is valid and is a registry key */ > Status = ObReferenceObjectByHandle(KeyHandle, > @@ -620,7 +668,8 @@ > ExGetPreviousMode(), > (PVOID*)&KeyObject, > NULL); > - if (!NT_SUCCESS(Status)) return Status; > + if (!NT_SUCCESS(Status)) > + goto end; > > /* Make sure the name is aligned, not too long, and the data under > 4GB */ > if ( (ValueNameCopy.Length > 32767) || > @@ -628,8 +677,8 @@ > (DataSize > 0x80000000)) > { > /* Fail */ > - ObDereferenceObject(KeyObject); > - return STATUS_INVALID_PARAMETER; > + Status = STATUS_INVALID_PARAMETER; > + goto end; > } > > /* Ignore any null characters at the end */ > @@ -644,14 +693,14 @@ > if (KeyObject->KeyControlBlock->ExtFlags & CM_KCB_READ_ONLY_KEY) > { > /* Fail */ > - ObDereferenceObject(KeyObject); > - return STATUS_ACCESS_DENIED; > + Status = STATUS_ACCESS_DENIED; > + goto end; > } > > /* Setup callback */ > PostOperationInfo.Object = (PVOID)KeyObject; > SetValueKeyInfo.Object = (PVOID)KeyObject; > - SetValueKeyInfo.ValueName = ValueName; > + SetValueKeyInfo.ValueName = &ValueNameCopy; > SetValueKeyInfo.TitleIndex = TitleIndex; > SetValueKeyInfo.Type = Type; > SetValueKeyInfo.Data = Data; > @@ -673,8 +722,13 @@ > PostOperationInfo.Status = Status; > CmiCallRegisteredCallbacks(RegNtPostSetValueKey, &PostOperationInfo); > > +end: > /* Dereference and return status */ > - ObDereferenceObject(KeyObject); > + if (KeyObject) > + ObDereferenceObject(KeyObject); > + ReleaseCapturedUnicodeString(&ValueNameCopy, PreviousMode); > + if ((PreviousMode != KernelMode) && Data) > + ExFreePoolWithTag(Data, TAG_CM); > return Status; > } > > > Modified: trunk/reactos/ntoskrnl/config/powerpc/cmhardwr.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/config/powerpc/cm… > > ============================================================================== > --- trunk/reactos/ntoskrnl/config/powerpc/cmhardwr.c [iso-8859-1] > (original) > +++ trunk/reactos/ntoskrnl/config/powerpc/cmhardwr.c [iso-8859-1] Tue > Jul 8 18:42:20 2014 > @@ -267,7 +267,7 @@ > > /* Set the value */ > RtlInitUnicodeString(&ValueName, L"PhysicalAddressExtension"); > - NtSetValueKey(KeyHandle, > + ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_DWORD, > @@ -465,7 +465,7 @@ > > /* Add it to the registry */ > RtlInitUnicodeString(&ValueName, > L"ProcessorNameString"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_SZ, > @@ -488,7 +488,7 @@ > > /* Add it to the registry */ > RtlInitUnicodeString(&ValueName, L"VendorIdentifier"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_SZ, > @@ -504,7 +504,7 @@ > { > /* Add them to the registry */ > RtlInitUnicodeString(&ValueName, L"FeatureSet"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_DWORD, > @@ -517,7 +517,7 @@ > { > /* Add it to the registry */ > RtlInitUnicodeString(&ValueName, L"~MHz"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_DWORD, > @@ -530,7 +530,7 @@ > { > /* Add it to the registry */ > RtlInitUnicodeString(&ValueName, L"Update Signature"); > - Status = NtSetValueKey(KeyHandle, > + Status = ZwSetValueKey(KeyHandle, > &ValueName, > 0, > REG_BINARY, > @@ -623,7 +623,7 @@ > > /* Write the date into the registry */ > RtlInitUnicodeString(&ValueName, L"SystemBiosDate"); > - Status = NtSetValueKey(SystemHandle, > + Status = ZwSetValueKey(SystemHandle, > &ValueName, > 0, > REG_SZ, > @@ -645,7 +645,7 @@ > if (NT_SUCCESS(Status)) > { > /* Save it to the registry */ > - Status = NtSetValueKey(BiosHandle, > + Status = ZwSetValueKey(BiosHandle, > &ValueName, > 0, > REG_SZ, > @@ -705,7 +705,7 @@ > > /* Write the BIOS Version to the registry */ > RtlInitUnicodeString(&ValueName, L"SystemBiosVersion"); > - Status = NtSetValueKey(SystemHandle, > + Status = ZwSetValueKey(SystemHandle, > &ValueName, > 0, > REG_MULTI_SZ, > @@ -749,7 +749,7 @@ > > /* Write the date into the registry */ > RtlInitUnicodeString(&ValueName, L"VideoBiosDate"); > - Status = NtSetValueKey(SystemHandle, > + Status = ZwSetValueKey(SystemHandle, > &ValueName, > 0, > REG_SZ, > @@ -804,7 +804,7 @@ > > /* Write the BIOS Version to the registry */ > RtlInitUnicodeString(&ValueName, L"VideoBiosVersion"); > - Status = NtSetValueKey(SystemHandle, > + Status = ZwSetValueKey(SystemHandle, > &ValueName, > 0, > REG_MULTI_SZ, > > Modified: trunk/reactos/ntoskrnl/ex/init.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ex/init.c?rev=637… > > ============================================================================== > --- trunk/reactos/ntoskrnl/ex/init.c [iso-8859-1] (original) > +++ trunk/reactos/ntoskrnl/ex/init.c [iso-8859-1] Tue Jul 8 18:42:20 > 2014 > @@ -1853,7 +1853,7 @@ > { > /* Write the safe boot type */ > RtlInitUnicodeString(&KeyName, L"OptionValue"); > - NtSetValueKey(OptionHandle, > + ZwSetValueKey(OptionHandle, > &KeyName, > 0, > REG_DWORD, > @@ -1866,7 +1866,7 @@ > /* Remember this for later */ > Disposition = TRUE; > RtlInitUnicodeString(&KeyName, L"UseAlternateShell"); > - NtSetValueKey(OptionHandle, > + ZwSetValueKey(OptionHandle, > &KeyName, > 0, > REG_DWORD, > > Modified: trunk/reactos/ntoskrnl/io/iomgr/bootlog.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/bootlog.… > > ============================================================================== > --- trunk/reactos/ntoskrnl/io/iomgr/bootlog.c [iso-8859-1] (original) > +++ trunk/reactos/ntoskrnl/io/iomgr/bootlog.c [iso-8859-1] Tue Jul 8 > 18:42:20 2014 > @@ -135,7 +135,7 @@ > > if (!NT_SUCCESS(Status)) > { > - DPRINT1("NtSetValueKey() failed (Status %lx)\n", Status); > + DPRINT1("ZwSetValueKey() failed (Status %lx)\n", Status); > } > else > { > > >

6 6

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-dev