Ros-diffs August 2015

ros-diffs@reactos.org

15 participants
301 discussions

[bfreisen] 68685: [TOOLS][HHPCOMP] Initial commit of hhpcomp, our new HTML Help Project (*.hhp) compiler
by bfreisen＠svn.reactos.org 11 Aug '15

11 Aug '15

Author: bfreisen Date: Tue Aug 11 13:37:36 2015 New Revision: 68685 URL: http://svn.reactos.org/svn/reactos?rev=68685&view=rev Log: [TOOLS][HHPCOMP] Initial commit of hhpcomp, our new HTML Help Project (*.hhp) compiler Added: trunk/reactos/tools/hhpcomp/ trunk/reactos/tools/hhpcomp/CMakeLists.txt (with props) trunk/reactos/tools/hhpcomp/COPYING (with props) trunk/reactos/tools/hhpcomp/chmc/ trunk/reactos/tools/hhpcomp/chmc/chm.h (with props) trunk/reactos/tools/hhpcomp/chmc/chmc.c (with props) trunk/reactos/tools/hhpcomp/chmc/chmc.h (with props) trunk/reactos/tools/hhpcomp/chmc/encint.h (with props) trunk/reactos/tools/hhpcomp/chmc/err.c (with props) trunk/reactos/tools/hhpcomp/chmc/err.h (with props) trunk/reactos/tools/hhpcomp/chmc/list.h (with props) trunk/reactos/tools/hhpcomp/hhp_reader.cpp (with props) trunk/reactos/tools/hhpcomp/hhp_reader.h (with props) trunk/reactos/tools/hhpcomp/hhpcomp.cpp (with props) trunk/reactos/tools/hhpcomp/lzx_compress/ trunk/reactos/tools/hhpcomp/lzx_compress/ChangeLog (with props) trunk/reactos/tools/hhpcomp/lzx_compress/lz_nonslide.c (with props) trunk/reactos/tools/hhpcomp/lzx_compress/lz_nonslide.h (with props) trunk/reactos/tools/hhpcomp/lzx_compress/lzx_compress.h (with props) trunk/reactos/tools/hhpcomp/lzx_compress/lzx_config.h (with props) trunk/reactos/tools/hhpcomp/lzx_compress/lzx_constants.h (with props) trunk/reactos/tools/hhpcomp/lzx_compress/lzx_layer.c (with props) trunk/reactos/tools/hhpcomp/utils.cpp (with props) trunk/reactos/tools/hhpcomp/utils.h (with props) Modified: trunk/reactos/tools/CMakeLists.txt [This mail would be too long, it was shortened to contain the URLs only.] Modified: trunk/reactos/tools/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/CMakeLists.txt?rev=6… Added: trunk/reactos/tools/hhpcomp/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/CMakeLists.t… Added: trunk/reactos/tools/hhpcomp/COPYING URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/COPYING?rev=… Added: trunk/reactos/tools/hhpcomp/chmc/chm.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/chmc/chm.h?r… Added: trunk/reactos/tools/hhpcomp/chmc/chmc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/chmc/chmc.c?… Added: trunk/reactos/tools/hhpcomp/chmc/chmc.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/chmc/chmc.h?… Added: trunk/reactos/tools/hhpcomp/chmc/encint.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/chmc/encint.… Added: trunk/reactos/tools/hhpcomp/chmc/err.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/chmc/err.c?r… Added: trunk/reactos/tools/hhpcomp/chmc/err.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/chmc/err.h?r… Added: trunk/reactos/tools/hhpcomp/chmc/list.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/chmc/list.h?… Added: trunk/reactos/tools/hhpcomp/hhp_reader.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/hhp_reader.c… Added: trunk/reactos/tools/hhpcomp/hhp_reader.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/hhp_reader.h… Added: trunk/reactos/tools/hhpcomp/hhpcomp.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/hhpcomp.cpp?… Added: trunk/reactos/tools/hhpcomp/lzx_compress/ChangeLog URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/lzx_compress… Added: trunk/reactos/tools/hhpcomp/lzx_compress/lz_nonslide.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/lzx_compress… Added: trunk/reactos/tools/hhpcomp/lzx_compress/lz_nonslide.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/lzx_compress… Added: trunk/reactos/tools/hhpcomp/lzx_compress/lzx_compress.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/lzx_compress… Added: trunk/reactos/tools/hhpcomp/lzx_compress/lzx_config.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/lzx_compress… Added: trunk/reactos/tools/hhpcomp/lzx_compress/lzx_constants.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/lzx_compress… Added: trunk/reactos/tools/hhpcomp/lzx_compress/lzx_layer.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/lzx_compress… Added: trunk/reactos/tools/hhpcomp/utils.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/utils.cpp?re… Added: trunk/reactos/tools/hhpcomp/utils.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/hhpcomp/utils.h?rev=…

