- Ros-dev - ReactOS.org

Re: [ros-dev] [ros-diffs] [akhaldi] 67242: [THEMEUI] Add this module for .msstyle file type association. By Lee Schroeder. CORE-5984
by Timo Kreuzer 18 Apr '15

18 Apr '15

Just as a reminder: we can put registry stuff, that is related to a special module, into a separate file together with the module itself now. This one looks a bit like it should go into the the themeui folder. Am 18.04.2015 um 11:52 schrieb akhaldi(a)svn.reactos.org: > Modified: trunk/reactos/boot/bootdata/hivecls.inf > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/bootdata/hivecls.inf?… > ============================================================================== > --- trunk/reactos/boot/bootdata/hivecls.inf [iso-8859-1] (original) > +++ trunk/reactos/boot/bootdata/hivecls.inf [iso-8859-1] Sat Apr 18 09:52:23 2015 > @@ -265,6 +265,18 @@ > HKCR,"sysfile","NoOpen",0x00000000,"" > HKCR,"sysfile","FriendlyTypeName",0x00020000,"@%SystemRoot%\system32\shell32.dll,-171" > HKCR,"sysfile\DefaultIcon","",0x00020000,"%SystemRoot%\system32\shell32.dll,-154" > + > +; MS Styles (Themes) > +HKCR,".msstyles","",0x00000000,"msstylesfile" > +HKCR,"msstylesfile","",0x00000000,"Visual Style File" > +HKCR,"msstylesfile\DefaultIcon","","0x00020000","%SystemRoot%\system32\themeui.dll,-1" > +HKCR,"msstylesfile\shell\open\command","",0x00000000,"%SystemRoot%\system32\rundll32.exe shell32.dll,Control_RunDLL %SystemRoot%\system32\desk.cpl desk,@Appearance /Action:OpenMSTheme file:""%1""" > + > +; Theme File > +HKCR,".theme","",0x00000000,"themefile" > +HKCR,"themefile","",0x00000000,"Theme File" > +HKCR,"themefile\DefaultIcon","","0x00020000","%SystemRoot%\system32\themeui.dll,-1" > +HKCR,"themefile\shell\open\command","",0x00000000,"%SystemRoot%\system32\rundll32.exe shell32.dll,Control_RunDLL %SystemRoot%\system32\desk.cpl desk,@Appearance /Action:OpenTheme /file:""%1""" > > ; URL shortcuts (e.g. used in favorites folder of IExplorer) > HKCR,".url","",0x00000000,"InternetShortcut" > >

1 0

Re: [ros-dev] [ros-diffs] [cwittich] 67177: [SNDREC32] do not delete objects which are still selected into a DC
by Timo Kreuzer 12 Apr '15

12 Apr '15

Just for the record: it's perfectly fine and safe to do that. They will be kept alive as long as they are selected and will be deleted, once a different object is selected or the DC is deleted. Am 12.04.2015 um 18:29 schrieb cwittich(a)svn.reactos.org: > Author: cwittich > Date: Sun Apr 12 16:29:36 2015 > New Revision: 67177 > > URL: http://svn.reactos.org/svn/reactos?rev=67177&view=rev > Log: > [SNDREC32] > do not delete objects which are still selected into a DC > > Modified: > trunk/reactos/base/applications/sndrec32/sndrec32.cpp > > Modified: trunk/reactos/base/applications/sndrec32/sndrec32.cpp > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/sndrec32… > ============================================================================== > --- trunk/reactos/base/applications/sndrec32/sndrec32.cpp [iso-8859-1] (original) > +++ trunk/reactos/base/applications/sndrec32/sndrec32.cpp [iso-8859-1] Sun Apr 12 16:29:36 2015 > @@ -352,6 +352,7 @@ > PAINTSTRUCT ps; > HDC hdc; > HPEN pen; > + HPEN oldpen; > > unsigned int max_h = (cli.bottom / 2); > unsigned int samples; > @@ -366,7 +367,7 @@ > /* Initialize hdc objects */ > hdc = BeginPaint(hWnd, &ps); > pen = (HPEN)CreatePen(PS_SOLID, 1, WAVEBAR_COLOR); > - SelectObject(hdc, (HBRUSH)pen); > + oldpen = (HPEN) SelectObject(hdc, (HBRUSH)pen); > if (AUD_OUT->current_status() == snd::WAVEOUT_PLAYING) > { > samples = AUD_OUT->tot_samples_buf(); > @@ -412,6 +413,7 @@ > LineTo(hdc, WAVEBAR_CX, cli.bottom / 2); > } > > + SelectObject(hdc, oldpen); > DeleteObject( pen ); > EndPaint( hWnd, &ps ); > break; > @@ -437,6 +439,7 @@ > PAINTSTRUCT ps; > HDC hdc; > HFONT font; > + HFONT oldfont; > long long slid_samp = 0; > > /* Checking for global pointers to buffer and io audio devices */ > @@ -774,7 +777,7 @@ > case WM_PAINT: > hdc = BeginPaint(hWnd, &ps); > font = CreateFontIndirect(&s_info.lfMenuFont); > - SelectObject(hdc, font); > + oldfont = (HFONT) SelectObject(hdc, font); > SetBkMode(hdc, TRANSPARENT); > > if (AUD_IN->current_status() == snd::WAVEIN_RECORDING) > @@ -877,6 +880,7 @@ > _tcslen(str_tmp), > 0); > > + SelectObject(hdc, oldfont); > DeleteObject(font); > EndPaint(hWnd, &ps); > break; > > >

1 0

Re: [ros-dev] [ros-diffs] [tkreuzer] 67140: [RTL] Improve RtlImageNtHeaderEx: - Fix signed/unsigned mismatch when comparing NT header offset - Simplify overflow checks - Add missing overflow-into-systemspace check CR-77 / C...
by Alex Ionescu 11 Apr '15

11 Apr '15

The Windows implementation of the API is different in user vs kernel-mode. We should probably try to remain compatible. Best regards, Alex Ionescu On Fri, Apr 10, 2015 at 12:21 PM, <tkreuzer(a)svn.reactos.org> wrote: > Author: tkreuzer > Date: Fri Apr 10 19:21:22 2015 > New Revision: 67140 > > URL: http://svn.reactos.org/svn/reactos?rev=67140&view=rev > Log: > [RTL] > Improve RtlImageNtHeaderEx: > - Fix signed/unsigned mismatch when comparing NT header offset > - Simplify overflow checks > - Add missing overflow-into-systemspace check > CR-77 / CORE-8091 #resolve > > Modified: > trunk/reactos/boot/freeldr/freeldr/lib/rtl/libsupp.c > trunk/reactos/dll/ntdll/rtl/libsupp.c > trunk/reactos/include/ndk/amd64/mmtypes.h > trunk/reactos/include/ndk/i386/mmtypes.h > trunk/reactos/lib/rtl/image.c > trunk/reactos/lib/rtl/rtlp.h > trunk/reactos/ntoskrnl/include/internal/amd64/mm.h > trunk/reactos/ntoskrnl/mm/ARM3/miarm.h > > Modified: trunk/reactos/boot/freeldr/freeldr/lib/rtl/libsupp.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/lib/r… > > ============================================================================== > --- trunk/reactos/boot/freeldr/freeldr/lib/rtl/libsupp.c > [iso-8859-1] (original) > +++ trunk/reactos/boot/freeldr/freeldr/lib/rtl/libsupp.c > [iso-8859-1] Fri Apr 10 19:21:22 2015 > @@ -19,6 +19,8 @@ > */ > > #include <freeldr.h> > + > +PVOID MmHighestUserAddress = (PVOID)MI_HIGHEST_USER_ADDRESS; > > #if DBG > VOID FASTCALL > > Modified: trunk/reactos/dll/ntdll/rtl/libsupp.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/dll/ntdll/rtl/libsupp.c?re… > > ============================================================================== > --- trunk/reactos/dll/ntdll/rtl/libsupp.c [iso-8859-1] (original) > +++ trunk/reactos/dll/ntdll/rtl/libsupp.c [iso-8859-1] Fri Apr 10 > 19:21:22 2015 > @@ -16,6 +16,7 @@ > > SIZE_T RtlpAllocDeallocQueryBufferSize = PAGE_SIZE; > PTEB LdrpTopLevelDllBeingLoadedTeb = NULL; > +PVOID MmHighestUserAddress = (PVOID)MI_HIGHEST_USER_ADDRESS; > > /* FUNCTIONS > ***************************************************************/ > > > Modified: trunk/reactos/include/ndk/amd64/mmtypes.h > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/include/ndk/amd64/mmtypes.… > > ============================================================================== > --- trunk/reactos/include/ndk/amd64/mmtypes.h [iso-8859-1] (original) > +++ trunk/reactos/include/ndk/amd64/mmtypes.h [iso-8859-1] Fri Apr 10 > 19:21:22 2015 > @@ -36,6 +36,11 @@ > #define MM_ALLOCATION_GRANULARITY 0x10000 > #define MM_ALLOCATION_GRANULARITY_SHIFT 16L > #define MM_PAGE_FRAME_NUMBER_SIZE 52 > + > +// > +// User space range limit > +// > +#define MI_HIGHEST_USER_ADDRESS (PVOID)0x000007FFFFFEFFFFULL > > // > // Address of the shared user page > > Modified: trunk/reactos/include/ndk/i386/mmtypes.h > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/include/ndk/i386/mmtypes.h… > > ============================================================================== > --- trunk/reactos/include/ndk/i386/mmtypes.h [iso-8859-1] (original) > +++ trunk/reactos/include/ndk/i386/mmtypes.h [iso-8859-1] Fri Apr 10 > 19:21:22 2015 > @@ -35,6 +35,11 @@ > #define MM_ALLOCATION_GRANULARITY 0x10000 > #define MM_ALLOCATION_GRANULARITY_SHIFT 16L > #define MM_PAGE_FRAME_NUMBER_SIZE 20 > + > +// > +// User space range limit > +// > +#define MI_HIGHEST_USER_ADDRESS (PVOID)0x7FFEFFFF > > // > // Address of the shared user page > > Modified: trunk/reactos/lib/rtl/image.c > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/image.c?rev=67140&… > > ============================================================================== > --- trunk/reactos/lib/rtl/image.c [iso-8859-1] (original) > +++ trunk/reactos/lib/rtl/image.c [iso-8859-1] Fri Apr 10 19:21:22 > 2015 > @@ -137,32 +137,42 @@ > */ > NTSTATUS > NTAPI > -RtlImageNtHeaderEx(IN ULONG Flags, > - IN PVOID Base, > - IN ULONG64 Size, > - OUT PIMAGE_NT_HEADERS *OutHeaders) > +RtlImageNtHeaderEx( > + _In_ ULONG Flags, > + _In_ PVOID Base, > + _In_ ULONG64 Size, > + _Out_ PIMAGE_NT_HEADERS *OutHeaders) > { > PIMAGE_NT_HEADERS NtHeaders; > PIMAGE_DOS_HEADER DosHeader; > BOOLEAN WantsRangeCheck; > + ULONG NtHeaderOffset; > > /* You must want NT Headers, no? */ > - if (!OutHeaders) return STATUS_INVALID_PARAMETER; > + if (OutHeaders == NULL) > + { > + DPRINT1("OutHeaders is NULL\n"); > + return STATUS_INVALID_PARAMETER; > + } > > /* Assume failure */ > *OutHeaders = NULL; > > /* Validate Flags */ > - if (Flags &~ RTL_IMAGE_NT_HEADER_EX_FLAG_NO_RANGE_CHECK) > - { > - DPRINT1("Invalid flag combination... check for new API flags?\n"); > + if (Flags & ~RTL_IMAGE_NT_HEADER_EX_FLAG_NO_RANGE_CHECK) > + { > + DPRINT1("Invalid flags: 0x%lx\n", Flags); > return STATUS_INVALID_PARAMETER; > } > > /* Validate base */ > - if (!(Base) || (Base == (PVOID)-1)) return STATUS_INVALID_PARAMETER; > - > - /* Check if the caller wants validation */ > + if ((Base == NULL) || (Base == (PVOID)-1)) > + { > + DPRINT1("Invalid base address: %p\n", Base); > + return STATUS_INVALID_PARAMETER; > + } > + > + /* Check if the caller wants range checks */ > WantsRangeCheck = !(Flags & > RTL_IMAGE_NT_HEADER_EX_FLAG_NO_RANGE_CHECK); > if (WantsRangeCheck) > { > @@ -179,56 +189,56 @@ > if (DosHeader->e_magic != IMAGE_DOS_SIGNATURE) > { > /* Not a valid COFF */ > - DPRINT1("Not an MZ file\n"); > + DPRINT1("Invalid image DOS signature!\n"); > + return STATUS_INVALID_IMAGE_FORMAT; > + } > + > + /* Get the offset to the NT headers (and copy from LONG to ULONG) */ > + NtHeaderOffset = DosHeader->e_lfanew; > + > + /* The offset must not be larger than 256MB, as a hard-coded check. > + In Windows this check is only done in user mode, not in kernel > mode, > + but it shouldn't harm to have it anyway. Note that without this > check, > + other overflow checks would become necessary! */ > + if (NtHeaderOffset >= (256 * 1024 * 1024)) > + { > + /* Fail */ > + DPRINT1("NT headers offset is larger than 256MB!\n"); > return STATUS_INVALID_IMAGE_FORMAT; > } > > /* Check if the caller wants validation */ > if (WantsRangeCheck) > { > - /* The offset should fit in the passsed-in size */ > - if (DosHeader->e_lfanew >= Size) > + /* Make sure the file header fits into the size */ > + if ((NtHeaderOffset + > + RTL_SIZEOF_THROUGH_FIELD(IMAGE_NT_HEADERS, FileHeader)) >= > Size) > { > /* Fail */ > - DPRINT1("e_lfanew is larger than PE file\n"); > + DPRINT1("NT headers beyond image size!\n"); > return STATUS_INVALID_IMAGE_FORMAT; > } > - > - /* It shouldn't be past 4GB either */ > - if (DosHeader->e_lfanew >= > - (MAXULONG - sizeof(IMAGE_DOS_SIGNATURE) - > sizeof(IMAGE_FILE_HEADER))) > - { > - /* Fail */ > - DPRINT1("e_lfanew is larger than 4GB\n"); > + } > + > + /* Now get a pointer to the NT Headers */ > + NtHeaders = (PIMAGE_NT_HEADERS)((ULONG_PTR)Base + NtHeaderOffset); > + > + /* Check if the mapping is in user space */ > + if (Base <= MmHighestUserAddress) > + { > + /* Make sure we don't overflow into kernel space */ > + if ((PVOID)(NtHeaders + 1) > MmHighestUserAddress) > + { > + DPRINT1("Image overflows from user space into kernel > space!\n"); > return STATUS_INVALID_IMAGE_FORMAT; > } > - > - /* And the whole file shouldn't overflow past 4GB */ > - if ((DosHeader->e_lfanew + > - sizeof(IMAGE_DOS_SIGNATURE) - sizeof(IMAGE_FILE_HEADER)) >= > Size) > - { > - /* Fail */ > - DPRINT1("PE is larger than 4GB\n"); > - return STATUS_INVALID_IMAGE_FORMAT; > - } > - } > - > - /* The offset also can't be larger than 256MB, as a hard-coded check > */ > - if (DosHeader->e_lfanew >= (256 * 1024 * 1024)) > - { > - /* Fail */ > - DPRINT1("PE offset is larger than 256MB\n"); > - return STATUS_INVALID_IMAGE_FORMAT; > - } > - > - /* Now it's safe to get the NT Headers */ > - NtHeaders = (PIMAGE_NT_HEADERS)((ULONG_PTR)Base + > DosHeader->e_lfanew); > + } > > /* Verify the PE Signature */ > if (NtHeaders->Signature != IMAGE_NT_SIGNATURE) > { > /* Fail */ > - DPRINT1("PE signature missing\n"); > + DPRINT1("Invalid image NT signature!\n"); > return STATUS_INVALID_IMAGE_FORMAT; > } > > @@ -245,6 +255,10 @@ > RtlImageNtHeader(IN PVOID Base) > { > PIMAGE_NT_HEADERS NtHeader; > + > + ULONG c = 1; > + ULONG s = FIELD_OFFSET(IMAGE_OPTIONAL_HEADER32, DataDirectory[c]); > + (void)s; > > /* Call the new API */ > RtlImageNtHeaderEx(RTL_IMAGE_NT_HEADER_EX_FLAG_NO_RANGE_CHECK, > > Modified: trunk/reactos/lib/rtl/rtlp.h > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/rtlp.h?rev=67140&r… > > ============================================================================== > --- trunk/reactos/lib/rtl/rtlp.h [iso-8859-1] (original) > +++ trunk/reactos/lib/rtl/rtlp.h [iso-8859-1] Fri Apr 10 19:21:22 > 2015 > @@ -34,6 +34,8 @@ > > #define RVA(m, b) ((PVOID)((ULONG_PTR)(b) + (ULONG_PTR)(m))) > > +extern PVOID MmHighestUserAddress; > + > NTSTATUS > NTAPI > RtlpSafeCopyMemory( > > Modified: trunk/reactos/ntoskrnl/include/internal/amd64/mm.h > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/include/internal/… > > ============================================================================== > --- trunk/reactos/ntoskrnl/include/internal/amd64/mm.h [iso-8859-1] > (original) > +++ trunk/reactos/ntoskrnl/include/internal/amd64/mm.h [iso-8859-1] Fri > Apr 10 19:21:22 2015 > @@ -5,7 +5,6 @@ > > /* Memory layout base addresses */ > #define MI_LOWEST_VAD_ADDRESS (PVOID)0x000000007FF00000ULL > -#define MI_HIGHEST_USER_ADDRESS (PVOID)0x000007FFFFFEFFFFULL > #define MI_USER_PROBE_ADDRESS (PVOID)0x000007FFFFFF0000ULL > #define MI_DEFAULT_SYSTEM_RANGE_START (PVOID)0xFFFF080000000000ULL > #define MI_REAL_SYSTEM_RANGE_START 0xFFFF800000000000ULL > > Modified: trunk/reactos/ntoskrnl/mm/ARM3/miarm.h > URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/miarm.h?r… > > ============================================================================== > --- trunk/reactos/ntoskrnl/mm/ARM3/miarm.h [iso-8859-1] (original) > +++ trunk/reactos/ntoskrnl/mm/ARM3/miarm.h [iso-8859-1] Fri Apr 10 > 19:21:22 2015 > @@ -29,7 +29,6 @@ > > #define MI_SYSTEM_VIEW_SIZE (32 * _1MB) > > -#define MI_HIGHEST_USER_ADDRESS (PVOID)0x7FFEFFFF > #define MI_USER_PROBE_ADDRESS (PVOID)0x7FFF0000 > #define MI_DEFAULT_SYSTEM_RANGE_START (PVOID)0x80000000 > #define MI_SYSTEM_CACHE_WS_START (PVOID)0xC0C00000 > > >

1 0

Re: [ros-dev] [ros-diffs] [spetreolle] 67053: [FREELDR] In a quest to better registry, don't break VSSolution builds. freeldr_pe is not in the same directory and copy doesn't care if you ask to concatenate C:\tomatoes, it a...
by Timo Kreuzer 07 Apr '15

07 Apr '15

