Ros-diffs

ros-diffs@reactos.org

81390 discussions

[tfaber] 67491: [NTOS:IO] - Free the correct event pointer when out of memory in IopDeviceFsIoControl
by tfaber＠svn.reactos.org 01 May '15

01 May '15

Author: tfaber Date: Fri May 1 12:00:50 2015 New Revision: 67491 URL: http://svn.reactos.org/svn/reactos?rev=67491&view=rev Log: [NTOS:IO] - Free the correct event pointer when out of memory in IopDeviceFsIoControl Modified: trunk/reactos/ntoskrnl/io/iomgr/iofunc.c Modified: trunk/reactos/ntoskrnl/io/iomgr/iofunc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/iofunc.c… ============================================================================== --- trunk/reactos/ntoskrnl/io/iomgr/iofunc.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/io/iomgr/iofunc.c [iso-8859-1] Fri May 1 12:00:50 2015 @@ -346,7 +346,7 @@ /* Allocate IRP */ Irp = IoAllocateIrp(DeviceObject->StackSize, FALSE); - if (!Irp) return IopCleanupFailedIrp(FileObject, Event, NULL); + if (!Irp) return IopCleanupFailedIrp(FileObject, EventObject, NULL); /* Setup the IRP */ Irp->UserIosb = IoStatusBlock;

1 0

[tfaber] 67490: [NTOS:IO] - Don't leak event object reference when out of memory in NtReadFile/NtWriteFile
by tfaber＠svn.reactos.org 01 May '15

01 May '15

Author: tfaber Date: Fri May 1 11:57:51 2015 New Revision: 67490 URL: http://svn.reactos.org/svn/reactos?rev=67490&view=rev Log: [NTOS:IO] - Don't leak event object reference when out of memory in NtReadFile/NtWriteFile Modified: trunk/reactos/ntoskrnl/io/iomgr/iofunc.c Modified: trunk/reactos/ntoskrnl/io/iomgr/iofunc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/iofunc.c… ============================================================================== --- trunk/reactos/ntoskrnl/io/iomgr/iofunc.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/io/iomgr/iofunc.c [iso-8859-1] Fri May 1 11:57:51 2015 @@ -2359,7 +2359,7 @@ /* Allocate the IRP */ Irp = IoAllocateIrp(DeviceObject->StackSize, FALSE); - if (!Irp) return IopCleanupFailedIrp(FileObject, NULL, NULL); + if (!Irp) return IopCleanupFailedIrp(FileObject, EventObject, NULL); /* Set the IRP */ Irp->Tail.Overlay.OriginalFileObject = FileObject; @@ -3279,7 +3279,7 @@ /* Allocate the IRP */ Irp = IoAllocateIrp(DeviceObject->StackSize, FALSE); - if (!Irp) return IopCleanupFailedIrp(FileObject, NULL, NULL); + if (!Irp) return IopCleanupFailedIrp(FileObject, EventObject, NULL); /* Set the IRP */ Irp->Tail.Overlay.OriginalFileObject = FileObject;

1 0

[tfaber] 67489: [NTOS:IO] - IopCompleteRequest: in case of an exception, fail the IRP as shown by kmtest:IoReadWrite CORE-9624
by tfaber＠svn.reactos.org 01 May '15

01 May '15

Author: tfaber Date: Fri May 1 11:52:32 2015 New Revision: 67489 URL: http://svn.reactos.org/svn/reactos?rev=67489&view=rev Log: [NTOS:IO] - IopCompleteRequest: in case of an exception, fail the IRP as shown by kmtest:IoReadWrite CORE-9624 Modified: trunk/reactos/ntoskrnl/io/iomgr/irp.c Modified: trunk/reactos/ntoskrnl/io/iomgr/irp.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/irp.c?re… ============================================================================== --- trunk/reactos/ntoskrnl/io/iomgr/irp.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/io/iomgr/irp.c [iso-8859-1] Fri May 1 11:52:32 2015 @@ -293,7 +293,8 @@ } _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) { - /* Do nothing */ + /* Fail the IRP */ + Irp->IoStatus.Status = _SEH2_GetExceptionCode(); } _SEH2_END; }

1 0

[tfaber] 67488: [KMTESTS:IO] - Fix GCC build
by tfaber＠svn.reactos.org 01 May '15

01 May '15

Author: tfaber Date: Fri May 1 11:07:40 2015 New Revision: 67488 URL: http://svn.reactos.org/svn/reactos?rev=67488&view=rev Log: [KMTESTS:IO] - Fix GCC build Modified: trunk/rostests/kmtests/ntos_io/IoReadWrite_drv.c Modified: trunk/rostests/kmtests/ntos_io/IoReadWrite_drv.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/ntos_io/IoReadWri… ============================================================================== --- trunk/rostests/kmtests/ntos_io/IoReadWrite_drv.c [iso-8859-1] (original) +++ trunk/rostests/kmtests/ntos_io/IoReadWrite_drv.c [iso-8859-1] Fri May 1 11:07:40 2015 @@ -200,6 +200,7 @@ _Out_ PIO_STATUS_BLOCK IoStatus, _In_ PDEVICE_OBJECT DeviceObject) { + TestLastFastWriteKey = LockKey; UNIMPLEMENTED; return FALSE; }

1 0

[tfaber] 67487: [KMTESTS:IO] - Add a test for the interaction between NtReadFile and a file system driver CORE-9624
by tfaber＠svn.reactos.org 01 May '15

01 May '15

