[NTDLL_APITEST]
[reactos.git] / rostests / apitests / ntdll / NtMapViewOfSection.c
index dbf8909..f191327 100644 (file)
@@ -1,23 +1,14 @@
+/*
+ * PROJECT:         ReactOS API Tests
+ * LICENSE:         LGPLv2.1+ - See COPYING.LIB in the top level directory
+ * PURPOSE:         Test for NtMapViewOfSection
+ * PROGRAMMERS:     Timo Kreuzer
+ */
 
+#include <apitest.h>
+#include <strsafe.h>
 #define WIN32_NO_STATUS
-#include <wine/test.h>
 #include <ndk/ntndk.h>
-#include <pseh/pseh2.h>
-
-NTSYSAPI
-NTSTATUS
-NTAPI
-NtMapViewOfSection(
-    HANDLE SectionHandle,
-       HANDLE ProcessHandle,
-       PVOID *BaseAddress,
-       ULONG_PTR ZeroBits,
-       SIZE_T CommitSize,
-       PLARGE_INTEGER SectionOffset,
-       PSIZE_T ViewSize,
-       SECTION_INHERIT InheritDisposition,
-       ULONG AllocationType,
-       ULONG Protect);
 
 void
 Test_PageFileSection(void)
@@ -25,10 +16,11 @@ Test_PageFileSection(void)
     NTSTATUS Status;
     HANDLE SectionHandle;
     LARGE_INTEGER MaximumSize, SectionOffset;
-    PVOID BaseAddress;
+    PVOID BaseAddress, BaseAddress2;
     SIZE_T ViewSize;
+    ULONG OldProtect;
 
-    /* Create a page file backed section */
+    /* Create a page file backed section with SEC_COMMIT */
     MaximumSize.QuadPart = 0x20000;
     Status = NtCreateSection(&SectionHandle,
                              SECTION_ALL_ACCESS,
@@ -37,7 +29,7 @@ Test_PageFileSection(void)
                              PAGE_READWRITE,
                              SEC_COMMIT,
                              NULL);
-    ok(NT_SUCCESS(Status), "NtCreateSection failed with Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
     if (!NT_SUCCESS(Status))
         return;
 
@@ -55,8 +47,7 @@ Test_PageFileSection(void)
                                 ViewShare,
                                 0,
                                 PAGE_READWRITE);
-    ok(Status == STATUS_MAPPED_ALIGNMENT,
-       "NtMapViewOfSection returned wrong Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_MAPPED_ALIGNMENT);
 
     /* Try to map a page with execute rights */
     BaseAddress = (PVOID)0x30000000;
@@ -72,10 +63,25 @@ Test_PageFileSection(void)
                                 ViewShare,
                                 0,
                                 PAGE_EXECUTE_READWRITE);
-    ok(Status == STATUS_SECTION_PROTECTION,
-       "NtMapViewOfSection returned wrong Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_SECTION_PROTECTION);
 
-    /* Map 2 pages, not comitting them */
+    /* Try to map 2 pages with MEM_COMMIT */
+    BaseAddress = (PVOID)0x30000000;
+    SectionOffset.QuadPart = 0;
+    ViewSize = 0x2000;
+    Status = NtMapViewOfSection(SectionHandle,
+                                NtCurrentProcess(),
+                                &BaseAddress,
+                                0,
+                                PAGE_SIZE,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                MEM_COMMIT,
+                                PAGE_READWRITE);
+    ok_ntstatus(Status, STATUS_INVALID_PARAMETER_9);
+
+    /* Map 2 pages, without MEM_COMMIT */
     BaseAddress = (PVOID)0x30000000;
     SectionOffset.QuadPart = 0;
     ViewSize = 0x2000;
@@ -89,9 +95,18 @@ Test_PageFileSection(void)
                                 ViewShare,
                                 0,
                                 PAGE_READWRITE);
-    ok(NT_SUCCESS(Status), "NtMapViewOfSection failed with Status %lx\n", Status);
-    if (!NT_SUCCESS(Status))
-        return;
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* We must be able to access the memory */
+    _SEH2_TRY
+    {
+        *(PULONG)BaseAddress = 1;
+    }
+    _SEH2_EXCEPT(1)
+    {
+        ok(FALSE, "Got an exception\n");
+    }
+    _SEH2_END;
 
     /* Commit a page in the section */
     BaseAddress = (PVOID)0x30000000;
@@ -102,12 +117,15 @@ Test_PageFileSection(void)
                                      &ViewSize,
                                      MEM_COMMIT,
                                      PAGE_READWRITE);