1 0

[cwittich] 68684: [WIN32SS] %wZ expects the address of the string
by cwittich＠svn.reactos.org 11 Aug '15

11 Aug '15

Author: cwittich Date: Tue Aug 11 10:49:54 2015 New Revision: 68684 URL: http://svn.reactos.org/svn/reactos?rev=68684&view=rev Log: [WIN32SS] %wZ expects the address of the string Modified: trunk/reactos/win32ss/user/ntuser/class.c Modified: trunk/reactos/win32ss/user/ntuser/class.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/class.… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/class.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/class.c [iso-8859-1] Tue Aug 11 10:49:54 2015 @@ -2657,7 +2657,7 @@ // If null instance use client. if (!hInstance) hInstance = hModClient; - TRACE("GetClassInfo(%wZ, %p)\n", SafeClassName, hInstance); + TRACE("GetClassInfo(%wZ, %p)\n", &SafeClassName, hInstance); /* NOTE: Need exclusive lock because getting the wndproc might require the creation of a call procedure handle */

1 0

[dreimer] 68683: [MAIN] Mouse wheel new icon by Henry Tang Ih CORE-9590 #resolve #comment Committed, thx ^^
by dreimer＠svn.reactos.org 11 Aug '15

11 Aug '15

Author: dreimer Date: Tue Aug 11 10:36:49 2015 New Revision: 68683 URL: http://svn.reactos.org/svn/reactos?rev=68683&view=rev Log: [MAIN] Mouse wheel new icon by Henry Tang Ih CORE-9590 #resolve #comment Committed, thx ^^ Modified: trunk/reactos/dll/cpl/main/resources/mouse_wheel.ico Modified: trunk/reactos/dll/cpl/main/resources/mouse_wheel.ico URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/cpl/main/resources/mou… ============================================================================== Binary files - no diff available.

1 0

[dreimer] 68682: [FONTS] Update DejaVu fonts to v2.35.
by dreimer＠svn.reactos.org 11 Aug '15

11 Aug '15