Author: tfaber Date: Fri May 1 11:03:21 2015 New Revision: 67487 URL: http://svn.reactos.org/svn/reactos?rev=67487&view=rev Log: [KMTESTS:IO] - Add a test for the interaction between NtReadFile and a file system driver CORE-9624 Added: trunk/rostests/kmtests/ntos_io/IoReadWrite.h (with props) trunk/rostests/kmtests/ntos_io/IoReadWrite_drv.c (with props) trunk/rostests/kmtests/ntos_io/IoReadWrite_user.c (with props) Modified: trunk/rostests/kmtests/CMakeLists.txt trunk/rostests/kmtests/include/kmt_platform.h trunk/rostests/kmtests/kmtest/testlist.c trunk/rostests/kmtests/ntos_io/CMakeLists.txt Modified: trunk/rostests/kmtests/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/CMakeLists.txt?re… ============================================================================== --- trunk/rostests/kmtests/CMakeLists.txt [iso-8859-1] (original) +++ trunk/rostests/kmtests/CMakeLists.txt [iso-8859-1] Fri May 1 11:03:21 2015 @@ -109,6 +109,7 @@ example/Example_user.c kernel32/FindFile_user.c ntos_io/IoDeviceObject_user.c + ntos_io/IoReadWrite_user.c tcpip/TcpIp_user.c ${COMMON_SOURCE} @@ -132,6 +133,7 @@ example_drv iodeviceobject_drv iohelper_drv + ioreadwrite_drv tcpip_drv) add_custom_target(kmtest_all) Modified: trunk/rostests/kmtests/include/kmt_platform.h URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/include/kmt_platf… ============================================================================== --- trunk/rostests/kmtests/include/kmt_platform.h [iso-8859-1] (original) +++ trunk/rostests/kmtests/include/kmt_platform.h [iso-8859-1] Fri May 1 11:03:21 2015 @@ -34,6 +34,7 @@ #include <winreg.h> #include <winsvc.h> #include <ndk/cmfuncs.h> +#include <ndk/iofuncs.h> #include <ndk/obfuncs.h> #include <ndk/rtlfuncs.h> #include <ndk/mmfuncs.h> Modified: trunk/rostests/kmtests/kmtest/testlist.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/kmtest/testlist.c… ============================================================================== --- trunk/rostests/kmtests/kmtest/testlist.c [iso-8859-1] (original) +++ trunk/rostests/kmtests/kmtest/testlist.c [iso-8859-1] Fri May 1 11:03:21 2015 @@ -10,6 +10,7 @@ KMT_TESTFUNC Test_Example; KMT_TESTFUNC Test_FindFile; KMT_TESTFUNC Test_IoDeviceObject; +KMT_TESTFUNC Test_IoReadWrite; KMT_TESTFUNC Test_RtlAvlTree; KMT_TESTFUNC Test_RtlException; KMT_TESTFUNC Test_RtlIntSafe; @@ -27,6 +28,7 @@ { "-Example", Test_Example }, { "FindFile", Test_FindFile }, { "IoDeviceObject", Test_IoDeviceObject }, + { "IoReadWrite", Test_IoReadWrite }, { "RtlAvlTree", Test_RtlAvlTree }, { "RtlException", Test_RtlException }, { "RtlIntSafe", Test_RtlIntSafe }, Modified: trunk/rostests/kmtests/ntos_io/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/ntos_io/CMakeList… ============================================================================== --- trunk/rostests/kmtests/ntos_io/CMakeLists.txt [iso-8859-1] (original) +++ trunk/rostests/kmtests/ntos_io/CMakeLists.txt [iso-8859-1] Fri May 1 11:03:21 2015 @@ -30,3 +30,18 @@ add_target_compile_definitions(iohelper_drv KMT_STANDALONE_DRIVER) #add_pch(iohelper_drv ../include/kmt_test.h) add_cd_file(TARGET iohelper_drv DESTINATION reactos/bin FOR all) + +# +# IoReadWrite +# +list(APPEND IOREADWRITE_DRV_SOURCE + ../kmtest_drv/kmtest_standalone.c + IoReadWrite_drv.c) + +add_library(ioreadwrite_drv SHARED ${IOREADWRITE_DRV_SOURCE}) +set_module_type(ioreadwrite_drv kernelmodedriver) +target_link_libraries(ioreadwrite_drv kmtest_printf ${PSEH_LIB}) +add_importlibs(ioreadwrite_drv ntoskrnl hal) +add_target_compile_definitions(ioreadwrite_drv KMT_STANDALONE_DRIVER) +#add_pch(ioreadwrite_drv ../include/kmt_test.h) +add_cd_file(TARGET ioreadwrite_drv DESTINATION reactos/bin FOR all) Added: trunk/rostests/kmtests/ntos_io/IoReadWrite.h URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/ntos_io/IoReadWri… ============================================================================== --- trunk/rostests/kmtests/ntos_io/IoReadWrite.h (added) +++ trunk/rostests/kmtests/ntos_io/IoReadWrite.h [iso-8859-1] Fri May 1 11:03:21 2015 @@ -0,0 +1,81 @@ +/* + * PROJECT: ReactOS kernel-mode tests + * LICENSE: LGPLv2.1+ - See COPYING.LIB in the top level directory + * PURPOSE: Read/Write operations test declarations + * PROGRAMMER: Thomas Faber <thomas.faber(a)reactos.org> + */ + +#ifndef _KMTEST_IOREADFILE_H_ +#define _KMTEST_IOREADFILE_H_ + +#define TEST_FILE_SIZE 17 + +#define KEY_SUCCEED 0x00 +#define KEY_FAIL_MISALIGNED 0x81 +#define KEY_FAIL_OVERFLOW 0x82 +#define KEY_FAIL_PARTIAL 0x83 +#define KEY_FAIL_BUSY 0x84 +#define KEY_FAIL_VERIFY_REQUIRED 0x85 + +#define KEY_FAIL_UNSUCCESSFUL 0xc1 +#define KEY_FAIL_NOT_IMPLEMENTED 0xc2 +#define KEY_FAIL_ACCESS_VIOLATION 0xc3 +#define KEY_FAIL_IN_PAGE_ERROR 0xc4 +#define KEY_FAIL_EOF 0xc5 +#define KEY_FAIL_ACCESS_DENIED 0xc6 +#define KEY_FAIL_MISALIGNED_ERROR 0xc7 +#define KEY_RESULT_MASK 0xff + +#define KEY_NEXT(key) ( (key) == KEY_FAIL_MISALIGNED_ERROR ? 0xff : \ + (key) == KEY_FAIL_VERIFY_REQUIRED ? KEY_FAIL_UNSUCCESSFUL : \ + (key) == KEY_SUCCEED ? KEY_FAIL_MISALIGNED : \ + (key) + 1 ) +#define KEY_ERROR(key) (((key) & 0xc0) == 0xc0) +static +NTSTATUS +TestGetReturnStatus( + _In_ ULONG LockKey) +{ + switch (LockKey & KEY_RESULT_MASK) + { + case KEY_SUCCEED: + return STATUS_SUCCESS; + + case KEY_FAIL_MISALIGNED: + return STATUS_DATATYPE_MISALIGNMENT; + case KEY_FAIL_OVERFLOW: + return STATUS_BUFFER_OVERFLOW; + case KEY_FAIL_PARTIAL: + return STATUS_PARTIAL_COPY; + case KEY_FAIL_BUSY: + return STATUS_DEVICE_BUSY; + case KEY_FAIL_VERIFY_REQUIRED: + return STATUS_VERIFY_REQUIRED; + + case KEY_FAIL_UNSUCCESSFUL: + return STATUS_UNSUCCESSFUL; + case KEY_FAIL_NOT_IMPLEMENTED: + return STATUS_NOT_IMPLEMENTED; + case KEY_FAIL_ACCESS_VIOLATION: + return STATUS_ACCESS_VIOLATION; + case KEY_FAIL_IN_PAGE_ERROR: + return STATUS_IN_PAGE_ERROR; + case KEY_FAIL_EOF: + return STATUS_END_OF_FILE; + case KEY_FAIL_ACCESS_DENIED: + return STATUS_ACCESS_DENIED; + case KEY_FAIL_MISALIGNED_ERROR: + return STATUS_DATATYPE_MISALIGNMENT_ERROR; + default: + ok(0, "Key = %lx\n", LockKey); + return STATUS_INVALID_PARAMETER; + } +} + +#define KEY_USE_FASTIO 0x100 +#define KEY_RETURN_PENDING 0x200 + +#define KEY_DATA(c) (((c) & 0xff) << 24) +#define KEY_GET_DATA(key) ((key) >> 24) + +#endif /* !defined _KMTEST_IOREADFILE_H_ */ Propchange: trunk/rostests/kmtests/ntos_io/IoReadWrite.h ------------------------------------------------------------------------------ svn:eol-style = native Added: trunk/rostests/kmtests/ntos_io/IoReadWrite_drv.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/ntos_io/IoReadWri… ============================================================================== --- trunk/rostests/kmtests/ntos_io/IoReadWrite_drv.c (added) +++ trunk/rostests/kmtests/ntos_io/IoReadWrite_drv.c [iso-8859-1] Fri May 1 11:03:21 2015 @@ -0,0 +1,312 @@ +/* + * PROJECT: ReactOS kernel-mode tests + * LICENSE: LGPLv2.1+ - See COPYING.LIB in the top level directory + * PURPOSE: Test driver for Read/Write operations + * PROGRAMMER: Thomas Faber <thomas.faber(a)reactos.org> + */ + +#include <kmt_test.h> +#include "IoReadWrite.h" + +#define NDEBUG +#include <debug.h> + +typedef struct _TEST_FCB +{ + FSRTL_ADVANCED_FCB_HEADER Header; + SECTION_OBJECT_POINTERS SectionObjectPointers; + FAST_MUTEX HeaderMutex; + BOOLEAN Cached; +} TEST_FCB, *PTEST_FCB; + +static KMT_IRP_HANDLER TestIrpHandler; +static FAST_IO_READ TestFastIoRead; +static FAST_IO_WRITE TestFastIoWrite; + +static FAST_IO_DISPATCH TestFastIoDispatch; +static ULONG TestLastFastReadKey; +static ULONG TestLastFastWriteKey; +static PFILE_OBJECT TestFileObject; +static PDEVICE_OBJECT TestDeviceObject; + +NTSTATUS +TestEntry( + _In_ PDRIVER_OBJECT DriverObject, + _In_ PCUNICODE_STRING RegistryPath, + _Out_ PCWSTR *DeviceName, + _Inout_ INT *Flags) +{ + NTSTATUS Status = STATUS_SUCCESS; + + PAGED_CODE(); + + UNREFERENCED_PARAMETER(RegistryPath); + + *DeviceName = L"IoReadWrite"; + *Flags = TESTENTRY_NO_EXCLUSIVE_DEVICE | TESTENTRY_BUFFERED_IO_DEVICE; + + TestFastIoDispatch.FastIoRead = TestFastIoRead; + TestFastIoDispatch.FastIoWrite = TestFastIoWrite; + DriverObject->FastIoDispatch = &TestFastIoDispatch; + + KmtRegisterIrpHandler(IRP_MJ_CREATE, NULL, TestIrpHandler); + KmtRegisterIrpHandler(IRP_MJ_CLEANUP, NULL, TestIrpHandler); + KmtRegisterIrpHandler(IRP_MJ_READ, NULL, TestIrpHandler); + //KmtRegisterIrpHandler(IRP_MJ_WRITE, NULL, TestIrpHandler); + + return Status; +} + +VOID +TestUnload( + _In_ PDRIVER_OBJECT DriverObject) +{ + PAGED_CODE(); +} + +static +BOOLEAN +NTAPI +TestAcquireForLazyWrite( + _In_ PVOID Context, + _In_ BOOLEAN Wait) +{ + ok_eq_pointer(Context, NULL); + ok(0, "Unexpected call to AcquireForLazyWrite\n"); + return TRUE; +} + +static +VOID +NTAPI +TestReleaseFromLazyWrite( + _In_ PVOID Context) +{ + ok_eq_pointer(Context, NULL); + ok(0, "Unexpected call to ReleaseFromLazyWrite\n"); +} + +static +BOOLEAN +NTAPI +TestAcquireForReadAhead( + _In_ PVOID Context, + _In_ BOOLEAN Wait) +{ + ok_eq_pointer(Context, NULL); + ok(0, "Unexpected call to AcquireForReadAhead\n"); + return TRUE; +} + +static +VOID +NTAPI +TestReleaseFromReadAhead( + _In_ PVOID Context) +{ + ok_eq_pointer(Context, NULL); + ok(0, "Unexpected call to ReleaseFromReadAhead\n"); +} + +static +NTSTATUS +TestCommonRead( + _In_ PVOID Buffer, + _In_ ULONG Length, + _In_ LONGLONG FileOffset, + _In_ ULONG LockKey, + _Out_ PIO_STATUS_BLOCK IoStatus) +{ + if (FileOffset >= TEST_FILE_SIZE) + { + trace("FileOffset %I64d > file size\n", FileOffset); + IoStatus->Status = STATUS_END_OF_FILE; + IoStatus->Information = 0; + } + else if (Length == 0 || Buffer == NULL) + { + IoStatus->Status = STATUS_BUFFER_OVERFLOW; + IoStatus->Information = TEST_FILE_SIZE - FileOffset; + } + else + { + Length = min(Length, TEST_FILE_SIZE - FileOffset); + RtlFillMemory(Buffer, Length, KEY_GET_DATA(LockKey)); + IoStatus->Status = TestGetReturnStatus(LockKey); + IoStatus->Information = Length; + } + if (LockKey & KEY_RETURN_PENDING) + return STATUS_PENDING; + return IoStatus->Status; +} + +static +BOOLEAN +NTAPI +TestFastIoRead( + _In_ PFILE_OBJECT FileObject, + _In_ PLARGE_INTEGER FileOffset, + _In_ ULONG Length, + _In_ BOOLEAN Wait, + _In_ ULONG LockKey, + _Out_ PVOID Buffer, + _Out_ PIO_STATUS_BLOCK IoStatus, + _In_ PDEVICE_OBJECT DeviceObject) +{ + PTEST_FCB Fcb; + NTSTATUS Status; + + //trace("FastIoRead: %p %lx %I64d+%lu -> %p\n", FileObject, LockKey, FileOffset->QuadPart, Length, Buffer); + ok_eq_pointer(FileObject, TestFileObject); + ok_bool_true(Wait, "Wait is"); + ok_eq_pointer(DeviceObject, TestDeviceObject); + Fcb = FileObject->FsContext; + ok_bool_true(Fcb->Cached, "Cached is"); + + TestLastFastReadKey = LockKey; + ok((ULONG_PTR)Buffer < MM_USER_PROBE_ADDRESS, "Buffer is %p\n", Buffer); + ok((ULONG_PTR)FileOffset > MM_USER_PROBE_ADDRESS, "FileOffset is %p\n", FileOffset); + ok((ULONG_PTR)IoStatus > MM_USER_PROBE_ADDRESS, "IoStatus is %p\n", IoStatus); + _SEH2_TRY + { + Status = TestCommonRead(Buffer, Length, FileOffset->QuadPart, LockKey, IoStatus); + } + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) + { + IoStatus->Status = _SEH2_GetExceptionCode(); + return FALSE; + } + _SEH2_END; + + if (Status == STATUS_PENDING) + return FALSE; + + if (LockKey & KEY_USE_FASTIO) + return TRUE; + else + return FALSE; +} + +static +BOOLEAN +NTAPI +TestFastIoWrite( + _In_ PFILE_OBJECT FileObject, + _In_ PLARGE_INTEGER FileOffset, + _In_ ULONG Length, + _In_ BOOLEAN Wait, + _In_ ULONG LockKey, + _In_ PVOID Buffer, + _Out_ PIO_STATUS_BLOCK IoStatus, + _In_ PDEVICE_OBJECT DeviceObject) +{ + UNIMPLEMENTED; + return FALSE; +} + +static +NTSTATUS +TestIrpHandler( + _In_ PDEVICE_OBJECT DeviceObject, + _In_ PIRP Irp, + _In_ PIO_STACK_LOCATION IoStack) +{ + NTSTATUS Status; + PTEST_FCB Fcb; + CACHE_MANAGER_CALLBACKS Callbacks; + CACHE_UNINITIALIZE_EVENT CacheUninitEvent; + + PAGED_CODE(); + + DPRINT("IRP %x/%x\n", IoStack->MajorFunction, IoStack->MinorFunction); + ASSERT(IoStack->MajorFunction == IRP_MJ_CREATE || + IoStack->MajorFunction == IRP_MJ_CLEANUP || + IoStack->MajorFunction == IRP_MJ_READ || + IoStack->MajorFunction == IRP_MJ_WRITE); + + Status = STATUS_NOT_SUPPORTED; + Irp->IoStatus.Information = 0; + + if (IoStack->MajorFunction == IRP_MJ_CREATE) + { + if (IoStack->FileObject->FileName.Length >= 2 * sizeof(WCHAR)) + { + TestDeviceObject = DeviceObject; + TestFileObject = IoStack->FileObject; + } + Fcb = ExAllocatePoolWithTag(NonPagedPool, sizeof(*Fcb), 'FwrI'); + RtlZeroMemory(Fcb, sizeof(*Fcb)); + ExInitializeFastMutex(&Fcb->HeaderMutex); + FsRtlSetupAdvancedHeader(&Fcb->Header, &Fcb->HeaderMutex); + Fcb->Header.AllocationSize.QuadPart = TEST_FILE_SIZE; + Fcb->Header.FileSize.QuadPart = TEST_FILE_SIZE; + Fcb->Header.ValidDataLength.QuadPart = TEST_FILE_SIZE; + IoStack->FileObject->FsContext = Fcb; + IoStack->FileObject->SectionObjectPointer = &Fcb->SectionObjectPointers; + if (IoStack->FileObject->FileName.Length >= 2 * sizeof(WCHAR) && + IoStack->FileObject->FileName.Buffer[1] != 'N') + { + Fcb->Cached = TRUE; + Callbacks.AcquireForLazyWrite = TestAcquireForLazyWrite; + Callbacks.ReleaseFromLazyWrite = TestReleaseFromLazyWrite; + Callbacks.AcquireForReadAhead = TestAcquireForReadAhead; + Callbacks.ReleaseFromReadAhead = TestReleaseFromReadAhead; + CcInitializeCacheMap(IoStack->FileObject, + (PCC_FILE_SIZES)&Fcb->Header.AllocationSize, + FALSE, + &Callbacks, + NULL); + } + Irp->IoStatus.Information = FILE_OPENED; + Status = STATUS_SUCCESS; + } + else if (IoStack->MajorFunction == IRP_MJ_CLEANUP) + { + KeInitializeEvent(&CacheUninitEvent.Event, NotificationEvent, FALSE); + CcUninitializeCacheMap(IoStack->FileObject, NULL, &CacheUninitEvent); + KeWaitForSingleObject(&CacheUninitEvent.Event, Executive, KernelMode, FALSE, NULL); + Fcb = IoStack->FileObject->FsContext; + ExFreePoolWithTag(Fcb, 'FwrI'); + IoStack->FileObject->FsContext = NULL; + Status = STATUS_SUCCESS; + } + else if (IoStack->MajorFunction == IRP_MJ_READ) + { + //trace("IRP_MJ_READ: %p %lx %I64d+%lu -> %p\n", IoStack->FileObject, IoStack->Parameters.Read.Key, IoStack->Parameters.Read.ByteOffset.QuadPart, IoStack->Parameters.Read.Length, Irp->AssociatedIrp.SystemBuffer); + ok_eq_pointer(DeviceObject, TestDeviceObject); + ok_eq_pointer(IoStack->FileObject, TestFileObject); + Fcb = IoStack->FileObject->FsContext; + if (Fcb->Cached) + ok_eq_hex(IoStack->Parameters.Read.Key, TestLastFastReadKey); + ok(Irp->AssociatedIrp.SystemBuffer == NULL || + (ULONG_PTR)Irp->AssociatedIrp.SystemBuffer > MM_USER_PROBE_ADDRESS, + "Buffer is %p\n", + Irp->AssociatedIrp.SystemBuffer); + Status = TestCommonRead(Irp->AssociatedIrp.SystemBuffer, + IoStack->Parameters.Read.Length, + IoStack->Parameters.Read.ByteOffset.QuadPart, + IoStack->Parameters.Read.Key, + &Irp->IoStatus); + } + else if (IoStack->MajorFunction == IRP_MJ_WRITE) + { + ok_eq_pointer(DeviceObject, TestDeviceObject); + ok_eq_pointer(IoStack->FileObject, TestFileObject); + UNIMPLEMENTED; + Status = STATUS_NOT_IMPLEMENTED; + } + + if (Status == STATUS_PENDING) + { + IoMarkIrpPending(Irp); + IoCompleteRequest(Irp, IO_NO_INCREMENT); + Status = STATUS_PENDING; + } + else + { + Irp->IoStatus.Status = Status; + IoCompleteRequest(Irp, IO_NO_INCREMENT); + } + + return Status; +} Propchange: trunk/rostests/kmtests/ntos_io/IoReadWrite_drv.c ------------------------------------------------------------------------------ svn:eol-style = native Added: trunk/rostests/kmtests/ntos_io/IoReadWrite_user.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/ntos_io/IoReadWri… ============================================================================== --- trunk/rostests/kmtests/ntos_io/IoReadWrite_user.c (added) +++ trunk/rostests/kmtests/ntos_io/IoReadWrite_user.c [iso-8859-1] Fri May 1 11:03:21 2015 @@ -0,0 +1,237 @@ +/* + * PROJECT: ReactOS kernel-mode tests + * LICENSE: LGPLv2.1+ - See COPYING.LIB in the top level directory + * PURPOSE: Test for Read/Write operations + * PROGRAMMER: Thomas Faber <thomas.faber(a)reactos.org> + */ + +#include <kmt_test.h> +#include "IoReadWrite.h" + +static +VOID +TestRead( + _In_ HANDLE FileHandle, + _In_ BOOLEAN Cached, + _In_ BOOLEAN UseFastIo, + _In_ BOOLEAN ReturnPending) +{ + NTSTATUS Status; + IO_STATUS_BLOCK IoStatus; + HANDLE EventHandle; + UCHAR Buffer[32]; + LARGE_INTEGER Offset; + ULONG BaseKey, StatusKey, Key; + DWORD WaitStatus; + + BaseKey = (UseFastIo ? KEY_USE_FASTIO : 0) | + (ReturnPending ? KEY_RETURN_PENDING : 0); + + EventHandle = CreateEventW(NULL, TRUE, FALSE, NULL); + ok(EventHandle != NULL, "CreateEvent failed with %lu\n", GetLastError()); + + for (StatusKey = KEY_SUCCEED ; StatusKey != 0xff; StatusKey = KEY_NEXT(StatusKey)) + { + trace("\tSTATUS KEY: %lx\n", StatusKey); + ResetEvent(EventHandle); + RtlFillMemory(&IoStatus, sizeof(IoStatus), 0x55); + Key = BaseKey | StatusKey | KEY_DATA(0x11); + Offset.QuadPart = 0; + Status = NtReadFile(FileHandle, + EventHandle, + NULL, + NULL, + &IoStatus, + NULL, + 0, + &Offset, + &Key); + WaitStatus = WaitForSingleObject(EventHandle, 0); + if (ReturnPending) + ok_eq_hex(Status, STATUS_ACCESS_VIOLATION); + else + ok_eq_hex(Status, STATUS_BUFFER_OVERFLOW); + if (Cached && UseFastIo && !ReturnPending) + { + ok_eq_ulong(WaitStatus, WAIT_OBJECT_0); + ok_eq_hex(IoStatus.Status, STATUS_BUFFER_OVERFLOW); + ok_eq_ulongptr(IoStatus.Information, TEST_FILE_SIZE); + } + else + { + ok_eq_ulong(WaitStatus, WAIT_TIMEOUT); + ok_eq_hex(IoStatus.Status, 0x55555555); + ok_eq_ulongptr(IoStatus.Information, (ULONG_PTR)0x5555555555555555); + } + + KmtStartSeh() + ResetEvent(EventHandle); + RtlFillMemory(&IoStatus, sizeof(IoStatus), 0x55); + Key = BaseKey | StatusKey | KEY_DATA(0x22); + Offset.QuadPart = 0; + Status = NtReadFile(FileHandle, + EventHandle, + NULL, + NULL, + &IoStatus, + NULL, + sizeof(Buffer), + &Offset, + &Key); + WaitStatus = WaitForSingleObject(EventHandle, 0); + ok_eq_ulong(WaitStatus, WAIT_TIMEOUT); + ok_eq_hex(Status, STATUS_ACCESS_VIOLATION); + ok_eq_hex(IoStatus.Status, 0x55555555); + ok_eq_ulongptr(IoStatus.Information, (ULONG_PTR)0x5555555555555555); + KmtEndSeh(STATUS_SUCCESS); + + ResetEvent(EventHandle); + RtlFillMemory(&IoStatus, sizeof(IoStatus), 0x55); + RtlFillMemory(Buffer, sizeof(Buffer), 0x55); + Key = BaseKey | StatusKey | KEY_DATA(0x33); + Offset.QuadPart = 0; + Status = NtReadFile(FileHandle, + EventHandle, + NULL, + NULL, + &IoStatus, + Buffer, + 0, + &Offset, + &Key); + WaitStatus = WaitForSingleObject(EventHandle, 0); + if (ReturnPending) + ok_eq_hex(Status, STATUS_ACCESS_VIOLATION); + else + ok_eq_hex(Status, STATUS_BUFFER_OVERFLOW); + if (Cached && UseFastIo && !ReturnPending) + { + ok_eq_ulong(WaitStatus, WAIT_OBJECT_0); + ok_eq_hex(IoStatus.Status, STATUS_BUFFER_OVERFLOW); + ok_eq_ulongptr(IoStatus.Information, TEST_FILE_SIZE); + } + else + { + ok_eq_ulong(WaitStatus, WAIT_TIMEOUT); + ok_eq_hex(IoStatus.Status, 0x55555555); + ok_eq_ulongptr(IoStatus.Information, (ULONG_PTR)0x5555555555555555); + } + ok_eq_uint(Buffer[0], 0x55); + + ResetEvent(EventHandle); + RtlFillMemory(&IoStatus, sizeof(IoStatus), 0x55); + RtlFillMemory(Buffer, sizeof(Buffer), 0x55); + Key = BaseKey | StatusKey | KEY_DATA(0x44); + Offset.QuadPart = 0; + Status = NtReadFile(FileHandle, + EventHandle, + NULL, + NULL, + &IoStatus, + Buffer, + sizeof(Buffer), + &Offset, + &Key); + WaitStatus = WaitForSingleObject(EventHandle, 0); + ok_eq_hex(Status, TestGetReturnStatus(StatusKey)); + if ((Cached && UseFastIo && !ReturnPending && + (StatusKey == KEY_SUCCEED || StatusKey == KEY_FAIL_OVERFLOW || StatusKey == KEY_FAIL_EOF)) || + !KEY_ERROR(StatusKey)) + { + ok_eq_ulong(WaitStatus, WAIT_OBJECT_0); + ok_eq_hex(IoStatus.Status, TestGetReturnStatus(StatusKey)); + ok_eq_ulongptr(IoStatus.Information, TEST_FILE_SIZE); + } + else + { + ok_eq_ulong(WaitStatus, WAIT_TIMEOUT); + ok_eq_hex(IoStatus.Status, 0x55555555); + ok_eq_ulongptr(IoStatus.Information, (ULONG_PTR)0x5555555555555555); + } + if ((StatusKey != KEY_FAIL_VERIFY_REQUIRED && !KEY_ERROR(StatusKey)) || + Cached) + { + ok_eq_uint(Buffer[0], 0x44); + ok_eq_uint(Buffer[TEST_FILE_SIZE - 1], 0x44); + ok_eq_uint(Buffer[TEST_FILE_SIZE], 0x55); + } + else + { + ok_eq_uint(Buffer[0], 0x55); + } + } +} + +START_TEST(IoReadWrite) +{ + HANDLE FileHandle; + UNICODE_STRING CachedFileName = RTL_CONSTANT_STRING(L"\\Device\\Kmtest-IoReadWrite\\Cached"); + UNICODE_STRING NonCachedFileName = RTL_CONSTANT_STRING(L"\\Device\\Kmtest-IoReadWrite\\NonCached"); + OBJECT_ATTRIBUTES ObjectAttributes; + IO_STATUS_BLOCK IoStatus; + NTSTATUS Status; + + KmtLoadDriver(L"IoReadWrite", FALSE); + KmtOpenDriver(); + + RtlFillMemory(&IoStatus, sizeof(IoStatus), 0x55); + InitializeObjectAttributes(&ObjectAttributes, + &NonCachedFileName, + OBJ_CASE_INSENSITIVE, + NULL, + NULL); + Status = NtOpenFile(&FileHandle, + FILE_ALL_ACCESS, + &ObjectAttributes, + &IoStatus, + 0, + FILE_NON_DIRECTORY_FILE | + FILE_SYNCHRONOUS_IO_NONALERT); + ok_eq_hex(Status, STATUS_SUCCESS); + if (!skip(NT_SUCCESS(Status), "No file\n")) + { + ok_eq_hex(IoStatus.Status, STATUS_SUCCESS); + ok_eq_ulongptr(IoStatus.Information, FILE_OPENED); + trace("Non-Cached, no FastIo, direct return\n"); + TestRead(FileHandle, FALSE, FALSE, FALSE); + trace("Non-Cached, allow FastIo, direct return\n"); + TestRead(FileHandle, FALSE, TRUE, FALSE); + trace("Non-Cached, no FastIo, pending return\n"); + TestRead(FileHandle, FALSE, FALSE, TRUE); + trace("Non-Cached, allow FastIo, pending return\n"); + TestRead(FileHandle, FALSE, TRUE, TRUE); + NtClose(FileHandle); + } + + RtlFillMemory(&IoStatus, sizeof(IoStatus), 0x55); + InitializeObjectAttributes(&ObjectAttributes, + &CachedFileName, + OBJ_CASE_INSENSITIVE, + NULL, + NULL); + Status = NtOpenFile(&FileHandle, + FILE_ALL_ACCESS, + &ObjectAttributes, + &IoStatus, + 0, + FILE_NON_DIRECTORY_FILE | + FILE_SYNCHRONOUS_IO_NONALERT); + ok_eq_hex(Status, STATUS_SUCCESS); + if (!skip(NT_SUCCESS(Status), "No file\n")) + { + ok_eq_hex(IoStatus.Status, STATUS_SUCCESS); + ok_eq_ulongptr(IoStatus.Information, FILE_OPENED); + trace("Cached, no FastIo, direct return\n"); + TestRead(FileHandle, TRUE, FALSE, FALSE); + trace("Cached, allow FastIo, direct return\n"); + TestRead(FileHandle, TRUE, TRUE, FALSE); + trace("Cached, no FastIo, pending return\n"); + TestRead(FileHandle, TRUE, FALSE, TRUE); + trace("Cached, allow FastIo, pending return\n"); + TestRead(FileHandle, TRUE, TRUE, TRUE); + NtClose(FileHandle); + } + + KmtCloseDriver(); + KmtUnloadDriver(); +} Propchange: trunk/rostests/kmtests/ntos_io/IoReadWrite_user.c ------------------------------------------------------------------------------ svn:eol-style = native