The comment "# Retrieve the full path to the generated file of the 'freeldr_pe' target" Doesn't explain why it is done. It rather looks like someone is doing something pointless and documenting in a comment that he's doing the pointless. ${CMAKE_CURRENT_BINARY_DIR} should be the "full path to the generated file". So it seems completely pointless to add extra magic to get that file path. Your commit message implies that this change fixes VSSolution builds, but it also doesn't explain it. In fact I don't even understand that sentence. If you don't add a proper comment to that thing, why it is needed, I promise I will have forgotten about this in a year and remove it again. :) Thanks, Timo Am 04.04.2015 um 22:33 schrieb spetreolle(a)svn.reactos.org: > Author: spetreolle > Date: Sat Apr 4 20:33:18 2015 > New Revision: 67053 > > URL: http://svn.reactos.org/svn/reactos?rev=67053&view=rev > Log: > [FREELDR] > In a quest to better registry, > don't break VSSolution builds. > freeldr_pe is not in the same directory and copy doesn't care if you ask to concatenate C:\tomatoes, it already has the first file. > > Modified: > trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt > > Modified: trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/CMake… > ============================================================================== > --- trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt [iso-8859-1] (original) > +++ trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt [iso-8859-1] Sat Apr 4 20:33:18 2015 > @@ -226,10 +226,13 @@ > add_dependencies(freeldr_pe asm) > add_dependencies(freeldr_pe_dbg asm) > > +# Retrieve the full path to the generated file of the 'freeldr_pe' target > +get_target_property(_freeldr_pe_output_file freeldr_pe LOCATION) > + > concatenate_files( > ${CMAKE_CURRENT_BINARY_DIR}/freeldr.sys > ${CMAKE_CURRENT_BINARY_DIR}/frldr16.bin > - ${CMAKE_CURRENT_BINARY_DIR}/freeldr_pe.dll) > + ${_freeldr_pe_output_file}) > > add_custom_target(freeldr ALL DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/freeldr.sys) > > @@ -240,7 +243,8 @@ > concatenate_files( > ${CMAKE_CURRENT_BINARY_DIR}/setupldr.sys > ${CMAKE_CURRENT_BINARY_DIR}/frldr16.bin > - ${CMAKE_CURRENT_BINARY_DIR}/freeldr_pe.dll) > + ${_freeldr_pe_output_file}) > > add_custom_target(setupldr ALL DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/setupldr.sys) > add_cd_file(TARGET setupldr FILE ${CMAKE_CURRENT_BINARY_DIR}/setupldr.sys DESTINATION loader NO_CAB FOR bootcd regtest) > + > > >

3 2

old.reactos.org reaching end-of-life
by Pierre Schweitzer 03 Apr '15

03 Apr '15

Dear all, On the 3rd of April 2015, the old.reactos.org website will be taken offline after several years of good services. If you have anything to do on it (like gathering data or whatever) please make sure you do before that date. With my best regards, -- Pierre Schweitzer <pierre(a)reactos.org> System & Network Administrator Senior Kernel Developer ReactOS Deutschland e.V.

1 1

Re: [ros-dev] [ros-diffs] [tkreuzer] 65893: [GDI32] Import metafile code from wine. The wine code is unmodified (please keep it that way) and used through a wrapper function dispatcher and some helpers. Fixes 1931 wine tests.
by Thomas Faber 31 Mar '15

31 Mar '15