Author: dreimer Date: Tue Aug 11 10:22:58 2015 New Revision: 68682 URL: http://svn.reactos.org/svn/reactos?rev=68682&view=rev Log: [FONTS] Update DejaVu fonts to v2.35. Modified: trunk/reactos/media/doc/3rd Party Files.txt trunk/reactos/media/fonts/DejaVuSans-Bold.ttf trunk/reactos/media/fonts/DejaVuSans-BoldOblique.ttf trunk/reactos/media/fonts/DejaVuSans-ExtraLight.ttf trunk/reactos/media/fonts/DejaVuSans-Oblique.ttf trunk/reactos/media/fonts/DejaVuSans.ttf trunk/reactos/media/fonts/DejaVuSansCondensed-Bold.ttf trunk/reactos/media/fonts/DejaVuSansCondensed-BoldOblique.ttf trunk/reactos/media/fonts/DejaVuSansCondensed-Oblique.ttf trunk/reactos/media/fonts/DejaVuSansCondensed.ttf trunk/reactos/media/fonts/DejaVuSansMono-Bold.ttf trunk/reactos/media/fonts/DejaVuSansMono-BoldOblique.ttf trunk/reactos/media/fonts/DejaVuSansMono-Oblique.ttf trunk/reactos/media/fonts/DejaVuSansMono.ttf trunk/reactos/media/fonts/DejaVuSerif-Bold.ttf trunk/reactos/media/fonts/DejaVuSerif-BoldItalic.ttf trunk/reactos/media/fonts/DejaVuSerif-Italic.ttf trunk/reactos/media/fonts/DejaVuSerif.ttf trunk/reactos/media/fonts/DejaVuSerifCondensed-Bold.ttf trunk/reactos/media/fonts/DejaVuSerifCondensed-BoldItalic.ttf trunk/reactos/media/fonts/DejaVuSerifCondensed-Italic.ttf trunk/reactos/media/fonts/DejaVuSerifCondensed.ttf trunk/reactos/media/fonts/doc/DejaVu/AUTHORS.txt trunk/reactos/media/fonts/doc/DejaVu/NEWS.txt trunk/reactos/media/fonts/doc/DejaVu/README.txt trunk/reactos/media/fonts/doc/DejaVu/langcover-sans.txt trunk/reactos/media/fonts/doc/DejaVu/langcover.txt trunk/reactos/media/fonts/doc/DejaVu/status.txt trunk/reactos/media/fonts/doc/DejaVu/unicover-sans.txt trunk/reactos/media/fonts/doc/DejaVu/unicover.txt [This mail would be too long, it was shortened to contain the URLs only.] Modified: trunk/reactos/media/doc/3rd Party Files.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/doc/3rd%20Party%20Fi… Modified: trunk/reactos/media/fonts/DejaVuSans-Bold.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSans-Bol… Modified: trunk/reactos/media/fonts/DejaVuSans-BoldOblique.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSans-Bol… Modified: trunk/reactos/media/fonts/DejaVuSans-ExtraLight.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSans-Ext… Modified: trunk/reactos/media/fonts/DejaVuSans-Oblique.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSans-Obl… Modified: trunk/reactos/media/fonts/DejaVuSans.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSans.ttf… Modified: trunk/reactos/media/fonts/DejaVuSansCondensed-Bold.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSansCond… Modified: trunk/reactos/media/fonts/DejaVuSansCondensed-BoldOblique.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSansCond… Modified: trunk/reactos/media/fonts/DejaVuSansCondensed-Oblique.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSansCond… Modified: trunk/reactos/media/fonts/DejaVuSansCondensed.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSansCond… Modified: trunk/reactos/media/fonts/DejaVuSansMono-Bold.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSansMono… Modified: trunk/reactos/media/fonts/DejaVuSansMono-BoldOblique.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSansMono… Modified: trunk/reactos/media/fonts/DejaVuSansMono-Oblique.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSansMono… Modified: trunk/reactos/media/fonts/DejaVuSansMono.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSansMono… Modified: trunk/reactos/media/fonts/DejaVuSerif-Bold.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSerif-Bo… Modified: trunk/reactos/media/fonts/DejaVuSerif-BoldItalic.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSerif-Bo… Modified: trunk/reactos/media/fonts/DejaVuSerif-Italic.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSerif-It… Modified: trunk/reactos/media/fonts/DejaVuSerif.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSerif.tt… Modified: trunk/reactos/media/fonts/DejaVuSerifCondensed-Bold.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSerifCon… Modified: trunk/reactos/media/fonts/DejaVuSerifCondensed-BoldItalic.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSerifCon… Modified: trunk/reactos/media/fonts/DejaVuSerifCondensed-Italic.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSerifCon… Modified: trunk/reactos/media/fonts/DejaVuSerifCondensed.ttf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/DejaVuSerifCon… Modified: trunk/reactos/media/fonts/doc/DejaVu/AUTHORS.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/doc/DejaVu/AUT… Modified: trunk/reactos/media/fonts/doc/DejaVu/NEWS.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/doc/DejaVu/NEW… Modified: trunk/reactos/media/fonts/doc/DejaVu/README.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/doc/DejaVu/REA… Modified: trunk/reactos/media/fonts/doc/DejaVu/langcover-sans.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/doc/DejaVu/lan… Modified: trunk/reactos/media/fonts/doc/DejaVu/langcover.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/doc/DejaVu/lan… Modified: trunk/reactos/media/fonts/doc/DejaVu/status.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/doc/DejaVu/sta… Modified: trunk/reactos/media/fonts/doc/DejaVu/unicover-sans.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/doc/DejaVu/uni… Modified: trunk/reactos/media/fonts/doc/DejaVu/unicover.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/fonts/doc/DejaVu/uni…

1 0

[cwittich] 68681: [BROWSEUI] addendum to 68672
by cwittich＠svn.reactos.org 11 Aug '15

11 Aug '15

Author: cwittich Date: Tue Aug 11 09:17:07 2015 New Revision: 68681 URL: http://svn.reactos.org/svn/reactos?rev=68681&view=rev Log: [BROWSEUI] addendum to 68672 Modified: trunk/reactos/dll/win32/browseui/addressband.cpp Modified: trunk/reactos/dll/win32/browseui/addressband.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/browseui/address… ============================================================================== --- trunk/reactos/dll/win32/browseui/addressband.cpp [iso-8859-1] (original) +++ trunk/reactos/dll/win32/browseui/addressband.cpp [iso-8859-1] Tue Aug 11 09:17:07 2015 @@ -95,7 +95,7 @@ if (pdbi->dwMask & DBIM_TITLE) { if (!LoadStringW(_AtlBaseModule.GetResourceInstance(), IDS_ADDRESSBANDLABEL, pdbi->wszTitle, _countof(pdbi->wszTitle))) - return E_FAIL; + return HRESULT_FROM_WIN32(GetLastError()); } if (pdbi->dwMask & DBIM_MODEFLAGS)

1 0

[cwittich] 68680: [MSAFD] addendum to 68673
by cwittich＠svn.reactos.org 11 Aug '15

11 Aug '15