1 0

[tfaber] 67486: [KMTEST] - Add TESTENTRY_NO_READONLY_DEVICE and TESTENTRY_BUFFERED_IO_DEVICE flags for standalone drivers
by tfaber＠svn.reactos.org 01 May '15

01 May '15

Author: tfaber Date: Fri May 1 10:52:37 2015 New Revision: 67486 URL: http://svn.reactos.org/svn/reactos?rev=67486&view=rev Log: [KMTEST] - Add TESTENTRY_NO_READONLY_DEVICE and TESTENTRY_BUFFERED_IO_DEVICE flags for standalone drivers Modified: trunk/rostests/kmtests/include/kmt_test.h trunk/rostests/kmtests/kmtest_drv/kmtest_standalone.c Modified: trunk/rostests/kmtests/include/kmt_test.h URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/include/kmt_test.… ============================================================================== --- trunk/rostests/kmtests/include/kmt_test.h [iso-8859-1] (original) +++ trunk/rostests/kmtests/include/kmt_test.h [iso-8859-1] Fri May 1 10:52:37 2015 @@ -109,6 +109,8 @@ TESTENTRY_NO_REGISTER_DISPATCH = 2, TESTENTRY_NO_REGISTER_UNLOAD = 4, TESTENTRY_NO_EXCLUSIVE_DEVICE = 8, + TESTENTRY_NO_READONLY_DEVICE = 16, + TESTENTRY_BUFFERED_IO_DEVICE = 32, } KMT_TESTENTRY_FLAGS; NTSTATUS TestEntry(IN PDRIVER_OBJECT DriverObject, IN PCUNICODE_STRING RegistryPath, OUT PCWSTR *DeviceName, IN OUT INT *Flags); Modified: trunk/rostests/kmtests/kmtest_drv/kmtest_standalone.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/kmtest_drv/kmtest… ============================================================================== --- trunk/rostests/kmtests/kmtest_drv/kmtest_standalone.c [iso-8859-1] (original) +++ trunk/rostests/kmtests/kmtest_drv/kmtest_standalone.c [iso-8859-1] Fri May 1 10:52:37 2015 @@ -121,7 +121,8 @@ RtlAppendUnicodeToString(&DeviceName, DeviceNameSuffix); Status = IoCreateDevice(DriverObject, 0, &DeviceName, FILE_DEVICE_UNKNOWN, - FILE_DEVICE_SECURE_OPEN | FILE_READ_ONLY_DEVICE, + FILE_DEVICE_SECURE_OPEN | + (Flags & TESTENTRY_NO_READONLY_DEVICE ? 0 : FILE_READ_ONLY_DEVICE), Flags & TESTENTRY_NO_EXCLUSIVE_DEVICE ? FALSE : TRUE, &TestDeviceObject); @@ -130,6 +131,9 @@ DPRINT1("Could not create device object %wZ\n", &DeviceName); goto cleanup; } + + if (Flags & TESTENTRY_BUFFERED_IO_DEVICE) + TestDeviceObject->Flags |= DO_BUFFERED_IO; DPRINT("DriverEntry. Created DeviceObject %p\n", TestDeviceObject);