-    ok(NT_SUCCESS(Status), "NtAllocateVirtualMemory failed with Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Try to decommit a page in the section */
     Status = NtFreeVirtualMemory(NtCurrentProcess(),
                                  &BaseAddress,
                                  &ViewSize,
                                  MEM_DECOMMIT);
-    ok(Status == STATUS_UNABLE_TO_DELETE_SECTION, "NtFreeVirtualMemory returned wrong Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_UNABLE_TO_DELETE_SECTION);
+
     /* Try to commit a range larger than the section */
     BaseAddress = (PVOID)0x30000000;
     ViewSize = 0x3000;
@@ -117,8 +135,7 @@ Test_PageFileSection(void)
                                      &ViewSize,
                                      MEM_COMMIT,
                                      PAGE_READWRITE);
-    ok(Status == STATUS_CONFLICTING_ADDRESSES,
-       "NtAllocateVirtualMemory failed with wrong Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_CONFLICTING_ADDRESSES);
 
     /* Try to commit a page after the section */
     BaseAddress = (PVOID)0x30002000;
@@ -129,7 +146,7 @@ Test_PageFileSection(void)
                                      &ViewSize,
                                      MEM_COMMIT,
                                      PAGE_READWRITE);
-    ok(!NT_SUCCESS(Status), "NtAllocateVirtualMemory Should fail\n");
+    ok_ntstatus(Status, STATUS_CONFLICTING_ADDRESSES);
 
     /* Try to allocate a page after the section */
     BaseAddress = (PVOID)0x30002000;
@@ -140,7 +157,7 @@ Test_PageFileSection(void)
                                      &ViewSize,
                                      MEM_RESERVE | MEM_COMMIT,
                                      PAGE_READWRITE);
-    ok(!NT_SUCCESS(Status), "NtAllocateVirtualMemory should fail\n");
+    ok_ntstatus(Status, STATUS_CONFLICTING_ADDRESSES);
 
     /* Need to go to next 64k boundary */
     BaseAddress = (PVOID)0x30010000;
@@ -151,7 +168,7 @@ Test_PageFileSection(void)
                                      &ViewSize,
                                      MEM_RESERVE | MEM_COMMIT,
                                      PAGE_READWRITE);
-    ok(NT_SUCCESS(Status), "NtAllocateVirtualMemory failed with Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
     if (!NT_SUCCESS(Status))
         return;
 
@@ -164,15 +181,14 @@ Test_PageFileSection(void)
                                  MEM_RELEASE);
     ok(NT_SUCCESS(Status), "NtFreeVirtualMemory failed with Status %lx\n", Status);
 
-    /* Free the section mapping */
+    /* Try to release the section mapping with NtFreeVirtualMemory */
     BaseAddress = (PVOID)0x30000000;
     ViewSize = 0x1000;
     Status = NtFreeVirtualMemory(NtCurrentProcess(),
                                  &BaseAddress,
                                  &ViewSize,
                                  MEM_RELEASE);
-    ok(Status == STATUS_UNABLE_TO_DELETE_SECTION,
-       "NtFreeVirtualMemory failed with wrong Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_UNABLE_TO_DELETE_SECTION);
 
     /* Commit a page in the section */
     BaseAddress = (PVOID)0x30001000;
@@ -183,7 +199,7 @@ Test_PageFileSection(void)
                                      &ViewSize,
                                      MEM_COMMIT,
                                      PAGE_READWRITE);
-    ok(NT_SUCCESS(Status), "NtAllocateVirtualMemory failed with Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
 
     /* Try to decommit the page */
     BaseAddress = (PVOID)0x30001000;
@@ -192,8 +208,7 @@ Test_PageFileSection(void)
                                  &BaseAddress,
                                  &ViewSize,
                                  MEM_DECOMMIT);
-    ok(Status == STATUS_UNABLE_TO_DELETE_SECTION,
-       "NtFreeVirtualMemory failed with wrong Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_UNABLE_TO_DELETE_SECTION);
 
     BaseAddress = UlongToPtr(0x40000000);
     SectionOffset.QuadPart = 0;
@@ -208,11 +223,11 @@ Test_PageFileSection(void)
                                 ViewShare,
                                 0,
                                 PAGE_READWRITE);
-    ok(NT_SUCCESS(Status), "NtMapViewOfSection failed with Status %lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
     if (!NT_SUCCESS(Status))
         return;
 
-    ok(BaseAddress == UlongToPtr(0x40000000), "Invalid BaseAddress: %p", BaseAddress);
+    ok(BaseAddress == UlongToPtr(0x40000000), "Invalid BaseAddress: %p\n", BaseAddress);
 
     BaseAddress = (PVOID)0x40080000;
     SectionOffset.QuadPart = 0x10000;
@@ -227,11 +242,9 @@ Test_PageFileSection(void)
                                 ViewShare,
                                 0,
                                 PAGE_READWRITE);
-    ok(NT_SUCCESS(Status), "NtMapViewOfSection failed with Status %lx\n", Status);
-    if (!NT_SUCCESS(Status))
-        return;
+    ok_ntstatus(Status, STATUS_SUCCESS);
 
