Ros-diffs June 2015

ros-diffs@reactos.org

13 participants
343 discussions

[aandrejevic] 68027: [FAST486] opgroups.c: We shouldn't restore SP in the opcode handler as that might interfere with triple faults. Also, in this case it's also wrong because the exception procedu...
by aandrejevic＠svn.reactos.org 05 Jun '15

05 Jun '15

Author: aandrejevic Date: Fri Jun 5 20:03:55 2015 New Revision: 68027 URL: http://svn.reactos.org/svn/reactos?rev=68027&view=rev Log: [FAST486] opgroups.c: We shouldn't restore SP in the opcode handler as that might interfere with triple faults. Also, in this case it's also wrong because the exception procedure already restores the stack pointer. common.c: Fix a comment, no code changes. Modified: trunk/reactos/lib/fast486/common.c trunk/reactos/lib/fast486/opgroups.c Modified: trunk/reactos/lib/fast486/common.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/common.c?rev=6… ============================================================================== --- trunk/reactos/lib/fast486/common.c [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/common.c [iso-8859-1] Fri Jun 5 20:03:55 2015 @@ -579,7 +579,7 @@ /* Restore the IP to the saved IP */ State->InstPtr = State->SavedInstPtr; - /* Restore the SP to the saved SP */ + /* Restore the SP to the saved SP */ State->GeneralRegs[FAST486_REG_ESP] = State->SavedStackPtr; /* Get the interrupt vector */ Modified: trunk/reactos/lib/fast486/opgroups.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/opgroups.c?rev… ============================================================================== --- trunk/reactos/lib/fast486/opgroups.c [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/opgroups.c [iso-8859-1] Fri Jun 5 20:03:55 2015 @@ -494,10 +494,7 @@ if (!Fast486ParseModRegRm(State, AddressSize, &ModRegRm)) { - /* Exception occurred - restore SP */ - if (OperandSize) State->GeneralRegs[FAST486_REG_ESP].Long -= sizeof(ULONG); - else State->GeneralRegs[FAST486_REG_ESP].LowWord -= sizeof(USHORT); - + /* Exception occurred */ return; }

1 0

[ekohl] 68026: [SYSSETUP] - Add missing VolumeClassInstaller export. - Implement TapeClassInstaller and VolumeClassInstaller.
by ekohl＠svn.reactos.org 05 Jun '15

05 Jun '15

Author: ekohl Date: Fri Jun 5 19:09:41 2015 New Revision: 68026 URL: http://svn.reactos.org/svn/reactos?rev=68026&view=rev Log: [SYSSETUP] - Add missing VolumeClassInstaller export. - Implement TapeClassInstaller and VolumeClassInstaller. Modified: trunk/reactos/dll/win32/syssetup/classinst.c trunk/reactos/dll/win32/syssetup/syssetup.spec Modified: trunk/reactos/dll/win32/syssetup/classinst.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/syssetup/classin… ============================================================================== --- trunk/reactos/dll/win32/syssetup/classinst.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/syssetup/classinst.c [iso-8859-1] Fri Jun 5 19:09:41 2015 @@ -1,7 +1,7 @@ /* * PROJECT: ReactOS system libraries * LICENSE: GPL - See COPYING in the top level directory - * FILE: drivers/storage/mountmgr/database.c + * FILE: dll/win32/syssetup/classinst.c * PURPOSE: Class installers * PROGRAMMERS: Copyright 2006 Hervé Poussineau (hpoussin(a)reactos.org) */ @@ -99,6 +99,7 @@ IN HDEVINFO DeviceInfoSet, IN PSP_DEVINFO_DATA DeviceInfoData OPTIONAL) { + DPRINT("HdcClassInstaller()\n"); return ERROR_DI_DO_DEFAULT; } @@ -200,7 +201,7 @@ /* - * @unimplemented + * @implemented */ DWORD WINAPI @@ -209,12 +210,23 @@ IN HDEVINFO DeviceInfoSet, IN PSP_DEVINFO_DATA DeviceInfoData OPTIONAL) { - switch (InstallFunction) - { - default: - DPRINT("Install function %u ignored\n", InstallFunction); - return ERROR_DI_DO_DEFAULT; - } + DPRINT("TapeClassInstaller()\n"); + return ERROR_DI_DO_DEFAULT; +} + + +/* + * @implemented + */ +DWORD +WINAPI +VolumeClassInstaller( + IN DI_FUNCTION InstallFunction, + IN HDEVINFO DeviceInfoSet, + IN PSP_DEVINFO_DATA DeviceInfoData OPTIONAL) +{ + DPRINT("VolumeClassInstaller()\n"); + return ERROR_DI_DO_DEFAULT; } /* EOF */ Modified: trunk/reactos/dll/win32/syssetup/syssetup.spec URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/syssetup/syssetu… ============================================================================== --- trunk/reactos/dll/win32/syssetup/syssetup.spec [iso-8859-1] (original) +++ trunk/reactos/dll/win32/syssetup/syssetup.spec [iso-8859-1] Fri Jun 5 19:09:41 2015 @@ -82,5 +82,6 @@ @ stub UpdatePnpDeviceDrivers @ stub UpgradePrinters @ stub ViewSetupActionLog +@ stdcall VolumeClassInstaller(long ptr ptr) @ stub pSetupDebugPrint @ stub pSetuplogSfcError

1 0

[cfinck] 68025: [PRINTING] Add an include directory for all printing components containing definitions not found in the public headers. For now, that's just PackStrings in spoolss.h. [SPOOLSS_APITE...
by cfinck＠svn.reactos.org 05 Jun '15

05 Jun '15