1 0

[tfaber] 67485: [NTOS:IO] - Add missing SEH around user buffer access in IopCompleteRequest - Remove a redundant condition CORE-9624
by tfaber＠svn.reactos.org 01 May '15

01 May '15

Author: tfaber Date: Fri May 1 10:49:50 2015 New Revision: 67485 URL: http://svn.reactos.org/svn/reactos?rev=67485&view=rev Log: [NTOS:IO] - Add missing SEH around user buffer access in IopCompleteRequest - Remove a redundant condition CORE-9624 Modified: trunk/reactos/ntoskrnl/io/iomgr/irp.c Modified: trunk/reactos/ntoskrnl/io/iomgr/irp.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/irp.c?re… ============================================================================== --- trunk/reactos/ntoskrnl/io/iomgr/irp.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/io/iomgr/irp.c [iso-8859-1] Fri May 1 10:49:50 2015 @@ -284,10 +284,18 @@ (Irp->IoStatus.Status != STATUS_VERIFY_REQUIRED) && !(NT_ERROR(Irp->IoStatus.Status))) { - /* Copy the buffer back to the user */ - RtlCopyMemory(Irp->UserBuffer, - Irp->AssociatedIrp.SystemBuffer, - Irp->IoStatus.Information); + _SEH2_TRY + { + /* Copy the buffer back to the user */ + RtlCopyMemory(Irp->UserBuffer, + Irp->AssociatedIrp.SystemBuffer, + Irp->IoStatus.Information); + } + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) + { + /* Do nothing */ + } + _SEH2_END; } /* Also check if we should de-allocate it */ @@ -317,10 +325,9 @@ * (but warnings are OK!), or if it was completed with an error, but * did return from a pending I/O Operation and is not synchronous. */ - if (!(NT_ERROR(Irp->IoStatus.Status)) || - (NT_ERROR(Irp->IoStatus.Status) && - (Irp->PendingReturned) && - !(IsIrpSynchronous(Irp, FileObject)))) + if (!NT_ERROR(Irp->IoStatus.Status) || + (Irp->PendingReturned && + !IsIrpSynchronous(Irp, FileObject))) { /* Get any information we need from the FO before we kill it */ if ((FileObject) && (FileObject->CompletionContext))