Hey Timo, I was just about to sync enhmetafile.c with Wine and found that your original import had the following diff: --- win32ss/gdi/gdi32/wine/enhmetafile.c (revision 67001) +++ win32ss/gdi/gdi32/wine/enhmetafile.c (working copy) @@ -546,8 +546,6 @@ if (!SetWorldTransform(hdc, &final_trans)) { - __debugbreak(); - SetWorldTransform(hdc, &final_trans); ERR("World transform failed!\n"); } } @@ -1232,12 +1230,10 @@ case EMR_EXTSELECTCLIPRGN: { const EMREXTSELECTCLIPRGN *lpRgn = (const EMREXTSELECTCLIPRGN *)mr; - const RGNDATA *pRgnData = (const RGNDATA *)lpRgn->RgnData; - DWORD dwSize = sizeof(RGNDATAHEADER) + pRgnData->rdh.nCount * sizeof(RECT); HRGN hRgn = 0; if (mr->nSize >= sizeof(*lpRgn) + sizeof(RGNDATAHEADER)) - hRgn = ExtCreateRegion( &info->init_transform, dwSize, pRgnData ); + hRgn = ExtCreateRegion( &info->init_transform, 0, (const RGNDATA *)lpRgn->RgnData ); ExtSelectClipRgn(hdc, hRgn, (INT)(lpRgn->iMode)); /* ExtSelectClipRgn created a copy of the region */ (this is the reverse diff, i.e. the one that would get it back in sync with Wine) My guess is I should just remove this difference and get it completely synced (i.e. apply the above patch)? What do you think? On 2014-12-29 15:25, tkreuzer(a)svn.reactos.org wrote: > Author: tkreuzer > Date: Mon Dec 29 14:25:30 2014 > New Revision: 65893 > > URL: http://svn.reactos.org/svn/reactos?rev=65893&view=rev > Log: > [GDI32] > Import metafile code from wine. The wine code is unmodified (please keep it that way) and used through a wrapper function dispatcher and some helpers. Fixes 1931 wine tests. > > Added: > trunk/reactos/win32ss/gdi/gdi32/include/wine/ (with props) > trunk/reactos/win32ss/gdi/gdi32/include/wine/gdi_driver.h (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/ (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/CMakeLists.txt (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/enhmetafile.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/ (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/bitblt.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/dc.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/enhmetafiledrv.h (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/graphics.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/init.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/objects.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/gdi_private.h (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/metafile.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/ (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/bitblt.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/dc.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/graphics.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/init.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/metafiledrv.h (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/objects.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/text.c (with props) > trunk/reactos/win32ss/gdi/gdi32/wine/rosglue.c (with props) > Modified: > trunk/reactos/win32ss/gdi/gdi32/CMakeLists.txt > trunk/reactos/win32ss/gdi/gdi32/include/gdi32p.h > trunk/reactos/win32ss/gdi/gdi32/misc/misc.c > trunk/reactos/win32ss/gdi/gdi32/objects/arc.c > trunk/reactos/win32ss/gdi/gdi32/objects/bitmap.c > trunk/reactos/win32ss/gdi/gdi32/objects/coord.c > trunk/reactos/win32ss/gdi/gdi32/objects/dc.c > trunk/reactos/win32ss/gdi/gdi32/objects/enhmfile.c > trunk/reactos/win32ss/gdi/gdi32/objects/gdiobj.c > trunk/reactos/win32ss/gdi/gdi32/objects/metafile.c > trunk/reactos/win32ss/gdi/gdi32/objects/painting.c > trunk/reactos/win32ss/gdi/gdi32/objects/palette.c > trunk/reactos/win32ss/gdi/gdi32/objects/path.c > trunk/reactos/win32ss/gdi/gdi32/objects/region.c > trunk/reactos/win32ss/gdi/gdi32/objects/text.c > trunk/reactos/win32ss/include/ntgdihdl.h > > [This mail would be too long, it was shortened to contain the URLs only.] > > Modified: trunk/reactos/win32ss/gdi/gdi32/CMakeLists.txt > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/CMakeLis… > > Modified: trunk/reactos/win32ss/gdi/gdi32/include/gdi32p.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/include/… > > Added: trunk/reactos/win32ss/gdi/gdi32/include/wine/gdi_driver.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/include/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/misc/misc.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/misc/mis… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/arc.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/bitmap.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/coord.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/dc.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/enhmfile.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/gdiobj.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/metafile.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/painting.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/palette.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/path.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/region.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Modified: trunk/reactos/win32ss/gdi/gdi32/objects/text.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/objects/… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/CMakeLists.txt > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/CMa… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/enhmetafile.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/enh… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/bitblt.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/enh… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/dc.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/enh… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/enhmetafiledrv.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/enh… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/graphics.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/enh… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/init.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/enh… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/enhmfdrv/objects.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/enh… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/gdi_private.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/gdi… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/metafile.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/met… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/bitblt.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/mfd… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/dc.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/mfd… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/graphics.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/mfd… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/init.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/mfd… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/metafiledrv.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/mfd… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/objects.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/mfd… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/mfdrv/text.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/mfd… > > Added: trunk/reactos/win32ss/gdi/gdi32/wine/rosglue.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/gdi32/wine/ros… > > Modified: trunk/reactos/win32ss/include/ntgdihdl.h > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/include/ntgdihdl.h…

3 3

Re: [ros-dev] [ros-diffs] [gadamopoulos] 66988: [EXPLORER] - Fix showing the website when Help is selected from the start menu.
by Pierre Schweitzer 31 Mar '15

31 Mar '15

Just a side (minor) remark. On 31/03/2015 00:28, gadamopoulos(a)svn.reactos.org wrote: > Modified: trunk/reactos/base/shell/explorer/explorer.rc > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/shell/explorer/explor… > ============================================================================== > --- trunk/reactos/base/shell/explorer/explorer.rc [iso-8859-1] (original) > +++ trunk/reactos/base/shell/explorer/explorer.rc [iso-8859-1] Mon Mar 30 22:28:08 2015 > @@ -28,7 +28,7 @@ > STRINGTABLE > BEGIN > //IDS_HELP_COMMAND "helpctr.exe>-FromStartHelp" > - IDS_HELP_COMMAND "https://www.reactos.org/" > + IDS_HELP_COMMAND "http://www.reactos.org/" This is to be reverted once ReactOS fully supports SSL. -- Pierre Schweitzer <pierre at reactos.org> System & Network Administrator Senior Kernel Developer ReactOS Deutschland e.V.

1 0

Re: [ros-dev] [ros-diffs] [gadamopoulos] 66992: [BROWSEUI] - CProgressDialog: Allocate big enough buffers needed for the text messages during initialization and don't reallocate them at all. Should stop thrashing the heap w...
by Thomas Faber 31 Mar '15

31 Mar '15

On 2015-03-31 00:50, gadamopoulos(a)svn.reactos.org wrote: > --- trunk/reactos/dll/win32/browseui/CProgressDialog.cpp [iso-8859-1] (original) > +++ trunk/reactos/dll/win32/browseui/CProgressDialog.cpp [iso-8859-1] Mon Mar 30 22:50:34 2015 > CProgressDialog::CProgressDialog() > { > + this->cancelMsg[0] = this->title[0]; Did you mean this->cancelMsg[0] = this->title[0] = 0;? (or UNICODE_NULL for that matter) > static void set_buffer(LPWSTR *buffer, LPCWSTR string) > { > + > + ULONG len = max((wcslen(string) + 1)*sizeof(WCHAR), BUFFER_SIZE); > StringCchCopyW(*buffer, len, string); This makes no sense. I think you simply want StringCchCopyW(*buffer, len, BUFFER_SIZE); Thanks! -Thomas

1 1

Re: [ros-dev] [ros-diffs] [hbelusca] 66946: [CMAKE] Improve the concatenate_files macro introduced by Timo in r52179 + r52182 and hackfixed in r53914 . Notice that now it can take an arbitrary number of files to be concaten...
by Thomas Faber 29 Mar '15

29 Mar '15

I don't see how creating a new host-tool is an improvement over using OS builtins? On 2015-03-29 03:53, hbelusca(a)svn.reactos.org wrote: > Author: hbelusca > Date: Sun Mar 29 01:53:10 2015 > New Revision: 66946 > > URL: http://svn.reactos.org/svn/reactos?rev=66946&view=rev > Log: > [CMAKE] > Improve the concatenate_files macro introduced by Timo in r52179 + r52182 and hackfixed in r53914 . Notice that now it can take an arbitrary number of files to be concatenated. Hence you need to specify first the output file, and all the rest is the different files. > > [FREELDR] > Use the improved concatenate_files macro. > > Modified: > trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt > trunk/reactos/cmake/CMakeMacros.cmake > > Modified: trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/CMake… > ============================================================================== > --- trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt [iso-8859-1] (original) > +++ trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt [iso-8859-1] Sun Mar 29 01:53:10 2015 > @@ -226,10 +226,13 @@ > add_dependencies(freeldr_pe asm) > add_dependencies(freeldr_pe_dbg asm) > > +# Retrieve the full path to the generated file of the 'freeldr_pe' target > +get_target_property(_freeldr_pe_output_file freeldr_pe LOCATION) > + > concatenate_files( > + ${CMAKE_CURRENT_BINARY_DIR}/freeldr.sys > ${CMAKE_CURRENT_BINARY_DIR}/frldr16.bin > - freeldr_pe > - ${CMAKE_CURRENT_BINARY_DIR}/freeldr.sys) > + ${_freeldr_pe_output_file}) > > add_custom_target(freeldr ALL DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/freeldr.sys) > > @@ -238,9 +241,9 @@ > add_cd_file(TARGET freeldr FILE ${CMAKE_CURRENT_BINARY_DIR}/freeldr.sys DESTINATION loader NO_CAB NOT_IN_HYBRIDCD FOR livecd hybridcd NAME_ON_CD setupldr.sys) > > concatenate_files( > + ${CMAKE_CURRENT_BINARY_DIR}/setupldr.sys > ${CMAKE_CURRENT_BINARY_DIR}/frldr16.bin > - freeldr_pe > - ${CMAKE_CURRENT_BINARY_DIR}/setupldr.sys) > + ${_freeldr_pe_output_file}) > > add_custom_target(setupldr ALL DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/setupldr.sys) > add_cd_file(TARGET setupldr FILE ${CMAKE_CURRENT_BINARY_DIR}/setupldr.sys DESTINATION loader NO_CAB FOR bootcd regtest) > > Modified: trunk/reactos/cmake/CMakeMacros.cmake > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/cmake/CMakeMacros.cmake?re… > ============================================================================== > --- trunk/reactos/cmake/CMakeMacros.cmake [iso-8859-1] (original) > +++ trunk/reactos/cmake/CMakeMacros.cmake [iso-8859-1] Sun Mar 29 01:53:10 2015 > @@ -504,33 +504,12 @@ > endfunction() > endif() > > -if(CMAKE_HOST_SYSTEM_NAME STREQUAL "Windows") > - macro(to_win_path _cmake_path _native_path) > - string(REPLACE "/" "\\" ${_native_path} "${_cmake_path}") > - endmacro() > - > - # yeah the parameter mess sucks, but thats what works... > - function(concatenate_files _file1 _target2 _output) > - get_target_property(_file2 ${_target2} LOCATION) > - to_win_path("${_file1}" _real_file1) > - to_win_path("${_file2}" _real_file2) > - to_win_path("${_output}" _real_output) > - add_custom_command( > - OUTPUT ${_output} > - COMMAND cmd.exe /C "copy /Y /B ${_real_file1} + ${_real_file2} ${_real_output} > nul" > - DEPENDS ${_file1} > - DEPENDS ${_target2}) > - endfunction() > -else() > - macro(concatenate_files _file1 _target2 _output) > - get_target_property(_file2 ${_target2} LOCATION) > - add_custom_command( > - OUTPUT ${_output} > - COMMAND cat ${_file1} ${_file2} > ${_output} > - DEPENDS ${_file1} > - DEPENDS ${_target2}) > - endmacro() > -endif() > +macro(concatenate_files _output) > + add_custom_command( > + OUTPUT ${_output} > + COMMAND native-cat ${ARGN} > ${_output} > + DEPENDS ${ARGN} native-cat) > +endmacro() > > function(add_importlibs _module) > add_dependency_node(${_module}) > >

6 7

Re: [ros-dev] [ros-diffs] [hbelusca] 66903: [NTVDM] - Fix some english. - Validity checks for DosGetSftEntry returned pointer added. - Use unsigned indices for for-loops indices that are always positive.
by Pierre Schweitzer 26 Mar '15

26 Mar '15

On 03/26/2015 03:52 PM, hbelusca(a)svn.reactos.org wrote: > Modified: trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/bios.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/mvdm/ntvdm/dos/… > ============================================================================== > --- trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/bios.c [iso-8859-1] (original) > +++ trunk/reactos/subsystems/mvdm/ntvdm/dos/dos32krnl/bios.c [iso-8859-1] Thu Mar 26 14:52:16 2015 > @@ -56,6 +56,13 @@ > BOOLEAN DosCheckInput(VOID) > { > PDOS_SFT_ENTRY SftEntry = DosGetSftEntry(DOS_INPUT_HANDLE); > + > + if (SftEntry == NULL) > + { > + /* Invalid handle */ > + DosLastError = ERROR_INVALID_HANDLE; // ERROR_FILE_NOT_FOUND Hum... Why? Is the error code wrong and should be ERROR_FILE_NOT_FOUND? And then, why setting this other error code? Or is the error code right, and then, what's the purpose of such comment? Or do you have a doubt about the right error code to use? Be it one or the other? I come back to my eternal comment: don't comment for yourself. Comment for the others, you're not alone here. So, make it explicit. Cheers, -- Pierre Schweitzer <pierre(a)reactos.org> System & Network Administrator Senior Kernel Developer ReactOS Deutschland e.V.

3 2

Win2k3 end of life
by Huw Campbell 26 Mar '15

26 Mar '15

Hey, Win2k3 is end of support in 6 months, which will mean that the symbol server will go down (as it has done for XP). At the very least, we should scrape the symbol server so we can continue to check and debug. Or move to NT6...

4 3

Status Meeting (March 2015)
by Aleksey Bragin 26 Mar '15

26 Mar '15

Hello, Let me invite you to the monthly status meeting taking place today 26th of March, 19:00 UTC, as always. 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. And join Mumble! Regards, Aleksey Bragin

1 0

Re: [ros-dev] [ros-diffs] [tkreuzer] 66893: [WIN32K] Rewrite brush code in C++
by Ged Murphy 26 Mar '15

26 Mar '15

Timo and C++?? Hell must be a bit chilly today :) -----Original Message----- From: Ros-diffs [mailto:ros-diffs-bounces@reactos.org] On Behalf Of tkreuzer(a)svn.reactos.org Sent: 25 March 2015 22:38 To: ros-diffs(a)reactos.org Subject: [ros-diffs] [tkreuzer] 66893: [WIN32K] Rewrite brush code in C++ Author: tkreuzer Date: Wed Mar 25 22:38:20 2015 New Revision: 66893 URL: http://svn.reactos.org/svn/reactos?rev=66893&view=rev Log: [WIN32K] Rewrite brush code in C++

1 0

msi:msi Wine Test hanging under Windows Server 2003
by Colin Finck 22 Mar '15

22 Mar '15

Hi all, With my latest log flush change to rosautotest, you see that it's msi:msi which is hanging in our Windows Server 2003 WHS-Testbot. See https://build.reactos.org/builders/WHS-Testbot/builds/30/steps/test/logs/st… The actual hang looks like this: http://fs2.directupload.net/images/150322/tsr72fu5.png This is the only bug that prevents us from running the full Wine Test Suite under Windows Server 2003. Fixes are welcome :) Cheers, Colin

1 0

Jira & Fisheye downtime
by Pierre Schweitzer 19 Mar '15

19 Mar '15

Dear all, Due to DB maintenance, Jira & Fisheye will be down in a few minutes. There's no ETA for up status. Sorry for the caused inconvenience, -- Pierre Schweitzer <pierre(a)reactos.org> System & Network Administrator Senior Kernel Developer ReactOS Deutschland e.V.

1 1

Re: [ros-dev] Ros-dev Digest, Vol 127, Issue 29
by Fedor Zaitsev 15 Mar '15

15 Mar '15

Thank you, with revision 66721 everything is ok with building ReactOS > Hi Fedor, > >> Hi. I am trying to compile ReactOS on my macbook with OSX 10.10 on it, but I got an error. Building tools compiled quite well (with some warmings, but I think that it is ok), but when I call "ninja all? I got error: > Can you please try with r66700 and report back? > > Cheers, > Amine.

1 0

Re: [ros-dev] [ros-diffs] [dreimer] 66690: [RAPPS] lack of a proxy configuration by Peter Hater. German translation updated by me. CORE-4852 #resolve #comment Committed, thx for help.
by Colin Finck 15 Mar '15

15 Mar '15

dreimer(a)svn.reactos.org wrote: > [RAPPS] lack of a proxy configuration by Peter Hater. I don't think we need a separate proxy configuration per application. We should rather keep things consistent and always use the system proxy set in the "Internet Options" control panel applet. Unless someone has any objections, I suggest reverting this. - Colin

4 4

Error with compiling ReactOS on OSX 10.10
by Fedor Zaitsev 14 Mar '15

14 Mar '15

Hi. I am trying to compile ReactOS on my macbook with OSX 10.10 on it, but I got an error. Building tools compiled quite well (with some warmings, but I think that it is ok), but when I call "ninja all» I got error: bash-3.2$ cd ../../output-MinGW-i386/reactos bash-3.2$ ninja [544/8381] Building C object dll/openg...Files/mesa_main.dir/api_arrayelt.c.obj FAILED: /usr/local/RosBE/i386/bin/i686-w64-mingw32-gcc -DBUILD_GL32 -DFEATURE_GL=1 -DKDBG=1 -DOPENGL32_USE_TLS -DUSE_3DNOW_ASM -DUSE_COMPILER_EXCEPTIONS -DUSE_MMX_ASM -DUSE_SSE_ASM -DUSE_X86_ASM -DWIN32 -DWINVER=0x502 -D_DLL -D_GDI32_ -D_GLAPI_NO_EXPORTS -D_M_IX86 -D_SEH_ENABLE_TRACE -D_SETUPAPI_VER=0x502 -D_USE_32BIT_TIME_T -D_USE_PSEH3=1 -D_WIN32_IE=0x600 -D_WIN32_WINDOWS=0x502 -D_WIN32_WINNT=0x502 -D_WINDOWS -D_X86_ -D__REACTOS__ -D__i386__ -D_inline=__inline -Di386 -Wa,--compress-debug-sections -pipe -fms-extensions -fno-strict-aliasing -mstackrealign -Wold-style-declaration -Wdeclaration-after-statement -fdebug-prefix-map="/Users/lobster/documents/RosBE/reactos"=ReactOS -gdwarf-2 -gstrict-dwarf -femit-struct-debug-detailed=none -feliminate-unused-debug-symbols -march=pentium -mtune=i686 -Werror -Wall -Wpointer-arith -Wno-char-subscripts -Wno-multichar -Wno-unused-value -Wno-maybe-uninitialized -Wno-error=unused-but-set-variable -Wno-error=type-limits -O1 -fno-optimize-sibling-calls -fno-omit-frame-pointer -mpreferred-stack-boundary=3 -fno-set-stack-executable -Winvalid-pch -Werror=invalid-pch -g -Idll/opengl/mesa/main -I../../dll/opengl/mesa/main -I../../include -I../../include/psdk -I../../include/dxsdk -Iinclude -Iinclude/psdk -Iinclude/dxsdk -Iinclude/reactos -I../../include/crt -I../../include/ddk -I../../include/ndk -I../../include/reactos -I../../include/reactos/libs -I../../dll/opengl/mesa/. -Wno-error -Wno-type-limits -include /Users/lobster/documents/RosBE/reactos/output-MinGW-i386/reactos/dll/opengl/mesa/main/mesa_main_pch.h -MMD -MT dll/opengl/mesa/main/CMakeFiles/mesa_main.dir/api_arrayelt.c.obj -MF dll/opengl/mesa/main/CMakeFiles/mesa_main.dir/api_arrayelt.c.obj.d -o dll/opengl/mesa/main/CMakeFiles/mesa_main.dir/api_arrayelt.c.obj -c ../../dll/opengl/mesa/main/api_arrayelt.c cc1: warning: /Users/lobster/documents/RosBE/reactos/output-MinGW-i386/reactos/dll/opengl/mesa/main/mesa_main_pch.h.gch: had text segment at different address [enabled by default] cc1: error: one or more PCH files were found, but they were invalid cc1: fatal error: /Users/lobster/documents/RosBE/reactos/output-MinGW-i386/reactos/dll/opengl/mesa/main/mesa_main_pch.h: No such file or directory compilation terminated. … … and so on What I am doing wrong? I have googled, but I don’t found anything relevant Thanks

3 4

Re: [ros-dev] [ros-diffs] [aandrejevic] 66666: [NTVDM][FAST486] - Implement VDDInstallMemoryHook and VDDDeInstallMemoryHook using page guards. - Implement another API for memory hooks that should be faster than page guards ...
by Aleksey Bragin 14 Mar '15

14 Mar '15

Allright, so NTVDM is evil. Pretty good choice, as I wouldn't want to taint kernel or other important part of the system by 66666 commit made on Friday 13th... ;-) Regards, Aleksey Bragin On 13.03.2015 20:57, aandrejevic(a)svn.reactos.org wrote: > Author: aandrejevic > Date: Fri Mar 13 17:57:51 2015 > New Revision: 66666 > > URL: http://svn.reactos.org/svn/reactos?rev=66666&view=rev > Log: > [NTVDM][FAST486] > - Implement VDDInstallMemoryHook and VDDDeInstallMemoryHook using page guards. > - Implement another API for memory hooks that should be faster than page guards > (for NTVDM only). > - Adjust the VGA and EMS memory handlers to use this method. > - In Fast486, implement a method that will allow us to "rewind" the current instruction, > in case it was interrupted by a memory hook page fault. > - Use a memory hook to protect the BIOS ROM from being written to. >

4 4

Re: [ros-dev] [ros-diffs] [dreimer] 66690: [RAPPS] lack of a proxy configuration by Peter Hater. German translation updated by me. CORE-4852 #resolve #comment Committed, thx for help.
by Colin Finck 14 Mar '15

14 Mar '15

dreimer(a)svn.reactos.org wrote: > [RAPPS] lack of a proxy configuration by Peter Hater. I don't think we need a separate proxy configuration per application. We should rather keep things consistent and always use the system proxy set in the "Internet Options" control panel applet. Unless someone has any objections, I suggest reverting this. - Colin

1 0

ReactOS versioning
by Timo Kreuzer 11 Mar '15

11 Mar '15

Hi, Here's the promised suggestion regarding how we handle versioning problems in reactos. It has some relationship to the tree restructure. Since some time we now run into issues with our targeted Windows version. This is both wine dlls, as well as applications that refuse to run due to reactos being limited to Windows server 2003 SP2. I think many of us, me included, see more in ReactOS than an academic research project, or a nice way for 3rd party companies to cheaply get insight into how the Windows kernel works. So we are interested in making it an actually useful operating system. To achieve this goal, it is obviously important to make it run modern Windows applications. The current approach of pure Windows 2003 Server SP2 compatibility on user mode side is a dead end. Our target OS version is starting to become a fossil. With time more and more applications will simply refuse to work on it. Even wine DLLs start to require Vista APIs and their number will most likely increase. So what can we do? It is obvious, that we cannot instantaniously switch all user mode to Windows 7/8/10 compatibility, due to the amount of required work, especially regarding missing kernel features. The wine approach is just adding whatever is needed, creating a Windows version chimera. It has already been discussed here and shown to be a problem, since it can easily fool applications into believing they run on Vista or Windows 7, making them demand all the modern features, which we cannot provide, thus failing to run, while they would run flawlessly, when being provided a pure Windows 2003 environment, restricting itself to this functionality. So this is also not a very good way, either. So the conclusion is, that we need a mechanism, that allows us to control this, providing individual applications with what they require, while leaving others in a more restricted environment. And at the same time allowing our internal/wine DLLs to make use of higher version functionality. Suggested approach: 1. We need a method to specify which application should be run in which environment. We should probably use the same mechanism that is used on Windows. Compatibility information is stored in a registry key HKCU\Software\Microsodt\Windows NT\CurrentVersion\AppCompatFlags\... The trick is to make this easy / transparent for the user. A right-click -> properties -> compatibility approach should for now probably be the easiest thing, even if it requires the user to actively make this setting. A larger app compatibility database would be nice, but it would be difficult to figure out what application is being run. And it's also a problem to maintain such a list. Potential solutions: detect failures to load due to missing imports and app crashes and invoke a "compatibility assistent" in that case. Detect first-run of a new application and try to identify it, either based on a hash or based on PE version information. 2. We need a way to provide the application transparently with the environment we want to give it. In terms of DLL exports this could be done on the loader side, making it chose the right DLL, potentially adding a suffix to the DLL name or selecting a different folder other than system32. While this will most likely work good in the majority of cases, it is not 100% transparent. Therefore a mechanism in the kernel, using file system redirection, like it already exists on 64 bit Windows for WoW64, seems to be a more promising approach. The file system redirection would redirect system32 into merged folders, containing the version specific DLLs, while everything that is not existing in this folder will be taken from the original system32. Potential naming scheme: system32.601 system32.602, etc. 3. We need a method to create and maintain the required DLLs for different OS versions. Preferably avoiding bloat by sharing common code in common "parent" DLLs. But also allowing to still plug the DLLs into the related Windows version for testing. This can be tricky. I suggest a DLL import forwarding scheme. This is both to avoid bloat, i.e. avoid to compile and deploy all full blown DLLs for all OS versions, as well as creating a better organized system. So each DLL, lets say ADVAPI32, as located in different version specific system32 folders, would mainly/only consist of forwarders to a "parent" DLL. On Windows we can see this being developed similarly, using "api sets" and redirections made by the loader. Cloning this mechanism 1:1 might not be the right thing though, since it does not address all our requirements. So instead I suggest proving our own custom "parent DLLs". While these could be organized the same way as on Windows 2003, this is probably not optimal. Instead I suggest merging stuff together into 1 or few DLLs (similar to how stuff was combined in kernelbase.dll) This might looks like this (note, that the names are just quickly made up names, I don't claim that they are good) - user32/gdi32 -> ros-win32-core.dll - kernel32/advapi32 -> ros-kernel-base.dll - msvcr* -> ros-crt.dll - ntdll -> ros-ntdll (the kernel would need to load this one) This also allows our DLLs to make use of higher version APIs, by linking them to the parent DLL. Now this obviously introduces a problem when trying to run individual DLLs within a Windows system. To still be able to do this for testing purposes, we need to make sure that they still run there. First, if they import from a custom ros-* DLL, it won't run on Windows without that DLL. So we need the possibility to either statically link these functions, or compile a "helper DLL", that contains these functions, so our DLLs can be run on Windows. If we used a common parent DLL, this also creates the problem of DLL initialization. e.g. the DLL parent for kernel32 and advapi32 would do the initialization for both of these, so this would not be suitable to use when replacing only one of the DLLs. Instead DLL initialization could be done by calling a specific initialization function in the parent DLL from the child DLL. So kernel32!DllMain would call ros-kernel-base!DllMain_kernel32, passing the original parameters as well as a version number, that the parent DLL can use to do version specific initialization. This way The parent DLL would only do the kernel32 initialization, when the related kernel32 child DLL was used, the advapi32 initialization would not be done. There is still a problem: relocation. So we would need to make sure we chose base addresses that still allow us to plug the stuff into Windows without causing everything to relocate, which often simply doesn't work. As an alternative, we should provide a compile time switch to compile specific DLLs in a self-contained way. In terms of structure we could use the MS api-sets as a base for static libraries. Then we can link these either into the parent DLLs like ros-kernel-base.dll or - when a compile switch is given - to link together fully self contained DLLs. I am really not interested in answers like "This is not what WIndows does!", "THIS CANNOT WORK!!!", "You are a ***** even suggesting this", "What if application x parses the import table and disassembles the DLL to hook into internal functions, ...": I am only interested in *constructive* comments. Everything else: -> /dev/null Thanks, Timo

14 25

Re: [ros-dev] [ros-diffs] [akhaldi] 66660: [ADVAPI32_WINETEST] Sync with Wine Staging 1.7.37. CORE-9246
by Thomas Faber 10 Mar '15

10 Mar '15

On 2015-03-10 11:10, akhaldi(a)svn.reactos.org wrote: > +if(NOT MSVC) > + add_target_compile_flags(advapi32_winetest "-Wno-format") > +endif() Why are we getting format warnings? Is __WINESRC__ not set correctly?

2 2

Re: [ros-dev] [ros-diffs] [akhaldi] 66658: [NTDLL_WINETEST] Sync with Wine Staging 1.7.37. CORE-9246
by Jerome Gardou 10 Mar '15

10 Mar '15

Hello Amine, Although I agreed to sync our stuff with wine staging, I am not sure it is a good idea to use their tests. My understanding is that patches present in the wine staging tree implement missing features but did not yet pass the wine quality requirement. Although it might be worth it for modules, I'm not sure this is a good idea to include tests which were not proven as good enough. Simply put: I don't want to chase unexisting bugs just because we're using a flaky test suite. Regards Jérôme Le 10/03/2015 11:10, akhaldi(a)svn.reactos.org a écrit : > Author: akhaldi > Date: Tue Mar 10 10:10:04 2015 > New Revision: 66658 > > URL: http://svn.reactos.org/svn/reactos?rev=66658&view=rev > Log: > [NTDLL_WINETEST] Sync with Wine Staging 1.7.37. CORE-9246 > > Modified: > trunk/rostests/winetests/ntdll/directory.c > trunk/rostests/winetests/ntdll/exception.c > trunk/rostests/winetests/ntdll/file.c > trunk/rostests/winetests/ntdll/info.c > trunk/rostests/winetests/ntdll/om.c > trunk/rostests/winetests/ntdll/reg.c > trunk/rostests/winetests/ntdll/rtl.c > trunk/rostests/winetests/ntdll/string.c > > Modified: trunk/rostests/winetests/ntdll/directory.c > URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/ntdll/directory… > ============================================================================== > --- trunk/rostests/winetests/ntdll/directory.c [iso-8859-1] (original) > +++ trunk/rostests/winetests/ntdll/directory.c [iso-8859-1] Tue Mar 10 10:10:04 2015 > @@ -51,7 +51,6 @@ > > /* The attribute sets to test */ > static struct testfile_s { > - BOOL todo; /* set if it doesn't work on wine yet */ > BOOL attr_done; /* set if attributes were tested for this file already */ > const DWORD attr; /* desired attribute */ > const char *name; /* filename to use */ > @@ -60,13 +59,13 @@ > int nfound; /* How many were found (expect 1) */ > WCHAR nameW[20]; /* unicode version of name (filled in later) */ > } testfiles[] = { > - { 0, 0, FILE_ATTRIBUTE_NORMAL, "n.tmp", NULL, "normal" }, > - { 1, 0, FILE_ATTRIBUTE_HIDDEN, "h.tmp", NULL, "hidden" }, > - { 1, 0, FILE_ATTRIBUTE_SYSTEM, "s.tmp", NULL, "system" }, > - { 0, 0, FILE_ATTRIBUTE_DIRECTORY, "d.tmp", NULL, "directory" }, > - { 0, 0, FILE_ATTRIBUTE_DIRECTORY, ".", NULL, ". directory" }, > - { 0, 0, FILE_ATTRIBUTE_DIRECTORY, "..", NULL, ".. directory" }, > - { 0, 0, 0, NULL } > + { 0, FILE_ATTRIBUTE_NORMAL, "n.tmp", NULL, "normal" }, > + { 0, FILE_ATTRIBUTE_HIDDEN, "h.tmp", NULL, "hidden" }, > + { 0, FILE_ATTRIBUTE_SYSTEM, "s.tmp", NULL, "system" }, > + { 0, FILE_ATTRIBUTE_DIRECTORY, "d.tmp", NULL, "directory" }, > + { 0, FILE_ATTRIBUTE_DIRECTORY, ".", NULL, ". directory" }, > + { 0, FILE_ATTRIBUTE_DIRECTORY, "..", NULL, ".. directory" }, > + { 0, 0, NULL } > }; > static const int max_test_dir_size = 20; /* size of above plus some for .. etc */ > > @@ -147,12 +146,7 @@ > if (namelen != len || memcmp(nameW, testfiles[i].nameW, len*sizeof(WCHAR))) > continue; > if (!testfiles[i].attr_done) { > - if (testfiles[i].todo) { > - todo_wine > - ok (attrib == (testfiles[i].attr & attribmask), "file %s: expected %s (%x), got %x (is your linux new enough?)\n", testfiles[i].name, testfiles[i].description, testfiles[i].attr, attrib); > - } else { > - ok (attrib == (testfiles[i].attr & attribmask), "file %s: expected %s (%x), got %x (is your linux new enough?)\n", testfiles[i].name, testfiles[i].description, testfiles[i].attr, attrib); > - } > + ok (attrib == (testfiles[i].attr & attribmask), "file %s: expected %s (%x), got %x (is your linux new enough?)\n", testfiles[i].name, testfiles[i].description, testfiles[i].attr, attrib); > testfiles[i].attr_done = TRUE; > } > testfiles[i].nfound++; > > Modified: trunk/rostests/winetests/ntdll/exception.c > URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/ntdll/exception… > ============================================================================== > --- trunk/rostests/winetests/ntdll/exception.c [iso-8859-1] (original) > +++ trunk/rostests/winetests/ntdll/exception.c [iso-8859-1] Tue Mar 10 10:10:04 2015 > @@ -46,11 +46,14 @@ > static PVOID (WINAPI *pRtlUnwind)(PVOID, PVOID, PEXCEPTION_RECORD, PVOID); > static PVOID (WINAPI *pRtlAddVectoredExceptionHandler)(ULONG first, PVECTORED_EXCEPTION_HANDLER func); > static ULONG (WINAPI *pRtlRemoveVectoredExceptionHandler)(PVOID handler); > +static PVOID (WINAPI *pRtlAddVectoredContinueHandler)(ULONG first, PVECTORED_EXCEPTION_HANDLER func); > +static ULONG (WINAPI *pRtlRemoveVectoredContinueHandler)(PVOID handler); > static NTSTATUS (WINAPI *pNtReadVirtualMemory)(HANDLE, const void*, void*, SIZE_T, SIZE_T*); > static NTSTATUS (WINAPI *pNtTerminateProcess)(HANDLE handle, LONG exit_code); > static NTSTATUS (WINAPI *pNtQueryInformationProcess)(HANDLE, PROCESSINFOCLASS, PVOID, ULONG, PULONG); > static NTSTATUS (WINAPI *pNtSetInformationProcess)(HANDLE, PROCESSINFOCLASS, PVOID, ULONG); > static BOOL (WINAPI *pIsWow64Process)(HANDLE, PBOOL); > +static NTSTATUS (WINAPI *pNtClose)(HANDLE); > > #if defined(__x86_64__) > static BOOLEAN (CDECL *pRtlAddFunctionTable)(RUNTIME_FUNCTION*, DWORD, DWORD64); > @@ -936,6 +939,16 @@ > /* here we handle exception */ > } > } > + else if (stage == 7 || stage == 8) > + { > + ok(de.u.Exception.ExceptionRecord.ExceptionCode == EXCEPTION_INVALID_HANDLE, > + "unexpected exception code %08x, expected %08x\n", de.u.Exception.ExceptionRecord.ExceptionCode, > + EXCEPTION_INVALID_HANDLE); > + ok(de.u.Exception.ExceptionRecord.NumberParameters == 0, > + "unexpected number of parameters %d, expected 0\n", de.u.Exception.ExceptionRecord.NumberParameters); > + > + if (stage == 8) continuestatus = DBG_EXCEPTION_NOT_HANDLED; > + } > else > ok(FALSE, "unexpected stage %x\n", stage); > > @@ -1695,6 +1708,7 @@ > > #endif /* __x86_64__ */ > > +#if defined(__i386__) || defined(__x86_64__) > static DWORD outputdebugstring_exceptions; > > static LONG CALLBACK outputdebugstring_vectored_handler(EXCEPTION_POINTERS *ExceptionInfo) > @@ -1713,7 +1727,7 @@ > return EXCEPTION_CONTINUE_SEARCH; > } > > -static void test_outputdebugstring(DWORD numexc, BOOL todo) > +static void test_outputdebugstring(DWORD numexc) > { > PVOID vectored_handler; > > @@ -1728,13 +1742,8 @@ > > outputdebugstring_exceptions = 0; > OutputDebugStringA("Hello World"); > - if (todo) > - todo_wine > - ok(outputdebugstring_exceptions == numexc, "OutputDebugStringA generated %d exceptions, expected %d\n", > - outputdebugstring_exceptions, numexc); > - else > - ok(outputdebugstring_exceptions == numexc, "OutputDebugStringA generated %d exceptions, expected %d\n", > - outputdebugstring_exceptions, numexc); > + ok(outputdebugstring_exceptions == numexc, "OutputDebugStringA generated %d exceptions, expected %d\n", > + outputdebugstring_exceptions, numexc); > > pRtlRemoveVectoredExceptionHandler(vectored_handler); > } > @@ -1788,6 +1797,91 @@ > pRtlRemoveVectoredExceptionHandler(vectored_handler); > } > > +static DWORD invalid_handle_exceptions; > + > +static LONG CALLBACK invalid_handle_vectored_handler(EXCEPTION_POINTERS *ExceptionInfo) > +{ > + PEXCEPTION_RECORD rec = ExceptionInfo->ExceptionRecord; > + trace("vect. handler %08x addr:%p\n", rec->ExceptionCode, rec->ExceptionAddress); > + > + ok(rec->ExceptionCode == EXCEPTION_INVALID_HANDLE, "ExceptionCode is %08x instead of %08x\n", > + rec->ExceptionCode, EXCEPTION_INVALID_HANDLE); > + ok(rec->NumberParameters == 0, "ExceptionParameters is %d instead of 0\n", rec->NumberParameters); > + > + invalid_handle_exceptions++; > + return (rec->ExceptionCode == EXCEPTION_INVALID_HANDLE) ? EXCEPTION_CONTINUE_EXECUTION : EXCEPTION_CONTINUE_SEARCH; > +} > + > +static void test_closehandle(DWORD numexc) > +{ > + PVOID vectored_handler; > + NTSTATUS status; > + DWORD res; > + > + if (!pRtlAddVectoredExceptionHandler || !pRtlRemoveVectoredExceptionHandler || !pRtlRaiseException) > + { > + skip("RtlAddVectoredExceptionHandler or RtlRemoveVectoredExceptionHandler or RtlRaiseException not found\n"); > + return; > + } > + > + vectored_handler = pRtlAddVectoredExceptionHandler(TRUE, &invalid_handle_vectored_handler); > + ok(vectored_handler != 0, "RtlAddVectoredExceptionHandler failed\n"); > + > + invalid_handle_exceptions = 0; > + res = CloseHandle((HANDLE)0xdeadbeef); > + ok(!res, "CloseHandle(0xdeadbeef) unexpectedly succeeded\n"); > + ok(GetLastError() == ERROR_INVALID_HANDLE, "wrong error code %d instead of %d", > + GetLastError(), ERROR_INVALID_HANDLE); > + ok(invalid_handle_exceptions == numexc, "CloseHandle generated %d exceptions, expected %d\n", > + invalid_handle_exceptions, numexc); > + > + invalid_handle_exceptions = 0; > + status = pNtClose((HANDLE)0xdeadbeef); > + ok(status == STATUS_INVALID_HANDLE, "NtClose(0xdeadbeef) returned status %08x\n", status); > + ok(invalid_handle_exceptions == numexc, "NtClose generated %d exceptions, expected %d\n", > + invalid_handle_exceptions, numexc); > + > + pRtlRemoveVectoredExceptionHandler(vectored_handler); > +} > + > +static void test_vectored_continue_handler(void) > +{ > + PVOID handler1, handler2; > + ULONG ret; > + > + if (!pRtlAddVectoredContinueHandler || !pRtlRemoveVectoredContinueHandler) > + { > + skip("RtlAddVectoredContinueHandler or RtlRemoveVectoredContinueHandler not found\n"); > + return; > + } > + > + handler1 = pRtlAddVectoredContinueHandler(TRUE, (void *)0xdeadbeef); > + ok(handler1 != 0, "RtlAddVectoredContinueHandler failed\n"); > + > + handler2 = pRtlAddVectoredContinueHandler(TRUE, (void *)0xdeadbeef); > + ok(handler2 != 0, "RtlAddVectoredContinueHandler failed\n"); > + ok(handler1 != handler2, "RtlAddVectoredContinueHandler returned same handler\n"); > + > + if (pRtlRemoveVectoredExceptionHandler) > + { > + ret = pRtlRemoveVectoredExceptionHandler(handler1); > + ok(!ret, "RtlRemoveVectoredExceptionHandler succeeded\n"); > + } > + > + ret = pRtlRemoveVectoredContinueHandler(handler1); > + ok(ret, "RtlRemoveVectoredContinueHandler failed\n"); > + > + ret = pRtlRemoveVectoredContinueHandler(handler2); > + ok(ret, "RtlRemoveVectoredContinueHandler failed\n"); > + > + ret = pRtlRemoveVectoredContinueHandler(handler1); > + ok(!ret, "RtlRemoveVectoredContinueHandler succeeded\n"); > + > + ret = pRtlRemoveVectoredContinueHandler((void *)0x11223344); > + ok(!ret, "RtlRemoveVectoredContinueHandler succeeded\n"); > +} > +#endif /* defined(__i386__) || defined(__x86_64__) */ > + > START_TEST(exception) > { > HMODULE hntdll = GetModuleHandleA("ntdll.dll"); > @@ -1802,6 +1896,7 @@ > pNtGetContextThread = (void *)GetProcAddress( hntdll, "NtGetContextThread" ); > pNtSetContextThread = (void *)GetProcAddress( hntdll, "NtSetContextThread" ); > pNtReadVirtualMemory = (void *)GetProcAddress( hntdll, "NtReadVirtualMemory" ); > + pNtClose = (void *)GetProcAddress( hntdll, "NtClose" ); > pRtlUnwind = (void *)GetProcAddress( hntdll, "RtlUnwind" ); > pRtlRaiseException = (void *)GetProcAddress( hntdll, "RtlRaiseException" ); > pNtTerminateProcess = (void *)GetProcAddress( hntdll, "NtTerminateProcess" ); > @@ -1809,6 +1904,10 @@ > "RtlAddVectoredExceptionHandler" ); > pRtlRemoveVectoredExceptionHandler = (void *)GetProcAddress( hntdll, > "RtlRemoveVectoredExceptionHandler" ); > + pRtlAddVectoredContinueHandler = (void *)GetProcAddress( hntdll, > + "RtlAddVectoredContinueHandler" ); > + pRtlRemoveVectoredContinueHandler = (void *)GetProcAddress( hntdll, > + "RtlRemoveVectoredContinueHandler" ); > pNtQueryInformationProcess = (void*)GetProcAddress( hntdll, > "NtQueryInformationProcess" ); > pNtSetInformationProcess = (void*)GetProcAddress( hntdll, > @@ -1858,13 +1957,17 @@ > run_rtlraiseexception_test(EXCEPTION_BREAKPOINT); > run_rtlraiseexception_test(EXCEPTION_INVALID_HANDLE); > test_stage = 3; > - test_outputdebugstring(0, FALSE); > + test_outputdebugstring(0); > test_stage = 4; > - test_outputdebugstring(2, TRUE); /* is this a Windows bug? */ > + test_outputdebugstring(2); > test_stage = 5; > test_ripevent(0); > test_stage = 6; > test_ripevent(1); > + test_stage = 7; > + test_closehandle(0); > + test_stage = 8; > + test_closehandle(1); > } > else > skip( "RtlRaiseException not found\n" ); > @@ -1876,8 +1979,10 @@ > test_unwind(); > test_exceptions(); > test_rtlraiseexception(); > - test_outputdebugstring(1, FALSE); > + test_outputdebugstring(1); > test_ripevent(1); > + test_closehandle(0); > + test_vectored_continue_handler(); > test_debugger(); > test_simd_exceptions(); > test_fpu_exceptions(); > @@ -1894,8 +1999,10 @@ > pRtlLookupFunctionEntry = (void *)GetProcAddress( hntdll, > "RtlLookupFunctionEntry" ); > > - test_outputdebugstring(1, FALSE); > + test_outputdebugstring(1); > test_ripevent(1); > + test_closehandle(0); > + test_vectored_continue_handler(); > test_virtual_unwind(); > > if (pRtlAddFunctionTable && pRtlDeleteFunctionTable && pRtlInstallFunctionTableCallback && pRtlLookupFunctionEntry) > > Modified: trunk/rostests/winetests/ntdll/file.c > URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/ntdll/file.c?re… > ============================================================================== > --- trunk/rostests/winetests/ntdll/file.c [iso-8859-1] (original) > +++ trunk/rostests/winetests/ntdll/file.c [iso-8859-1] Tue Mar 10 10:10:04 2015 > @@ -37,6 +37,33 @@ > #include "winternl.h" > #include "winuser.h" > #include "wine/winioctl.h" > + > +/* FIXME */ > +typedef struct _REPARSE_DATA_BUFFER { > + ULONG ReparseTag; > + USHORT ReparseDataLength; > + USHORT Reserved; > + _ANONYMOUS_UNION union { > + struct { > + USHORT SubstituteNameOffset; > + USHORT SubstituteNameLength; > + USHORT PrintNameOffset; > + USHORT PrintNameLength; > + ULONG Flags; > + WCHAR PathBuffer[1]; > + } SymbolicLinkReparseBuffer; > + struct { > + USHORT SubstituteNameOffset; > + USHORT SubstituteNameLength; > + USHORT PrintNameOffset; > + USHORT PrintNameLength; > + WCHAR PathBuffer[1]; > + } MountPointReparseBuffer; > + struct { > + UCHAR DataBuffer[1]; > + } GenericReparseBuffer; > + } DUMMYUNIONNAME; > +} REPARSE_DATA_BUFFER, *PREPARSE_DATA_BUFFER; > > #ifndef IO_COMPLETION_ALL_ACCESS > #define IO_COMPLETION_ALL_ACCESS 0x001F0003 > @@ -1234,7 +1261,7 @@ > memset(&fbi, 0, sizeof(fbi)); > res = pNtQueryInformationFile(h, &io, &fbi, sizeof fbi, FileBasicInformation); > ok ( res == STATUS_SUCCESS, "can't get attributes\n"); > - todo_wine ok ( (fbi.FileAttributes & attrib_mask) == FILE_ATTRIBUTE_SYSTEM, "attribute %x not FILE_ATTRIBUTE_SYSTEM\n", fbi.FileAttributes ); > + ok ( (fbi.FileAttributes & attrib_mask) == FILE_ATTRIBUTE_SYSTEM, "attribute %x not FILE_ATTRIBUTE_SYSTEM (ok in old linux without xattr)\n", fbi.FileAttributes ); > > /* Then HIDDEN */ > memset(&fbi, 0, sizeof(fbi)); > @@ -1247,7 +1274,7 @@ > memset(&fbi, 0, sizeof(fbi)); > res = pNtQueryInformationFile(h, &io, &fbi, sizeof fbi, FileBasicInformation); > ok ( res == STATUS_SUCCESS, "can't get attributes\n"); > - todo_wine ok ( (fbi.FileAttributes & attrib_mask) == FILE_ATTRIBUTE_HIDDEN, "attribute %x not FILE_ATTRIBUTE_HIDDEN\n", fbi.FileAttributes ); > + ok ( (fbi.FileAttributes & attrib_mask) == FILE_ATTRIBUTE_HIDDEN, "attribute %x not FILE_ATTRIBUTE_HIDDEN (ok in old linux without xattr)\n", fbi.FileAttributes ); > > /* Check NORMAL last of all (to make sure we can clear attributes) */ > memset(&fbi, 0, sizeof(fbi)); > @@ -1304,7 +1331,7 @@ > memset(&fai_buf.fai, 0, sizeof(fai_buf.fai)); > res = pNtQueryInformationFile(h, &io, &fai_buf.fai, sizeof fai_buf, FileAllInformation); > ok ( res == STATUS_SUCCESS, "can't get attributes, res %x\n", res); > - todo_wine ok ( (fai_buf.fai.BasicInformation.FileAttributes & attrib_mask) == FILE_ATTRIBUTE_SYSTEM, "attribute %x not FILE_ATTRIBUTE_SYSTEM\n", fai_buf.fai.BasicInformation.FileAttributes ); > + ok ( (fai_buf.fai.BasicInformation.FileAttributes & attrib_mask) == FILE_ATTRIBUTE_SYSTEM, "attribute %x not FILE_ATTRIBUTE_SYSTEM (ok in old linux without xattr)\n", fai_buf.fai.BasicInformation.FileAttributes ); > > /* Then HIDDEN */ > memset(&fai_buf.fai.BasicInformation, 0, sizeof(fai_buf.fai.BasicInformation)); > @@ -1317,7 +1344,7 @@ > memset(&fai_buf.fai, 0, sizeof(fai_buf.fai)); > res = pNtQueryInformationFile(h, &io, &fai_buf.fai, sizeof fai_buf, FileAllInformation); > ok ( res == STATUS_SUCCESS, "can't get attributes\n"); > - todo_wine ok ( (fai_buf.fai.BasicInformation.FileAttributes & attrib_mask) == FILE_ATTRIBUTE_HIDDEN, "attribute %x not FILE_ATTRIBUTE_HIDDEN\n", fai_buf.fai.BasicInformation.FileAttributes ); > + ok ( (fai_buf.fai.BasicInformation.FileAttributes & attrib_mask) == FILE_ATTRIBUTE_HIDDEN, "attribute %x not FILE_ATTRIBUTE_HIDDEN (ok in old linux without xattr)\n", fai_buf.fai.BasicInformation.FileAttributes ); > > /* Check NORMAL last of all (to make sure we can clear attributes) */ > memset(&fai_buf.fai.BasicInformation, 0, sizeof(fai_buf.fai.BasicInformation)); > @@ -1371,7 +1398,6 @@ > ok( res == STATUS_INVALID_INFO_CLASS || res == STATUS_NOT_IMPLEMENTED, "Unexpected NtQueryInformationFile result (expected STATUS_INVALID_INFO_CLASS, got %x)\n", res ); > fdi.DoDeleteFile = TRUE; > res = pNtSetInformationFile( handle, &io, &fdi, sizeof fdi, FileDispositionInformation ); > - todo_wine > ok( res == STATUS_ACCESS_DENIED, "unexpected FileDispositionInformation result (expected STATUS_ACCESS_DENIED, got %x)\n", res ); > CloseHandle( handle ); > fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > @@ -1384,12 +1410,24 @@ > ok( handle != INVALID_HANDLE_VALUE, "failed to create temp file\n" ); > fdi.DoDeleteFile = TRUE; > res = pNtSetInformationFile( handle, &io, &fdi, sizeof fdi, FileDispositionInformation ); > - todo_wine > ok( res == STATUS_SUCCESS, "unexpected FileDispositionInformation result (expected STATUS_SUCCESS, got %x)\n", res ); > CloseHandle( handle ); > fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > - todo_wine > ok( fileDeleted, "File should have been deleted\n" ); > + DeleteFileA( buffer ); > + > + /* cannot set disposition on readonly file */ > + GetTempFileNameA( tmp_path, "dis", 0, buffer ); > + DeleteFileA( buffer ); > + handle = CreateFileA(buffer, GENERIC_WRITE | DELETE, 0, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_READONLY, 0); > + ok( handle != INVALID_HANDLE_VALUE, "failed to create temp file\n" ); > + fdi.DoDeleteFile = TRUE; > + res = pNtSetInformationFile( handle, &io, &fdi, sizeof fdi, FileDispositionInformation ); > + ok( res == STATUS_CANNOT_DELETE, "unexpected FileDispositionInformation result (expected STATUS_CANNOT_DELETE, got %x)\n", res ); > + CloseHandle( handle ); > + fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > + ok( !fileDeleted, "File shouldn't have been deleted\n" ); > + SetFileAttributesA( buffer, FILE_ATTRIBUTE_NORMAL ); > DeleteFileA( buffer ); > > /* cannot set disposition on readonly file */ > @@ -1402,6 +1440,21 @@ > ok( res == STATUS_CANNOT_DELETE, "unexpected FileDispositionInformation result (expected STATUS_CANNOT_DELETE, got %x)\n", res ); > CloseHandle( handle ); > fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > + todo_wine > + ok( !fileDeleted, "File shouldn't have been deleted\n" ); > + SetFileAttributesA( buffer, FILE_ATTRIBUTE_NORMAL ); > + DeleteFileA( buffer ); > + > + /* cannot set disposition on readonly file */ > + GetTempFileNameA( tmp_path, "dis", 0, buffer ); > + DeleteFileA( buffer ); > + handle = CreateFileA(buffer, GENERIC_WRITE | DELETE, 0, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_READONLY, 0); > + ok( handle != INVALID_HANDLE_VALUE, "failed to create temp file\n" ); > + fdi.DoDeleteFile = TRUE; > + res = pNtSetInformationFile( handle, &io, &fdi, sizeof fdi, FileDispositionInformation ); > + ok( res == STATUS_CANNOT_DELETE, "unexpected FileDispositionInformation result (expected STATUS_CANNOT_DELETE, got %x)\n", res ); > + CloseHandle( handle ); > + fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > ok( !fileDeleted, "File shouldn't have been deleted\n" ); > SetFileAttributesA( buffer, FILE_ATTRIBUTE_NORMAL ); > DeleteFileA( buffer ); > @@ -1412,11 +1465,9 @@ > ok( handle != INVALID_HANDLE_VALUE, "failed to create temp file\n" ); > fdi.DoDeleteFile = TRUE; > res = pNtSetInformationFile( handle, &io, &fdi, sizeof fdi, FileDispositionInformation ); > - todo_wine > ok( res == STATUS_SUCCESS, "unexpected FileDispositionInformation result (expected STATUS_SUCCESS, got %x)\n", res ); > fdi.DoDeleteFile = FALSE; > res = pNtSetInformationFile( handle, &io, &fdi, sizeof fdi, FileDispositionInformation ); > - todo_wine > ok( res == STATUS_SUCCESS, "unexpected FileDispositionInformation result (expected STATUS_SUCCESS, got %x)\n", res ); > CloseHandle( handle ); > fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > @@ -1429,7 +1480,6 @@ > ok( handle != INVALID_HANDLE_VALUE, "failed to create temp file\n" ); > fdi.DoDeleteFile = FALSE; > res = pNtSetInformationFile( handle, &io, &fdi, sizeof fdi, FileDispositionInformation ); > - todo_wine > ok( res == STATUS_SUCCESS, "unexpected FileDispositionInformation result (expected STATUS_SUCCESS, got %x)\n", res ); > CloseHandle( handle ); > fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > @@ -1444,7 +1494,6 @@ > CloseHandle( handle ); > fdi.DoDeleteFile = FALSE; > res = pNtSetInformationFile( handle2, &io, &fdi, sizeof fdi, FileDispositionInformation ); > - todo_wine > ok( res == STATUS_SUCCESS, "unexpected FileDispositionInformation result (expected STATUS_SUCCESS, got %x)\n", res ); > CloseHandle( handle2 ); > fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > @@ -1459,11 +1508,9 @@ > ok( handle != INVALID_HANDLE_VALUE, "failed to open a directory\n" ); > fdi.DoDeleteFile = TRUE; > res = pNtSetInformationFile( handle, &io, &fdi, sizeof fdi, FileDispositionInformation ); > - todo_wine > ok( res == STATUS_SUCCESS, "unexpected FileDispositionInformation result (expected STATUS_SUCCESS, got %x)\n", res ); > CloseHandle( handle ); > fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > - todo_wine > ok( fileDeleted, "Directory should have been deleted\n" ); > RemoveDirectoryA( buffer ); > > @@ -1476,7 +1523,6 @@ > RemoveDirectoryA( buffer ); > fdi.DoDeleteFile = FALSE; > res = pNtSetInformationFile( handle, &io, &fdi, sizeof fdi, FileDispositionInformation ); > - todo_wine > ok( res == STATUS_SUCCESS, "unexpected FileDispositionInformation result (expected STATUS_SUCCESS, got %x)\n", res ); > CloseHandle( handle ); > fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > @@ -1501,6 +1547,7 @@ > buffer[dirpos] = '\0'; > CloseHandle( handle ); > fileDeleted = GetFileAttributesA( buffer ) == INVALID_FILE_ATTRIBUTES && GetLastError() == ERROR_FILE_NOT_FOUND; > + todo_wine > ok( !fileDeleted, "Directory shouldn't have been deleted\n" ); > RemoveDirectoryA( buffer ); > } > @@ -2683,6 +2730,178 @@ > CloseHandle(hfile); > } > > +static INT build_reparse_buffer(WCHAR *filename, REPARSE_DATA_BUFFER **pbuffer) > +{ > + REPARSE_DATA_BUFFER *buffer; > + INT buffer_len, string_len; > + WCHAR *dest; > + > + string_len = (lstrlenW(filename)+1)*sizeof(WCHAR); > + buffer_len = FIELD_OFFSET(REPARSE_DATA_BUFFER, MountPointReparseBuffer.PathBuffer[1]) + string_len; > + buffer = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, buffer_len); > + buffer->ReparseTag = IO_REPARSE_TAG_MOUNT_POINT; > + buffer->ReparseDataLength = sizeof(buffer->MountPointReparseBuffer) + string_len; > + buffer->MountPointReparseBuffer.SubstituteNameLength = string_len - sizeof(WCHAR); > + buffer->MountPointReparseBuffer.PrintNameOffset = string_len; > + dest = &buffer->MountPointReparseBuffer.PathBuffer[0]; > + memcpy(dest, filename, string_len); > + *pbuffer = buffer; > + return buffer_len; > +} > + > +static void test_junction_points(void) > +{ > + static const WCHAR junctionW[] = {'\\','j','u','n','c','t','i','o','n',0}; > + WCHAR path[MAX_PATH], junction_path[MAX_PATH], target_path[MAX_PATH]; > + static const WCHAR targetW[] = {'\\','t','a','r','g','e','t',0}; > + FILE_BASIC_INFORMATION old_attrib, new_attrib; > + static const WCHAR fooW[] = {'f','o','o',0}; > + static WCHAR volW[] = {'c',':','\\',0}; > + REPARSE_GUID_DATA_BUFFER guid_buffer; > + static const WCHAR dotW[] = {'.',0}; > + REPARSE_DATA_BUFFER *buffer = NULL; > + DWORD dwret, dwLen, dwFlags, err; > + INT buffer_len, string_len; > + IO_STATUS_BLOCK iosb; > + UNICODE_STRING nameW; > + HANDLE hJunction; > + WCHAR *dest; > + BOOL bret; > + > + /* Create a temporary folder for the junction point tests */ > + GetTempFileNameW(dotW, fooW, 0, path); > + DeleteFileW(path); > + if (!CreateDirectoryW(path, NULL)) > + { > + win_skip("Unable to create a temporary junction point directory.\n"); > + return; > + } > + > + /* Check that the volume this folder is located on supports junction points */ > + pRtlDosPathNameToNtPathName_U(path, &nameW, NULL, NULL); > + volW[0] = nameW.Buffer[4]; > + pRtlFreeUnicodeString( &nameW ); > + GetVolumeInformationW(volW, 0, 0, 0, &dwLen, &dwFlags, 0, 0); > + if (!(dwFlags & FILE_SUPPORTS_REPARSE_POINTS)) > + { > + skip("File system does not support junction points.\n"); > + RemoveDirectoryW(path); > + return; > + } > + > + /* Create the folder to be replaced by a junction point */ > + lstrcpyW(junction_path, path); > + lstrcatW(junction_path, junctionW); > + bret = CreateDirectoryW(junction_path, NULL); > + ok(bret, "Failed to create junction point directory.\n"); > + > + /* Create a destination folder for the junction point to target */ > + lstrcpyW(target_path, path); > + lstrcatW(target_path, targetW); > + bret = CreateDirectoryW(target_path, NULL); > + ok(bret, "Failed to create junction point target directory.\n"); > + pRtlDosPathNameToNtPathName_U(target_path, &nameW, NULL, NULL); > + > + /* Create the junction point */ > + hJunction = CreateFileW(junction_path, GENERIC_READ | GENERIC_WRITE, 0, 0, OPEN_EXISTING, > + FILE_FLAG_BACKUP_SEMANTICS | FILE_FLAG_OPEN_REPARSE_POINT, 0); > + if (hJunction == INVALID_HANDLE_VALUE) > + { > + win_skip("Failed to open junction point directory handle (0x%x).\n", GetLastError()); > + goto cleanup; > + } > + dwret = NtQueryInformationFile(hJunction, &iosb, &old_attrib, sizeof(old_attrib), FileBasicInformation); > + ok(dwret == STATUS_SUCCESS, "Failed to get junction point folder's attributes (0x%x).\n", dwret); > + buffer_len = build_reparse_buffer(nameW.Buffer, &buffer); > + bret = DeviceIoControl(hJunction, FSCTL_SET_REPARSE_POINT, (LPVOID)buffer, buffer_len, NULL, 0, &dwret, 0); > + ok(bret, "Failed to create junction point! (0x%x)\n", GetLastError()); > + > + /* Check the file attributes of the junction point */ > + dwret = GetFileAttributesW(junction_path); > + ok(dwret != (DWORD)~0, "Junction point doesn't exist (attributes: 0x%x)!\n", dwret); > + ok(dwret & FILE_ATTRIBUTE_REPARSE_POINT, "File is not a junction point! (attributes: %d)\n", dwret); > + > + /* Read back the junction point */ > + HeapFree(GetProcessHeap(), 0, buffer); > + buffer_len = sizeof(*buffer) + MAX_PATH*sizeof(WCHAR); > + buffer = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, buffer_len); > + bret = DeviceIoControl(hJunction, FSCTL_GET_REPARSE_POINT, NULL, 0, (LPVOID)buffer, buffer_len, &dwret, 0); > + string_len = buffer->MountPointReparseBuffer.SubstituteNameLength; > + dest = &buffer->MountPointReparseBuffer.PathBuffer[buffer->MountPointReparseBuffer.SubstituteNameOffset/sizeof(WCHAR)]; > + ok(bret, "Failed to read junction point!\n"); > + ok((memcmp(dest, nameW.Buffer, string_len) == 0), "Junction point destination does not match ('%s' != '%s')!\n", > + wine_dbgstr_w(dest), wine_dbgstr_w(nameW.Buffer)); > + > + /* Delete the junction point */ > + memset(&old_attrib, 0x00, sizeof(old_attrib)); > + old_attrib.LastAccessTime.QuadPart = 0x200deadcafebeef; > + dwret = NtSetInformationFile(hJunction, &iosb, &old_attrib, sizeof(old_attrib), FileBasicInformation); > + ok(dwret == STATUS_SUCCESS, "Failed to set junction point folder's attributes (0x%x).\n", dwret); > + memset(&guid_buffer, 0x00, sizeof(guid_buffer)); > + guid_buffer.ReparseTag = IO_REPARSE_TAG_MOUNT_POINT; > + bret = DeviceIoControl(hJunction, FSCTL_DELETE_REPARSE_POINT, (LPVOID)&guid_buffer, > + REPARSE_GUID_DATA_BUFFER_HEADER_SIZE, NULL, 0, &dwret, 0); > + ok(bret, "Failed to delete junction point! (0x%x)\n", GetLastError()); > + memset(&new_attrib, 0x00, sizeof(new_attrib)); > + dwret = NtQueryInformationFile(hJunction, &iosb, &new_attrib, sizeof(new_attrib), FileBasicInformation); > + ok(dwret == STATUS_SUCCESS, "Failed to get junction point folder's attributes (0x%x).\n", dwret); > + ok(old_attrib.LastAccessTime.QuadPart == new_attrib.LastAccessTime.QuadPart, > + "Junction point folder's access time does not match (0x%llx != 0x%llx).\n", > + new_attrib.LastAccessTime.QuadPart, old_attrib.LastAccessTime.QuadPart); > + CloseHandle(hJunction); > + > + /* Check deleting a junction point as if it were a directory */ > + HeapFree(GetProcessHeap(), 0, buffer); > + hJunction = CreateFileW(junction_path, GENERIC_READ | GENERIC_WRITE, 0, 0, OPEN_EXISTING, > + FILE_FLAG_BACKUP_SEMANTICS | FILE_FLAG_OPEN_REPARSE_POINT, 0); > + buffer_len = build_reparse_buffer(nameW.Buffer, &buffer); > + bret = DeviceIoControl(hJunction, FSCTL_SET_REPARSE_POINT, (LPVOID)buffer, buffer_len, NULL, 0, &dwret, 0); > + ok(bret, "Failed to create junction point! (0x%x)\n", GetLastError()); > + CloseHandle(hJunction); > + bret = RemoveDirectoryW(junction_path); > + ok(bret, "Failed to delete junction point as directory!\n"); > + dwret = GetFileAttributesW(junction_path); > + ok(dwret == (DWORD)~0, "Junction point still exists (attributes: 0x%x)!\n", dwret); > + > + /* Check deleting a junction point as if it were a file */ > + HeapFree(GetProcessHeap(), 0, buffer); > + bret = CreateDirectoryW(junction_path, NULL); > + ok(bret, "Failed to create junction point target directory.\n"); > + hJunction = CreateFileW(junction_path, GENERIC_READ | GENERIC_WRITE, 0, 0, OPEN_EXISTING, > + FILE_FLAG_BACKUP_SEMANTICS | FILE_FLAG_OPEN_REPARSE_POINT, 0); > + buffer_len = build_reparse_buffer(nameW.Buffer, &buffer); > + bret = DeviceIoControl(hJunction, FSCTL_SET_REPARSE_POINT, (LPVOID)buffer, buffer_len, NULL, 0, &dwret, 0); > + ok(bret, "Failed to create junction point! (0x%x)\n", GetLastError()); > + CloseHandle(hJunction); > + bret = DeleteFileW(junction_path); > + ok(!bret, "Succeeded in deleting junction point as file!\n"); > + err = GetLastError(); > + ok(err == ERROR_ACCESS_DENIED, "Expected last error 0x%x for DeleteFile on junction point (actually 0x%x)!\n", > + ERROR_ACCESS_DENIED, err); > + dwret = GetFileAttributesW(junction_path); > + ok(dwret != (DWORD)~0, "Junction point doesn't exist (attributes: 0x%x)!\n", dwret); > + ok(dwret & FILE_ATTRIBUTE_REPARSE_POINT, "File is not a junction point! (attributes: 0x%x)\n", dwret); > + > + /* Test deleting a junction point's target */ > + dwret = GetFileAttributesW(junction_path); > + ok(dwret == 0x410 || broken(dwret == 0x430) /* win2k */, > + "Unexpected junction point attributes (0x%x != 0x410)!\n", dwret); > + bret = RemoveDirectoryW(target_path); > + ok(bret, "Failed to delete junction point target!\n"); > + bret = CreateDirectoryW(target_path, NULL); > + ok(bret, "Failed to create junction point target directory.\n"); > + > +cleanup: > + /* Cleanup */ > + pRtlFreeUnicodeString( &nameW ); > + HeapFree(GetProcessHeap(), 0, buffer); > + bret = RemoveDirectoryW(junction_path); > + ok(bret, "Failed to remove temporary junction point directory!\n"); > + bret = RemoveDirectoryW(target_path); > + ok(bret, "Failed to remove temporary target directory!\n"); > + RemoveDirectoryW(path); > +} > + > START_TEST(file) > { > HMODULE hkernel32 = GetModuleHandleA("kernel32.dll"); > @@ -2736,4 +2955,5 @@ > test_file_disposition_information(); > test_query_volume_information_file(); > test_query_attribute_information_file(); > + test_junction_points(); > } > > Modified: trunk/rostests/winetests/ntdll/info.c > URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/ntdll/info.c?re… > ============================================================================== > --- trunk/rostests/winetests/ntdll/info.c [iso-8859-1] (original) > +++ trunk/rostests/winetests/ntdll/info.c [iso-8859-1] Tue Mar 10 10:10:04 2015 > @@ -737,6 +737,84 @@ > HeapFree(GetProcessHeap(), 0, ppi); > } > > +static void test_query_process_wow64(void) > +{ > + NTSTATUS status; > + ULONG ReturnLength; > + ULONG_PTR pbi[2], dummy; > + > + memset(&dummy, 0xcc, sizeof(dummy)); > + > + /* Do not give a handle and buffer */ > + status = pNtQueryInformationProcess(NULL, ProcessWow64Information, NULL, 0, NULL); > + ok( status == STATUS_INFO_LENGTH_MISMATCH, "Expected STATUS_INFO_LENGTH_MISMATCH, got %08x\n", status); > + > + /* Use a correct info class and buffer size, but still no handle and buffer */ > + status = pNtQueryInformationProcess(NULL, ProcessWow64Information, NULL, sizeof(ULONG_PTR), NULL); > + ok( status == STATUS_ACCESS_VIOLATION || status == STATUS_INVALID_HANDLE, > + "Expected STATUS_ACCESS_VIOLATION or STATUS_INVALID_HANDLE, got %08x\n", status); > + > + /* Use a correct info class, buffer size and handle, but no buffer */ > + status = pNtQueryInformationProcess(GetCurrentProcess(), ProcessWow64Information, NULL, sizeof(ULONG_PTR), NULL); > + ok( status == STATUS_ACCESS_VIOLATION , "Expected STATUS_ACCESS_VIOLATION, got %08x\n", status); > + > + /* Use a correct info class, buffer and buffer size, but no handle */ > + pbi[0] = pbi[1] = dummy; > + status = pNtQueryInformationProcess(NULL, ProcessWow64Information, pbi, sizeof(ULONG_PTR), NULL); > + ok( status == STATUS_INVALID_HANDLE, "Expected STATUS_INVALID_HANDLE, got %08x\n", status); > + ok( pbi[0] == dummy, "pbi[0] changed to %lx\n", pbi[0]); > + ok( pbi[1] == dummy, "pbi[1] changed to %lx\n", pbi[1]); > + > + /* Use a greater buffer size */ > + pbi[0] = pbi[1] = dummy; > + status = pNtQueryInformationProcess(NULL, ProcessWow64Information, pbi, sizeof(ULONG_PTR) + 1, NULL); > + ok( status == STATUS_INFO_LENGTH_MISMATCH, "Expected STATUS_INFO_LENGTH_MISMATCH, got %08x\n", status); > + ok( pbi[0] == dummy, "pbi[0] changed to %lx\n", pbi[0]); > + ok( pbi[1] == dummy, "pbi[1] changed to %lx\n", pbi[1]); > + > + /* Use no ReturnLength */ > + pbi[0] = pbi[1] = dummy; > + status = pNtQueryInformationProcess(GetCurrentProcess(), ProcessWow64Information, pbi, sizeof(ULONG_PTR), NULL); > + ok( status == STATUS_SUCCESS, "Expected STATUS_SUCCESS, got %08x\n", status); > + trace( "Platform is_wow64 %d, ProcessInformation of ProcessWow64Information %lx\n", is_wow64, pbi[0]); > + ok( is_wow64 == (pbi[0] != 0), "is_wow64 %x, pbi[0] %lx\n", is_wow64, pbi[0]); > + ok( pbi[0] != dummy, "pbi[0] %lx\n", pbi[0]); > + ok( pbi[1] == dummy, "pbi[1] changed to %lx\n", pbi[1]); > + /* Test written size on 64 bit by checking high 32 bit buffer */ > + if (sizeof(ULONG_PTR) > sizeof(DWORD)) > + { > + DWORD *ptr = (DWORD *)pbi; > + ok( ptr[1] != (DWORD)dummy, "ptr[1] unchanged!\n"); > + } > + > + /* Finally some correct calls */ > + pbi[0] = pbi[1] = dummy; > + ReturnLength = 0xdeadbeef; > + status = pNtQueryInformationProcess(GetCurrentProcess(), ProcessWow64Information, pbi, sizeof(ULONG_PTR), &ReturnLength); > + ok( status == STATUS_SUCCESS, "Expected STATUS_SUCCESS, got %08x\n", status); > + ok( is_wow64 == (pbi[0] != 0), "is_wow64 %x, pbi[0] %lx\n", is_wow64, pbi[0]); > + ok( pbi[1] == dummy, "pbi[1] changed to %lx\n", pbi[1]); > + ok( ReturnLength == sizeof(ULONG_PTR), "Inconsistent length %d\n", ReturnLength); > + > + /* Everything is correct except a too small buffer size */ > + pbi[0] = pbi[1] = dummy; > + ReturnLength = 0xdeadbeef; > + status = pNtQueryInformationProcess(GetCurrentProcess(), ProcessWow64Information, pbi, sizeof(ULONG_PTR) - 1, &ReturnLength); > + ok( status == STATUS_INFO_LENGTH_MISMATCH, "Expected STATUS_INFO_LENGTH_MISMATCH, got %08x\n", status); > + ok( pbi[0] == dummy, "pbi[0] changed to %lx\n", pbi[0]); > + ok( pbi[1] == dummy, "pbi[1] changed to %lx\n", pbi[1]); > + todo_wine ok( ReturnLength == 0xdeadbeef, "Expected 0xdeadbeef, got %d\n", ReturnLength); > + > + /* Everything is correct except a too large buffer size */ > + pbi[0] = pbi[1] = dummy; > + ReturnLength = 0xdeadbeef; > + status = pNtQueryInformationProcess(GetCurrentProcess(), ProcessWow64Information, pbi, sizeof(ULONG_PTR) + 1, &ReturnLength); > + ok( status == STATUS_INFO_LENGTH_MISMATCH, "Expected STATUS_INFO_LENGTH_MISMATCH, got %08x\n", status); > + ok( pbi[0] == dummy, "pbi[0] changed to %lx\n", pbi[0]); > + ok( pbi[1] == dummy, "pbi[1] changed to %lx\n", pbi[1]); > + todo_wine ok( ReturnLength == 0xdeadbeef, "Expected 0xdeadbeef, got %d\n", ReturnLength); > +} > + > static void test_query_process_basic(void) > { > NTSTATUS status; > @@ -1714,6 +1792,10 @@ > trace("Starting test_query_process_handlecount()\n"); > test_query_process_handlecount(); > > + /* 0x1A ProcessWow64Information */ > + trace("Starting test_query_process_wow64()\n"); > + test_query_process_wow64(); > + > /* 0x1B ProcessImageFileName */ > trace("Starting test_query_process_image_file_name()\n"); > test_query_process_image_file_name(); > > Modified: trunk/rostests/winetests/ntdll/om.c > URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/ntdll/om.c?rev=… > ============================================================================== > --- trunk/rostests/winetests/ntdll/om.c [iso-8859-1] (original) > +++ trunk/rostests/winetests/ntdll/om.c [iso-8859-1] Tue Mar 10 10:10:04 2015 > @@ -835,14 +835,14 @@ > status = pNtOpenEvent(&Event2, GENERIC_ALL, &attr); > ok( status == STATUS_SUCCESS, "NtOpenEvent failed %08x\n", status ); > > - status = pNtClose(Event); > + pNtClose(Event); > > status = pNtQueryEvent(Event2, EventBasicInformation, &info, sizeof(info), NULL); > ok( status == STATUS_SUCCESS, "NtQueryEvent failed %08x\n", status ); > ok( info.EventType == 1 && info.EventState == 0, > "NtQueryEvent failed, expected 1 0, got %d %d\n", info.EventType, info.EventState ); > > - status = pNtClose(Event2); > + pNtClose(Event2); > } > > static const WCHAR keyed_nameW[] = {'\\','B','a','s','e','N','a','m','e','d','O','b','j','e','c','t','s', > > Modified: trunk/rostests/winetests/ntdll/reg.c > URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/ntdll/reg.c?rev… > ============================================================================== > --- trunk/rostests/winetests/ntdll/reg.c [iso-8859-1] (original) > +++ trunk/rostests/winetests/ntdll/reg.c [iso-8859-1] Tue Mar 10 10:10:04 2015 > @@ -129,11 +129,14 @@ > static NTSTATUS (WINAPI * pNtCreateKey)( PHANDLE retkey, ACCESS_MASK access, const OBJECT_ATTRIBUTES *attr, > ULONG TitleIndex, const UNICODE_STRING *class, ULONG options, > PULONG dispos ); > +static NTSTATUS (WINAPI * pNtQueryKey)(HANDLE,KEY_INFORMATION_CLASS,PVOID,ULONG,PULONG); > +static NTSTATUS (WINAPI * pNtQueryLicenseValue)(const UNICODE_STRING *,ULONG *,PVOID,ULONG,ULONG *); > static NTSTATUS (WINAPI * pNtQueryValueKey)(HANDLE,const UNICODE_STRING *,KEY_VALUE_INFORMATION_CLASS,void *,DWORD,DWORD *); > static NTSTATUS (WINAPI * pNtSetValueKey)(HANDLE, const PUNICODE_STRING, ULONG, > ULONG, const void*, ULONG ); > static NTSTATUS (WINAPI * pNtQueryInformationProcess)(HANDLE,PROCESSINFOCLASS,PVOID,ULONG,PULONG); > static NTSTATUS (WINAPI * pRtlFormatCurrentUserKeyPath)(PUNICODE_STRING); > +static LONG (WINAPI * pRtlCompareUnicodeString)(const PUNICODE_STRING,const PUNICODE_STRING,BOOLEAN); > static BOOLEAN (WINAPI * pRtlCreateUnicodeString)(PUNICODE_STRING, LPCWSTR); > static LPVOID (WINAPI * pRtlReAllocateHeap)(IN PVOID, IN ULONG, IN PVOID, IN ULONG); > static NTSTATUS (WINAPI * pRtlAppendUnicodeToString)(PUNICODE_STRING, PCWSTR); > @@ -166,7 +169,6 @@ > NTDLL_GET_PROC(RtlCreateUnicodeStringFromAsciiz) > NTDLL_GET_PROC(RtlCreateUnicodeString) > NTDLL_GET_PROC(RtlFreeUnicodeString) > - NTDLL_GET_PROC(NtDeleteValueKey) > NTDLL_GET_PROC(RtlQueryRegistryValues) > NTDLL_GET_PROC(RtlCheckRegistryKey) > NTDLL_GET_PROC(RtlOpenCurrentUser) > @@ -175,11 +177,13 @@ > NTDLL_GET_PROC(NtCreateKey) > NTDLL_GET_PROC(NtFlushKey) > NTDLL_GET_PROC(NtDeleteKey) > + NTDLL_GET_PROC(NtQueryKey) > NTDLL_GET_PROC(NtQueryValueKey) > NTDLL_GET_PROC(NtQueryInformationProcess) > NTDLL_GET_PROC(NtSetValueKey) > NTDLL_GET_PROC(NtOpenKey) > NTDLL_GET_PROC(RtlFormatCurrentUserKeyPath) > + NTDLL_GET_PROC(RtlCompareUnicodeString) > NTDLL_GET_PROC(RtlReAllocateHeap) > NTDLL_GET_PROC(RtlAppendUnicodeToString) > NTDLL_GET_PROC(RtlUnicodeStringToAnsiString) > @@ -188,6 +192,10 @@ > NTDLL_GET_PROC(RtlZeroMemory) > NTDLL_GET_PROC(RtlpNtQueryValueKey) > NTDLL_GET_PROC(RtlOpenCurrentUser) > + > + /* optional functions */ > + pNtQueryLicenseValue = (void *)GetProcAddress(hntdll, "NtQueryLicenseValue"); > + > return TRUE; > } > #undef NTDLL_GET_PROC > @@ -640,6 +648,180 @@ > > status = pNtDeleteKey(hkey); > ok(status == STATUS_SUCCESS, "NtDeleteKey Failed: 0x%08x\n", status); > +} > + > +static void test_NtQueryLicenseKey(void) > +{ > + static const WCHAR emptyW[] = {'E','M','P','T','Y',0}; > + UNICODE_STRING name; > + WORD buffer[32]; > + NTSTATUS status; > + ULONG type, len; > + DWORD value; > + > + if (!pNtQueryLicenseValue) > + { > + win_skip("NtQueryLicenseValue not found, skipping tests\n"); > + return; > + } > + > + type = 0xdead; > + len = 0xbeef; > + memset(&name, 0, sizeof(name)); > + status = pNtQueryLicenseValue(&name, &type, buffer, sizeof(buffer), &len); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + ok(len == 0xbeef, "expected unmodified value for len, got %u\n", len); > + > + /* test with empty key */ > + pRtlCreateUnicodeStringFromAsciiz(&name, ""); > + > + type = 0xdead; > + len = 0xbeef; > + status = pNtQueryLicenseValue(NULL, &type, buffer, sizeof(buffer), &len); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + ok(len == 0xbeef, "expected unmodified value for len, got %u\n", len); > + > + type = 0xdead; > + status = pNtQueryLicenseValue(&name, &type, buffer, sizeof(buffer), NULL); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + > + len = 0xbeef; > + status = pNtQueryLicenseValue(&name, NULL, buffer, sizeof(buffer), &len); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(len == 0xbeef, "expected unmodified value for len, got %u\n", len); > + > + type = 0xdead; > + len = 0xbeef; > + status = pNtQueryLicenseValue(&name, &type, buffer, sizeof(buffer), &len); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + ok(len == 0xbeef, "expected unmodified value for len, got %u\n", len); > + > + pRtlFreeUnicodeString(&name); > + > + /* test with nonexistent licence key */ > + pRtlCreateUnicodeStringFromAsciiz(&name, "Nonexistent-License-Value"); > + > + type = 0xdead; > + len = 0xbeef; > + status = pNtQueryLicenseValue(NULL, &type, buffer, sizeof(buffer), &len); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + ok(len == 0xbeef, "expected unmodified value for len, got %u\n", len); > + > + type = 0xdead; > + status = pNtQueryLicenseValue(&name, &type, buffer, sizeof(buffer), NULL); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + > + len = 0xbeef; > + status = pNtQueryLicenseValue(&name, NULL, buffer, sizeof(buffer), &len); > + ok(status == STATUS_OBJECT_NAME_NOT_FOUND, "NtQueryLicenseValue returned %08x, expected STATUS_OBJECT_NAME_NOT_FOUND\n", status); > + ok(len == 0xbeef, "expected unmodified value for len, got %u\n", len); > + > + type = 0xdead; > + len = 0xbeef; > + status = pNtQueryLicenseValue(&name, &type, buffer, sizeof(buffer), &len); > + ok(status == STATUS_OBJECT_NAME_NOT_FOUND, "NtQueryLicenseValue unexpected suceeded\n"); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + ok(len == 0xbeef, "expected unmodified value for len, got %u\n", len); > + > + pRtlFreeUnicodeString(&name); > + > + /* test with REG_SZ license key */ > + pRtlCreateUnicodeStringFromAsciiz(&name, "Kernel-MUI-Language-Allowed"); > + > + type = 0xdead; > + len = 0xbeef; > + status = pNtQueryLicenseValue(NULL, &type, buffer, sizeof(buffer), &len); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + ok(len == 0xbeef, "expected unmodified value for len, got %u\n", len); > + > + type = 0xdead; > + status = pNtQueryLicenseValue(&name, &type, buffer, sizeof(buffer), NULL); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + > + type = 0xdead; > + len = 0; > + status = pNtQueryLicenseValue(&name, &type, buffer, 0, &len); > + ok(status == STATUS_BUFFER_TOO_SMALL, "NtQueryLicenseValue returned %08x, expected STATUS_BUFFER_TOO_SMALL\n", status); > + ok(type == REG_SZ, "expected type = REG_SZ, got %u\n", type); > + ok(len == sizeof(emptyW), "expected len = %u, got %u\n", (DWORD)sizeof(emptyW), len); > + > + len = 0; > + status = pNtQueryLicenseValue(&name, NULL, buffer, 0, &len); > + ok(status == STATUS_BUFFER_TOO_SMALL, "NtQueryLicenseValue returned %08x, expected STATUS_BUFFER_TOO_SMALL\n", status); > + ok(len == sizeof(emptyW), "expected len = %u, got %u\n", (DWORD)sizeof(emptyW), len); > + > + type = 0xdead; > + len = 0; > + memset(buffer, 0x11, sizeof(buffer)); > + status = pNtQueryLicenseValue(&name, &type, buffer, sizeof(buffer), &len); > + ok(status == STATUS_SUCCESS, "NtQueryLicenseValue returned %08x, expected STATUS_SUCCESS\n", status); > + ok(type == REG_SZ, "expected type = REG_SZ, got %u\n", type); > + ok(len == sizeof(emptyW), "expected len = %u, got %u\n", (DWORD)sizeof(emptyW), len); > + ok(!memcmp(buffer, emptyW, sizeof(emptyW)), "unexpected buffer content\n"); > + > + type = 0xdead; > + len = 0; > + memset(buffer, 0x11, sizeof(buffer)); > + status = pNtQueryLicenseValue(&name, &type, buffer, 2, &len); > + ok(status == STATUS_BUFFER_TOO_SMALL, "NtQueryLicenseValue returned %08x, expected STATUS_BUFFER_TOO_SMALL\n", status); > + ok(type == REG_SZ, "expected type REG_SZ, got %u\n", type); > + ok(len == sizeof(emptyW), "expected len = %u, got %u\n", (DWORD)sizeof(emptyW), len); > + ok(buffer[0] == 0x1111, "expected buffer[0] = 0x1111, got %u\n", buffer[0]); > + > + pRtlFreeUnicodeString(&name); > + > + /* test with REG_DWORD license key */ > + pRtlCreateUnicodeStringFromAsciiz(&name, "Kernel-MUI-Number-Allowed"); > + > + type = 0xdead; > + len = 0xbeef; > + status = pNtQueryLicenseValue(NULL, &type, &value, sizeof(value), &len); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + ok(len == 0xbeef, "expected unmodified value for len, got %u\n", len); > + > + type = 0xdead; > + status = pNtQueryLicenseValue(&name, &type, &value, sizeof(value), NULL); > + ok(status == STATUS_INVALID_PARAMETER, "NtQueryLicenseValue returned %08x, expected STATUS_INVALID_PARAMETER\n", status); > + ok(type == 0xdead, "expected unmodified value for type, got %u\n", type); > + > + type = 0xdead; > + len = 0; > + status = pNtQueryLicenseValue(&name, &type, &value, 0, &len); > + ok(status == STATUS_BUFFER_TOO_SMALL, "NtQueryLicenseValue returned %08x, expected STATUS_BUFFER_TOO_SMALL\n", status); > + ok(type == REG_DWORD, "expected type = REG_DWORD, got %u\n", type); > + ok(len == sizeof(value), "expected len = %u, got %u\n", (DWORD)sizeof(value), len); > + > + len = 0; > + status = pNtQueryLicenseValue(&name, NULL, &value, 0, &len); > + ok(status == STATUS_BUFFER_TOO_SMALL, "NtQueryLicenseValue returned %08x, expected STATUS_BUFFER_TOO_SMALL\n", status); > + ok(len == sizeof(value), "expected len = %u, got %u\n", (DWORD)sizeof(value), len); > + > + type = 0xdead; > + len = 0; > + value = 0xdeadbeef; > + status = pNtQueryLicenseValue(&name, &type, &value, sizeof(value), &len); > + ok(status == STATUS_SUCCESS, "NtQueryLicenseValue returned %08x, expected STATUS_SUCCESS\n", status); > + ok(type == REG_DWORD, "expected type = REG_DWORD, got %u\n", type); > + ok(len == sizeof(value), "expected len = %u, got %u\n", (DWORD)sizeof(value), len); > + ok(value != 0xdeadbeef, "expected value != 0xdeadbeef\n"); > + > + type = 0xdead; > + len = 0; > + status = pNtQueryLicenseValue(&name, &type, &value, 2, &len); > + ok(status == STATUS_BUFFER_TOO_SMALL, "NtQueryLicenseValue returned %08x, expected STATUS_BUFFER_TOO_SMALL\n", status); > + ok(type == REG_DWORD, "expected type REG_DWORD, got %u\n", type); > + ok(len == sizeof(value), "expected len = %u, got %u\n", (DWORD)sizeof(value), len); > + > + pRtlFreeUnicodeString(&name); > } > > static void test_RtlpNtQueryValueKey(void) > @@ -1277,6 +1459,51 @@ > pNtClose(key); > } > > +static void test_NtQueryKey(void) > +{ > + HANDLE key; > + NTSTATUS status; > + OBJECT_ATTRIBUTES attr; > + ULONG length, len; > + KEY_NAME_INFORMATION *info = NULL; > + UNICODE_STRING str; > + > + InitializeObjectAttributes(&attr, &winetestpath, 0, 0, 0); > + status = pNtOpenKey(&key, KEY_READ, &attr); > + ok(status == STATUS_SUCCESS, "NtOpenKey Failed: 0x%08x\n", status); > + > + status = pNtQueryKey(key, KeyNameInformation, NULL, 0, &length); > + if (status == STATUS_INVALID_PARAMETER) { > + win_skip("KeyNameInformation is not supported\n"); > + pNtClose(key); > + return; > + } > + todo_wine ok(status == STATUS_BUFFER_TOO_SMALL, "NtQueryKey Failed: 0x%08x\n", status); > + info = HeapAlloc(GetProcessHeap(), 0, length); > + > + /* non-zero buffer size, but insufficient */ > + status = pNtQueryKey(key, KeyNameInformation, info, sizeof(*info), &len); > + ok(status == STATUS_BUFFER_OVERFLOW, "NtQueryKey Failed: 0x%08x\n", status); > + ok(length == len, "got %d, expected %d\n", len, length); > + ok(info->NameLength == winetestpath.Length, "got %d, expected %d\n", > + info->NameLength, winetestpath.Length); > + > + /* correct buffer size */ > + status = pNtQueryKey(key, KeyNameInformation, info, length, &len); > + ok(status == STATUS_SUCCESS, "NtQueryKey Failed: 0x%08x\n", status); > + ok(length == len, "got %d, expected %d\n", len, length); > + > + str.Buffer = info->Name; > + str.Length = info->NameLength; > + ok(pRtlCompareUnicodeString(&winetestpath, &str, TRUE) == 0, > + "got %s, expected %s\n", > + wine_dbgstr_wn(str.Buffer, str.Length/sizeof(WCHAR)), > + wine_dbgstr_wn(winetestpath.Buffer, winetestpath.Length/sizeof(WCHAR))); > + > + HeapFree(GetProcessHeap(), 0, info); > + pNtClose(key); > +} > + > START_TEST(reg) > { > static const WCHAR winetest[] = {'\\','W','i','n','e','T','e','s','t',0}; > @@ -1297,7 +1524,9 @@ > test_RtlQueryRegistryValues(); > test_RtlpNtQueryValueKey(); > test_NtFlushKey(); > + test_NtQueryKey(); > test_NtQueryValueKey(); > + test_NtQueryLicenseKey(); > test_long_value_name(); > test_NtDeleteKey(); > test_symlinks(); > > Modified: trunk/rostests/winetests/ntdll/rtl.c > URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/ntdll/rtl.c?rev… > ============================================================================== > --- trunk/rostests/winetests/ntdll/rtl.c [iso-8859-1] (original) > +++ trunk/rostests/winetests/ntdll/rtl.c [iso-8859-1] Tue Mar 10 10:10:04 2015 > @@ -62,6 +62,8 @@ > > /* Function ptrs for ntdll calls */ > static HMODULE hntdll = 0; > +static PVOID (WINAPI *pWinSqmStartSession)(PVOID unknown1, DWORD unknown2, DWORD unknown3); > +static NTSTATUS (WINAPI *pWinSqmEndSession)(PVOID unknown1); > static SIZE_T (WINAPI *pRtlCompareMemory)(LPCVOID,LPCVOID,SIZE_T); > static SIZE_T (WINAPI *pRtlCompareMemoryUlong)(PULONG, SIZE_T, ULONG); > static NTSTATUS (WINAPI *pRtlDeleteTimer)(HANDLE, HANDLE, HANDLE); > @@ -89,9 +91,14 @@ > static CHAR * (WINAPI *pRtlIpv4AddressToStringA)(const IN_ADDR *, LPSTR); > static NTSTATUS (WINAPI *pRtlIpv4AddressToStringExA)(const IN_ADDR *, USHORT, LPSTR, PULONG); > static NTSTATUS (WINAPI *pRtlIpv4StringToAddressA)(PCSTR, BOOLEAN, PCSTR *, IN_ADDR *); > +static NTSTATUS (WINAPI *pRtlIpv4StringToAddressExA)(PCSTR, BOOLEAN, IN_ADDR *, PUSHORT); > static NTSTATUS (WINAPI *pLdrAddRefDll)(ULONG, HMODULE); > static NTSTATUS (WINAPI *pLdrLockLoaderLock)(ULONG, ULONG*, ULONG_PTR*); > static NTSTATUS (WINAPI *pLdrUnlockLoaderLock)(ULONG, ULONG_PTR); > +static NTSTATUS (WINAPI *pRtlGetCompressionWorkSpaceSize)(USHORT, PULONG, PULONG); > +static NTSTATUS (WINAPI *pRtlDecompressBuffer)(USHORT, PUCHAR, ULONG, const UCHAR*, ULONG, PULONG); > +static NTSTATUS (WINAPI *pRtlDecompressFragment)(USHORT, PUCHAR, ULONG, const UCHAR*, ULONG, ULONG, PULONG, PVOID); > +static NTSTATUS (WINAPI *pRtlCompressBuffer)(USHORT, const UCHAR*, ULONG, PUCHAR, ULONG, ULONG, PULONG, PVOID); > > static HMODULE hkernel32 = 0; > static BOOL (WINAPI *pIsWow64Process)(HANDLE, PBOOL); > @@ -109,6 +116,8 @@ > hntdll = LoadLibraryA("ntdll.dll"); > ok(hntdll != 0, "LoadLibrary failed\n"); > if (hntdll) { > + pWinSqmStartSession = (void *)GetProcAddress(hntdll, "WinSqmStartSession"); > + pWinSqmEndSession = (void *)GetProcAddress(hntdll, "WinSqmEndSession"); > pRtlCompareMemory = (void *)GetProcAddress(hntdll, "RtlCompareMemory"); > pRtlCompareMemoryUlong = (void *)GetProcAddress(hntdll, "RtlCompareMemoryUlong"); > pRtlDeleteTimer = (void *)GetProcAddress(hntdll, "RtlDeleteTimer"); > @@ -136,9 +145,14 @@ > pRtlIpv4AddressToStringA = (void *)GetProcAddress(hntdll, "RtlIpv4AddressToStringA"); > pRtlIpv4AddressToStringExA = (void *)GetProcAddress(hntdll, "RtlIpv4AddressToStringExA"); > pRtlIpv4StringToAddressA = (void *)GetProcAddress(hntdll, "RtlIpv4StringToAddressA"); > + pRtlIpv4StringToAddressExA = (void *)GetProcAddress(hntdll, "RtlIpv4StringToAddressExA"); > pLdrAddRefDll = (void *)GetProcAddress(hntdll, "LdrAddRefDll"); > pLdrLockLoaderLock = (void *)GetProcAddress(hntdll, "LdrLockLoaderLock"); > pLdrUnlockLoaderLock = (void *)GetProcAddress(hntdll, "LdrUnlockLoaderLock"); > + pRtlGetCompressionWorkSpaceSize = (void *)GetProcAddress(hntdll, "RtlGetCompressionWorkSpaceSize"); > + pRtlDecompressBuffer = (void *)GetProcAddress(hntdll, "RtlDecompressBuffer"); > + pRtlDecompressFragment = (void *)GetProcAddress(hntdll, "RtlDecompressFragment"); > + pRtlCompressBuffer = (void *)GetProcAddress(hntdll, "RtlCompressBuffer"); > } > hkernel32 = LoadLibraryA("kernel32.dll"); > ok(hkernel32 != 0, "LoadLibrary failed\n"); > @@ -148,6 +162,46 @@ > strcpy((char*)src_aligned_block, src_src); > ok(strlen(src) == 15, "Source must be 16 bytes long!\n"); > } > + > +#ifdef __i386__ > +const char stdcall3_thunk[] = > + "\x56" /* push %esi */ > + "\x89\xE6" /* mov %esp, %esi */ > + "\xFF\x74\x24\x14" /* pushl 20(%esp) */ > + "\xFF\x74\x24\x14" /* pushl 20(%esp) */ > + "\xFF\x74\x24\x14" /* pushl 20(%esp) */ > + "\xFF\x54\x24\x14" /* calll 20(%esp) */ > + "\x89\xF0" /* mov %esi, %eax */ > + "\x29\xE0" /* sub %esp, %eax */ > + "\x89\xF4" /* mov %esi, %esp */ > + "\x5E" /* pop %esi */ > + "\xC2\x10\x00" /* ret $16 */ > +; > + > +static INT (WINAPI *call_stdcall_func3)(PVOID func, PVOID arg0, DWORD arg1, DWORD arg2) = NULL; > + > +static void test_WinSqm(void) > +{ > + INT args; > + > + if (!pWinSqmStartSession) > + { > + win_skip("WinSqmStartSession() is not available\n"); > + return; > + } > + > + call_stdcall_func3 = (void*) VirtualAlloc( NULL, sizeof(stdcall3_thunk) - 1, MEM_COMMIT, > + PAGE_EXECUTE_READWRITE ); > + memcpy( call_stdcall_func3, stdcall3_thunk, sizeof(stdcall3_thunk) - 1 ); > + > + args = 3 - call_stdcall_func3( pWinSqmStartSession, NULL, 0, 0 ) / 4; > + ok(args == 3, "WinSqmStartSession expected to take %d arguments instead of 3\n", args); > + args = 3 - call_stdcall_func3( pWinSqmEndSession, NULL, 0, 0 ) / 4; > + ok(args == 1, "WinSqmEndSession expected to take %d arguments instead of 1\n", args); > + > + VirtualFree( call_stdcall_func3, 0, MEM_RELEASE ); > +} > +#endif > > #define COMP(str1,str2,cmplen,len) size = pRtlCompareMemory(str1, str2, cmplen); \ > ok(size == len, "Expected %ld, got %ld\n", size, (SIZE_T)len) > @@ -1492,6 +1546,94 @@ > } > } > > +static void test_RtlIpv4StringToAddressEx(void) > +{ > + NTSTATUS res; > + IN_ADDR ip, expected_ip; > + USHORT port; > + struct > + { > + PCSTR address; > + NTSTATUS res; > + int ip[4]; > + USHORT port; > + } tests[] = > + { > + { "", STATUS_INVALID_PARAMETER, { -1 }, 0xdead }, > + { " ", STATUS_INVALID_PARAMETER, { -1 }, 0xdead }, > + { "1.1.1.1:", STATUS_INVALID_PARAMETER, { 1, 1, 1, 1 }, 0xdead }, > + { "1.1.1.1+", STATUS_INVALID_PARAMETER, { 1, 1, 1, 1 }, 0xdead }, > + { "1.1.1.1:1", STATUS_SUCCESS, { 1, 1, 1, 1 }, 0x100 }, > + { "0.0.0.0:0", STATUS_INVALID_PARAMETER, { 0, 0, 0, 0 }, 0xdead }, > + { "0.0.0.0:1", STATUS_SUCCESS, { 0, 0, 0, 0 }, 0x100 }, > + { "1.2.3.4:65535", STATUS_SUCCESS, { 1, 2, 3, 4 }, 65535 }, > + { "1.2.3.4:65536", STATUS_INVALID_PARAMETER, { 1, 2, 3, 4 }, 0xdead }, > + { "1.2.3.4:0xffff", STATUS_SUCCESS, { 1, 2, 3, 4 }, 65535 }, > + { "1.2.3.4:0XfFfF", STATUS_SUCCESS, { 1, 2, 3, 4 }, 65535 }, > + { "1.2.3.4:011064", STATUS_SUCCESS, { 1, 2, 3, 4 }, 0x3412 }, > + { "1.2.3.4:1234a", STATUS_INVALID_PARAMETER, { 1, 2, 3, 4 }, 0xdead }, > + { "1.2.3.4:1234+", STATUS_INVALID_PARAMETER, { 1, 2, 3, 4 }, 0xdead }, > + { "1.2.3.4: 1234", STATUS_INVALID_PARAMETER, { 1, 2, 3, 4 }, 0xdead }, > + { "1.2.3.4:\t1234", STATUS_INVALID_PARAMETER, { 1, 2, 3, 4 }, 0xdead }, > + }; > + const int testcount = sizeof(tests) / sizeof(tests[0]); > + int i, Strict; > + > + if (!pRtlIpv4StringToAddressExA) > + { > + skip("RtlIpv4StringToAddressEx not available\n"); > + return; > + } > + > + /* do not crash, and do not touch the ip / port. */ > + ip.S_un.S_addr = 0xabababab; > + port = 0xdead; > + res = pRtlIpv4StringToAddressExA(NULL, FALSE, &ip, &port); > + ok(res == STATUS_INVALID_PARAMETER, "[null address] res = 0x%08x, expected 0x%08x\n", res, STATUS_INVALID_PARAMETER); > + ok(ip.S_un.S_addr == 0xabababab, "RtlIpv4StringToAddressExA should not touch the ip!, ip == %x\n", ip.S_un.S_addr); > + ok(port == 0xdead, "RtlIpv4StringToAddressExA should not touch the port!, port == %x\n", port); > + > + port = 0xdead; > + res = pRtlIpv4StringToAddressExA("1.1.1.1", FALSE, NULL, &port); > + ok(res == STATUS_INVALID_PARAMETER, "[null ip] res = 0x%08x, expected 0x%08x\n", res, STATUS_INVALID_PARAMETER); > + ok(port == 0xdead, "RtlIpv4StringToAddressExA should not touch the port!, port == %x\n", port); > + > + ip.S_un.S_addr = 0xabababab; > + port = 0xdead; > + res = pRtlIpv4StringToAddressExA("1.1.1.1", FALSE, &ip, NULL); > + ok(res == STATUS_INVALID_PARAMETER, "[null port] res = 0x%08x, expected 0x%08x\n", res, STATUS_INVALID_PARAMETER); > + ok(ip.S_un.S_addr == 0xabababab, "RtlIpv4StringToAddressExA should not touch the ip!, ip == %x\n", ip.S_un.S_addr); > + ok(port == 0xdead, "RtlIpv4StringToAddressExA should not touch the port!, port == %x\n", port); > + > + for (i = 0; i < testcount; i++) > + { > + /* Strict is only relevant for the ip address, so make sure that it does not influence the port */ > + for (Strict = 0; Strict < 2; Strict++) > + { > + ip.S_un.S_addr = 0xabababab; > + port = 0xdead; > + res = pRtlIpv4StringToAddressExA(tests[i].address, Strict, &ip, &port); > + if (tests[i].ip[0] == -1) > + { > + expected_ip.S_un.S_addr = 0xabababab; > + } > + else > + { > + expected_ip.S_un.S_un_b.s_b1 = tests[i].ip[0]; > + expected_ip.S_un.S_un_b.s_b2 = tests[i].ip[1]; > + expected_ip.S_un.S_un_b.s_b3 = tests[i].ip[2]; > + expected_ip.S_un.S_un_b.s_b4 = tests[i].ip[3]; > + } > + ok(res == tests[i].res, "[%s] res = 0x%08x, expected 0x%08x\n", > + tests[i].address, res, tests[i].res); > + ok(ip.S_un.S_addr == expected_ip.S_un.S_addr, "[%s] ip = %08x, expected %08x\n", > + tests[i].address, ip.S_un.S_addr, expected_ip.S_un.S_addr); > + ok(port == tests[i].port, "[%s] port = %u, expected %u\n", > + tests[i].address, port, tests[i].port); > + } > + } > +} > + > static void test_LdrAddRefDll(void) > { > HMODULE mod, mod2; > @@ -1599,9 +1741,744 @@ > pLdrUnlockLoaderLock(0, magic); > } > > +static void test_RtlGetCompressionWorkSpaceSize(void) > +{ > + ULONG compress_workspace, decompress_workspace; > + NTSTATUS status; > + > + if (!pRtlGetCompressionWorkSpaceSize) > + { > + win_skip("RtlGetCompressionWorkSpaceSize is not available\n"); > + return; > + } > + > + status = pRtlGetCompressionWorkSpaceSize(COMPRESSION_FORMAT_NONE, &compress_workspace, > + &decompress_workspace); > + ok(status == STATUS_INVALID_PARAMETER, "got wrong status 0x%08x\n", status); > + > + status = pRtlGetCompressionWorkSpaceSize(COMPRESSION_FORMAT_DEFAULT, &compress_workspace, > + &decompress_workspace); > + ok(status == STATUS_INVALID_PARAMETER, "got wrong status 0x%08x\n", status); > + > + status = pRtlGetCompressionWorkSpaceSize(0xFF, &compress_workspace, &decompress_workspace); > + ok(status == STATUS_UNSUPPORTED_COMPRESSION, "got wrong status 0x%08x\n", status); > + > + compress_workspace = decompress_workspace = 0xdeadbeef; > + status = pRtlGetCompressionWorkSpaceSize(COMPRESSION_FORMAT_LZNT1, &compress_workspace, > + &decompress_workspace); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + ok(compress_workspace != 0, "got wrong compress_workspace %d\n", compress_workspace); > + ok(decompress_workspace == 0x1000, "got wrong decompress_workspace %d\n", decompress_workspace); > + > + compress_workspace = decompress_workspace = 0xdeadbeef; > + status = pRtlGetCompressionWorkSpaceSize(COMPRESSION_FORMAT_LZNT1 | COMPRESSION_ENGINE_MAXIMUM, > + &compress_workspace, &decompress_workspace); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + ok(compress_workspace != 0, "got wrong compress_workspace %d\n", compress_workspace); > + ok(decompress_workspace == 0x1000, "got wrong decompress_workspace %d\n", decompress_workspace); > +} > + > +/* helper for test_RtlDecompressBuffer, checks if a chunk is incomplete */ > +static BOOL is_incomplete_chunk(const UCHAR *compressed, ULONG compressed_size, BOOL check_all) > +{ > + ULONG chunk_size; > + if (compressed_size <= sizeof(WORD)) > + return TRUE; > + while (compressed_size >= sizeof(WORD)) > + { > + chunk_size = (*(WORD *)compressed & 0xFFF) + 1; > + if (compressed_size < sizeof(WORD) + chunk_size) > + return TRUE; > + if (!check_all) > + break; > + compressed += sizeof(WORD) + chunk_size; > + compressed_size -= sizeof(WORD) + chunk_size; > + } > + return FALSE; > +} > + > +#define DECOMPRESS_BROKEN_TRUNCATED 1 > +#define DECOMPRESS_BROKEN_FRAGMENT0 2 > +#define DECOMPRESS_BROKEN_FRAGMENT1 4 > +#define DECOMPRESS_BROKEN_FRAGMENT4095 8 > + > +static void test_RtlDecompressBuffer(void) > +{ > + static const UCHAR test_multiple_chunks[] = {0x03, 0x30, 'W', 'i', 'n', 'e', > + 0x03, 0x30, 'W', 'i', 'n', 'e'}; > + static const struct > + { > + UCHAR compressed[32]; > + ULONG compressed_size; > + NTSTATUS status; > + UCHAR uncompressed[32]; > + ULONG uncompressed_size; > + DWORD broken_flags; > + } > + test_lznt[] = > + { > + /* 4 byte uncompressed chunk */ > + { > + {0x03, 0x30, 'W', 'i', 'n', 'e'}, > + 6, > + STATUS_SUCCESS, > + "Wine", > + 4, > + DECOMPRESS_BROKEN_FRAGMENT4095 | > + DECOMPRESS_BROKEN_FRAGMENT1 | > + DECOMPRESS_BROKEN_FRAGMENT0 > + }, > + /* 8 byte uncompressed chunk */ > + { > + {0x07, 0x30, 'W', 'i', 'n', 'e', 'W', 'i', 'n', 'e'}, > + 10, > + STATUS_SUCCESS, > + "WineWine", > + 8, > + DECOMPRESS_BROKEN_FRAGMENT4095 | > + DECOMPRESS_BROKEN_FRAGMENT1 | > + DECOMPRESS_BROKEN_FRAGMENT0 > + }, > + /* 4 byte compressed chunk */ > + { > + {0x04, 0xB0, 0x00, 'W', 'i', 'n', 'e'}, > + 7, > + STATUS_SUCCESS, > + "Wine", > + 4 > + }, > + /* 8 byte compressed chunk */ > + { > + {0x08, 0xB0, 0x00, 'W', 'i', 'n', 'e', 'W', 'i', 'n', 'e'}, > + 11, > + STATUS_SUCCESS, > + "WineWine", > + 8 > + }, > + /* compressed chunk using backwards reference */ > + { > + {0x06, 0xB0, 0x10, 'W', 'i', 'n', 'e', 0x01, 0x30}, > + 9, > + STATUS_SUCCESS, > + "WineWine", > + 8, > + DECOMPRESS_BROKEN_TRUNCATED > + }, > + /* compressed chunk using backwards reference with length > bytes_read */ > + { > + {0x06, 0xB0, 0x10, 'W', 'i', 'n', 'e', 0x05, 0x30}, > + 9, > + STATUS_SUCCESS, > + "WineWineWine", > + 12, > + DECOMPRESS_BROKEN_TRUNCATED > + }, > + /* same as above, but unused bits != 0 */ > + { > + {0x06, 0xB0, 0x30, 'W', 'i', 'n', 'e', 0x01, 0x30}, > + 9, > + STATUS_SUCCESS, > + "WineWine", > + 8, > + DECOMPRESS_BROKEN_TRUNCATED > + }, > + /* compressed chunk without backwards reference and unused bits != 0 */ > + { > + {0x01, 0xB0, 0x02, 'W'}, > + 4, > + STATUS_SUCCESS, > + "W", > + 1 > + }, > + /* termination sequence after first chunk */ > + { > + {0x03, 0x30, 'W', 'i', 'n', 'e', 0x00, 0x00, 0x03, 0x30, 'W', 'i', 'n', 'e'}, > + 14, > + STATUS_SUCCESS, > + "Wine", > + 4, > + DECOMPRESS_BROKEN_FRAGMENT4095 | > + DECOMPRESS_BROKEN_FRAGMENT1 | > + DECOMPRESS_BROKEN_FRAGMENT0 > + }, > + /* compressed chunk using backwards reference with 4 bit offset, 12 bit length */ > + { > + {0x14, 0xB0, 0x00, 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', > + 0x00, 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', > + 0x01, 0x01, 0xF0}, > + 23, > + STATUS_SUCCESS, > + "ABCDEFGHIJKLMNOPABCD", > + 20, > + DECOMPRESS_BROKEN_TRUNCATED > + }, > + /* compressed chunk using backwards reference with 5 bit offset, 11 bit length */ > + { > + {0x15, 0xB0, 0x00, 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', > + 0x00, 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', > + 0x02, 'A', 0x00, 0x78}, > + 24, > + STATUS_SUCCESS, > + "ABCDEFGHIJKLMNOPABCD", > + 20, > + DECOMPRESS_BROKEN_TRUNCATED > + }, > + /* uncompressed chunk with invalid magic */ > + { > + {0x03, 0x20, 'W', 'i', 'n', 'e'}, > + 6, > + STATUS_SUCCESS, > + "Wine", > + 4, > + DECOMPRESS_BROKEN_FRAGMENT4095 | > + DECOMPRESS_BROKEN_FRAGMENT1 | > + DECOMPRESS_BROKEN_FRAGMENT0 > + }, > + /* compressed chunk with invalid magic */ > + { > + {0x04, 0xA0, 0x00, 'W', 'i', 'n', 'e'}, > + 7, > + STATUS_SUCCESS, > + "Wine", > + 4 > + }, > + /* garbage byte after end of buffer */ > + { > + {0x00, 0xB0, 0x02, 0x01}, > + 4, > + STATUS_SUCCESS, > + "", > + 0 > + }, > + /* empty compressed chunk */ > + { > + {0x00, 0xB0, 0x00}, > + 3, > + STATUS_SUCCESS, > + "", > + 0 > + }, > + /* empty compressed chunk with unused bits != 0 */ > + { > + {0x00, 0xB0, 0x01}, > + 3, > + STATUS_SUCCESS, > + "", > + 0 > + }, > + /* empty input buffer */ > + { > + {0}, > + 0, > + STATUS_BAD_COMPRESSION_BUFFER, > + }, > + /* incomplete chunk header */ > + { > + {0x01}, > + 1, > + STATUS_BAD_COMPRESSION_BUFFER > + }, > + /* incomplete chunk header */ > + { > + {0x00, 0x30}, > + 2, > + STATUS_BAD_COMPRESSION_BUFFER > + }, > + /* compressed chunk with invalid backwards reference */ > + { > + {0x06, 0xB0, 0x10, 'W', 'i', 'n', 'e', 0x05, 0x40}, > + 9, > + STATUS_BAD_COMPRESSION_BUFFER > + }, > + /* compressed chunk with incomplete backwards reference */ > + { > + {0x05, 0xB0, 0x10, 'W', 'i', 'n', 'e', 0x05}, > + 8, > + STATUS_BAD_COMPRESSION_BUFFER > + }, > + /* incomplete uncompressed chunk */ > + { > + {0x07, 0x30, 'W', 'i', 'n', 'e'}, > + 6, > + STATUS_BAD_COMPRESSION_BUFFER > + }, > + /* incomplete compressed chunk */ > + { > + {0x08, 0xB0, 0x00, 'W', 'i', 'n', 'e'}, > + 7, > + STATUS_BAD_COMPRESSION_BUFFER > + }, > + /* two compressed chunks, the second one incomplete */ > + { > + {0x00, 0xB0, 0x02, 0x00, 0xB0}, > + 5, > + STATUS_BAD_COMPRESSION_BUFFER, > + } > + }; > + > + static UCHAR buf[0x2000], workspace[0x1000]; > + NTSTATUS status, expected_status; > + ULONG final_size; > + int i; > + > + if (!pRtlDecompressBuffer || !pRtlDecompressFragment) > + { > + win_skip("RtlDecompressBuffer or RtlDecompressFragment is not available\n"); > + return; > + } > + > + /* test compression format / engine */ > + final_size = 0xdeadbeef; > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_NONE, buf, sizeof(buf) - 1, test_lznt[0].compressed, > + test_lznt[0].compressed_size, &final_size); > + ok(status == STATUS_INVALID_PARAMETER, "got wrong status 0x%08x\n", status); > + ok(final_size == 0xdeadbeef, "got wrong final_size %d\n", final_size); > + > + final_size = 0xdeadbeef; > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_DEFAULT, buf, sizeof(buf) - 1, test_lznt[0].compressed, > + test_lznt[0].compressed_size, &final_size); > + ok(status == STATUS_INVALID_PARAMETER, "got wrong status 0x%08x\n", status); > + ok(final_size == 0xdeadbeef, "got wrong final_size %d\n", final_size); > + > + final_size = 0xdeadbeef; > + status = pRtlDecompressBuffer(0xFF, buf, sizeof(buf) - 1, test_lznt[0].compressed, > + test_lznt[0].compressed_size, &final_size); > + ok(status == STATUS_UNSUPPORTED_COMPRESSION, "got wrong status 0x%08x\n", status); > + ok(final_size == 0xdeadbeef, "got wrong final_size %d\n", final_size); > + > + /* regular tests for RtlDecompressBuffer */ > + for (i = 0; i < sizeof(test_lznt) / sizeof(test_lznt[0]); i++) > + { > + trace("Running test %d (compressed_size=%d, compressed_size=%d, status=%d)\n", > + i, test_lznt[i].compressed_size, test_lznt[i].compressed_size, test_lznt[i].status); > + > + /* test with very big buffer */ > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf, sizeof(buf), test_lznt[i].compressed, > + test_lznt[i].compressed_size, &final_size); > + ok(status == test_lznt[i].status, "%d: got wrong status 0x%08x\n", i, status); > + if (!status) > + { > + ok(final_size == test_lznt[i].uncompressed_size, > + "%d: got wrong final_size %d\n", i, final_size); > + ok(!memcmp(buf, test_lznt[i].uncompressed, test_lznt[i].uncompressed_size), > + "%d: got wrong decoded data\n", i); > + ok(buf[test_lznt[i].uncompressed_size] == 0x11, > + "%d: buf[%d] overwritten\n", i, test_lznt[i].uncompressed_size); > + } > + > + /* test that modifier for compression engine is ignored */ > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1 | COMPRESSION_ENGINE_MAXIMUM, buf, sizeof(buf), > + test_lznt[i].compressed, test_lznt[i].compressed_size, &final_size); > + ok(status == test_lznt[i].status, "%d: got wrong status 0x%08x\n", i, status); > + if (!status) > + { > + ok(final_size == test_lznt[i].uncompressed_size, > + "%d: got wrong final_size %d\n", i, final_size); > + ok(!memcmp(buf, test_lznt[i].uncompressed, test_lznt[i].uncompressed_size), > + "%d: got wrong decoded data\n", i); > + ok(buf[test_lznt[i].uncompressed_size] == 0x11, > + "%d: buf[%d] overwritten\n", i, test_lznt[i].uncompressed_size); > + } > + > + /* test with expected output size */ > + if (test_lznt[i].uncompressed_size > 0) > + { > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf, test_lznt[i].uncompressed_size, > + test_lznt[i].compressed, test_lznt[i].compressed_size, &final_size); > + ok(status == test_lznt[i].status, "%d: got wrong status 0x%08x\n", i, status); > + if (!status) > + { > + ok(final_size == test_lznt[i].uncompressed_size, > + "%d: got wrong final_size %d\n", i, final_size); > + ok(!memcmp(buf, test_lznt[i].uncompressed, test_lznt[i].uncompressed_size), > + "%d: got wrong decoded data\n", i); > + ok(buf[test_lznt[i].uncompressed_size] == 0x11, > + "%d: buf[%d] overwritten\n", i, test_lznt[i].uncompressed_size); > + } > + } > + > + /* test with smaller output size */ > + if (test_lznt[i].uncompressed_size > 1) > + { > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf, test_lznt[i].uncompressed_size - 1, > + test_lznt[i].compressed, test_lznt[i].compressed_size, &final_size); > + ok(status == test_lznt[i].status || broken(status == STATUS_BAD_COMPRESSION_BUFFER && > + (test_lznt[i].broken_flags & DECOMPRESS_BROKEN_TRUNCATED)), "%d: got wrong status 0x%08x\n", i, status); > + if (!status) > + { > + ok(final_size == test_lznt[i].uncompressed_size - 1, > + "%d: got wrong final_size %d\n", i, final_size); > + ok(!memcmp(buf, test_lznt[i].uncompressed, test_lznt[i].uncompressed_size - 1), > + "%d: got wrong decoded data\n", i); > + ok(buf[test_lznt[i].uncompressed_size - 1] == 0x11, > + "%d: buf[%d] overwritten\n", i, test_lznt[i].uncompressed_size - 1); > + } > + } > + > + /* test with zero output size */ > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf, 0, test_lznt[i].compressed, > + test_lznt[i].compressed_size, &final_size); > + if (is_incomplete_chunk(test_lznt[i].compressed, test_lznt[i].compressed_size, FALSE)) > + { > + ok(status == STATUS_BAD_COMPRESSION_BUFFER, "%d: got wrong status 0x%08x\n", i, status); > + } > + else > + { > + ok(status == STATUS_SUCCESS, "%d: got wrong status 0x%08x\n", i, status); > + ok(final_size == 0, "%d: got wrong final_size %d\n", i, final_size); > + ok(buf[0] == 0x11, "%d: buf[%d] overwritten\n", i, test_lznt[i].uncompressed_size); > + } > + > + /* test RtlDecompressBuffer with offset = 0 */ > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, sizeof(buf), test_lznt[i].compressed, > + test_lznt[i].compressed_size, 0, &final_size, workspace); > + ok(status == test_lznt[i].status || broken(status == STATUS_BAD_COMPRESSION_BUFFER && > + (test_lznt[i].broken_flags & DECOMPRESS_BROKEN_FRAGMENT0)), "%d: got wrong status 0x%08x\n", i, status); > + if (!status) > + { > + ok(final_size == test_lznt[i].uncompressed_size, > + "%d: got wrong final_size %d\n", i, final_size); > + ok(!memcmp(buf, test_lznt[i].uncompressed, test_lznt[i].uncompressed_size), > + "%d: got wrong decoded data\n", i); > + ok(buf[test_lznt[i].uncompressed_size] == 0x11, > + "%d: buf[%d] overwritten\n", i, test_lznt[i].uncompressed_size); > + } > + > + /* test RtlDecompressBuffer with offset = 1 */ > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, sizeof(buf), test_lznt[i].compressed, > + test_lznt[i].compressed_size, 1, &final_size, workspace); > + ok(status == test_lznt[i].status || broken(status == STATUS_BAD_COMPRESSION_BUFFER && > + (test_lznt[i].broken_flags & DECOMPRESS_BROKEN_FRAGMENT1)), "%d: got wrong status 0x%08x\n", i, status); > + if (!status) > + { > + if (test_lznt[i].uncompressed_size == 0) > + { > + todo_wine > + ok(final_size == 4095, > + "%d: got wrong final size %d\n", i, final_size); > + /* Buffer doesn't contain any useful value on Windows */ > + ok(buf[4095] == 0x11, > + "%d: buf[4095] overwritten\n", i); > + } > + else > + { > + ok(final_size == test_lznt[i].uncompressed_size - 1, > + "%d: got wrong final_size %d\n", i, final_size); > + ok(!memcmp(buf, test_lznt[i].uncompressed + 1, test_lznt[i].uncompressed_size - 1), > + "%d: got wrong decoded data\n", i); > + ok(buf[test_lznt[i].uncompressed_size - 1] == 0x11, > + "%d: buf[%d] overwritten\n", i, test_lznt[i].uncompressed_size - 1); > + } > + } > + > + /* test RtlDecompressBuffer with offset = 4095 */ > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, sizeof(buf), test_lznt[i].compressed, > + test_lznt[i].compressed_size, 4095, &final_size, workspace); > + ok(status == test_lznt[i].status || broken(status == STATUS_BAD_COMPRESSION_BUFFER && > + (test_lznt[i].broken_flags & DECOMPRESS_BROKEN_FRAGMENT4095)), "%d: got wrong status 0x%08x\n", i, status); > + if (!status) > + { > + todo_wine > + ok(final_size == 1, > + "%d: got wrong final size %d\n", i, final_size); > + todo_wine > + ok(buf[0] == 0, > + "%d: padding is not zero\n", i); > + ok(buf[1] == 0x11, > + "%d: buf[1] overwritten\n", i); > + } > + > + /* test RtlDecompressBuffer with offset = 4096 */ > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, sizeof(buf), test_lznt[i].compressed, > + test_lznt[i].compressed_size, 4096, &final_size, workspace); > + expected_status = is_incomplete_chunk(test_lznt[i].compressed, test_lznt[i].compressed_size, TRUE) ? > + test_lznt[i].status : STATUS_SUCCESS; > + ok(status == expected_status, "%d: got wrong status 0x%08x, expected 0x%08x\n", i, status, expected_status); > + if (!status) > + { > + ok(final_size == 0, > + "%d: got wrong final size %d\n", i, final_size); > + ok(buf[0] == 0x11, > + "%d: buf[4096] overwritten\n", i); > + } > + } > + > + /* test decoding of multiple chunks with pRtlDecompressBuffer */ > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf, sizeof(buf), test_multiple_chunks, > + sizeof(test_multiple_chunks), &final_size); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 4100, "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf, "Wine", 4), "got wrong decoded data at offset 0\n"); > + ok(buf[4] == 0 && buf[4095] == 0, "padding is not zero\n"); > + ok(!memcmp(buf + 4096, "Wine", 4), "got wrong decoded data at offset 4096\n"); > + ok(buf[4100] == 0x11, "buf[4100] overwritten\n"); > + } > + > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf, 4097, test_multiple_chunks, > + sizeof(test_multiple_chunks), &final_size); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 4097, "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf, "Wine", 4), "got wrong decoded data at offset 0\n"); > + ok(buf[4] == 0 && buf[4095] == 0, "padding is not zero\n"); > + ok(buf[4096], "got wrong decoded data at offset 4096\n"); > + ok(buf[4097] == 0x11, "buf[4097] overwritten\n"); > + } > + > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf, 4096, test_multiple_chunks, > + sizeof(test_multiple_chunks), &final_size); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 4, "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf, "Wine", 4), "got wrong decoded data at offset 0\n"); > + ok(buf[4] == 0x11, "buf[4] overwritten\n"); > + } > + > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf, 4, test_multiple_chunks, > + sizeof(test_multiple_chunks), &final_size); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 4, "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf, "Wine", 4), "got wrong decoded data at offset 0\n"); > + ok(buf[4] == 0x11, "buf[4] overwritten\n"); > + } > + > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf, 3, test_multiple_chunks, > + sizeof(test_multiple_chunks), &final_size); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 3, "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf, "Wine", 3), "got wrong decoded data at offset 0\n"); > + ok(buf[3] == 0x11, "buf[3] overwritten\n"); > + } > + > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf, 0, test_multiple_chunks, > + sizeof(test_multiple_chunks), &final_size); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 0, "got wrong final_size %d\n", final_size); > + ok(buf[0] == 0x11, "buf[0] overwritten\n"); > + } > + > + /* test multiple chunks in combination with RtlDecompressBuffer and offset=1 */ > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, sizeof(buf), test_multiple_chunks, > + sizeof(test_multiple_chunks), 1, &final_size, workspace); > + ok(status == STATUS_SUCCESS || broken(status == STATUS_BAD_COMPRESSION_BUFFER), > + "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 4099, "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf, "ine", 3), "got wrong decoded data at offset 0\n"); > + ok(buf[3] == 0 && buf[4094] == 0, "padding is not zero\n"); > + ok(!memcmp(buf + 4095, "Wine", 4), "got wrong decoded data at offset 4095\n"); > + ok(buf[4099] == 0x11, "buf[4099] overwritten\n"); > + } > + > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, 4096, test_multiple_chunks, > + sizeof(test_multiple_chunks), 1, &final_size, workspace); > + ok(status == STATUS_SUCCESS || broken(status == STATUS_BAD_COMPRESSION_BUFFER), > + "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 4096, "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf, "ine", 3), "got wrong decoded data at offset 0\n"); > + ok(buf[3] == 0 && buf[4094] == 0, "padding is not zero\n"); > + ok(buf[4095] == 'W', "got wrong decoded data at offset 4095\n"); > + ok(buf[4096] == 0x11, "buf[4096] overwritten\n"); > + } > + > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, 4095, test_multiple_chunks, > + sizeof(test_multiple_chunks), 1, &final_size, workspace); > + ok(status == STATUS_SUCCESS || broken(status == STATUS_BAD_COMPRESSION_BUFFER), > + "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 3, "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf, "ine", 3), "got wrong decoded data at offset 0\n"); > + ok(buf[4] == 0x11, "buf[4] overwritten\n"); > + } > + > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, 3, test_multiple_chunks, > + sizeof(test_multiple_chunks), 1, &final_size, workspace); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 3, "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf, "ine", 3), "got wrong decoded data at offset 0\n"); > + ok(buf[3] == 0x11, "buf[3] overwritten\n"); > + } > + > + /* test multiple chunks in combination with RtlDecompressBuffer and offset=4 */ > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, sizeof(buf), test_multiple_chunks, > + sizeof(test_multiple_chunks), 4, &final_size, workspace); > + ok(status == STATUS_SUCCESS || broken(status == STATUS_BAD_COMPRESSION_BUFFER), > + "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 4096, "got wrong final_size %d\n", final_size); > + ok(buf[0] == 0 && buf[4091] == 0, "padding is not zero\n"); > + ok(!memcmp(buf + 4092, "Wine", 4), "got wrong decoded data at offset 4092\n"); > + ok(buf[4096] == 0x11, "buf[4096] overwritten\n"); > + } > + > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, sizeof(buf), test_multiple_chunks, > + sizeof(test_multiple_chunks), 4095, &final_size, workspace); > + ok(status == STATUS_SUCCESS || broken(status == STATUS_BAD_COMPRESSION_BUFFER), > + "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 5, "got wrong final_size %d\n", final_size); > + ok(buf[0] == 0, "padding is not zero\n"); > + ok(!memcmp(buf + 1, "Wine", 4), "got wrong decoded data at offset 1\n"); > + ok(buf[5] == 0x11, "buf[5] overwritten\n"); > + } > + > + final_size = 0xdeadbeef; > + memset(buf, 0x11, sizeof(buf)); > + status = pRtlDecompressFragment(COMPRESSION_FORMAT_LZNT1, buf, sizeof(buf), test_multiple_chunks, > + sizeof(test_multiple_chunks), 4096, &final_size, workspace); > + ok(status == STATUS_SUCCESS || broken(status == STATUS_BAD_COMPRESSION_BUFFER), > + "got wrong status 0x%08x\n", status); > + if (!status) > + { > + ok(final_size == 4, "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf, "Wine", 4), "got wrong decoded data at offset 0\n"); > + ok(buf[4] == 0x11, "buf[4] overwritten\n"); > + } > + > +} > + > +static void test_RtlCompressBuffer(void) > +{ > + ULONG compress_workspace, decompress_workspace; > + static const UCHAR test_buffer[] = "WineWineWine"; > + static UCHAR buf1[0x1000], buf2[0x1000], *workspace; > + ULONG final_size, buf_size; > + NTSTATUS status; > + > + if (!pRtlCompressBuffer || !pRtlGetCompressionWorkSpaceSize) > + { > + win_skip("RtlCompressBuffer or RtlGetCompressionWorkSpaceSize is not available\n"); > + return; > + } > + > + compress_workspace = decompress_workspace = 0xdeadbeef; > + status = pRtlGetCompressionWorkSpaceSize(COMPRESSION_FORMAT_LZNT1, &compress_workspace, > + &decompress_workspace); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + ok(compress_workspace != 0, "got wrong compress_workspace %d\n", compress_workspace); > + > + workspace = HeapAlloc( GetProcessHeap(), 0, compress_workspace ); > + ok(workspace != NULL, "HeapAlloc failed %x\n", GetLastError()); > + > + /* test compression format / engine */ > + final_size = 0xdeadbeef; > + status = pRtlCompressBuffer(COMPRESSION_FORMAT_NONE, test_buffer, sizeof(test_buffer), > + buf1, sizeof(buf1) - 1, 4096, &final_size, workspace); > + ok(status == STATUS_INVALID_PARAMETER, "got wrong status 0x%08x\n", status); > + ok(final_size == 0xdeadbeef, "got wrong final_size %d\n", final_size); > + > + final_size = 0xdeadbeef; > + status = pRtlCompressBuffer(COMPRESSION_FORMAT_DEFAULT, test_buffer, sizeof(test_buffer), > + buf1, sizeof(buf1) - 1, 4096, &final_size, workspace); > + ok(status == STATUS_INVALID_PARAMETER, "got wrong status 0x%08x\n", status); > + ok(final_size == 0xdeadbeef, "got wrong final_size %d\n", final_size); > + > + final_size = 0xdeadbeef; > + status = pRtlCompressBuffer(0xFF, test_buffer, sizeof(test_buffer), > + buf1, sizeof(buf1) - 1, 4096, &final_size, workspace); > + ok(status == STATUS_UNSUPPORTED_COMPRESSION, "got wrong status 0x%08x\n", status); > + ok(final_size == 0xdeadbeef, "got wrong final_size %d\n", final_size); > + > + /* test compression */ > + final_size = 0xdeadbeef; > + memset(buf1, 0x11, sizeof(buf1)); > + status = pRtlCompressBuffer(COMPRESSION_FORMAT_LZNT1, test_buffer, sizeof(test_buffer), > + buf1, sizeof(buf1), 4096, &final_size, workspace); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + ok((*(WORD *)buf1 & 0x7000) == 0x3000, "no chunk signature found %04x\n", *(WORD *)buf1); > + buf_size = final_size; > + todo_wine > + ok(final_size < sizeof(test_buffer), "got wrong final_size %d\n", final_size); > + > + /* test decompression */ > + final_size = 0xdeadbeef; > + memset(buf2, 0x11, sizeof(buf2)); > + status = pRtlDecompressBuffer(COMPRESSION_FORMAT_LZNT1, buf2, sizeof(buf2), > + buf1, buf_size, &final_size); > + ok(status == STATUS_SUCCESS, "got wrong status 0x%08x\n", status); > + ok(final_size == sizeof(test_buffer), "got wrong final_size %d\n", final_size); > + ok(!memcmp(buf2, test_buffer, sizeof(test_buffer)), "got wrong decoded data\n"); > + ok(buf2[sizeof(test_buffer)] == 0x11, "buf[%u] overwritten\n", (DWORD)sizeof(test_buffer)); > + > + /* buffer too small */ > + final_size = 0xdeadbeef; > + memset(buf1, 0x11, sizeof(buf1)); > + status = pRtlCompressBuffer(COMPRESSION_FORMAT_LZNT1, test_buffer, sizeof(test_buffer), > + buf1, 4, 4096, &final_size, workspace); > + ok(status == STATUS_BUFFER_TOO_SMALL, "got wrong status 0x%08x\n", status); > + > + HeapFree(GetProcessHeap(), 0, workspace); > +} > + > START_TEST(rtl) > { > InitFunctionPtrs(); > + > +#ifdef __i386__ > + test_WinSqm(); > +#else > + skip("stdcall-style parameter checks are not supported on this platform.\n"); > +#endif > > test_RtlCompareMemory(); > test_RtlCompareMemoryUlong(); > @@ -1623,6 +2500,10 @@ > test_RtlIpv4AddressToString(); > test_RtlIpv4AddressToStringEx(); > test_RtlIpv4StringToAddress(); > + test_RtlIpv4StringToAddressEx(); > test_LdrAddRefDll(); > test_LdrLockLoaderLock(); > -} > + test_RtlGetCompressionWorkSpaceSize(); > + test_RtlDecompressBuffer(); > + test_RtlCompressBuffer(); > +} > > Modified: trunk/rostests/winetests/ntdll/string.c > URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/ntdll/string.c?… > ============================================================================== > --- trunk/rostests/winetests/ntdll/string.c [iso-8859-1] (original) > +++ trunk/rostests/winetests/ntdll/string.c [iso-8859-1] Tue Mar 10 10:10:04 2015 > @@ -1083,8 +1083,8 @@ > result = p_atoi64(str2longlong[test_num].str); > if (str2longlong[test_num].overflow) > ok(result == str2longlong[test_num].value || > - (result == (str2longlong[test_num].overflow == -1) ? > - ULL(0x80000000,0x00000000) : ULL(0x7fffffff,0xffffffff)), > + (result == ((str2longlong[test_num].overflow == -1) ? > + ULL(0x80000000,0x00000000) : ULL(0x7fffffff,0xffffffff))), > "(test %d): call failed: _atoi64(\"%s\") has result 0x%x%08x, expected: 0x%x%08x\n", > test_num, str2longlong[test_num].str, (DWORD)(result >> 32), (DWORD)result, > (DWORD)(str2longlong[test_num].value >> 32), (DWORD)str2longlong[test_num].value); > @@ -1108,8 +1108,8 @@ > result = p_wtoi64(uni.Buffer); > if (str2longlong[test_num].overflow) > ok(result == str2longlong[test_num].value || > - (result == (str2longlong[test_num].overflow == -1) ? > - ULL(0x80000000,0x00000000) : ULL(0x7fffffff,0xffffffff)), > + (result == ((str2longlong[test_num].overflow == -1) ? > + ULL(0x80000000,0x00000000) : ULL(0x7fffffff,0xffffffff))), > "(test %d): call failed: _atoi64(\"%s\") has result 0x%x%08x, expected: 0x%x%08x\n", > test_num, str2longlong[test_num].str, (DWORD)(result >> 32), (DWORD)result, > (DWORD)(str2longlong[test_num].value >> 32), (DWORD)str2longlong[test_num].value); > >

1 0

Re: [ros-dev] [ros-diffs] [tkreuzer] 66636: [DDK/XDK] - Fix some prototypes in wdm.h (PMDLX -> PMDL) - Fix annotations for ExFreePool(WithTag)
by Thomas Faber 10 Mar '15

10 Mar '15

On 2015-03-10 01:09, tkreuzer(a)svn.reactos.org wrote: > --- trunk/reactos/include/xdk/exfuncs.h [iso-8859-1] (original) > +++ trunk/reactos/include/xdk/exfuncs.h [iso-8859-1] Tue Mar 10 00:09:20 2015 > @@ -606,14 +606,14 @@ > VOID > NTAPI > ExFreePool( > - _In_ __drv_freesMem(Mem) PVOID P); > + _Pre_not __drv_freesMem(Mem) PVOID P); > > _IRQL_requires_max_(DISPATCH_LEVEL) > NTKERNELAPI > VOID > NTAPI > ExFreePoolWithTag( > - _In_ __drv_freesMem(Mem) PVOID P, > + _Pre_not __drv_freesMem(Mem) PVOID P, > _In_ ULONG Tag); > > _IRQL_requires_max_(DISPATCH_LEVEL) Are these typos (missing "null_")?

1 0

Undocumented interesting WinDBG commands
by Hermès BÉLUSCA - MAÏTO 09 Mar '15

09 Mar '15

Hi ! For your information, here are some undocumented but nevertheless interesting WinDBG commands: http://blogs.msdn.com/b/reiley/archive/2011/10/30/undocumented-windbg.aspx Cheers, Hermès.

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-dev