Author: cfinck Date: Fri Jun 5 18:16:51 2015 New Revision: 68025 URL: http://svn.reactos.org/svn/reactos?rev=68025&view=rev Log: [PRINTING] Add an include directory for all printing components containing definitions not found in the public headers. For now, that's just PackStrings in spoolss.h. [SPOOLSS_APITEST] Use the new spoolss.h header. [WINPRINT_APITEST] Add an API-Test for the winprint Print Processor, so far for its EnumPrintProcessorDatatypesW function. Tests succeed in Windows Server 2003. winspool.drv also provides functions that go into winprint.dll, but as these tests show, they behave slightly different in terms of error codes due to the involved RPC and routing. Windows Server 2003 has winprint functions in localspl.dll, so you have to copy its localspl.dll to winprint.dll for testing. [WINPRINT] - Use PackStrings to simplify the code. - Fix test failures. Added: branches/colins-printing-for-freedom/reactos/win32ss/printing/include/ (with props) branches/colins-printing-for-freedom/reactos/win32ss/printing/include/spoolss.h (with props) branches/colins-printing-for-freedom/rostests/apitests/winprint/ (with props) branches/colins-printing-for-freedom/rostests/apitests/winprint/CMakeLists.txt (with props) branches/colins-printing-for-freedom/rostests/apitests/winprint/EnumPrintProcessorDatatypesW.c (with props) branches/colins-printing-for-freedom/rostests/apitests/winprint/testlist.c (with props) Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/CMakeLists.txt branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/CMakeLists.txt branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/main.c branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/precomp.h branches/colins-printing-for-freedom/rostests/apitests/CMakeLists.txt branches/colins-printing-for-freedom/rostests/apitests/spoolss/CMakeLists.txt branches/colins-printing-for-freedom/rostests/apitests/spoolss/PackStrings.c Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/rea… ============================================================================== --- branches/colins-printing-for-freedom/reactos/win32ss/printing/CMakeLists.txt [iso-8859-1] (original) +++ branches/colins-printing-for-freedom/reactos/win32ss/printing/CMakeLists.txt [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -1,3 +1,5 @@ +include_directories(include) + add_subdirectory(base) #add_subdirectory(drivers) add_subdirectory(monitors) Propchange: branches/colins-printing-for-freedom/reactos/win32ss/printing/include/ ------------------------------------------------------------------------------ --- bugtraq:logregex (added) +++ bugtraq:logregex Fri Jun 5 18:16:51 2015 @@ -0,0 +1,2 @@ +([Ii]ssue|[Bb]ug)s? #?(\d+)(,? ?#?(\d+))*(,? ?(and |or )?#?(\d+))? +(\d+) Propchange: branches/colins-printing-for-freedom/reactos/win32ss/printing/include/ ------------------------------------------------------------------------------ bugtraq:message = See issue #%BUGID% for more details. Propchange: branches/colins-printing-for-freedom/reactos/win32ss/printing/include/ ------------------------------------------------------------------------------ bugtraq:url = http://www.reactos.org/bugzilla/show_bug.cgi?id=%BUGID% Propchange: branches/colins-printing-for-freedom/reactos/win32ss/printing/include/ ------------------------------------------------------------------------------ tsvn:logminsize = 10 Added: branches/colins-printing-for-freedom/reactos/win32ss/printing/include/spoolss.h URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/rea… ============================================================================== --- branches/colins-printing-for-freedom/reactos/win32ss/printing/include/spoolss.h (added) +++ branches/colins-printing-for-freedom/reactos/win32ss/printing/include/spoolss.h [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -0,0 +1,14 @@ +/* + * PROJECT: ReactOS Printing Include files + * LICENSE: GNU LGPLv2 or any later version as published by the Free Software Foundation + * PURPOSE: Undocumented APIs of the Spooler Router "spoolss.dll" + * COPYRIGHT: Copyright 2015 Colin Finck <colin(a)reactos.org> + */ + +#ifndef _REACTOS_SPOOLSS_H +#define _REACTOS_SPOOLSS_H + +PBYTE WINAPI +PackStrings(PCWSTR* pSource, PBYTE pDest, PDWORD DestOffsets, PBYTE pEnd); + +#endif Propchange: branches/colins-printing-for-freedom/reactos/win32ss/printing/include/spoolss.h ------------------------------------------------------------------------------ svn:eol-style = native Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/rea… ============================================================================== --- branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/CMakeLists.txt [iso-8859-1] (original) +++ branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/CMakeLists.txt [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -13,6 +13,6 @@ set_module_type(winprint win32dll UNICODE) target_link_libraries(winprint wine) -add_importlibs(winprint kernel32 msvcrt ntdll) +add_importlibs(winprint spoolss msvcrt kernel32 ntdll) add_pch(winprint precomp.h SOURCE) add_cd_file(TARGET winprint DESTINATION reactos/system32/spool/prtprocs/w32x86 FOR all) Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/main.c URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/rea… ============================================================================== --- branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/main.c [iso-8859-1] (original) +++ branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/main.c [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -48,18 +48,14 @@ BOOL WINAPI EnumPrintProcessorDatatypesW(LPWSTR pName, LPWSTR pPrintProcessorName, DWORD Level, LPBYTE pDatatypes, DWORD cbBuf, LPDWORD pcbNeeded, LPDWORD pcReturned) { - DATATYPES_INFO_1W DatatypesInfo1; DWORD cbDatatype; - PBYTE pCurrentOutputDatatype; - PBYTE pCurrentOutputDatatypeInfo; + DWORD dwOffsets[_countof(pwszDatatypes)]; PCWSTR* pCurrentDatatype; + PDWORD pCurrentOffset = dwOffsets; // Sanity checks if (Level != 1 || !pcbNeeded || !pcReturned) - { - SetLastError(ERROR_INVALID_PARAMETER); return FALSE; - } // Count the required buffer size and the number of datatypes. *pcbNeeded = 0; @@ -69,7 +65,12 @@ { cbDatatype = (wcslen(*pCurrentDatatype) + 1) * sizeof(WCHAR); *pcbNeeded += sizeof(DATATYPES_INFO_1W) + cbDatatype; + + // Also calculate the offset in the output buffer of the pointer to this datatype string. + *pCurrentOffset = *pcReturned * sizeof(DATATYPES_INFO_1W) + FIELD_OFFSET(DATATYPES_INFO_1W, pName); + *pcReturned++; + pCurrentOffset++; } // Check if the supplied buffer is large enough. @@ -79,25 +80,16 @@ return FALSE; } - // Put the datatype strings right after the last DATATYPES_INFO_1W structure. - pCurrentOutputDatatypeInfo = pDatatypes; - pCurrentOutputDatatype = pDatatypes + *pcReturned * sizeof(DATATYPES_INFO_1W); + // Check if a buffer was supplied at all. + if (!pDatatypes) + { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } // Copy over all datatypes. - for (pCurrentDatatype = pwszDatatypes; *pCurrentDatatype; pCurrentDatatype++) - { - // Copy the datatype string into the output buffer. - cbDatatype = (wcslen(*pCurrentDatatype) + 1) * sizeof(WCHAR); - CopyMemory(pCurrentOutputDatatype, *pCurrentDatatype, cbDatatype); - - // Fill and copy the DATATYPES_INFO_1W structure belonging to this datatype. - DatatypesInfo1.pName = (PWSTR)pCurrentOutputDatatype; - CopyMemory(pCurrentOutputDatatypeInfo, &DatatypesInfo1, sizeof(DATATYPES_INFO_1W)); - - // Advance to the next DATATYPES_INFO_1W location and string location in the output buffer. - pCurrentOutputDatatype += cbDatatype; - pCurrentOutputDatatypeInfo += sizeof(DATATYPES_INFO_1W); - } + *pCurrentOffset = MAXDWORD; + PackStrings(pwszDatatypes, pDatatypes, dwOffsets, &pDatatypes[*pcbNeeded]); return TRUE; } Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/precomp.h URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/rea… ============================================================================== --- branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/precomp.h [iso-8859-1] (original) +++ branches/colins-printing-for-freedom/reactos/win32ss/printing/processors/winprint/precomp.h [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -9,6 +9,7 @@ #define _PRECOMP_H #define WIN32_NO_STATUS +#include <stdlib.h> #include <windef.h> #include <winbase.h> #include <wingdi.h> @@ -16,6 +17,8 @@ #include <winspool.h> #include <winsplp.h> +#include <spoolss.h> + #include <wine/debug.h> WINE_DEFAULT_DEBUG_CHANNEL(winprint); Modified: branches/colins-printing-for-freedom/rostests/apitests/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/ros… ============================================================================== --- branches/colins-printing-for-freedom/rostests/apitests/CMakeLists.txt [iso-8859-1] (original) +++ branches/colins-printing-for-freedom/rostests/apitests/CMakeLists.txt [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -25,5 +25,6 @@ endif() add_subdirectory(winhttp) add_subdirectory(wininet) +add_subdirectory(winprint) add_subdirectory(winspool) add_subdirectory(ws2_32) Modified: branches/colins-printing-for-freedom/rostests/apitests/spoolss/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/ros… ============================================================================== --- branches/colins-printing-for-freedom/rostests/apitests/spoolss/CMakeLists.txt [iso-8859-1] (original) +++ branches/colins-printing-for-freedom/rostests/apitests/spoolss/CMakeLists.txt [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -1,3 +1,5 @@ + +include_directories(${REACTOS_SOURCE_DIR}/win32ss/printing/include) list(APPEND SOURCE PackStrings.c Modified: branches/colins-printing-for-freedom/rostests/apitests/spoolss/PackStrings.c URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/ros… ============================================================================== --- branches/colins-printing-for-freedom/rostests/apitests/spoolss/PackStrings.c [iso-8859-1] (original) +++ branches/colins-printing-for-freedom/rostests/apitests/spoolss/PackStrings.c [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -10,8 +10,7 @@ #define WIN32_NO_STATUS #include <windef.h> #include <winbase.h> - -PBYTE WINAPI PackStrings(PCWSTR*, PBYTE, PDWORD, PBYTE); +#include <spoolss.h> typedef struct _EXAMPLE_STRUCT { Propchange: branches/colins-printing-for-freedom/rostests/apitests/winprint/ ------------------------------------------------------------------------------ --- bugtraq:logregex (added) +++ bugtraq:logregex Fri Jun 5 18:16:51 2015 @@ -0,0 +1,2 @@ +([Ii]ssue|[Bb]ug)s? #?(\d+)(,? ?#?(\d+))*(,? ?(and |or )?#?(\d+))? +(\d+) Propchange: branches/colins-printing-for-freedom/rostests/apitests/winprint/ ------------------------------------------------------------------------------ bugtraq:message = See issue #%BUGID% for more details. Propchange: branches/colins-printing-for-freedom/rostests/apitests/winprint/ ------------------------------------------------------------------------------ bugtraq:url = http://www.reactos.org/bugzilla/show_bug.cgi?id=%BUGID% Added: branches/colins-printing-for-freedom/rostests/apitests/winprint/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/ros… ============================================================================== --- branches/colins-printing-for-freedom/rostests/apitests/winprint/CMakeLists.txt (added) +++ branches/colins-printing-for-freedom/rostests/apitests/winprint/CMakeLists.txt [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -0,0 +1,10 @@ + +list(APPEND SOURCE + EnumPrintProcessorDatatypesW.c + testlist.c) + +add_executable(winprint_apitest ${SOURCE}) +target_link_libraries(winprint_apitest wine ${PSEH_LIB}) +set_module_type(winprint_apitest win32cui) +add_importlibs(winprint_apitest winprint msvcrt kernel32 ntdll) +add_cd_file(TARGET winprint_apitest DESTINATION reactos/bin FOR all) Propchange: branches/colins-printing-for-freedom/rostests/apitests/winprint/CMakeLists.txt ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/colins-printing-for-freedom/rostests/apitests/winprint/EnumPrintProcessorDatatypesW.c URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/ros… ============================================================================== --- branches/colins-printing-for-freedom/rostests/apitests/winprint/EnumPrintProcessorDatatypesW.c (added) +++ branches/colins-printing-for-freedom/rostests/apitests/winprint/EnumPrintProcessorDatatypesW.c [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -0,0 +1,81 @@ +/* + * PROJECT: ReactOS Standard Print Processor API Tests + * LICENSE: GNU GPLv2 or any later version as published by the Free Software Foundation + * PURPOSE: Tests for EnumPrintProcessorDatatypesW + * COPYRIGHT: Copyright 2015 Colin Finck <colin(a)reactos.org> + */ + +#include <apitest.h> + +#define WIN32_NO_STATUS +#include <windef.h> +#include <winbase.h> +#include <wingdi.h> +#include <winspool.h> + +START_TEST(EnumPrintProcessorDatatypesW) +{ + DWORD cbNeeded; + DWORD cbTemp; + DWORD dwReturned; + PDATATYPES_INFO_1W pDatatypesInfo1; + + // Try with an invalid level. The error needs to be set by winspool, but not by the Print Processor. + SetLastError(0xDEADBEEF); + ok(!EnumPrintProcessorDatatypesW(NULL, NULL, 0, NULL, 0, NULL, NULL), "EnumPrintProcessorDatatypesW returns TRUE!\n"); + ok(GetLastError() == 0xDEADBEEF, "EnumPrintProcessorDatatypesW returns error %lu!\n", GetLastError()); + + // Now try with valid level, but no pcbNeeded and no pcReturned. The error needs to be set by RPC. + SetLastError(0xDEADBEEF); + ok(!EnumPrintProcessorDatatypesW(NULL, NULL, 1, NULL, 0, NULL, NULL), "EnumPrintProcessorDatatypesW returns TRUE!\n"); + ok(GetLastError() == 0xDEADBEEF, "EnumPrintProcessorDatatypesW returns error %lu!\n", GetLastError()); + + // Now try with pcbNeeded and pcReturned, but give no Print Processor. Show that winprint actually ignores the given Print Processor. + SetLastError(0xDEADBEEF); + ok(!EnumPrintProcessorDatatypesW(NULL, NULL, 1, NULL, 0, &cbNeeded, &dwReturned), "EnumPrintProcessorDatatypesW returns TRUE!\n"); + ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "EnumPrintProcessorDatatypesW returns error %lu!\n", GetLastError()); + ok(cbNeeded > 0, "cbNeeded is 0!\n"); + ok(dwReturned == 0, "dwReturned is %lu!\n", dwReturned); + + // Same error has to occur when looking for an invalid Print Processor. + SetLastError(0xDEADBEEF); + ok(!EnumPrintProcessorDatatypesW(NULL, L"invalid", 1, NULL, 0, &cbNeeded, &dwReturned), "EnumPrintProcessorDatatypesW returns TRUE!\n"); + ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "EnumPrintProcessorDatatypesW returns error %lu!\n", GetLastError()); + ok(cbNeeded > 0, "cbNeeded is 0!\n"); + ok(dwReturned == 0, "dwReturned is %lu!\n", dwReturned); + + // Now get the required buffer size by supplying all information. This needs to fail with ERROR_INSUFFICIENT_BUFFER. + SetLastError(0xDEADBEEF); + ok(!EnumPrintProcessorDatatypesW(NULL, L"winprint", 1, NULL, 0, &cbNeeded, &dwReturned), "EnumPrintProcessorDatatypesW returns TRUE!\n"); + ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "EnumPrintProcessorDatatypesW returns error %lu!\n", GetLastError()); + ok(cbNeeded > 0, "cbNeeded is 0!\n"); + ok(dwReturned == 0, "dwReturned is %lu!\n", dwReturned); + + // Same error has to occur with a size to small. + SetLastError(0xDEADBEEF); + ok(!EnumPrintProcessorDatatypesW(NULL, L"winprint", 1, NULL, 1, &cbNeeded, &dwReturned), "EnumPrintProcessorDatatypesW returns TRUE!\n"); + ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "EnumPrintersW returns error %lu!\n", GetLastError()); + ok(cbNeeded > 0, "cbNeeded is 0!\n"); + ok(dwReturned == 0, "dwReturned is %lu!\n", dwReturned); + + // Now provide the demanded size, but no buffer. Show that winprint returns a different error than the same function in winspool. + SetLastError(0xDEADBEEF); + ok(!EnumPrintProcessorDatatypesW(NULL, L"winprint", 1, NULL, cbNeeded, &cbTemp, &dwReturned), "EnumPrintProcessorDatatypesW returns TRUE!\n"); + ok(GetLastError() == ERROR_INVALID_PARAMETER, "EnumPrintProcessorDatatypesW returns error %lu!\n", GetLastError()); + ok(cbTemp == cbNeeded, "cbTemp is %lu!\n", cbTemp); + ok(dwReturned == 0, "dwReturned is %lu!\n", dwReturned); + + // This also has to fail the same way when no Print Processor was given at all. + SetLastError(0xDEADBEEF); + ok(!EnumPrintProcessorDatatypesW(NULL, NULL, 1, NULL, cbNeeded, &cbTemp, &dwReturned), "EnumPrintProcessorDatatypesW returns TRUE!\n"); + ok(GetLastError() == ERROR_INVALID_PARAMETER, "EnumPrintProcessorDatatypesW returns error %lu!\n", GetLastError()); + ok(cbTemp == cbNeeded, "cbTemp is %lu!\n", cbTemp); + ok(dwReturned == 0, "dwReturned is %lu!\n", dwReturned); + + // Finally use the function as intended and aim for success! Show that winprint doesn't modify the error code at all. + pDatatypesInfo1 = HeapAlloc(GetProcessHeap(), 0, cbNeeded); + SetLastError(0xDEADBEEF); + ok(EnumPrintProcessorDatatypesW(NULL, L"winprint", 1, (PBYTE)pDatatypesInfo1, cbNeeded, &cbNeeded, &dwReturned), "EnumPrintProcessorDatatypesW returns FALSE!\n"); + ok(GetLastError() == 0xDEADBEEF, "EnumPrintProcessorDatatypesW returns error %lu!\n", GetLastError()); + HeapFree(GetProcessHeap(), 0, pDatatypesInfo1); +} Propchange: branches/colins-printing-for-freedom/rostests/apitests/winprint/EnumPrintProcessorDatatypesW.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/colins-printing-for-freedom/rostests/apitests/winprint/testlist.c URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/ros… ============================================================================== --- branches/colins-printing-for-freedom/rostests/apitests/winprint/testlist.c (added) +++ branches/colins-printing-for-freedom/rostests/apitests/winprint/testlist.c [iso-8859-1] Fri Jun 5 18:16:51 2015 @@ -0,0 +1,32 @@ +/* + * PROJECT: ReactOS Standard Print Processor API Tests + * LICENSE: GNU GPLv2 or any later version as published by the Free Software Foundation + * PURPOSE: Test list + * COPYRIGHT: Copyright 2015 Colin Finck <colin(a)reactos.org> + */ + +/* + * These tests are developed and tested against the Windows Server 2003 counterpart of winprint. + * While ReactOS implements the Standard Print Processor in a separate module winprint.dll, + * Windows Server 2003 puts it into the Local Print Spooler localspl.dll. + * + * To test against Windows, simply run these tests under Windows Server 2003, but copy its + * localspl.dll to winprint.dll in advance. + * + * winspool.drv also provides functions that go into winprint.dll, but as these tests show, + * they behave slightly different in terms of error codes due to the involved RPC and routing. + */ + +#define __ROS_LONG64__ + +#define STANDALONE +#include <apitest.h> + +extern void func_EnumPrintProcessorDatatypesW(void); + +const struct test winetest_testlist[] = +{ + { "EnumPrintProcessorDatatypesW", func_EnumPrintProcessorDatatypesW }, + + { 0, 0 } +}; Propchange: branches/colins-printing-for-freedom/rostests/apitests/winprint/testlist.c ------------------------------------------------------------------------------ svn:eol-style = native