1 0

[dquintana] 67484: Note to self: review diffs and remove temporary changes BEFORE commiting (not after).
by dquintana＠svn.reactos.org 30 Apr '15

30 Apr '15

Author: dquintana Date: Thu Apr 30 22:09:11 2015 New Revision: 67484 URL: http://svn.reactos.org/svn/reactos?rev=67484&view=rev Log: Note to self: review diffs and remove temporary changes BEFORE commiting (not after). Modified: trunk/reactos/dll/win32/fmifs/precomp.h trunk/reactos/dll/win32/framedyn/chstring.cpp trunk/reactos/drivers/network/ndis/include/ndissys.h trunk/reactos/include/ddk/stdunk.h trunk/reactos/include/psdk/kcom.h trunk/reactos/lib/sdk/cpprt/i386/cpprt.s Modified: trunk/reactos/dll/win32/fmifs/precomp.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/fmifs/precomp.h?… ============================================================================== --- trunk/reactos/dll/win32/fmifs/precomp.h [iso-8859-1] (original) +++ trunk/reactos/dll/win32/fmifs/precomp.h [iso-8859-1] Thu Apr 30 22:09:11 2015 @@ -13,10 +13,9 @@ /* INCLUDES ******************************************************************/ -#define WIN32_NO_STATUS - #include <stdio.h> +#define WIN32_NO_STATUS /* PSDK/NDK Headers */ #include <windef.h> Modified: trunk/reactos/dll/win32/framedyn/chstring.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/framedyn/chstrin… ============================================================================== --- trunk/reactos/dll/win32/framedyn/chstring.cpp [iso-8859-1] (original) +++ trunk/reactos/dll/win32/framedyn/chstring.cpp [iso-8859-1] Thu Apr 30 22:09:11 2015 @@ -54,19 +54,6 @@ } } -#if _MSC_VER >= 51900 -void __cdecl operator delete(void* ptr, unsigned int) -{ - // In Windows 2k3, they check for ptr being null. - // ISO, POSIX and even MSDN explains that it is allowed - // to call free with NULL pointer... - if (ptr) - { - free(ptr); - } -} -#endif - // Implement our own new operator so that we can throw our own exception in case // of allocation failure. // It could have been done using set_new_handler(), but well. MS guys didn't do it Modified: trunk/reactos/drivers/network/ndis/include/ndissys.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/ndis/inclu… ============================================================================== --- trunk/reactos/drivers/network/ndis/include/ndissys.h [iso-8859-1] (original) +++ trunk/reactos/drivers/network/ndis/include/ndissys.h [iso-8859-1] Thu Apr 30 22:09:11 2015 @@ -9,13 +9,6 @@ */ #ifndef __NDISSYS_H #define __NDISSYS_H - -/* portability fixes */ -#ifdef _M_AMD64 -#define KfReleaseSpinLock KeReleaseSpinLock -#define KefAcquireSpinLockAtDpcLevel KeAcquireSpinLockAtDpcLevel -#define KefReleaseSpinLockFromDpcLevel KeReleaseSpinLockFromDpcLevel -#endif #include <ndis.h> @@ -61,4 +54,11 @@ ExGetCurrentProcessorCpuUsage( PULONG CpuUsage); +/* portability fixes */ +#ifdef _M_AMD64 +#define KfReleaseSpinLock KeReleaseSpinLock +#define KefAcquireSpinLockAtDpcLevel KeAcquireSpinLockAtDpcLevel +#define KefReleaseSpinLockFromDpcLevel KeReleaseSpinLockFromDpcLevel +#endif + #endif /* __NDISSYS_H */ Modified: trunk/reactos/include/ddk/stdunk.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ddk/stdunk.h?rev=6… ============================================================================== --- trunk/reactos/include/ddk/stdunk.h [iso-8859-1] (original) +++ trunk/reactos/include/ddk/stdunk.h [iso-8859-1] Thu Apr 30 22:09:11 2015 @@ -206,7 +206,7 @@ inline void __cdecl operator delete( - PVOID ptr, UINT unk) + PVOID ptr, UINT) { ExFreePool(ptr); } Modified: trunk/reactos/include/psdk/kcom.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/kcom.h?rev=67… ============================================================================== --- trunk/reactos/include/psdk/kcom.h [iso-8859-1] (original) +++ trunk/reactos/include/psdk/kcom.h [iso-8859-1] Thu Apr 30 22:09:11 2015 @@ -235,7 +235,7 @@ } inline void __cdecl operator delete( - PVOID pVoid, UINT unk) + PVOID pVoid, UINT) { if (pVoid) ExFreePool(pVoid); } Modified: trunk/reactos/lib/sdk/cpprt/i386/cpprt.s URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/sdk/cpprt/i386/cpprt.s… ============================================================================== --- trunk/reactos/lib/sdk/cpprt/i386/cpprt.s [iso-8859-1] (original) +++ trunk/reactos/lib/sdk/cpprt/i386/cpprt.s [iso-8859-1] Thu Apr 30 22:09:11 2015 @@ -27,7 +27,7 @@ ; void __stdcall `eh vector constructor iterator'(void *,unsigned int,int,void (__thiscall*)(void *),void (__thiscall*)(void *)) DEFINE_ALIAS ??_L@YGXPAXIHP6EX0@Z1@Z, ?MSVCRTEX_eh_vector_constructor_iterator@@YGXPAXIHP6EX0@Z1@Z -; void __stdcall `eh vector constructor iterator'(void *,unsigned int,int,void (__thiscall*)(void *),void (__thiscall*)(void *)) +; void __stdcall `eh vector constructor iterator'(void *,unsigned int,unsigned int,void (__thiscall*)(void *),void (__thiscall*)(void *)) DEFINE_ALIAS ??_L@YGXPAXIIP6EX0@Z1@Z, ?MSVCRTEX_eh_vector_constructor_iterator@@YGXPAXIHP6EX0@Z1@Z ; void __stdcall `eh vector destructor iterator'(void *,unsigned int,int,void (__thiscall*)(void *)) @@ -38,6 +38,5 @@ ; void __cdecl operator delete(void *,unsigned int) DEFINE_ALIAS ??3@YAXPAXI@Z, ??3@YAXPAX@Z -DEFINE_ALIAS ??3@YAXPAXII@Z, ??3@YAXPAX@Z END