Author: cwittich Date: Tue Aug 11 09:05:45 2015 New Revision: 68680 URL: http://svn.reactos.org/svn/reactos?rev=68680&view=rev Log: [MSAFD] addendum to 68673 Modified: trunk/reactos/dll/win32/msafd/misc/helpers.c Modified: trunk/reactos/dll/win32/msafd/misc/helpers.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/msafd/misc/helpe… ============================================================================== --- trunk/reactos/dll/win32/msafd/misc/helpers.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/msafd/misc/helpers.c [iso-8859-1] Tue Aug 11 09:05:45 2015 @@ -40,8 +40,8 @@ PLIST_ENTRY Helpers; INT Status; - TRACE("AddressFamily %p, SocketType %p, Protocol %p, Group %u, Flags %lx, HelperDllContext %p, HeplperDllData %p, Events %p\n", - AddressFamily, SocketType, Protocol, Group, Flags, HelperDllContext, HelperDllData, Events); + TRACE("AddressFamily %p, SocketType %p, Protocol %p, Group %u, Flags %lx, TransportName %p, HelperDllContext %p, HelperDllData %p, Events %p\n", + AddressFamily, SocketType, Protocol, Group, Flags, TransportName, HelperDllContext, HelperDllData, Events); /* Check in our Current Loaded Helpers */ for (Helpers = SockHelpersListHead.Flink;

1 0

[tfaber] 68679: [CMAKE] - Remove hackfix for VLC player, relocating wsock32.dll works fine now. Reverts r61898. CORE-8384 #resolve
by tfaber＠svn.reactos.org 11 Aug '15

11 Aug '15

Author: tfaber Date: Tue Aug 11 09:02:25 2015 New Revision: 68679 URL: http://svn.reactos.org/svn/reactos?rev=68679&view=rev Log: [CMAKE] - Remove hackfix for VLC player, relocating wsock32.dll works fine now. Reverts r61898. CORE-8384 #resolve Modified: trunk/reactos/cmake/baseaddress.cmake Modified: trunk/reactos/cmake/baseaddress.cmake URL: http://svn.reactos.org/svn/reactos/trunk/reactos/cmake/baseaddress.cmake?re… ============================================================================== --- trunk/reactos/cmake/baseaddress.cmake [iso-8859-1] (original) +++ trunk/reactos/cmake/baseaddress.cmake [iso-8859-1] Tue Aug 11 09:02:25 2015 @@ -283,7 +283,7 @@ set(baseaddress_ws2_32_new 0x70790000) set(baseaddress_wshirda 0x70760000) set(baseaddress_wshtcpip 0x70730000) -set(baseaddress_wsock32 0x78700000) # FIXME: hackfix for VLC player +set(baseaddress_wsock32 0x70700000) set(baseaddress_wtsapi32 0x706d0000) set(baseaddress_wuapi 0x70690000) set(baseaddress_xinput1_1 0x70660000)

1 0