1 0

[akhaldi] 68024: [NTOSKRNL] Fix NtQuerySystemInformation not returning correct ResultLength for SystemFileCacheInformation. Fixes tons of ntdll:info tests. Brought to you by Tetsui Ohkubo. CORE-9757
by akhaldi＠svn.reactos.org 05 Jun '15

05 Jun '15

Author: akhaldi Date: Fri Jun 5 17:57:28 2015 New Revision: 68024 URL: http://svn.reactos.org/svn/reactos?rev=68024&view=rev Log: [NTOSKRNL] Fix NtQuerySystemInformation not returning correct ResultLength for SystemFileCacheInformation. Fixes tons of ntdll:info tests. Brought to you by Tetsui Ohkubo. CORE-9757 Modified: trunk/reactos/ntoskrnl/ex/sysinfo.c Modified: trunk/reactos/ntoskrnl/ex/sysinfo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ex/sysinfo.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/ex/sysinfo.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/ex/sysinfo.c [iso-8859-1] Fri Jun 5 17:57:28 2015 @@ -1291,9 +1291,10 @@ { SYSTEM_FILECACHE_INFORMATION *Sci = (SYSTEM_FILECACHE_INFORMATION *) Buffer; - if (Size < sizeof(SYSTEM_FILECACHE_INFORMATION)) - { - *ReqSize = sizeof(SYSTEM_FILECACHE_INFORMATION); + *ReqSize = sizeof(SYSTEM_FILECACHE_INFORMATION); + + if (Size < *ReqSize) + { return STATUS_INFO_LENGTH_MISMATCH; }

1 0

[cfinck] 68023: [SPOOLSS] In an attempt to simplify the code for further functions that return information structures, implement and document the undocumented PackStrings function. Hints were taken...
by cfinck＠svn.reactos.org 05 Jun '15

05 Jun '15

Author: cfinck Date: Fri Jun 5 15:52:39 2015 New Revision: 68023 URL: http://svn.reactos.org/svn/reactos?rev=68023&view=rev Log: [SPOOLSS] In an attempt to simplify the code for further functions that return information structures, implement and document the undocumented PackStrings function. Hints were taken from an XP DDK sample and own testing. [SPOOLSS_APITEST] Add some tests for PackStrings that succeed under Windows Server 2003. Added: branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/tools.c (with props) branches/colins-printing-for-freedom/rostests/apitests/spoolss/ (with props) branches/colins-printing-for-freedom/rostests/apitests/spoolss/CMakeLists.txt (with props) branches/colins-printing-for-freedom/rostests/apitests/spoolss/PackStrings.c (with props) branches/colins-printing-for-freedom/rostests/apitests/spoolss/testlist.c (with props) Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/CMakeLists.txt branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/spoolss.spec branches/colins-printing-for-freedom/rostests/apitests/CMakeLists.txt Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/rea… ============================================================================== --- branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/CMakeLists.txt [iso-8859-1] (original) +++ branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/CMakeLists.txt [iso-8859-1] Fri Jun 5 15:52:39 2015 @@ -4,7 +4,8 @@ list(APPEND SOURCE context.c main.c - precomp.h) + precomp.h + tools.c) add_library(spoolss SHARED ${SOURCE} Modified: branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/spoolss.spec URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/rea… ============================================================================== --- branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/spoolss.spec [iso-8859-1] (original) +++ branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/spoolss.spec [iso-8859-1] Fri Jun 5 15:52:39 2015 @@ -106,7 +106,7 @@ @ stub OpenPrinterExW @ stub OpenPrinterPortW @ stdcall OpenPrinterW(wstr ptr ptr) -@ stub PackStrings +@ stdcall PackStrings(ptr ptr ptr ptr) @ stub PartialReplyPrinterChangeNotification @ stub PlayGdiScriptOnPrinterIC @ stub PrinterHandleRundown Added: branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/tools.c URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/rea… ============================================================================== --- branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/tools.c (added) +++ branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/tools.c [iso-8859-1] Fri Jun 5 15:52:39 2015 @@ -0,0 +1,70 @@ +/* + * PROJECT: ReactOS Spooler Router + * LICENSE: GNU LGPL v2.1 or any later version as published by the Free Software Foundation + * PURPOSE: Miscellaneous tool functions + * COPYRIGHT: Copyright 2015 Colin Finck <colin(a)reactos.org> + */ + +#include "precomp.h" + + +/** + * @name PackStrings + * + * Takes an array of Unicode strings and fills an output buffer with these strings at the end and pointers to each string at specific offsets. + * Useful helper for functions that copy an information structure including strings into a given buffer (like PRINTER_INFO_1). + * + * @param pSource + * The array of Unicode strings to copy. Needs to have at least as many elements as the DestOffsets array. + * + * @param pDest + * Pointer to the beginning of the output buffer. + * The caller is responsible for verifying that this buffer is large enough to hold all strings and pointers. + * + * @param DestOffsets + * Array of byte offsets in the output buffer. For each element of DestOffsets, the function will copy the address of the corresponding copied + * string of pSource to this location in the output buffer. If a string in pSource is NULL, the function will set the pointer address to NULL + * in the output buffer. + * Use macros like FIELD_OFFSET to calculate the offsets for this array. + * The last element of the array must have the value MAXDWORD to let the function detect the end of the array. + * + * @param pEnd + * Pointer to the end of the output buffer. That means the first element outside of the buffer given in pDest. + * + * @return + * Returns a pointer to the beginning of the strings in pDest. + * The strings are copied in reverse order, so this pointer will point to the last copied string of pSource. + */ +PBYTE WINAPI +PackStrings(PCWSTR* pSource, PBYTE pDest, PDWORD DestOffsets, PBYTE pEnd) +{ + DWORD cbString; + ULONG_PTR StringAddress; + + // Loop until we reach an element with offset set to MAXDWORD. + while (*DestOffsets != MAXDWORD) + { + StringAddress = 0; + + if (*pSource) + { + // Determine the length of the source string. + cbString = (wcslen(*pSource) + 1) * sizeof(WCHAR); + + // Copy it before the last string. + pEnd -= cbString; + StringAddress = (ULONG_PTR)pEnd; + CopyMemory(pEnd, *pSource, cbString); + } + + // Copy the address of the copied string to the location given by the offset. + CopyMemory(&pDest[*DestOffsets], &StringAddress, sizeof(ULONG_PTR)); + + // Advance to the next source string and destination offset. + pSource++; + DestOffsets++; + } + + // pEnd is now at the last string we copied. Return this value as a pointer to the beginning of all strings in the output buffer. + return pEnd; +} Propchange: branches/colins-printing-for-freedom/reactos/win32ss/printing/base/spoolss/tools.c ------------------------------------------------------------------------------ svn:eol-style = native Modified: branches/colins-printing-for-freedom/rostests/apitests/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/ros… ============================================================================== --- branches/colins-printing-for-freedom/rostests/apitests/CMakeLists.txt [iso-8859-1] (original) +++ branches/colins-printing-for-freedom/rostests/apitests/CMakeLists.txt [iso-8859-1] Fri Jun 5 15:52:39 2015 @@ -17,6 +17,7 @@ add_subdirectory(setupapi) add_subdirectory(shell32) add_subdirectory(psapi) +add_subdirectory(spoolss) add_subdirectory(user32) if(NOT ARCH STREQUAL "amd64") add_subdirectory(w32kdll) Propchange: branches/colins-printing-for-freedom/rostests/apitests/spoolss/ ------------------------------------------------------------------------------ --- bugtraq:logregex (added) +++ bugtraq:logregex Fri Jun 5 15:52:39 2015 @@ -0,0 +1,2 @@ +([Ii]ssue|[Bb]ug)s? #?(\d+)(,? ?#?(\d+))*(,? ?(and |or )?#?(\d+))? +(\d+) Propchange: branches/colins-printing-for-freedom/rostests/apitests/spoolss/ ------------------------------------------------------------------------------ bugtraq:message = See issue #%BUGID% for more details. Propchange: branches/colins-printing-for-freedom/rostests/apitests/spoolss/ ------------------------------------------------------------------------------ bugtraq:url = http://www.reactos.org/bugzilla/show_bug.cgi?id=%BUGID% Added: branches/colins-printing-for-freedom/rostests/apitests/spoolss/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/ros… ============================================================================== --- branches/colins-printing-for-freedom/rostests/apitests/spoolss/CMakeLists.txt (added) +++ branches/colins-printing-for-freedom/rostests/apitests/spoolss/CMakeLists.txt [iso-8859-1] Fri Jun 5 15:52:39 2015 @@ -0,0 +1,10 @@ + +list(APPEND SOURCE + PackStrings.c + testlist.c) + +add_executable(spoolss_apitest ${SOURCE}) +target_link_libraries(spoolss_apitest wine ${PSEH_LIB}) +set_module_type(spoolss_apitest win32cui) +add_importlibs(spoolss_apitest spoolss msvcrt kernel32 ntdll) +add_cd_file(TARGET spoolss_apitest DESTINATION reactos/bin FOR all) Propchange: branches/colins-printing-for-freedom/rostests/apitests/spoolss/CMakeLists.txt ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/colins-printing-for-freedom/rostests/apitests/spoolss/PackStrings.c URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/ros… ============================================================================== --- branches/colins-printing-for-freedom/rostests/apitests/spoolss/PackStrings.c (added) +++ branches/colins-printing-for-freedom/rostests/apitests/spoolss/PackStrings.c [iso-8859-1] Fri Jun 5 15:52:39 2015 @@ -0,0 +1,48 @@ +/* + * PROJECT: ReactOS Spooler Router API Tests + * LICENSE: GNU GPLv2 or any later version as published by the Free Software Foundation + * PURPOSE: Tests for PackStrings + * COPYRIGHT: Copyright 2015 Colin Finck <colin(a)reactos.org> + */ + +#include <apitest.h> + +#define WIN32_NO_STATUS +#include <windef.h> +#include <winbase.h> + +PBYTE WINAPI PackStrings(PCWSTR*, PBYTE, PDWORD, PBYTE); + +typedef struct _EXAMPLE_STRUCT +{ + PWSTR String1; + PWSTR String2; +} +EXAMPLE_STRUCT, *PEXAMPLE_STRUCT; + +START_TEST(PackStrings) +{ + PCWSTR Source1[] = { L"Test", L"String" }; + PCWSTR Source2[] = { L"Test", NULL }; + + BYTE Buffer[50]; + PBYTE pEnd; + PEXAMPLE_STRUCT pStruct = (PEXAMPLE_STRUCT)Buffer; + DWORD Offsets[] = { + FIELD_OFFSET(EXAMPLE_STRUCT, String1), + FIELD_OFFSET(EXAMPLE_STRUCT, String2), + MAXDWORD + }; + + // Try a usual case with two strings. Verify that they are copied in reverse order. + pEnd = PackStrings(Source1, Buffer, Offsets, &Buffer[sizeof(Buffer)]); + ok(wcscmp(pStruct->String1, Source1[0]) == 0, "String1 and Source1[0] don't match!\n"); + ok(wcscmp(pStruct->String2, Source1[1]) == 0, "String2 and Source1[1] don't match!\n"); + ok(wcscmp((PWSTR)pEnd, Source1[1]) == 0, "pEnd and Source1[1] don't match!\n"); + + // Now verify that the corresponding pointer is set to NULL if a string is NULL. + pEnd = PackStrings(Source2, Buffer, Offsets, &Buffer[sizeof(Buffer)]); + ok(wcscmp(pStruct->String1, Source2[0]) == 0, "String1 and Source2[0] don't match!\n"); + ok(!pStruct->String2, "String2 is %p!\n", pStruct->String2); + ok(wcscmp((PWSTR)pEnd, Source2[0]) == 0, "pEnd and Source2[0] don't match!\n"); +} Propchange: branches/colins-printing-for-freedom/rostests/apitests/spoolss/PackStrings.c ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/colins-printing-for-freedom/rostests/apitests/spoolss/testlist.c URL: http://svn.reactos.org/svn/reactos/branches/colins-printing-for-freedom/ros… ============================================================================== --- branches/colins-printing-for-freedom/rostests/apitests/spoolss/testlist.c (added) +++ branches/colins-printing-for-freedom/rostests/apitests/spoolss/testlist.c [iso-8859-1] Fri Jun 5 15:52:39 2015 @@ -0,0 +1,20 @@ +/* + * PROJECT: ReactOS Print Spooler Router API Tests + * LICENSE: GNU GPLv2 or any later version as published by the Free Software Foundation + * PURPOSE: Test list + * COPYRIGHT: Copyright 2015 Colin Finck <colin(a)reactos.org> + */ + +#define __ROS_LONG64__ + +#define STANDALONE +#include <apitest.h> + +extern void func_PackStrings(void); + +const struct test winetest_testlist[] = +{ + { "PackStrings", func_PackStrings }, + + { 0, 0 } +}; Propchange: branches/colins-printing-for-freedom/rostests/apitests/spoolss/testlist.c ------------------------------------------------------------------------------ svn:eol-style = native

1 0

[ekohl] 68022: [SYSSETUP] Add CoInstaller stubs.
by ekohl＠svn.reactos.org 05 Jun '15

05 Jun '15

Author: ekohl Date: Fri Jun 5 15:19:09 2015 New Revision: 68022 URL: http://svn.reactos.org/svn/reactos?rev=68022&view=rev Log: [SYSSETUP] Add CoInstaller stubs. Modified: trunk/reactos/dll/win32/syssetup/classinst.c trunk/reactos/dll/win32/syssetup/syssetup.spec Modified: trunk/reactos/dll/win32/syssetup/classinst.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/syssetup/classin… ============================================================================== --- trunk/reactos/dll/win32/syssetup/classinst.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/syssetup/classinst.c [iso-8859-1] Fri Jun 5 15:19:09 2015 @@ -35,6 +35,26 @@ */ DWORD WINAPI +CriticalDeviceCoInstaller( + IN DI_FUNCTION InstallFunction, + IN HDEVINFO DeviceInfoSet, + IN PSP_DEVINFO_DATA DeviceInfoData OPTIONAL, + IN OUT PCOINSTALLER_CONTEXT_DATA Context) +{ + switch (InstallFunction) + { + default: + DPRINT1("Install function %u ignored\n", InstallFunction); + return ERROR_SUCCESS; + } +} + + +/* + * @unimplemented + */ +DWORD +WINAPI DeviceBayClassInstaller( IN DI_FUNCTION InstallFunction, IN HDEVINFO DeviceInfoSet, @@ -50,6 +70,26 @@ /* + * @unimplemented + */ +DWORD +WINAPI +EisaUpHalCoInstaller( + IN DI_FUNCTION InstallFunction, + IN HDEVINFO DeviceInfoSet, + IN PSP_DEVINFO_DATA DeviceInfoData OPTIONAL, + IN OUT PCOINSTALLER_CONTEXT_DATA Context) +{ + switch (InstallFunction) + { + default: + DPRINT1("Install function %u ignored\n", InstallFunction); + return ERROR_SUCCESS; + } +} + + +/* * @implemented */ DWORD @@ -144,6 +184,26 @@ */ DWORD WINAPI +StorageCoInstaller( + IN DI_FUNCTION InstallFunction, + IN HDEVINFO DeviceInfoSet, + IN PSP_DEVINFO_DATA DeviceInfoData OPTIONAL, + IN OUT PCOINSTALLER_CONTEXT_DATA Context) +{ + switch (InstallFunction) + { + default: + DPRINT1("Install function %u ignored\n", InstallFunction); + return ERROR_SUCCESS; + } +} + + +/* + * @unimplemented + */ +DWORD +WINAPI TapeClassInstaller( IN DI_FUNCTION InstallFunction, IN HDEVINFO DeviceInfoSet, Modified: trunk/reactos/dll/win32/syssetup/syssetup.spec URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/syssetup/syssetu… ============================================================================== --- trunk/reactos/dll/win32/syssetup/syssetup.spec [iso-8859-1] (original) +++ trunk/reactos/dll/win32/syssetup/syssetup.spec [iso-8859-1] Fri Jun 5 15:19:09 2015 @@ -12,12 +12,12 @@ @ stub CreateLocalAdminAccount @ stub CreateLocalAdminAccountEx @ stub CreateLocalUserAccount -@ stub CriticalDeviceCoInstaller +@ stdcall CriticalDeviceCoInstaller(long ptr ptr ptr) @ stub DevInstallW @ stdcall DeviceBayClassInstaller(long ptr ptr) @ stub DiskPropPageProvider @ stub DoInstallComponentInfs -@ stub EisaUpHalCoInstaller +@ stdcall EisaUpHalCoInstaller(long ptr ptr ptr) @ stub EisaUpHalPropPageProvider @ stub GenerateName @ stdcall HdcClassInstaller(long ptr ptr) @@ -74,7 +74,7 @@ @ stub SetupShellSettings @ stdcall SetupStartService(wstr long) @ stub SetupUnregisterOsComponent -@ stub StorageCoInstaller +@ stdcall StorageCoInstaller(long ptr ptr ptr) @ stub SystemUpdateUserProfileDirectory @ stdcall TapeClassInstaller(long ptr ptr) @ stub TapePropPageProvider

1 0

[aandrejevic] 68021: [FAST486] - Restore the stack pointer on an exception. - Fix the parameter size for 286 call gates. Caught by V.
by aandrejevic＠svn.reactos.org 05 Jun '15

05 Jun '15

Author: aandrejevic Date: Fri Jun 5 14:02:24 2015 New Revision: 68021 URL: http://svn.reactos.org/svn/reactos?rev=68021&view=rev Log: [FAST486] - Restore the stack pointer on an exception. - Fix the parameter size for 286 call gates. Caught by V. Modified: trunk/reactos/include/reactos/libs/fast486/fast486.h trunk/reactos/lib/fast486/common.c trunk/reactos/lib/fast486/fast486.c Modified: trunk/reactos/include/reactos/libs/fast486/fast486.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/reactos/libs/fast4… ============================================================================== --- trunk/reactos/include/reactos/libs/fast486/fast486.h [iso-8859-1] (original) +++ trunk/reactos/include/reactos/libs/fast486/fast486.h [iso-8859-1] Fri Jun 5 14:02:24 2015 @@ -515,6 +515,7 @@ FAST486_REG GeneralRegs[FAST486_NUM_GEN_REGS]; FAST486_SEG_REG SegmentRegs[FAST486_NUM_SEG_REGS]; FAST486_REG InstPtr, SavedInstPtr; + FAST486_REG SavedStackPtr; FAST486_FLAGS_REG Flags; FAST486_TABLE_REG Gdtr, Idtr; FAST486_LDT_REG Ldtr; Modified: trunk/reactos/lib/fast486/common.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/common.c?rev=6… ============================================================================== --- trunk/reactos/lib/fast486/common.c [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/common.c [iso-8859-1] Fri Jun 5 14:02:24 2015 @@ -579,6 +579,9 @@ /* Restore the IP to the saved IP */ State->InstPtr = State->SavedInstPtr; + /* Restore the SP to the saved SP */ + State->GeneralRegs[FAST486_REG_ESP] = State->SavedStackPtr; + /* Get the interrupt vector */ if (!Fast486GetIntVector(State, ExceptionCode, &IdtEntry)) { @@ -1040,7 +1043,7 @@ OldEsp, FALSE, ParamBuffer, - Gate->ParamCount * sizeof(ULONG))) + Gate->ParamCount * (GateSize ? sizeof(ULONG) : sizeof(USHORT)))) { /* Exception occurred */ return FALSE; Modified: trunk/reactos/lib/fast486/fast486.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/fast486.c?rev=… ============================================================================== --- trunk/reactos/lib/fast486/fast486.c [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/fast486.c [iso-8859-1] Fri Jun 5 14:02:24 2015 @@ -61,7 +61,11 @@ { NextInst: /* Check if this is a new instruction */ - if (State->PrefixFlags == 0) State->SavedInstPtr = State->InstPtr; + if (State->PrefixFlags == 0) + { + State->SavedInstPtr = State->InstPtr; + State->SavedStackPtr = State->GeneralRegs[FAST486_REG_ESP]; + } /* Perform an instruction fetch */ if (!Fast486FetchByte(State, &Opcode))

1 0

[ekohl] 68020: [DEVMGR] Display class CoInstallers.
by ekohl＠svn.reactos.org 05 Jun '15

05 Jun '15

Author: ekohl Date: Fri Jun 5 13:47:39 2015 New Revision: 68020 URL: http://svn.reactos.org/svn/reactos?rev=68020&view=rev Log: [DEVMGR] Display class CoInstallers. Modified: trunk/reactos/dll/win32/devmgr/advprop.c Modified: trunk/reactos/dll/win32/devmgr/advprop.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/devmgr/advprop.c… ============================================================================== --- trunk/reactos/dll/win32/devmgr/advprop.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/devmgr/advprop.c [iso-8859-1] Fri Jun 5 13:47:39 2015 @@ -1069,6 +1069,96 @@ static VOID +DisplayClassCoinstallers(IN PDEVADVPROP_INFO dap, + IN HWND hwndListView) +{ + HDEVINFO DeviceInfoSet; + PSP_DEVINFO_DATA DeviceInfoData; + WCHAR szClassGuid[45]; + HKEY hKey = INVALID_HANDLE_VALUE; + DWORD dwSize; + DWORD dwType; + LPBYTE lpBuffer = NULL; + LPWSTR lpStr; + INT index; + INT len; + LONG lError; + + if (dap->CurrentDeviceInfoSet != INVALID_HANDLE_VALUE) + { + DeviceInfoSet = dap->CurrentDeviceInfoSet; + DeviceInfoData = &dap->CurrentDeviceInfoData; + } + else + { + DeviceInfoSet = dap->DeviceInfoSet; + DeviceInfoData = &dap->DeviceInfoData; + } + + dwSize = 45 * sizeof(WCHAR); + if (!SetupDiGetDeviceRegistryProperty(DeviceInfoSet, + DeviceInfoData, + SPDRP_CLASSGUID, + &dwType, + (LPBYTE)szClassGuid, + dwSize, + &dwSize)) + return; + + lError = RegOpenKeyExW(HKEY_LOCAL_MACHINE, + L"SYSTEM\\CurrentControlSet\\Control\\CoDeviceInstallers", + 0, + GENERIC_READ, + &hKey); + if (lError != ERROR_SUCCESS) + return; + + dwSize = 0; + lError = RegQueryValueEx(hKey, + szClassGuid, + NULL, + &dwType, + NULL, + &dwSize); + if (lError != ERROR_SUCCESS) + goto done; + + if (dwSize == 0) + goto done; + + lpBuffer = HeapAlloc(GetProcessHeap(), + HEAP_ZERO_MEMORY, + dwSize); + + RegQueryValueEx(hKey, + szClassGuid, + NULL, + &dwType, + lpBuffer, + &dwSize); + + lpStr = (LPWSTR)lpBuffer; + index = 0; + while (*lpStr != 0) + { + len = wcslen(lpStr) + 1; + + SetListViewText(hwndListView, index, lpStr); + + lpStr += len; + index++; + } + +done: + if (lpBuffer != NULL) + HeapFree(GetProcessHeap(), 0, lpBuffer); + + if (hKey != INVALID_HANDLE_VALUE) + RegCloseKey(hKey); +} + + +static VOID DisplayDeviceCoinstallers(IN PDEVADVPROP_INFO dap, IN HWND hwndListView) { @@ -1351,10 +1441,10 @@ L"Installer32"); break; -#if 0 case 18: /* Class Coinstaller */ + DisplayClassCoinstallers(dap, + hwndListView); break; -#endif case 19: /* Device Coinstaller */ DisplayDeviceCoinstallers(dap,

1 0

[cwittich] 68019: [SHELL32] replace some sizeof(var)/sizeof(WCHAR) by _countof
by cwittich＠svn.reactos.org 05 Jun '15

05 Jun '15

Author: cwittich Date: Fri Jun 5 08:11:25 2015 New Revision: 68019 URL: http://svn.reactos.org/svn/reactos?rev=68019&view=rev Log: [SHELL32] replace some sizeof(var)/sizeof(WCHAR) by _countof Modified: trunk/reactos/dll/win32/shell32/dialogs/drive.cpp Modified: trunk/reactos/dll/win32/shell32/dialogs/drive.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/dialogs/… ============================================================================== --- trunk/reactos/dll/win32/shell32/dialogs/drive.cpp [iso-8859-1] (original) +++ trunk/reactos/dll/win32/shell32/dialogs/drive.cpp [iso-8859-1] Fri Jun 5 08:11:25 2015 @@ -133,7 +133,7 @@ psh.phpage = hpsp; WCHAR wszName[256]; - if (GetVolumeInformationW(pwszDrive, wszName, sizeof(wszName) / sizeof(WCHAR), NULL, NULL, NULL, NULL, 0)) + if (GetVolumeInformationW(pwszDrive, wszName, _countof(wszName), NULL, NULL, NULL, NULL, 0)) { psh.pszCaption = wszName; psh.dwFlags |= PSH_PROPTITLE; @@ -230,7 +230,7 @@ return; } - if (LoadStringW(shell32_hInstance, IDS_DEFAULT_CLUSTER_SIZE, wszBuf, sizeof(wszBuf) / sizeof(WCHAR))) + if (LoadStringW(shell32_hInstance, IDS_DEFAULT_CLUSTER_SIZE, wszBuf, _countof(wszBuf))) { hDlgCtrl = GetDlgItem(hwndDlg, 28680); SendMessageW(hDlgCtrl, CB_RESETCONTENT, 0, 0); @@ -249,7 +249,7 @@ return; } - if (LoadStringW(shell32_hInstance, IDS_DEFAULT_CLUSTER_SIZE, wszBuf, sizeof(wszBuf) / sizeof(WCHAR))) + if (LoadStringW(shell32_hInstance, IDS_DEFAULT_CLUSTER_SIZE, wszBuf, _countof(wszBuf))) { hDlgCtrl = GetDlgItem(hwndDlg, 28680); SendMessageW(hDlgCtrl, CB_RESETCONTENT, 0, 0); @@ -269,7 +269,7 @@ } hDlgCtrl = GetDlgItem(hwndDlg, 28680); - if (LoadStringW(shell32_hInstance, IDS_DEFAULT_CLUSTER_SIZE, wszBuf, sizeof(wszBuf) / sizeof(WCHAR))) + if (LoadStringW(shell32_hInstance, IDS_DEFAULT_CLUSTER_SIZE, wszBuf, _countof(wszBuf))) { SendMessageW(hDlgCtrl, CB_RESETCONTENT, 0, 0); lIndex = SendMessageW(hDlgCtrl, CB_ADDSTRING, 0, (LPARAM)wszBuf); @@ -281,7 +281,7 @@ for (lIndex = 0; lIndex < 4; lIndex++) { TotalNumberOfBytes.QuadPart = ClusterSize; - if (StrFormatByteSizeW(TotalNumberOfBytes.QuadPart, wszBuf, sizeof(wszBuf) / sizeof(WCHAR))) + if (StrFormatByteSizeW(TotalNumberOfBytes.QuadPart, wszBuf, _countof(wszBuf))) { lIndex = SendMessageW(hDlgCtrl, CB_ADDSTRING, 0, (LPARAM)wszBuf); if (lIndex != CB_ERR) @@ -311,17 +311,17 @@ BOOLEAN Latest; HWND hwndFileSystems; - cchText = GetWindowTextW(hwndDlg, szText, sizeof(szText) / sizeof(WCHAR) - 1); + cchText = GetWindowTextW(hwndDlg, szText, _countof(szText) - 1); if (cchText < 0) cchText = 0; szText[cchText++] = L' '; szDrive[0] = pContext->Drive + L'A'; - if (GetVolumeInformationW(szDrive, &szText[cchText], (sizeof(szText) / sizeof(WCHAR)) - cchText, NULL, NULL, NULL, szFs, sizeof(szFs) / sizeof(WCHAR))) + if (GetVolumeInformationW(szDrive, &szText[cchText], _countof(szText) - cchText, NULL, NULL, NULL, szFs, _countof(szFs))) { if (szText[cchText] == UNICODE_NULL) { /* load default volume label */ - cchText += LoadStringW(shell32_hInstance, IDS_DRIVE_FIXED, &szText[cchText], (sizeof(szText) / sizeof(WCHAR)) - cchText); + cchText += LoadStringW(shell32_hInstance, IDS_DRIVE_FIXED, &szText[cchText], _countof(szText) - cchText); } else { @@ -338,7 +338,7 @@ if (GetDiskFreeSpaceExW(szDrive, &FreeBytesAvailableUser, &TotalNumberOfBytes, NULL)) { - if (StrFormatByteSizeW(TotalNumberOfBytes.QuadPart, szText, sizeof(szText) / sizeof(WCHAR))) + if (StrFormatByteSizeW(TotalNumberOfBytes.QuadPart, szText, _countof(szText))) { /* add drive capacity */ SendDlgItemMessageW(hwndDlg, 28673, CB_ADDSTRING, 0, (LPARAM)szText); @@ -441,7 +441,7 @@ return; } Length = SendMessageW(hDlgCtrl, CB_GETLBTEXTLEN, iSelIndex, 0); - if ((int)Length == CB_ERR || Length + 1 > sizeof(szFileSys) / sizeof(WCHAR)) + if ((int)Length == CB_ERR || Length + 1 > _countof(szFileSys)) { FIXME("\n"); return; @@ -449,17 +449,17 @@ /* retrieve the file system */ SendMessageW(hDlgCtrl, CB_GETLBTEXT, iSelIndex, (LPARAM)szFileSys); - szFileSys[(sizeof(szFileSys)/sizeof(WCHAR))-1] = L'\0'; + szFileSys[_countof(szFileSys)-1] = L'\0'; /* retrieve the volume label */ hDlgCtrl = GetWindow(hwndDlg, 28679); Length = SendMessageW(hDlgCtrl, WM_GETTEXTLENGTH, 0, 0); - if (Length + 1 > sizeof(szLabel) / sizeof(WCHAR)) + if (Length + 1 > _countof(szLabel)) { FIXME("\n"); return; } - SendMessageW(hDlgCtrl, WM_GETTEXT, sizeof(szLabel) / sizeof(WCHAR), (LPARAM)szLabel); + SendMessageW(hDlgCtrl, WM_GETTEXT, _countof(szLabel), (LPARAM)szLabel); szLabel[(sizeof(szLabel)/sizeof(WCHAR))-1] = L'\0'; /* check for quickformat */

1 0

[cwittich] 68018: [SHELL32] show/hide some controls on disk drive properties based on drive type
by cwittich＠svn.reactos.org 05 Jun '15

05 Jun '15

Author: cwittich Date: Fri Jun 5 07:38:17 2015 New Revision: 68018 URL: http://svn.reactos.org/svn/reactos?rev=68018&view=rev Log: [SHELL32] show/hide some controls on disk drive properties based on drive type Modified: trunk/reactos/dll/win32/shell32/dialogs/drvdefext.cpp Modified: trunk/reactos/dll/win32/shell32/dialogs/drvdefext.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/dialogs/… ============================================================================== --- trunk/reactos/dll/win32/shell32/dialogs/drvdefext.cpp [iso-8859-1] (original) +++ trunk/reactos/dll/win32/shell32/dialogs/drvdefext.cpp [iso-8859-1] Fri Jun 5 07:38:17 2015 @@ -308,9 +308,18 @@ default: IconId = IDI_SHELL_DRIVE; TypeStrId = IDS_DRIVE_FIXED; } - if (DriveType != DRIVE_FIXED) + if (DriveType == DRIVE_CDROM) + { + /* volume label textbox */ EnableWindow(GetDlgItem(hwndDlg, 14000), false); - + + /* disk compression */ + ShowWindow(GetDlgItem(hwndDlg, 14011), false); + + /* index */ + ShowWindow(GetDlgItem(hwndDlg, 14012), false); + } + HICON hIcon = (HICON)LoadImage(shell32_hInstance, MAKEINTRESOURCE(IconId), IMAGE_ICON, 32, 32, LR_SHARED); if (hIcon) SendDlgItemMessageW(hwndDlg, 14016, STM_SETICON, (WPARAM)hIcon, 0); @@ -367,6 +376,9 @@ GetDlgItemTextW(hwndDlg, 14009, wszFormat, _countof(wszFormat)); swprintf(wszBuf, wszFormat, m_wszDrive[0]); SetDlgItemTextW(hwndDlg, 14009, wszBuf); + + /* show disk cleanup button only for fixed drives */ + ShowWindow(GetDlgItem(hwndDlg, 14010), DriveType == DRIVE_FIXED); } INT_PTR CALLBACK

1 0

← Newer
1
...
28
29
30
31
32
33
34
35
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 June 2015