1 0

[dquintana] 67483: This commit brings support for compiling ReactOS with Visual Studio 2015 RC (and possibly the final release). [BUILD] msvc.cmake: Disable thread-local static initialization. CMak...
by dquintana＠svn.reactos.org 30 Apr '15

30 Apr '15

Author: dquintana Date: Thu Apr 30 21:48:26 2015 New Revision: 67483 URL: http://svn.reactos.org/svn/reactos?rev=67483&view=rev Log: This commit brings support for compiling ReactOS with Visual Studio 2015 RC (and possibly the final release). [BUILD] msvc.cmake: Disable thread-local static initialization. CMakeLists.txt: Disable PCH for VS2015. configure.cmd: Make it aware of cl.exe version 19.x [CPPRT] Add alias for the new variants of the delete operators. [BROWSEUI] [MFIFS] [FRAMEDYN] [NDIS] [DDK] [PSDK] [STLPORT] Add explicit declarations of the new delete operators for those modules that don't use the WITH_RUNTIME option. [WIDL] [WPP] Do not alias the snprintf family of functions to the _snprintf variants, since VS14 already declares them internally. Modified: trunk/reactos/CMakeLists.txt trunk/reactos/cmake/msvc.cmake trunk/reactos/configure.cmd trunk/reactos/dll/win32/browseui/utility.cpp trunk/reactos/dll/win32/fmifs/precomp.h trunk/reactos/dll/win32/framedyn/chstring.cpp trunk/reactos/drivers/network/ndis/include/ndissys.h trunk/reactos/include/ddk/stdunk.h trunk/reactos/include/psdk/kcom.h trunk/reactos/lib/3rdparty/stlport/test/eh/nc_alloc.cpp trunk/reactos/lib/sdk/cpprt/i386/cpprt.s trunk/reactos/tools/widl/CMakeLists.txt trunk/reactos/tools/wpp/CMakeLists.txt Modified: trunk/reactos/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/CMakeLists.txt?rev=67483&r… ============================================================================== --- trunk/reactos/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/CMakeLists.txt [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -137,7 +137,7 @@ add_definitions(-D_WINKD_=1) endif() - if(CMAKE_VERSION MATCHES "ReactOS") + if(CMAKE_VERSION MATCHES "ReactOS" AND MSVC_VERSION LESS 1900) set(PCH 1 CACHE BOOL "Whether to use precompiled headers") else() set(PCH 0 CACHE BOOL "Whether to use precompiled headers") Modified: trunk/reactos/cmake/msvc.cmake URL: http://svn.reactos.org/svn/reactos/trunk/reactos/cmake/msvc.cmake?rev=67483… ============================================================================== --- trunk/reactos/cmake/msvc.cmake [iso-8859-1] (original) +++ trunk/reactos/cmake/msvc.cmake [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -31,6 +31,11 @@ # VS 12+ requires /FS when used in parallel compilations if(MSVC_VERSION GREATER 1799 AND NOT MSVC_IDE) add_compile_flags("/FS") +endif () + +# VS14+ tries to use thread-safe initialization +if(MSVC_VERSION GREATER 1899) + add_compile_flags("/Zc:threadSafeInit-") endif () # Disable overly sensitive warnings as well as those that generally aren't Modified: trunk/reactos/configure.cmd URL: http://svn.reactos.org/svn/reactos/trunk/reactos/configure.cmd?rev=67483&r1… ============================================================================== --- trunk/reactos/configure.cmd [iso-8859-1] (original) +++ trunk/reactos/configure.cmd [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -56,6 +56,7 @@ cl 2>&1 | find "16.00." > NUL && set VS_VERSION=10 cl 2>&1 | find "17.00." > NUL && set VS_VERSION=11 cl 2>&1 | find "18.00." > NUL && set VS_VERSION=12 + cl 2>&1 | find "19.00." > NUL && set VS_VERSION=14 if not defined VS_VERSION ( echo Error: Visual Studio version too old or version detection failed. exit /b Modified: trunk/reactos/dll/win32/browseui/utility.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/browseui/utility… ============================================================================== --- trunk/reactos/dll/win32/browseui/utility.cpp [iso-8859-1] (original) +++ trunk/reactos/dll/win32/browseui/utility.cpp [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -9,3 +9,8 @@ { LocalFree(p); } + +void operator delete(void *p, unsigned int) +{ + LocalFree(p); +} Modified: trunk/reactos/dll/win32/fmifs/precomp.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/fmifs/precomp.h?… ============================================================================== --- trunk/reactos/dll/win32/fmifs/precomp.h [iso-8859-1] (original) +++ trunk/reactos/dll/win32/fmifs/precomp.h [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -13,9 +13,10 @@ /* INCLUDES ******************************************************************/ +#define WIN32_NO_STATUS + #include <stdio.h> -#define WIN32_NO_STATUS /* PSDK/NDK Headers */ #include <windef.h> Modified: trunk/reactos/dll/win32/framedyn/chstring.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/framedyn/chstrin… ============================================================================== --- trunk/reactos/dll/win32/framedyn/chstring.cpp [iso-8859-1] (original) +++ trunk/reactos/dll/win32/framedyn/chstring.cpp [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -54,6 +54,19 @@ } } +#if _MSC_VER >= 51900 +void __cdecl operator delete(void* ptr, unsigned int) +{ + // In Windows 2k3, they check for ptr being null. + // ISO, POSIX and even MSDN explains that it is allowed + // to call free with NULL pointer... + if (ptr) + { + free(ptr); + } +} +#endif + // Implement our own new operator so that we can throw our own exception in case // of allocation failure. // It could have been done using set_new_handler(), but well. MS guys didn't do it Modified: trunk/reactos/drivers/network/ndis/include/ndissys.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/ndis/inclu… ============================================================================== --- trunk/reactos/drivers/network/ndis/include/ndissys.h [iso-8859-1] (original) +++ trunk/reactos/drivers/network/ndis/include/ndissys.h [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -9,6 +9,13 @@ */ #ifndef __NDISSYS_H #define __NDISSYS_H + +/* portability fixes */ +#ifdef _M_AMD64 +#define KfReleaseSpinLock KeReleaseSpinLock +#define KefAcquireSpinLockAtDpcLevel KeAcquireSpinLockAtDpcLevel +#define KefReleaseSpinLockFromDpcLevel KeReleaseSpinLockFromDpcLevel +#endif #include <ndis.h> @@ -54,11 +61,4 @@ ExGetCurrentProcessorCpuUsage( PULONG CpuUsage); -/* portability fixes */ -#ifdef _M_AMD64 -#define KfReleaseSpinLock KeReleaseSpinLock -#define KefAcquireSpinLockAtDpcLevel KeAcquireSpinLockAtDpcLevel -#define KefReleaseSpinLockFromDpcLevel KeReleaseSpinLockFromDpcLevel -#endif - #endif /* __NDISSYS_H */ Modified: trunk/reactos/include/ddk/stdunk.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ddk/stdunk.h?rev=6… ============================================================================== --- trunk/reactos/include/ddk/stdunk.h [iso-8859-1] (original) +++ trunk/reactos/include/ddk/stdunk.h [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -204,6 +204,13 @@ ExFreePool(ptr); } +inline void __cdecl +operator delete( + PVOID ptr, UINT unk) +{ + ExFreePool(ptr); +} + #endif /* ALLOCATION_OPERATORS_DEFINED */ Modified: trunk/reactos/include/psdk/kcom.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/kcom.h?rev=67… ============================================================================== --- trunk/reactos/include/psdk/kcom.h [iso-8859-1] (original) +++ trunk/reactos/include/psdk/kcom.h [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -234,6 +234,12 @@ if (pVoid) ExFreePool(pVoid); } +inline void __cdecl operator delete( + PVOID pVoid, UINT unk) +{ + if (pVoid) ExFreePool(pVoid); +} + #endif /* _NEW_DELETE_OPERATORS_ */ #if defined(_SYS_GUID_OPERATOR_EQ_) Modified: trunk/reactos/lib/3rdparty/stlport/test/eh/nc_alloc.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/3rdparty/stlport/test/… ============================================================================== --- trunk/reactos/lib/3rdparty/stlport/test/eh/nc_alloc.cpp [iso-8859-1] (original) +++ trunk/reactos/lib/3rdparty/stlport/test/eh/nc_alloc.cpp [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -264,6 +264,14 @@ } } +#if defined (EH_DELETE_HAS_THROW_SPEC) +void _STLP_CALL operator delete(void* s, unsigned int) throw() +#else +void _STLP_CALL operator delete(void* s, unsigned int) +#endif +{ + ::operator delete(s); +} /*=================================================================================== ClearAllocationSet (private helper) Modified: trunk/reactos/lib/sdk/cpprt/i386/cpprt.s URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/sdk/cpprt/i386/cpprt.s… ============================================================================== --- trunk/reactos/lib/sdk/cpprt/i386/cpprt.s [iso-8859-1] (original) +++ trunk/reactos/lib/sdk/cpprt/i386/cpprt.s [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -27,7 +27,17 @@ ; void __stdcall `eh vector constructor iterator'(void *,unsigned int,int,void (__thiscall*)(void *),void (__thiscall*)(void *)) DEFINE_ALIAS ??_L@YGXPAXIHP6EX0@Z1@Z, ?MSVCRTEX_eh_vector_constructor_iterator@@YGXPAXIHP6EX0@Z1@Z +; void __stdcall `eh vector constructor iterator'(void *,unsigned int,int,void (__thiscall*)(void *),void (__thiscall*)(void *)) +DEFINE_ALIAS ??_L@YGXPAXIIP6EX0@Z1@Z, ?MSVCRTEX_eh_vector_constructor_iterator@@YGXPAXIHP6EX0@Z1@Z + ; void __stdcall `eh vector destructor iterator'(void *,unsigned int,int,void (__thiscall*)(void *)) DEFINE_ALIAS ??_M@YGXPAXIHP6EX0@Z@Z, ?MSVCRTEX_eh_vector_destructor_iterator@@YGXPAXIHP6EX0@Z@Z +; void __stdcall `eh vector destructor iterator'(void *,unsigned int,unsigned int,void (__thiscall*)(void *)) +DEFINE_ALIAS ??_M@YGXPAXIIP6EX0@Z@Z, ?MSVCRTEX_eh_vector_destructor_iterator@@YGXPAXIHP6EX0@Z@Z + +; void __cdecl operator delete(void *,unsigned int) +DEFINE_ALIAS ??3@YAXPAXI@Z, ??3@YAXPAX@Z +DEFINE_ALIAS ??3@YAXPAXII@Z, ??3@YAXPAX@Z + END Modified: trunk/reactos/tools/widl/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/widl/CMakeLists.txt?… ============================================================================== --- trunk/reactos/tools/widl/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/tools/widl/CMakeLists.txt [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -1,10 +1,12 @@ if(MSVC) - add_definitions(-Dsnprintf=_snprintf) + if(MSVC_VERSION LESS 1900) + add_definitions(-Dsnprintf=_snprintf) - # Add this definition for WDK only, VS 9 doesn't like that - if(DEFINED ENV{DDKBUILDENV}) - add_definitions(-Dvsnprintf=_vsnprintf) + # Add this definition for WDK only, VS 9 doesn't like that + if(DEFINED ENV{DDKBUILDENV}) + add_definitions(-Dvsnprintf=_vsnprintf) + endif() endif() list(APPEND SOURCE getopt.c) Modified: trunk/reactos/tools/wpp/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/wpp/CMakeLists.txt?r… ============================================================================== --- trunk/reactos/tools/wpp/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/tools/wpp/CMakeLists.txt [iso-8859-1] Thu Apr 30 21:48:26 2015 @@ -1,13 +1,15 @@ if(MSVC) - add_definitions( - -Dsnprintf=_snprintf - -Dstrtoull=_strtoui64 - -Dstrtoll=_strtoi64) + if(MSVC_VERSION LESS 1900) + add_definitions( + -Dsnprintf=_snprintf + -Dstrtoull=_strtoui64 + -Dstrtoll=_strtoi64) - # Add this definition for WDK only, VS 9 doesn't like that - if(DEFINED ENV{DDKBUILDENV}) - add_definitions(-Dvsnprintf=_vsnprintf) + # Add this definition for WDK only, VS 9 doesn't like that + if(DEFINED ENV{DDKBUILDENV}) + add_definitions(-Dvsnprintf=_vsnprintf) + endif() endif() endif()