[tfaber] 68678: [NTOS:MM] - Ignore IMAGE_SCN_TYPE_NOLOAD when loading image sections, as shown by ntdll_apitest:NtMapViewOfSection. This avoids errors when trying to relocate .rossym sections (sinc...
by tfaber＠svn.reactos.org 11 Aug '15

11 Aug '15

Author: tfaber Date: Tue Aug 11 08:47:14 2015 New Revision: 68678 URL: http://svn.reactos.org/svn/reactos?rev=68678&view=rev Log: [NTOS:MM] - Ignore IMAGE_SCN_TYPE_NOLOAD when loading image sections, as shown by ntdll_apitest:NtMapViewOfSection. This avoids errors when trying to relocate .rossym sections (since they were not loaded but Ldr tried to change their protection) CORE-8384 Modified: trunk/reactos/ntoskrnl/mm/section.c Modified: trunk/reactos/ntoskrnl/mm/section.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/section.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/mm/section.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/section.c [iso-8859-1] Tue Aug 11 08:47:14 2015 @@ -4216,13 +4216,10 @@ * and calculate the image base address */ for (i = 0; i < NrSegments; i++) { - if (!(SectionSegments[i].Image.Characteristics & IMAGE_SCN_TYPE_NOLOAD)) + if (Segment == &SectionSegments[i]) { - if (Segment == &SectionSegments[i]) - { - ImageBaseAddress = (char*)BaseAddress - (ULONG_PTR)SectionSegments[i].Image.VirtualAddress; - break; - } + ImageBaseAddress = (char*)BaseAddress - (ULONG_PTR)SectionSegments[i].Image.VirtualAddress; + break; } } if (i >= NrSegments) @@ -4232,18 +4229,15 @@ for (i = 0; i < NrSegments; i++) { - if (!(SectionSegments[i].Image.Characteristics & IMAGE_SCN_TYPE_NOLOAD)) + PVOID SBaseAddress = (PVOID) + ((char*)ImageBaseAddress + (ULONG_PTR)SectionSegments[i].Image.VirtualAddress); + + Status = MmUnmapViewOfSegment(AddressSpace, SBaseAddress); + if (!NT_SUCCESS(Status)) { - PVOID SBaseAddress = (PVOID) - ((char*)ImageBaseAddress + (ULONG_PTR)SectionSegments[i].Image.VirtualAddress); - - Status = MmUnmapViewOfSegment(AddressSpace, SBaseAddress); - if (!NT_SUCCESS(Status)) - { - DPRINT1("MmUnmapViewOfSegment failed for %p (Process %p) with %lx\n", - SBaseAddress, Process, Status); - NT_ASSERT(NT_SUCCESS(Status)); - } + DPRINT1("MmUnmapViewOfSegment failed for %p (Process %p) with %lx\n", + SBaseAddress, Process, Status); + NT_ASSERT(NT_SUCCESS(Status)); } } } @@ -4520,13 +4514,10 @@ ImageSize = 0; for (i = 0; i < NrSegments; i++) { - if (!(SectionSegments[i].Image.Characteristics & IMAGE_SCN_TYPE_NOLOAD)) - { - ULONG_PTR MaxExtent; - MaxExtent = (ULONG_PTR)(SectionSegments[i].Image.VirtualAddress + - SectionSegments[i].Length.QuadPart); - ImageSize = max(ImageSize, MaxExtent); - } + ULONG_PTR MaxExtent; + MaxExtent = (ULONG_PTR)(SectionSegments[i].Image.VirtualAddress + + SectionSegments[i].Length.QuadPart); + ImageSize = max(ImageSize, MaxExtent); } ImageSectionObject->ImageInformation.ImageFileSize = (ULONG)ImageSize; @@ -4570,25 +4561,22 @@ for (i = 0; i < NrSegments; i++) { - if (!(SectionSegments[i].Image.Characteristics & IMAGE_SCN_TYPE_NOLOAD)) + PVOID SBaseAddress = (PVOID) + ((char*)ImageBase + (ULONG_PTR)SectionSegments[i].Image.VirtualAddress); + MmLockSectionSegment(&SectionSegments[i]); + Status = MmMapViewOfSegment(AddressSpace, + Section, + &SectionSegments[i], + &SBaseAddress, + SectionSegments[i].Length.LowPart, + SectionSegments[i].Protection, + 0, + 0); + MmUnlockSectionSegment(&SectionSegments[i]); + if (!NT_SUCCESS(Status)) { - PVOID SBaseAddress = (PVOID) - ((char*)ImageBase + (ULONG_PTR)SectionSegments[i].Image.VirtualAddress); - MmLockSectionSegment(&SectionSegments[i]); - Status = MmMapViewOfSegment(AddressSpace, - Section, - &SectionSegments[i], - &SBaseAddress, - SectionSegments[i].Length.LowPart, - SectionSegments[i].Protection, - 0, - 0); - MmUnlockSectionSegment(&SectionSegments[i]); - if (!NT_SUCCESS(Status)) - { - MmUnlockAddressSpace(AddressSpace); - return(Status); - } + MmUnlockAddressSpace(AddressSpace); + return(Status); } }

1 0

[tfaber] 68677: [NTOS:MM] - When paging in data for an image section, first flush outstanding cached data since paging reads are noncached. Fixes failure to create section in ntdll_apitest:NtMapVie...
by tfaber＠svn.reactos.org 11 Aug '15

11 Aug '15

Author: tfaber Date: Tue Aug 11 08:21:36 2015 New Revision: 68677 URL: http://svn.reactos.org/svn/reactos?rev=68677&view=rev Log: [NTOS:MM] - When paging in data for an image section, first flush outstanding cached data since paging reads are noncached. Fixes failure to create section in ntdll_apitest:NtMapViewOfSection/Test_NoLoadSection - Gracefully handle out-of-pool situation CORE-8384 Modified: trunk/reactos/ntoskrnl/mm/section.c Modified: trunk/reactos/ntoskrnl/mm/section.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/section.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/mm/section.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/section.c [iso-8859-1] Tue Aug 11 08:21:36 2015 @@ -3236,6 +3236,12 @@ BufferSize = Length + OffsetAdjustment; BufferSize = PAGE_ROUND_UP(BufferSize); + /* Flush data since we're about to perform a non-cached read */ + CcFlushCache(FileObject->SectionObjectPointer, + &FileOffset, + BufferSize, + &Iosb); + /* * It's ok to use paged pool, because this is a temporary buffer only used in * the loading of executables. The assumption is that MmCreateSection is @@ -3247,7 +3253,7 @@ 'rXmM'); if (!Buffer) { - KeBugCheck(MEMORY_MANAGEMENT); + return STATUS_INSUFFICIENT_RESOURCES; } UsedSize = 0;

1 0

[tfaber] 68676: [NTDLL_APITEST] - Add tests for mapping and relocating an image with an IMAGE_SCN_TYPE_NOLOAD section CORE-8384
by tfaber＠svn.reactos.org 11 Aug '15

11 Aug '15

Author: tfaber Date: Tue Aug 11 07:21:33 2015 New Revision: 68676 URL: http://svn.reactos.org/svn/reactos?rev=68676&view=rev Log: [NTDLL_APITEST] - Add tests for mapping and relocating an image with an IMAGE_SCN_TYPE_NOLOAD section CORE-8384 Modified: trunk/rostests/apitests/ntdll/NtMapViewOfSection.c Modified: trunk/rostests/apitests/ntdll/NtMapViewOfSection.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/ntdll/NtMapViewO… ============================================================================== --- trunk/rostests/apitests/ntdll/NtMapViewOfSection.c [iso-8859-1] (original) +++ trunk/rostests/apitests/ntdll/NtMapViewOfSection.c [iso-8859-1] Tue Aug 11 07:21:33 2015 @@ -3,6 +3,7 @@ * LICENSE: LGPLv2.1+ - See COPYING.LIB in the top level directory * PURPOSE: Test for NtMapViewOfSection * PROGRAMMERS: Timo Kreuzer + * Thomas Faber */ #include <apitest.h> @@ -1073,7 +1074,281 @@ ok(((ULONG_PTR)BaseAddress1 - (ULONG_PTR)BaseAddress2) == 0x10000, "Invalid addresses: BaseAddress1=%p, BaseAddress2=%p\n", BaseAddress1, BaseAddress2); #endif - +} + +#define BYTES4(x) x, x, x, x +#define BYTES8(x) BYTES4(x), BYTES4(x) +#define BYTES16(x) BYTES8(x), BYTES8(x) +#define BYTES32(x) BYTES16(x), BYTES16(x) +#define BYTES64(x) BYTES32(x), BYTES32(x) +#define BYTES128(x) BYTES64(x), BYTES64(x) +#define BYTES256(x) BYTES128(x), BYTES128(x) +#define BYTES512(x) BYTES256(x), BYTES256(x) +#define BYTES1024(x) BYTES512(x), BYTES512(x) + +static struct _MY_IMAGE_FILE +{ + IMAGE_DOS_HEADER doshdr; + WORD stub[32]; + IMAGE_NT_HEADERS32 nthdrs; + IMAGE_SECTION_HEADER text_header; + IMAGE_SECTION_HEADER rossym_header; + IMAGE_SECTION_HEADER rsrc_header; + BYTE pad[16]; + BYTE text_data[0x400]; + BYTE rossym_data[0x400]; + BYTE rsrc_data[0x400]; +} ImageFile = +{ + /* IMAGE_DOS_HEADER */ + { + IMAGE_DOS_SIGNATURE, 144, 3, 0, 4, 0, 0xFFFF, 0, 0xB8, 0, 0, 0, 0x40, + 0, { 0 }, 0, 0, { 0 }, 0x80 + }, + /* binary to print "This program cannot be run in DOS mode." */ + { + 0x1F0E, 0x0EBA, 0xB400, 0xCD09, 0xB821, 0x4C01, 0x21CD, 0x6854, 0x7369, + 0x7020, 0x6F72, 0x7267, 0x6D61, 0x6320, 0x6E61, 0x6F6E, 0x2074, 0x6562, + 0x7220, 0x6E75, 0x6920, 0x206E, 0x4F44, 0x2053, 0x6F6D, 0x6564, 0x0D2E, + 0x0A0D, 0x0024, 0x0000, 0x0000, 0x0000 + }, + /* IMAGE_NT_HEADERS32 */ + { + IMAGE_NT_SIGNATURE, /* Signature */ + /* IMAGE_FILE_HEADER */ + { + IMAGE_FILE_MACHINE_I386, /* Machine */ + 3, /* NumberOfSections */ + 0x47EFDF09, /* TimeDateStamp */ + 0, /* PointerToSymbolTable */ + 0, /* NumberOfSymbols */ + 0xE0, /* SizeOfOptionalHeader */ + IMAGE_FILE_32BIT_MACHINE | IMAGE_FILE_LOCAL_SYMS_STRIPPED | + IMAGE_FILE_LINE_NUMS_STRIPPED | IMAGE_FILE_EXECUTABLE_IMAGE | + IMAGE_FILE_DLL, /* Characteristics */ + }, + /* IMAGE_OPTIONAL_HEADER32 */ + { + IMAGE_NT_OPTIONAL_HDR32_MAGIC, /* Magic */ + 8, /* MajorLinkerVersion */ + 0, /* MinorLinkerVersion */ + 0x400, /* SizeOfCode */ + 0x000, /* SizeOfInitializedData */ + 0, /* SizeOfUninitializedData */ + 0x2000, /* AddressOfEntryPoint */ + 0x2000, /* BaseOfCode */ + 0x0000, /* BaseOfData */ + 0x400000, /* ImageBase */ + 0x2000, /* SectionAlignment */ + 0x200, /* FileAlignment */ + 4, /* MajorOperatingSystemVersion */ + 0, /* MinorOperatingSystemVersion */ + 0, /* MajorImageVersion */ + 0, /* MinorImageVersion */ + 4, /* MajorSubsystemVersion */ + 0, /* MinorSubsystemVersion */ + 0, /* Win32VersionValue */ + 0x8000, /* SizeOfImage */ + 0x200, /* SizeOfHeaders */ + 0x0, /* CheckSum */ + IMAGE_SUBSYSTEM_WINDOWS_CUI, /* Subsystem */ + IMAGE_DLLCHARACTERISTICS_DYNAMIC_BASE | + IMAGE_DLLCHARACTERISTICS_NO_SEH | + IMAGE_DLLCHARACTERISTICS_NX_COMPAT, /* DllCharacteristics */ + 0x100000, /* SizeOfStackReserve */ + 0x1000, /* SizeOfStackCommit */ + 0x100000, /* SizeOfHeapReserve */ + 0x1000, /* SizeOfHeapCommit */ + 0, /* LoaderFlags */ + 0x10, /* NumberOfRvaAndSizes */ + /* IMAGE_DATA_DIRECTORY */ + { + { 0 }, /* Export Table */ + { 0 }, /* Import Table */ + { 0 }, /* Resource Table */ + { 0 }, /* Exception Table */ + { 0 }, /* Certificate Table */ + { 0 }, /* Base Relocation Table */ + { 0 }, /* Debug */ + { 0 }, /* Copyright */ + { 0 }, /* Global Ptr */ + { 0 }, /* TLS Table */ + { 0 }, /* Load Config Table */ + { 0 }, /* Bound Import */ + { 0 }, /* IAT */ + { 0 }, /* Delay Import Descriptor */ + { 0 }, /* CLI Header */ + { 0 } /* Reserved */ + } + } + }, + /* IMAGE_SECTION_HEADER */ + { + ".text", /* Name */ + { 0x394 }, /* Misc.VirtualSize */ + 0x2000, /* VirtualAddress */ + 0x400, /* SizeOfRawData */ + 0x200, /* PointerToRawData */ + 0, /* PointerToRelocations */ + 0, /* PointerToLinenumbers */ + 0, /* NumberOfRelocations */ + 0, /* NumberOfLinenumbers */ + IMAGE_SCN_MEM_READ | IMAGE_SCN_MEM_EXECUTE | + IMAGE_SCN_CNT_CODE, /* Characteristics */ + }, + /* IMAGE_SECTION_HEADER */ + { + ".rossym", /* Name */ + { 0x100 }, /* Misc.VirtualSize */ + 0x4000, /* VirtualAddress */ + 0x400, /* SizeOfRawData */ + 0x600, /* PointerToRawData */ + 0, /* PointerToRelocations */ + 0, /* PointerToLinenumbers */ + 0, /* NumberOfRelocations */ + 0, /* NumberOfLinenumbers */ + IMAGE_SCN_MEM_READ | IMAGE_SCN_TYPE_NOLOAD, /* Characteristics */ + }, + /* IMAGE_SECTION_HEADER */ + { + ".rsrc", /* Name */ + { 0x100 }, /* Misc.VirtualSize */ + 0x6000, /* VirtualAddress */ + 0x400, /* SizeOfRawData */ + 0xA00, /* PointerToRawData */ + 0, /* PointerToRelocations */ + 0, /* PointerToLinenumbers */ + 0, /* NumberOfRelocations */ + 0, /* NumberOfLinenumbers */ + IMAGE_SCN_CNT_INITIALIZED_DATA | IMAGE_SCN_MEM_READ, /* Characteristics */ + }, + /* fill */ + { 0 }, + /* text */ + { 0xc3, 0 }, + /* rossym */ + { 0x01, 0x23, 0x45, 0x67, 0x89, 0xab, 0xcd, 0xef, + BYTES8(0xaa), + BYTES16(0xbb), + BYTES32(0xcc), + BYTES64(0xdd), + BYTES64(0xee), + BYTES64(0xff), + }, + /* rsrc */ + { 0 }, +}; + +C_ASSERT(FIELD_OFFSET(struct _MY_IMAGE_FILE, text_data) == 0x200); +C_ASSERT(FIELD_OFFSET(struct _MY_IMAGE_FILE, rossym_data) == 0x600); +C_ASSERT(FIELD_OFFSET(struct _MY_IMAGE_FILE, rsrc_data) == 0xa00); + +static +void +Test_NoLoadSection(BOOL Relocate) +{ + NTSTATUS Status; + WCHAR TempPath[MAX_PATH]; + WCHAR FileName[MAX_PATH]; + HANDLE Handle; + HANDLE SectionHandle; + LARGE_INTEGER SectionOffset; + PVOID BaseAddress; + SIZE_T ViewSize; + ULONG Written; + ULONG Length; + BOOL Success; + + Length = GetTempPathW(MAX_PATH, TempPath); + ok(Length != 0, "GetTempPathW failed with %lu\n", GetLastError()); + Length = GetTempFileNameW(TempPath, L"nta", 0, FileName); + ok(Length != 0, "GetTempFileNameW failed with %lu\n", GetLastError()); + Handle = CreateFileW(FileName, + FILE_ALL_ACCESS, + 0, + NULL, + CREATE_ALWAYS, + 0, + NULL); + if (Handle == INVALID_HANDLE_VALUE) + { + skip("Failed to create temp file %ls, error %lu\n", FileName, GetLastError()); + return; + } + if (Relocate) + { + ok((ULONG_PTR)GetModuleHandle(NULL) <= 0x80000000, "Module at %p\n", GetModuleHandle(NULL)); + ImageFile.nthdrs.OptionalHeader.ImageBase = (ULONG)(ULONG_PTR)GetModuleHandle(NULL); + } + else + { + ImageFile.nthdrs.OptionalHeader.ImageBase = 0xe400000; + } + + Success = WriteFile(Handle, + &ImageFile, + sizeof(ImageFile), + &Written, + NULL); + ok(Success == TRUE, "WriteFile failed with %lu\n", GetLastError()); + ok(Written == sizeof(ImageFile), "WriteFile wrote %lu bytes\n", Written); + + Status = NtCreateSection(&SectionHandle, + SECTION_ALL_ACCESS, + NULL, + NULL, + PAGE_EXECUTE_READWRITE, + SEC_IMAGE, + Handle); + ok_ntstatus(Status, STATUS_SUCCESS); + + if (NT_SUCCESS(Status)) + { + /* Map the section with */ + BaseAddress = NULL; + SectionOffset.QuadPart = 0; + ViewSize = 0; + Status = NtMapViewOfSection(SectionHandle, + NtCurrentProcess(), + &BaseAddress, + 0, + 0, + &SectionOffset, + &ViewSize, + ViewShare, + 0, + PAGE_READWRITE); + if (Relocate) + ok_ntstatus(Status, STATUS_IMAGE_NOT_AT_BASE); + else + ok_ntstatus(Status, STATUS_SUCCESS); + if (NT_SUCCESS(Status)) + { + PUCHAR Bytes = BaseAddress; +#define TEST_BYTE(n, v) StartSeh() ok_hex(Bytes[n], v); EndSeh(STATUS_SUCCESS); + TEST_BYTE(0x2000, 0xc3); + TEST_BYTE(0x2001, 0x00); + TEST_BYTE(0x4000, 0x01); + TEST_BYTE(0x4001, 0x23); + TEST_BYTE(0x4007, 0xef); + TEST_BYTE(0x4008, 0xaa); + TEST_BYTE(0x4010, 0xbb); + TEST_BYTE(0x4020, 0xcc); + TEST_BYTE(0x4040, 0xdd); + TEST_BYTE(0x4080, 0xee); + TEST_BYTE(0x40c0, 0xff); + TEST_BYTE(0x40ff, 0xff); + TEST_BYTE(0x4100, 0x00); + TEST_BYTE(0x41ff, 0x00); + Status = NtUnmapViewOfSection(NtCurrentProcess(), BaseAddress); + ok_ntstatus(Status, STATUS_SUCCESS); + } + Status = NtClose(SectionHandle); + ok_ntstatus(Status, STATUS_SUCCESS); + } + + CloseHandle(Handle); + DeleteFileW(FileName); } START_TEST(NtMapViewOfSection) @@ -1081,4 +1356,6 @@ Test_PageFileSection(); Test_ImageSection(); Test_BasedSection(); + Test_NoLoadSection(FALSE); + Test_NoLoadSection(TRUE); }

1 0

← Newer
1
...
18
19
20
21
22
23
24
...
31
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs August 2015