-    ok(BaseAddress == (PVOID)0x40080000, "Invalid BaseAddress: %p", BaseAddress);
+    ok(BaseAddress == (PVOID)0x40080000, "Invalid BaseAddress: %p\n", BaseAddress);
 
     /* Commit a page in the section */
     BaseAddress = (PVOID)0x40000000;
@@ -241,10 +254,397 @@ Test_PageFileSection(void)
                                      &ViewSize,
                                      MEM_COMMIT,
                                      PAGE_READWRITE);
-    ok(NT_SUCCESS(Status), "NtAllocateVirtualMemory failed with Status %lx\n", Status);
-    if (!NT_SUCCESS(Status))
-        return;
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Close the mapping */
+    Status = NtUnmapViewOfSection(NtCurrentProcess(), BaseAddress);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    BaseAddress = (PVOID)0x30000000;
+    Status = NtUnmapViewOfSection(NtCurrentProcess(), BaseAddress);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    Status = NtClose(SectionHandle);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Create a page file backed section, but only reserved */
+    MaximumSize.QuadPart = 0x20000;
+    Status = NtCreateSection(&SectionHandle,
+                             SECTION_ALL_ACCESS,
+                             NULL,
+                             &MaximumSize,
+                             PAGE_READWRITE,
+                             SEC_RESERVE,
+                             NULL);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Try to map 1 page, passing MEM_RESERVE */
+    BaseAddress = NULL;
+    SectionOffset.QuadPart = 0;
+    ViewSize = PAGE_SIZE;
+    Status = NtMapViewOfSection(SectionHandle,
+                                NtCurrentProcess(),
+                                &BaseAddress,
+                                0,
+                                PAGE_SIZE,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                MEM_RESERVE,
+                                PAGE_READWRITE);
+    ok_ntstatus(Status, STATUS_INVALID_PARAMETER_9);
+
+    /* Try to map 1 page using MEM_COMMIT */
+    BaseAddress = NULL;
+    SectionOffset.QuadPart = 0;
+    ViewSize = PAGE_SIZE;
+    Status = NtMapViewOfSection(SectionHandle,
+                                NtCurrentProcess(),
+                                &BaseAddress,
+                                0,
+                                PAGE_SIZE,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                MEM_COMMIT,
+                                PAGE_READWRITE);
+    ok_ntstatus(Status, STATUS_INVALID_PARAMETER_9);
+
+    /* Map 2 pages, but commit 1 */
+    BaseAddress = NULL;
+    SectionOffset.QuadPart = 0;
+    ViewSize = 2 * PAGE_SIZE;
+    Status = NtMapViewOfSection(SectionHandle,
+                                NtCurrentProcess(),
+                                &BaseAddress,
+                                0,
+                                PAGE_SIZE,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                0,
+                                PAGE_READWRITE);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* We must be able to access the 1st page */
+    Status = STATUS_SUCCESS;
+    _SEH2_TRY
+    {
+        *(PUCHAR)BaseAddress = 1;
+    }
+    _SEH2_EXCEPT(1)
+    {
+        Status = _SEH2_GetExceptionCode();
+    }
+    _SEH2_END;
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* We must not be able to access the 2nd page */
+    Status = STATUS_SUCCESS;
+    _SEH2_TRY
+    {
+        *((PUCHAR)BaseAddress + PAGE_SIZE) = 1;
+    }
+    _SEH2_EXCEPT(1)
+    {
+        Status = _SEH2_GetExceptionCode();
+    }
+    _SEH2_END;
+    ok_ntstatus(Status, STATUS_ACCESS_VIOLATION);
+
+    /* Map the 2 pages again into a different memory location */
+    BaseAddress2 = NULL;
+    Status = NtMapViewOfSection(SectionHandle,
+                                NtCurrentProcess(),
+                                &BaseAddress2,
+                                0,
+                                0,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                0,
+                                PAGE_READWRITE);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Commit a the 2nd page in the 2nd memory location */
+    BaseAddress2 = (PUCHAR)BaseAddress2 + PAGE_SIZE;
+    ViewSize = PAGE_SIZE;
+    Status = NtAllocateVirtualMemory(NtCurrentProcess(),
+                                     &BaseAddress2,
+                                     0,
+                                     &ViewSize,
+                                     MEM_COMMIT,
+                                     PAGE_READONLY);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Try to commit again (the already committed page) */
+    Status = NtAllocateVirtualMemory(NtCurrentProcess(),
+                                     &BaseAddress2,
+                                     0,
+                                     &ViewSize,
+                                     MEM_COMMIT,
+                                     PAGE_READONLY);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* We must be able to access the memory in the 2nd page of the 1st memory location */
+    Status = STATUS_SUCCESS;
+    _SEH2_TRY
+    {
+        *((PUCHAR)BaseAddress + PAGE_SIZE) = 2;
+    }
+    _SEH2_EXCEPT(1)
+    {
+        Status = _SEH2_GetExceptionCode();
+    }
+    _SEH2_END;
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    ok(*(PULONG)BaseAddress2 == 2, "Value in memory was wrong\n");
+
+    /* Close the mapping */
+    Status = NtUnmapViewOfSection(NtCurrentProcess(), BaseAddress);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    Status = NtUnmapViewOfSection(NtCurrentProcess(), (PUCHAR)BaseAddress2 - PAGE_SIZE);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    Status = NtClose(SectionHandle);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Try to create a 512 GB page file backed section with committed pages */
+    MaximumSize.QuadPart = 0x8000000000;
+    Status = NtCreateSection(&SectionHandle,
+                             SECTION_ALL_ACCESS,
+                             NULL,
+                             &MaximumSize,
+                             PAGE_READWRITE,
+                             SEC_COMMIT,
+                             NULL);
+    ok_ntstatus(Status, STATUS_COMMITMENT_LIMIT);
+
+    /* Try to create a huge page file backed section with PAGE_NOACCESS protection */
+    MaximumSize.QuadPart = 0x8000000000;
+    Status = NtCreateSection(&SectionHandle,
+                             SECTION_ALL_ACCESS,
+                             NULL,
+                             &MaximumSize,
+                             PAGE_NOACCESS,
+                             SEC_COMMIT,
+                             NULL);
+    ok_ntstatus(Status, STATUS_INVALID_PAGE_PROTECTION);
+
+    /* Try to create a very huge page file backed section, but only reserved */
+    MaximumSize.QuadPart = 0x80000000000;
+    Status = NtCreateSection(&SectionHandle,
+                             SECTION_ALL_ACCESS,
+                             NULL,
+                             &MaximumSize,
+                             PAGE_READWRITE,
+                             SEC_RESERVE,
+                             NULL);
+#ifdef _WIN64
+    ok_ntstatus(Status, STATUS_INSUFFICIENT_RESOURCES);
+#else
+    /* WoW64 returns STATUS_INSUFFICIENT_RESOURCES */
+    ok((Status == STATUS_INSUFFICIENT_RESOURCES) || (Status == STATUS_SECTION_TOO_BIG),
+       "got wrong Status: 0x%lx\n", Status);
+#endif
+
+    /* Try to create a even huger page file backed section, but only reserved */
+    MaximumSize.QuadPart = 0x800000000000;
+    Status = NtCreateSection(&SectionHandle,
+                             SECTION_ALL_ACCESS,
+                             NULL,
+                             &MaximumSize,
+                             PAGE_READWRITE,
+                             SEC_RESERVE,
+                             NULL);
+    ok_ntstatus(Status, STATUS_SECTION_TOO_BIG);
+
+    /* Create a 8 GB page file backed section, but only reserved */
+    MaximumSize.QuadPart = 0x200000000;
+    Status = NtCreateSection(&SectionHandle,
+                             SECTION_ALL_ACCESS,
+                             NULL,
+                             &MaximumSize,
+                             PAGE_READWRITE,
+                             SEC_RESERVE,
+                             NULL);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Pass a too large region size */
+    BaseAddress = NULL;
+    SectionOffset.QuadPart = 0;
+    ViewSize = MAXULONG_PTR;
+    Status = NtMapViewOfSection(SectionHandle,
+                                NtCurrentProcess(),
+                                &BaseAddress,
+                                0,
+                                0,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                0,
+                                PAGE_READWRITE);
+#ifdef _WIN64
+    ok_ntstatus(Status, STATUS_INVALID_PARAMETER_3);
+#else
+    /* WoW64 returns STATUS_INVALID_PARAMETER_4 */
+    ok((Status == STATUS_INVALID_PARAMETER_4) || (Status == STATUS_INVALID_PARAMETER_3),
+       "got wrong Status: 0x%lx\n", Status);
+#endif
+
+    /* Pass 0 region size */
+    BaseAddress = NULL;
+    SectionOffset.QuadPart = 0;
+    ViewSize = 0;
+    Status = NtMapViewOfSection(SectionHandle,
+                                NtCurrentProcess(),
+                                &BaseAddress,
+                                0,
+                                0,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                0,
+                                PAGE_READWRITE);
+#ifdef _WIN64
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    ok(ViewSize == 0x200000000, "wrong ViewSize: 0x%Ix\n", ViewSize);
+#else
+    /* WoW64 returns STATUS_NO_MEMORY */
+    ok((Status == STATUS_NO_MEMORY) || (Status == STATUS_INVALID_VIEW_SIZE),
+       "got wrong Status: 0x%lx\n", Status);
+    ok(ViewSize == 0, "wrong ViewSize: 0x%Ix\n", ViewSize);
+#endif
+
+    /* Map with PAGE_NOACCESS */
+    BaseAddress = NULL;
+    SectionOffset.QuadPart = 0;
+    ViewSize = 0x20000000;
+    Status = NtMapViewOfSection(SectionHandle,
+                                NtCurrentProcess(),
+                                &BaseAddress,
+                                0,
+                                0,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                0,
+                                PAGE_NOACCESS);
+    ok_ntstatus(Status, STATUS_SUCCESS);
 
+    /* Try to change protection to read/write */
+    ViewSize = 0x1000;
+    OldProtect = -1;
+    BaseAddress2 = BaseAddress;
+    Status = NtProtectVirtualMemory(NtCurrentProcess(), &BaseAddress2, &ViewSize, PAGE_READWRITE, &OldProtect);
+    ok_ntstatus(Status, STATUS_SECTION_PROTECTION);
+    // Windows 2003 returns bogus
+    //ok(OldProtect == PAGE_READWRITE, "Wrong protection returned: %u\n", OldProtect);
+
+    /* Test read access */
+    Status = STATUS_SUCCESS;
+    _SEH2_TRY
+    {
+        (void)(*(volatile char*)BaseAddress2);
+    }
+    _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
+    {
+        Status = _SEH2_GetExceptionCode();
+    }
+    _SEH2_END;
+    ok_ntstatus(Status, STATUS_ACCESS_VIOLATION);
+
+    /* Try to change protection to read/write */
+    ViewSize = 0x1000;
+    OldProtect = -1;
+    BaseAddress2 = BaseAddress;
+    Status = NtProtectVirtualMemory(NtCurrentProcess(), &BaseAddress2, &ViewSize, PAGE_READWRITE, &OldProtect);
+    ok_ntstatus(Status, STATUS_SECTION_PROTECTION);
+#ifdef _WIN64
+    ok(OldProtect == 0, "Wrong protection returned: 0x%lx\n", OldProtect);
+#else
+    // Windows 2003 returns bogus
+#endif
+
+    /* Try to change protection to readonly */
+    ViewSize = 0x1000;
+    OldProtect = -1;
+    BaseAddress2 = BaseAddress;
+    Status = NtProtectVirtualMemory(NtCurrentProcess(), &BaseAddress2, &ViewSize, PAGE_READONLY, &OldProtect);
+    ok_ntstatus(Status, STATUS_SECTION_PROTECTION);
+#ifdef _WIN64
+    //ok(OldProtect == 0, "Wrong protection returned: 0x%lx\n", OldProtect);
+#else
+    // Windows 2003 returns bogus
+#endif
+
+    /* Commit a page */
+    ViewSize = 0x1000;
+    BaseAddress2 = BaseAddress;
+    Status = NtAllocateVirtualMemory(NtCurrentProcess(), &BaseAddress2, 0, &ViewSize, MEM_COMMIT, PAGE_READONLY);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    ok(BaseAddress2 == BaseAddress, "Invalid base address: %p\n", BaseAddress2);
+
+    /* Commit the page again */
+    ViewSize = 0x1000;
+    BaseAddress2 = BaseAddress;
+    Status = NtAllocateVirtualMemory(NtCurrentProcess(), &BaseAddress2, 0, &ViewSize, MEM_COMMIT, PAGE_READONLY);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    ok(BaseAddress2 == BaseAddress, "Invalid base address: %p\n", BaseAddress2);
+
+    /* Test read access */
+    Status = STATUS_SUCCESS;
+    _SEH2_TRY
+    {
+        (void)(*(volatile char*)BaseAddress2);
+    }
+    _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
+    {
+        Status = _SEH2_GetExceptionCode();
+    }
+    _SEH2_END;
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Test write access */
+    Status = STATUS_SUCCESS;
+    _SEH2_TRY
+    {
+        *(char*)BaseAddress2 = 1;
+    }
+    _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
+    {
+        Status = _SEH2_GetExceptionCode();
+    }
+    _SEH2_END;
+    ok_ntstatus(Status, STATUS_ACCESS_VIOLATION);
+
+    /* Update protection to PAGE_READWRITE */
+    ViewSize = 0x1000;
+    BaseAddress2 = BaseAddress;
+    Status = NtAllocateVirtualMemory(NtCurrentProcess(), &BaseAddress2, 0, &ViewSize, MEM_COMMIT, PAGE_READWRITE);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Test write access */
+    Status = STATUS_SUCCESS;
+    _SEH2_TRY
+    {
+        *(char*)BaseAddress2 = 1;
+    }
+    _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
+    {
+        Status = _SEH2_GetExceptionCode();
+    }
+    _SEH2_END;
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Update protection to PAGE_EXECUTE_READWRITE (1 page) */
+    ViewSize = 0x1000;
+    BaseAddress2 = BaseAddress;
+    Status = NtAllocateVirtualMemory(NtCurrentProcess(), &BaseAddress2, 0, &ViewSize, MEM_COMMIT, PAGE_EXECUTE_READWRITE);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    Status = NtUnmapViewOfSection(NtCurrentProcess(), BaseAddress);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    Status = NtClose(SectionHandle);
+    ok_ntstatus(Status, STATUS_SUCCESS);
 }
 
 void
@@ -254,11 +654,15 @@ Test_ImageSection(void)
     NTSTATUS Status;
     OBJECT_ATTRIBUTES FileObjectAttributes;
     IO_STATUS_BLOCK IoStatusBlock;
+    WCHAR TestDllPath[MAX_PATH];
     HANDLE FileHandle, DataSectionHandle, ImageSectionHandle;
     PVOID DataBase, ImageBase;
     SIZE_T ViewSize;
 
-    if (!RtlDosPathNameToNtPathName_U(L"testdata\\test.dll",
+    GetModuleFileNameW(NULL, TestDllPath, RTL_NUMBER_OF(TestDllPath));
+    wcsrchr(TestDllPath, L'\\')[1] = UNICODE_NULL;
+    StringCbCatW(TestDllPath, sizeof(TestDllPath), L"testdata\\test.dll");
+    if (!RtlDosPathNameToNtPathName_U(TestDllPath,
                                       &FileName,
                                       NULL,
                                       NULL))
@@ -279,7 +683,12 @@ Test_ImageSection(void)
                         &IoStatusBlock,
                         FILE_SHARE_READ,
                         FILE_SYNCHRONOUS_IO_NONALERT);
-    ok(Status == STATUS_SUCCESS, "NtOpenFile failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    if (!NT_SUCCESS(Status))
+    {
+        skip("Failed to open file\n");
+        return;
+    }
 
     /* Create a data section with write access */
     Status = NtCreateSection(&DataSectionHandle,
@@ -289,9 +698,15 @@ Test_ImageSection(void)
                              PAGE_READWRITE, // SectionPageProtection
                              SEC_COMMIT, // AllocationAttributes
                              FileHandle);
-    ok(Status == STATUS_SUCCESS, "NtCreateSection failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    if (!NT_SUCCESS(Status))
+    {
+        skip("Failed to create data section\n");
+        NtClose(FileHandle);
+        return;
+    }
 
-    /* Map the data section */
+    /* Map the data section as flat mapping */
     DataBase = NULL;
     ViewSize = 0;
     Status = NtMapViewOfSection(DataSectionHandle,
@@ -304,7 +719,15 @@ Test_ImageSection(void)
                                 ViewShare,
                                 0,
                                 PAGE_READWRITE);
-    ok(Status == STATUS_SUCCESS, "NtMapViewOfSection failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    //ok(ViewSize == 0x3f95cc48, "ViewSize wrong: 0x%lx\n");
+    if (!NT_SUCCESS(Status))
+    {
+        skip("Failed to map view of data section\n");
+        NtClose(DataSectionHandle);
+        NtClose(FileHandle);
+        return;
+    }
 
     /* Check the original data */
     ok(*(ULONG*)DataBase == 0x00905a4d, "Header not ok\n");
@@ -326,7 +749,8 @@ Test_ImageSection(void)
                              PAGE_READWRITE, // SectionPageProtection
                              SEC_IMAGE, // AllocationAttributes
                              FileHandle);
-    ok(Status == STATUS_INVALID_IMAGE_NOT_MZ, "NtCreateSection failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_INVALID_IMAGE_NOT_MZ);
+    if (NT_SUCCESS(Status)) NtClose(ImageSectionHandle);
 
     /* Restore the original header */
     *(ULONG*)DataBase = 0x00905a4d;
@@ -343,7 +767,14 @@ Test_ImageSection(void)
                              PAGE_READWRITE, // SectionPageProtection
                              SEC_IMAGE, // AllocationAttributes
                              FileHandle);
-    ok(Status == STATUS_SUCCESS, "NtCreateSection failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    if (!NT_SUCCESS(Status))
+    {
+        skip("Failed to create image section\n");
+        NtClose(DataSectionHandle);
+        NtClose(FileHandle);
+        return;
+    }
 
     /* Map the image section */
     ImageBase = NULL;
@@ -351,18 +782,26 @@ Test_ImageSection(void)
     Status = NtMapViewOfSection(ImageSectionHandle,
                                 NtCurrentProcess(),
                                 &ImageBase,
-                                0,
-                                0,
-                                NULL,
+                                0, // ZeroBits
+                                0, // CommitSize
+                                NULL, // SectionOffset
                                 &ViewSize,
                                 ViewShare,
-                                0,
+                                0, // AllocationType
                                 PAGE_READONLY);
 #ifdef _M_IX86
-    ok(Status == STATUS_SUCCESS, "NtMapViewOfSection failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
 #else
-    ok(Status == STATUS_IMAGE_MACHINE_TYPE_MISMATCH, "NtMapViewOfSection failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_IMAGE_MACHINE_TYPE_MISMATCH);
 #endif
+    if (!NT_SUCCESS(Status))
+    {
+        skip("Failed to map view of image section\n");
+        NtClose(ImageSectionHandle);
+        NtClose(DataSectionHandle);
+        NtClose(FileHandle);
+        return;
+    }
 
     /* Check the header */
     ok(*(ULONG*)DataBase == 0x00905a4d, "Header not ok\n");
@@ -390,7 +829,7 @@ Test_ImageSection(void)
                                   &DataBase,
                                   &ViewSize,
                                   &IoStatusBlock);
-    ok(Status == STATUS_SUCCESS, "NtFlushVirtualMemory failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
 
     /* Check the data again */
     ok(*(ULONG*)ImageBase == 0x00905a4d, "Data should not be synced!\n");
@@ -414,7 +853,14 @@ Test_ImageSection(void)
                              PAGE_READWRITE, // SectionPageProtection
                              SEC_IMAGE, // AllocationAttributes
                              FileHandle);
-    ok(Status == STATUS_SUCCESS, "NtCreateSection failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    if (!NT_SUCCESS(Status))
+    {
+        skip("Failed to create image section\n");
+        NtClose(DataSectionHandle);
+        NtClose(FileHandle);
+        return;
+    }
 
     /* Map the image section again */
     ImageBase = NULL;
@@ -430,14 +876,23 @@ Test_ImageSection(void)
                                 0,
                                 PAGE_READONLY);
 #ifdef _M_IX86
-    ok(Status == STATUS_SUCCESS, "NtMapViewOfSection failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_SUCCESS);
 #else
-    ok(Status == STATUS_IMAGE_MACHINE_TYPE_MISMATCH, "NtMapViewOfSection failed, Status 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_IMAGE_MACHINE_TYPE_MISMATCH);
 #endif
+    if (!NT_SUCCESS(Status))
+    {
+        skip("Failed to map view of image section\n");
+        NtClose(ImageSectionHandle);
+        NtClose(DataSectionHandle);
+        NtClose(FileHandle);
+        return;
+    }
 
+    // This one doesn't always work, needs investigation
     /* Check the .data section again */
-    ok(*(ULONG*)((PUCHAR)ImageBase + 0x80000) == 0xf00dada,
-       "Data should be synced: 0x%lx!\n", *(ULONG*)((PUCHAR)ImageBase + 0x80000));
+    //ok(*(ULONG*)((PUCHAR)ImageBase + 0x80000) == 0xf00dada,
+    //   "Data should be synced: 0x%lx!\n", *(ULONG*)((PUCHAR)ImageBase + 0x80000));
 
     /* Restore the original data */
     *(ULONG*)((PUCHAR)DataBase + 0x800) = 0x12345678;
@@ -451,7 +906,7 @@ Test_ImageSection(void)
     DataBase = (PUCHAR)ImageBase + 0x20000;
     ViewSize = 0x1000;
     Status = NtAllocateVirtualMemory(NtCurrentProcess(), &DataBase, 0, &ViewSize, MEM_RESERVE, PAGE_NOACCESS);
-    ok(Status ==  STATUS_CONFLICTING_ADDRESSES, "Wrong Status: 0x%lx\n", Status);
+    ok_ntstatus(Status, STATUS_CONFLICTING_ADDRESSES);
 
     /* Cleanup */
     NtClose(FileHandle);
@@ -459,10 +914,171 @@ Test_ImageSection(void)
     NtUnmapViewOfSection(NtCurrentProcess(), ImageBase);
 }
 
+void
+Test_ImageSection2(void)
+{
+    UNICODE_STRING FileName;
+    NTSTATUS Status;
+    OBJECT_ATTRIBUTES FileObjectAttributes;
+    IO_STATUS_BLOCK IoStatusBlock;
+    HANDLE FileHandle, ImageSectionHandle;
+    PVOID ImageBase, BaseAddress;
+    SIZE_T ViewSize;
+    LARGE_INTEGER MaximumSize, SectionOffset;
+
+    if (!RtlDosPathNameToNtPathName_U(L"testdata\\nvoglv32.dll",
+                                      &FileName,
+                                      NULL,
+                                      NULL))
+    {
+        ok(0, "RtlDosPathNameToNtPathName_U failed\n");
+        return;
+    }
+
+    InitializeObjectAttributes(&FileObjectAttributes,
+                               &FileName,
+                               0,
+                               NULL,
+                               NULL);
+
+    Status = NtOpenFile(&FileHandle,
+                        GENERIC_READ|GENERIC_WRITE|SYNCHRONIZE,
+                        &FileObjectAttributes,
+                        &IoStatusBlock,
+                        FILE_SHARE_READ,
+                        FILE_SYNCHRONOUS_IO_NONALERT);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    printf("Opened file with handle %p\n", FileHandle);
+
+    /* Create a data section with write access */
+    MaximumSize.QuadPart = 0x20000;
+    Status = NtCreateSection(&ImageSectionHandle,
+                             SECTION_ALL_ACCESS, // DesiredAccess
+                             NULL, // ObjectAttributes
+                             &MaximumSize, // MaximumSize
+                             PAGE_READWRITE, // SectionPageProtection
+                             SEC_IMAGE, // AllocationAttributes
+                             FileHandle);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    printf("Created image section with handle %p\n", ImageSectionHandle);
+    //system("PAUSE");
+
+    /* Map the image section */
+    ImageBase = NULL;
+    ViewSize = 0x0000;
+    SectionOffset.QuadPart = 0x00000;
+    Status = NtMapViewOfSection(ImageSectionHandle,
+                                NtCurrentProcess(),
+                                &ImageBase,
+                                0,
+                                0,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                0,
+                                PAGE_READWRITE);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    printf("Mapped image section at %p, value in text section: %lx\n",
+           ImageBase, *((ULONG*)((PCHAR)ImageBase + 0x1196)));
+    system("PAUSE");
+
+    /* Try to allocate a page after the section */
+    BaseAddress = (PUCHAR)ImageBase + 0x10000;
+    ViewSize = 0x1000;
+    Status = NtAllocateVirtualMemory(NtCurrentProcess(),
+                                     &BaseAddress,
+                                     0,
+                                     &ViewSize,
+                                     MEM_RESERVE | MEM_COMMIT,
+                                     PAGE_READWRITE);
+    printf("allocation status: %lx\n", Status);
+    system("PAUSE");
+
+}
+
+// doesn't work with WoW64!
+void
+Test_BasedSection(void)
+{
+    NTSTATUS Status;
+    HANDLE SectionHandle1, SectionHandle2;
+    LARGE_INTEGER MaximumSize, SectionOffset;
+    PVOID BaseAddress1, BaseAddress2;
+    SIZE_T ViewSize;
+
+    /* Create a based section with SEC_COMMIT */
+    MaximumSize.QuadPart = 0x1000;
+    Status = NtCreateSection(&SectionHandle1,
+                             SECTION_ALL_ACCESS,
+                             NULL,
+                             &MaximumSize,
+                             PAGE_READWRITE,
+                             SEC_COMMIT | SEC_BASED,
+                             NULL);
+    ok_ntstatus(Status, STATUS_SUCCESS);
+
+    /* Map the 1st section */
+    BaseAddress1 = NULL;
+    SectionOffset.QuadPart = 0;
+    ViewSize = 0;
+    Status = NtMapViewOfSection(SectionHandle1,
+                                NtCurrentProcess(),
+                                &BaseAddress1,
+                                0,
+                                0,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                0,
+                                PAGE_READWRITE);
+#if 0 // WOW64?
+    ok_ntstatus(Status, STATUS_CONFLICTING_ADDRESSES);
+#else
+    ok_ntstatus(Status, STATUS_SUCCESS);
+#endif
+
+    /* Create a 2nd based section with SEC_COMMIT */
+    MaximumSize.QuadPart = 0x1000;
+    Status = NtCreateSection(&SectionHandle2,
+                             SECTION_ALL_ACCESS,
+                             NULL,
+                             &MaximumSize,
+                             PAGE_READWRITE,
+                             SEC_COMMIT | SEC_BASED,
+                             NULL);
+    ok_ntstatus(Status, STATUS_SUCCESS);//
+
+    /* Map the 2nd section */
+    BaseAddress2 = NULL;
+    SectionOffset.QuadPart = 0;
+    ViewSize = 0;
+    Status = NtMapViewOfSection(SectionHandle2,
+                                NtCurrentProcess(),
+                                &BaseAddress2,
+                                0,
+                                0,
+                                &SectionOffset,
+                                &ViewSize,
+                                ViewShare,
+                                0,
+                                PAGE_READWRITE);
+#if 0 // WOW64?
+    ok_ntstatus(Status, STATUS_CONFLICTING_ADDRESSES);
+#else
+    ok_ntstatus(Status, STATUS_SUCCESS);
+    ok((ULONG_PTR)BaseAddress2 < (ULONG_PTR)BaseAddress1,
+       "Invalid addresses: BaseAddress1=%p, BaseAddress2=%p\n", BaseAddress1, BaseAddress2);
+    ok(((ULONG_PTR)BaseAddress1 - (ULONG_PTR)BaseAddress2) == 0x10000,
+       "Invalid addresses: BaseAddress1=%p, BaseAddress2=%p\n", BaseAddress1, BaseAddress2);
+#endif
+
+}
 
 START_TEST(NtMapViewOfSection)
 {
     Test_PageFileSection();
     Test_ImageSection();
+    Test_BasedSection();
 }
-