1 0

[tkreuzer] 67482: [CRT/KERNEL32] Add some more ARM stubs
by tkreuzer＠svn.reactos.org 30 Apr '15

30 Apr '15

Author: tkreuzer Date: Thu Apr 30 21:07:08 2015 New Revision: 67482 URL: http://svn.reactos.org/svn/reactos?rev=67482&view=rev Log: [CRT/KERNEL32] Add some more ARM stubs Added: trunk/reactos/dll/win32/kernel32/client/arm/ (with props) trunk/reactos/dll/win32/kernel32/client/arm/fiber.S (with props) trunk/reactos/dll/win32/kernel32/client/arm/thread.S (with props) trunk/reactos/lib/sdk/crt/except/arm/__jump_unwind.s (with props) Propchange: trunk/reactos/dll/win32/kernel32/client/arm/ ------------------------------------------------------------------------------ --- bugtraq:logregex (added) +++ bugtraq:logregex Thu Apr 30 21:07:08 2015 @@ -0,0 +1 @@ +((CORE|ROSTESTS|ROSAPPS)-\d+)(,? ?((CORE|ROSTESTS|ROSAPPS)-\d+))*(,? ?(and |or )?((CORE|ROSTESTS|ROSAPPS)-\d+))? Propchange: trunk/reactos/dll/win32/kernel32/client/arm/ ------------------------------------------------------------------------------ bugtraq:message = See issue %BUGID% for more details. Propchange: trunk/reactos/dll/win32/kernel32/client/arm/ ------------------------------------------------------------------------------ bugtraq:url = https://jira.reactos.org/browse/%BUGID% Propchange: trunk/reactos/dll/win32/kernel32/client/arm/ ------------------------------------------------------------------------------ tsvn:logminsize = 10 Added: trunk/reactos/dll/win32/kernel32/client/arm/fiber.S URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/kernel32/client/… ============================================================================== --- trunk/reactos/dll/win32/kernel32/client/arm/fiber.S (added) +++ trunk/reactos/dll/win32/kernel32/client/arm/fiber.S [iso-8859-1] Thu Apr 30 21:07:08 2015 @@ -0,0 +1,24 @@ +/* + * COPYRIGHT: See COPYING in the top level directory + * PROJECT: ReactOS system libraries + * FILE: dll/win32//kernel32/client/arm/fiber.S + * PURPOSE: Fiber context switch code for the amd64 architecture + * PROGRAMMER: + */ + +#include <ksarm.h> + + TEXTAREA + + LEAF_ENTRY BaseFiberStartup + __assertfail + bx lr + LEAF_END BaseFiberStartup + + LEAF_ENTRY SwitchToFiber + __assertfail + bx lr + LEAF_END SwitchToFiber + + END +/* EOF */ Propchange: trunk/reactos/dll/win32/kernel32/client/arm/fiber.S ------------------------------------------------------------------------------ svn:eol-style = native Added: trunk/reactos/dll/win32/kernel32/client/arm/thread.S URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/kernel32/client/… ============================================================================== --- trunk/reactos/dll/win32/kernel32/client/arm/thread.S (added) +++ trunk/reactos/dll/win32/kernel32/client/arm/thread.S [iso-8859-1] Thu Apr 30 21:07:08 2015 @@ -0,0 +1,27 @@ +/* + * COPYRIGHT: See COPYING in the top level directory + * PROJECT: ReactOS system libraries + * FILE: lib/kernel32/thread/arm/thread.S + * PURPOSE: Thread and process start thunks + * PROGRAMMER: + */ + +#include <ksarm.h> + + IMPORT BaseThreadStartup + IMPORT BaseProcessStartup + + TEXTAREA + + LEAF_ENTRY BaseThreadStartupThunk + __assertfail + bx lr + LEAF_END BaseThreadStartupThunk + + LEAF_ENTRY BaseProcessStartThunk + __assertfail + bx lr + LEAF_END BaseProcessStartThunk + + END +/* EOF */ Propchange: trunk/reactos/dll/win32/kernel32/client/arm/thread.S ------------------------------------------------------------------------------ svn:eol-style = native Added: trunk/reactos/lib/sdk/crt/except/arm/__jump_unwind.s URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/sdk/crt/except/arm/__j… ============================================================================== --- trunk/reactos/lib/sdk/crt/except/arm/__jump_unwind.s (added) +++ trunk/reactos/lib/sdk/crt/except/arm/__jump_unwind.s [iso-8859-1] Thu Apr 30 21:07:08 2015 @@ -0,0 +1,21 @@ +/* + * COPYRIGHT: See COPYING in the top level directory + * PROJECT: ReactOS CRT library + * PURPOSE: Implementation of __jmp_unwind + * PROGRAMMER: Timo Kreuzer (timo.kreuzer(a)reactos.org) + */ + +/* INCLUDES ******************************************************************/ + +#include <kxarm.h> + +/* CODE **********************************************************************/ + TEXTAREA + + LEAF_ENTRY __jmp_unwind + __assertfail + bx lr + LEAF_END __jmp_unwind + + END +/* EOF */ Propchange: trunk/reactos/lib/sdk/crt/except/arm/__jump_unwind.s ------------------------------------------------------------------------------ svn:eol-style = native

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs