[reactos] 01/01: [SETUPLIB][REACTOS][USETUP] Turn setuplib into a DLL shared between TUI and GUI 1st-stage setups (#7523)
https://git.reactos.org/?p=reactos.git;a=commitdiff;h=d7c1d220b550c3cd8373f1...
commit d7c1d220b550c3cd8373f1c16bd929dc6a8368cd Author: Hermès Bélusca-Maïto hermes.belusca-maito@reactos.org AuthorDate: Fri Nov 22 21:45:06 2024 +0100 Commit: Hermès Bélusca-Maïto hermes.belusca-maito@reactos.org CommitDate: Mon Dec 2 23:05:38 2024 +0100
[SETUPLIB][REACTOS][USETUP] Turn setuplib into a DLL shared between TUI and GUI 1st-stage setups (#7523)
CORE-13525
Notes: - Most of the exported functions have been turned from default cdecl to explicit stdcall / "NTAPI". - The two InitializeSetup() phases have been collapsed to make the initialization simpler.
Average reductions (percentages; see PR #7523 for actual numbers):
x86 Debug builds: reactos.exe: 35.1% smss.exe : 39.8% Total (including setuplib.dll): 17.9%
x86 Release builds: reactos.exe: 22.3% smss.exe : 25.0% Total (including setuplib.dll): 10.6%
x64 Debug builds: reactos.exe: 40.6% smss.exe : 41.6% Total (including setuplib.dll): 20.0%
x64 Release builds: reactos.exe: 22.8% smss.exe : 22.3% Total (including setuplib.dll): 10.1% --- base/setup/lib/CMakeLists.txt | 14 +- base/setup/lib/bootsup.c | 4 +- base/setup/lib/bootsup.h | 2 + base/setup/lib/fsutil.c | 10 ++ base/setup/lib/fsutil.h | 8 + base/setup/lib/install.c | 4 +- base/setup/lib/install.h | 2 + base/setup/lib/precomp.h | 7 +- base/setup/lib/setuplib.c | 169 +++++++++++++-------- base/setup/lib/setuplib.h | 36 ++++- base/setup/lib/setuplib.rc | 16 ++ base/setup/lib/setuplib.spec | 83 ++++++++++ base/setup/lib/spapisup/fileqsup.c | 11 +- base/setup/lib/spapisup/fileqsup.h | 33 ++-- base/setup/lib/spapisup/infsupp.c | 15 +- base/setup/lib/spapisup/infsupp.h | 61 ++++---- base/setup/lib/utils/genlist.c | 9 ++ base/setup/lib/utils/genlist.h | 9 ++ base/setup/lib/utils/osdetect.c | 17 ++- base/setup/lib/utils/osdetect.h | 12 +- base/setup/lib/utils/partinfo.h | 4 +- base/setup/lib/utils/partlist.c | 8 + base/setup/lib/utils/partlist.h | 8 + base/setup/reactos/CMakeLists.txt | 5 +- base/setup/reactos/reactos.c | 9 +- base/setup/reactos/reactos.h | 2 - base/setup/reactos/spapisup/fileqsup.c | 15 +- base/setup/reactos/spapisup/infsupp.c | 23 +-- base/setup/usetup/CMakeLists.txt | 4 +- base/setup/usetup/spapisup/fileqsup.c | 15 +- base/setup/usetup/spapisup/infsupp.c | 23 +-- base/setup/usetup/usetup.c | 54 +++---- base/setup/usetup/usetup.h | 1 - modules/rostests/unittests/setuplib/CMakeLists.txt | 3 +- .../unittests/setuplib/IsValidInstallDirectory.c | 22 +-- 35 files changed, 459 insertions(+), 259 deletions(-)
diff --git a/base/setup/lib/CMakeLists.txt b/base/setup/lib/CMakeLists.txt index 836c9ef5967..eb8ef7925a8 100644 --- a/base/setup/lib/CMakeLists.txt +++ b/base/setup/lib/CMakeLists.txt @@ -3,9 +3,12 @@ add_definitions(${I18N_DEFS}) if(_WINKD_) add_definitions(-D_WINKD_) endif() +add_definitions(-D_SETUPLIB_)
include_directories(spapisup utils)
+spec2def(setuplib.dll setuplib.spec ADD_IMPORTLIB) + list(APPEND SOURCE spapisup/fileqsup.c spapisup/infsupp.c @@ -32,6 +35,15 @@ list(APPEND SOURCE setuplib.c precomp.h)
-add_library(setuplib ${SOURCE}) +add_library(setuplib SHARED + ${SOURCE} + setuplib.rc + ${CMAKE_CURRENT_BINARY_DIR}/setuplib.def) + add_pch(setuplib precomp.h SOURCE) add_dependencies(setuplib xdk) # psdk + +set_module_type(setuplib nativedll) +target_link_libraries(setuplib ext2lib vfatlib btrfslib ${PSEH_LIB}) +add_importlibs(setuplib ntdll) +add_cd_file(TARGET setuplib DESTINATION reactos/system32 NO_CAB FOR bootcd regtest) diff --git a/base/setup/lib/bootsup.c b/base/setup/lib/bootsup.c index 030ffd6e54c..4c57fc8218b 100644 --- a/base/setup/lib/bootsup.c +++ b/base/setup/lib/bootsup.c @@ -19,7 +19,7 @@ #include "bootcode.h" #include "fsutil.h"
-#include "setuplib.h" // HAXX for IsUnattendedSetup!! +#include "setuplib.h" // HACK for IsUnattendedSetup
#include "bootsup.h"
@@ -1658,6 +1658,7 @@ GetDeviceInfo( * @return An NTSTATUS code indicating success or failure. **/ NTSTATUS +NTAPI InstallBootManagerAndBootEntries( _In_ ARCHITECTURE_TYPE ArchType, _In_ PCUNICODE_STRING SystemRootPath, @@ -1813,6 +1814,7 @@ Quit: }
NTSTATUS +NTAPI InstallBootcodeToRemovable( _In_ ARCHITECTURE_TYPE ArchType, _In_ PCUNICODE_STRING RemovableRootPath, diff --git a/base/setup/lib/bootsup.h b/base/setup/lib/bootsup.h index faacff7fbb1..abbc20f0594 100644 --- a/base/setup/lib/bootsup.h +++ b/base/setup/lib/bootsup.h @@ -8,6 +8,7 @@ #pragma once
NTSTATUS +NTAPI InstallBootManagerAndBootEntries( _In_ ARCHITECTURE_TYPE ArchType, _In_ PCUNICODE_STRING SystemRootPath, @@ -16,6 +17,7 @@ InstallBootManagerAndBootEntries( _In_ ULONG_PTR Options);
NTSTATUS +NTAPI InstallBootcodeToRemovable( _In_ ARCHITECTURE_TYPE ArchType, _In_ PCUNICODE_STRING RemovableRootPath, diff --git a/base/setup/lib/fsutil.c b/base/setup/lib/fsutil.c index 875e8ec7f9d..2f4c99129c2 100644 --- a/base/setup/lib/fsutil.c +++ b/base/setup/lib/fsutil.c @@ -179,6 +179,7 @@ static FILE_SYSTEM RegisteredFileSystems[] =
/** QueryAvailableFileSystemFormat() **/ BOOLEAN +NTAPI GetRegisteredFileSystems( IN ULONG Index, OUT PCWSTR* FileSystemName) @@ -241,6 +242,7 @@ GetFileSystemByName(
/** ChkdskEx() **/ NTSTATUS +NTAPI ChkdskFileSystem_UStr( _In_ PUNICODE_STRING DriveRoot, _In_ PCWSTR FileSystemName, @@ -284,6 +286,7 @@ ChkdskFileSystem_UStr( }
NTSTATUS +NTAPI ChkdskFileSystem( _In_ PCWSTR DriveRoot, _In_ PCWSTR FileSystemName, @@ -308,6 +311,7 @@ ChkdskFileSystem(
/** FormatEx() **/ NTSTATUS +NTAPI FormatFileSystem_UStr( _In_ PUNICODE_STRING DriveRoot, _In_ PCWSTR FileSystemName, @@ -373,6 +377,7 @@ FormatFileSystem_UStr( }
NTSTATUS +NTAPI FormatFileSystem( _In_ PCWSTR DriveRoot, _In_ PCWSTR FileSystemName, @@ -754,6 +759,7 @@ Quit: //
NTSTATUS +NTAPI ChkdskVolume( _In_ PVOLINFO Volume, _In_ BOOLEAN FixErrors, @@ -778,6 +784,7 @@ ChkdskVolume( }
NTSTATUS +NTAPI ChkdskPartition( _In_ PPARTENTRY PartEntry, _In_ BOOLEAN FixErrors, @@ -801,6 +808,7 @@ ChkdskPartition( }
NTSTATUS +NTAPI FormatVolume( _In_ PVOLINFO Volume, _In_ PCWSTR FileSystemName, @@ -839,6 +847,7 @@ FormatVolume( }
NTSTATUS +NTAPI FormatPartition( _In_ PPARTENTRY PartEntry, _In_ PCWSTR FileSystemName, @@ -1084,6 +1093,7 @@ GetNextUnformattedVolume( }
BOOLEAN +NTAPI FsVolCommitOpsQueue( _In_ PPARTLIST PartitionList, _In_ PVOLENTRY SystemVolume, diff --git a/base/setup/lib/fsutil.h b/base/setup/lib/fsutil.h index 5af0a4265f5..a72ef381cc9 100644 --- a/base/setup/lib/fsutil.h +++ b/base/setup/lib/fsutil.h @@ -12,6 +12,7 @@
/** QueryAvailableFileSystemFormat() **/ BOOLEAN +NTAPI GetRegisteredFileSystems( IN ULONG Index, OUT PCWSTR* FileSystemName); @@ -19,6 +20,7 @@ GetRegisteredFileSystems(
/** ChkdskEx() **/ NTSTATUS +NTAPI ChkdskFileSystem_UStr( _In_ PUNICODE_STRING DriveRoot, _In_ PCWSTR FileSystemName, @@ -29,6 +31,7 @@ ChkdskFileSystem_UStr( _In_opt_ PFMIFSCALLBACK Callback);
NTSTATUS +NTAPI ChkdskFileSystem( _In_ PCWSTR DriveRoot, _In_ PCWSTR FileSystemName, @@ -41,6 +44,7 @@ ChkdskFileSystem(
/** FormatEx() **/ NTSTATUS +NTAPI FormatFileSystem_UStr( _In_ PUNICODE_STRING DriveRoot, _In_ PCWSTR FileSystemName, @@ -51,6 +55,7 @@ FormatFileSystem_UStr( _In_opt_ PFMIFSCALLBACK Callback);
NTSTATUS +NTAPI FormatFileSystem( _In_ PCWSTR DriveRoot, _In_ PCWSTR FileSystemName, @@ -109,6 +114,7 @@ InstallNtfsBootCode( //
NTSTATUS +NTAPI ChkdskPartition( _In_ PPARTENTRY PartEntry, _In_ BOOLEAN FixErrors, @@ -118,6 +124,7 @@ ChkdskPartition( _In_opt_ PFMIFSCALLBACK Callback);
NTSTATUS +NTAPI FormatPartition( _In_ PPARTENTRY PartEntry, _In_ PCWSTR FileSystemName, @@ -200,6 +207,7 @@ typedef FSVOL_OP _In_ ULONG_PTR Param2);
BOOLEAN +NTAPI FsVolCommitOpsQueue( _In_ PPARTLIST PartitionList, _In_ PVOLENTRY SystemVolume, diff --git a/base/setup/lib/install.c b/base/setup/lib/install.c index 72f3ea92181..a9bc1828db4 100644 --- a/base/setup/lib/install.c +++ b/base/setup/lib/install.c @@ -13,7 +13,7 @@ #include "filesup.h" #include "infsupp.h"
-#include "setuplib.h" // HAXX for USETUP_DATA!! +#include "setuplib.h" // HACK for USETUP_DATA
#include "install.h"
@@ -681,6 +681,7 @@ PrepareCopyInfFile( // #define USE_CABINET_INF
BOOLEAN // ERROR_NUMBER +NTAPI PrepareFileCopy( IN OUT PUSETUP_DATA pSetupData, IN PFILE_COPY_STATUS_ROUTINE StatusRoutine OPTIONAL) @@ -823,6 +824,7 @@ PrepareFileCopy( }
BOOLEAN +NTAPI DoFileCopy( IN OUT PUSETUP_DATA pSetupData, IN PSP_FILE_CALLBACK_W MsgHandler, diff --git a/base/setup/lib/install.h b/base/setup/lib/install.h index 8c3e84b3257..5b58a497726 100644 --- a/base/setup/lib/install.h +++ b/base/setup/lib/install.h @@ -27,11 +27,13 @@ PrepareCopyInfFile( #endif
BOOLEAN // ERROR_NUMBER +NTAPI PrepareFileCopy( IN OUT PUSETUP_DATA pSetupData, IN PFILE_COPY_STATUS_ROUTINE StatusRoutine OPTIONAL);
BOOLEAN +NTAPI DoFileCopy( IN OUT PUSETUP_DATA pSetupData, IN PSP_FILE_CALLBACK_W MsgHandler, diff --git a/base/setup/lib/precomp.h b/base/setup/lib/precomp.h index f347044d865..2195c55a9e6 100644 --- a/base/setup/lib/precomp.h +++ b/base/setup/lib/precomp.h @@ -29,10 +29,14 @@
#include <ntstrsafe.h>
- /* Filesystem headers */ #include <reactos/rosioctl.h> // For extra partition IDs
+ +#ifndef SPLIBAPI +#define SPLIBAPI +#endif + // ///* Internal Headers */ //#include "interface/consup.h" @@ -51,7 +55,6 @@ //#include "filesup.h" //#include "genlist.h"
- extern HANDLE ProcessHeap;
#include "errorcode.h" diff --git a/base/setup/lib/setuplib.c b/base/setup/lib/setuplib.c index 632db24ec29..862c9020719 100644 --- a/base/setup/lib/setuplib.c +++ b/base/setup/lib/setuplib.c @@ -22,9 +22,13 @@
/* GLOBALS ******************************************************************/
+HANDLE ProcessHeap; +BOOLEAN IsUnattendedSetup = FALSE; + /* FUNCTIONS ****************************************************************/
VOID +NTAPI CheckUnattendedSetup( IN OUT PUSETUP_DATA pSetupData) { @@ -199,6 +203,7 @@ Quit: }
VOID +NTAPI InstallSetupInfFile( IN OUT PUSETUP_DATA pSetupData) { @@ -669,6 +674,7 @@ LoadSetupInf( * @brief Find or set the active system partition. **/ BOOLEAN +NTAPI InitSystemPartition( /**/_In_ PPARTLIST PartitionList, /* HACK HACK! */ /**/_In_ PPARTENTRY InstallPartition, /* HACK HACK! */ @@ -768,6 +774,7 @@ InitSystemPartition( * Each path component must be a valid 8.3 name. **/ BOOLEAN +NTAPI IsValidInstallDirectory( _In_ PCWSTR InstallDir) { @@ -852,6 +859,7 @@ IsValidInstallDirectory(
NTSTATUS +NTAPI InitDestinationPaths( _Inout_ PUSETUP_DATA pSetupData, _In_ PCWSTR InstallationDir, @@ -1007,92 +1015,91 @@ InitDestinationPaths(
// NTSTATUS ERROR_NUMBER +NTAPI InitializeSetup( - IN OUT PUSETUP_DATA pSetupData, - IN ULONG InitPhase) + _Inout_ PUSETUP_DATA pSetupData, + _In_opt_ PSETUP_ERROR_ROUTINE ErrorRoutine, + _In_ PSPFILE_EXPORTS pSpFileExports, + _In_ PSPINF_EXPORTS pSpInfExports) { - if (InitPhase == 0) - { - RtlZeroMemory(pSetupData, sizeof(*pSetupData)); + ERROR_NUMBER Error; + NTSTATUS Status;
- /* Initialize error handling */ - pSetupData->LastErrorNumber = ERROR_SUCCESS; - pSetupData->ErrorRoutine = NULL; + IsUnattendedSetup = FALSE; + RtlZeroMemory(pSetupData, sizeof(*pSetupData));
- /* Initialize global unicode strings */ - RtlInitUnicodeString(&pSetupData->SourcePath, NULL); - RtlInitUnicodeString(&pSetupData->SourceRootPath, NULL); - RtlInitUnicodeString(&pSetupData->SourceRootDir, NULL); - RtlInitUnicodeString(&pSetupData->DestinationArcPath, NULL); - RtlInitUnicodeString(&pSetupData->DestinationPath, NULL); - RtlInitUnicodeString(&pSetupData->DestinationRootPath, NULL); - RtlInitUnicodeString(&pSetupData->SystemRootPath, NULL); + /* Initialize error handling */ + pSetupData->LastErrorNumber = ERROR_SUCCESS; + pSetupData->ErrorRoutine = ErrorRoutine;
- // FIXME: This is only temporary!! Must be removed later! - /***/RtlInitUnicodeString(&pSetupData->InstallPath, NULL);/***/ + /* Initialize global unicode strings */ + RtlInitUnicodeString(&pSetupData->SourcePath, NULL); + RtlInitUnicodeString(&pSetupData->SourceRootPath, NULL); + RtlInitUnicodeString(&pSetupData->SourceRootDir, NULL); + RtlInitUnicodeString(&pSetupData->DestinationArcPath, NULL); + RtlInitUnicodeString(&pSetupData->DestinationPath, NULL); + RtlInitUnicodeString(&pSetupData->DestinationRootPath, NULL); + RtlInitUnicodeString(&pSetupData->SystemRootPath, NULL);
- // - // TODO: Load and start SetupDD, and ask it for the information - // + // FIXME: This is only temporary!! Must be removed later! + /***/RtlInitUnicodeString(&pSetupData->InstallPath, NULL);/***/ + + /* Initialize SpFile and SpInf support */ + RtlCopyMemory(&SpFileExports, pSpFileExports, sizeof(SpFileExports)); + RtlCopyMemory(&SpInfExports, pSpInfExports, sizeof(SpInfExports));
- return ERROR_SUCCESS; + // + // TODO: Load and start SetupDD, and ask it for the information + // + + /* Get the source path and source root path */ + Status = GetSourcePaths(&pSetupData->SourcePath, + &pSetupData->SourceRootPath, + &pSetupData->SourceRootDir); + if (!NT_SUCCESS(Status)) + { + DPRINT1("GetSourcePaths() failed (Status 0x%08lx)\n", Status); + return ERROR_NO_SOURCE_DRIVE; } - else - if (InitPhase == 1) + DPRINT1("SourcePath (1): '%wZ'\n", &pSetupData->SourcePath); + DPRINT1("SourceRootPath (1): '%wZ'\n", &pSetupData->SourceRootPath); + DPRINT1("SourceRootDir (1): '%wZ'\n", &pSetupData->SourceRootDir); + + /* Set up default values */ + pSetupData->DestinationDiskNumber = 0; + pSetupData->DestinationPartitionNumber = 1; + pSetupData->BootLoaderLocation = 2; // Default to "System partition" + pSetupData->FormatPartition = 0; + pSetupData->AutoPartition = 0; + pSetupData->FsType = 0; + + /* Load 'txtsetup.sif' from the installation media */ + Error = LoadSetupInf(pSetupData); + if (Error != ERROR_SUCCESS) { - ERROR_NUMBER Error; - NTSTATUS Status; + DPRINT1("LoadSetupInf() failed (Error 0x%lx)\n", Error); + return Error; + } + DPRINT1("SourcePath (2): '%wZ'\n", &pSetupData->SourcePath); + DPRINT1("SourceRootPath (2): '%wZ'\n", &pSetupData->SourceRootPath); + DPRINT1("SourceRootDir (2): '%wZ'\n", &pSetupData->SourceRootDir);
- /* Get the source path and source root path */ - Status = GetSourcePaths(&pSetupData->SourcePath, - &pSetupData->SourceRootPath, - &pSetupData->SourceRootDir); - if (!NT_SUCCESS(Status)) - { - DPRINT1("GetSourcePaths() failed (Status 0x%08lx)\n", Status); - return ERROR_NO_SOURCE_DRIVE; - } - DPRINT1("SourcePath (1): '%wZ'\n", &pSetupData->SourcePath); - DPRINT1("SourceRootPath (1): '%wZ'\n", &pSetupData->SourceRootPath); - DPRINT1("SourceRootDir (1): '%wZ'\n", &pSetupData->SourceRootDir); - - /* Set up default values */ - pSetupData->DestinationDiskNumber = 0; - pSetupData->DestinationPartitionNumber = 1; - pSetupData->BootLoaderLocation = 2; // Default to "System partition" - pSetupData->FormatPartition = 0; - pSetupData->AutoPartition = 0; - pSetupData->FsType = 0; - - /* Load 'txtsetup.sif' from the installation media */ - Error = LoadSetupInf(pSetupData); - if (Error != ERROR_SUCCESS) - { - DPRINT1("LoadSetupInf() failed (Error 0x%lx)\n", Error); - return Error; - } - DPRINT1("SourcePath (2): '%wZ'\n", &pSetupData->SourcePath); - DPRINT1("SourceRootPath (2): '%wZ'\n", &pSetupData->SourceRootPath); - DPRINT1("SourceRootDir (2): '%wZ'\n", &pSetupData->SourceRootDir); - - /* Retrieve the target machine architecture type */ - // FIXME: This should be determined at runtime!! - // FIXME: Allow for (pre-)installing on an architecture - // different from the current one? + /* Retrieve the target machine architecture type */ + // FIXME: This should be determined at runtime!! + // FIXME: Allow for (pre-)installing on an architecture + // different from the current one? #if defined(SARCH_XBOX) - pSetupData->ArchType = ARCH_Xbox; + pSetupData->ArchType = ARCH_Xbox; // #elif defined(SARCH_PC98) #else // TODO: Arc, UEFI - pSetupData->ArchType = (IsNEC_98 ? ARCH_NEC98x86 : ARCH_PcAT); + pSetupData->ArchType = (IsNEC_98 ? ARCH_NEC98x86 : ARCH_PcAT); #endif
- return ERROR_SUCCESS; - } - return ERROR_SUCCESS; }
VOID +NTAPI FinishSetup( IN OUT PUSETUP_DATA pSetupData) { @@ -1143,6 +1150,7 @@ FinishSetup( * Calls SetMountedDeviceValues */ ERROR_NUMBER +NTAPI UpdateRegistry( IN OUT PUSETUP_DATA pSetupData, /**/IN BOOLEAN RepairUpdateFlag, /* HACK HACK! */ @@ -1396,4 +1404,31 @@ Cleanup: return ErrorNumber; }
+ +/* ENTRY-POINT ***************************************************************/ + +/* Declared in ndk/umfuncs.h */ +NTSTATUS +NTAPI +LdrDisableThreadCalloutsForDll( + _In_ PVOID BaseAddress); + +BOOL +NTAPI +DllMain( + _In_ HINSTANCE hDll, + _In_ ULONG dwReason, + _In_opt_ PVOID pReserved) +{ + UNREFERENCED_PARAMETER(pReserved); + + if (dwReason == DLL_PROCESS_ATTACH) + { + LdrDisableThreadCalloutsForDll(hDll); + ProcessHeap = RtlGetProcessHeap(); + } + + return TRUE; +} + /* EOF */ diff --git a/base/setup/lib/setuplib.h b/base/setup/lib/setuplib.h index a69f7bba877..d969e894e8a 100644 --- a/base/setup/lib/setuplib.h +++ b/base/setup/lib/setuplib.h @@ -7,6 +7,16 @@
#pragma once
+#ifdef __cplusplus +extern "C" { +#endif + +#ifndef _SETUPLIB_ +#define SPLIBAPI DECLSPEC_IMPORT +#else +#define SPLIBAPI +#endif + /* INCLUDES *****************************************************************/
/* Needed PSDK headers when using this library */ @@ -20,9 +30,9 @@
#endif
-/* NOTE: Please keep the header inclusion order! */ +extern SPLIBAPI BOOLEAN IsUnattendedSetup; // HACK
-extern HANDLE ProcessHeap; +/* NOTE: Please keep the header inclusion order! */
#include "errorcode.h" #include "spapisup/fileqsup.h" @@ -153,19 +163,17 @@ typedef struct _USETUP_DATA #include "install.h"
-// HACK!! -extern BOOLEAN IsUnattendedSetup; - - /* FUNCTIONS ****************************************************************/
#include "substset.h"
VOID +NTAPI CheckUnattendedSetup( IN OUT PUSETUP_DATA pSetupData);
VOID +NTAPI InstallSetupInfFile( IN OUT PUSETUP_DATA pSetupData);
@@ -182,6 +190,7 @@ LoadSetupInf( #define ERROR_SYSTEM_PARTITION_NOT_FOUND (ERROR_LAST_ERROR_CODE + 1)
BOOLEAN +NTAPI InitSystemPartition( /**/_In_ PPARTLIST PartitionList, /* HACK HACK! */ /**/_In_ PPARTENTRY InstallPartition, /* HACK HACK! */ @@ -200,10 +209,12 @@ InitSystemPartition( (isalnum(c) || (c) == L'.' || (c) == L'\' || (c) == L'-' || (c) == L'_')
BOOLEAN +NTAPI IsValidInstallDirectory( _In_ PCWSTR InstallDir);
NTSTATUS +NTAPI InitDestinationPaths( _Inout_ PUSETUP_DATA pSetupData, _In_ PCWSTR InstallationDir, @@ -211,11 +222,15 @@ InitDestinationPaths(
// NTSTATUS ERROR_NUMBER +NTAPI InitializeSetup( - IN OUT PUSETUP_DATA pSetupData, - IN ULONG InitPhase); + _Inout_ PUSETUP_DATA pSetupData, + _In_opt_ PSETUP_ERROR_ROUTINE ErrorRoutine, + _In_ PSPFILE_EXPORTS pSpFileExports, + _In_ PSPINF_EXPORTS pSpInfExports);
VOID +NTAPI FinishSetup( IN OUT PUSETUP_DATA pSetupData);
@@ -236,6 +251,7 @@ typedef VOID (__cdecl *PREGISTRY_STATUS_ROUTINE)(IN REGISTRY_STATUS, ...);
ERROR_NUMBER +NTAPI UpdateRegistry( IN OUT PUSETUP_DATA pSetupData, /**/IN BOOLEAN RepairUpdateFlag, /* HACK HACK! */ @@ -245,4 +261,8 @@ UpdateRegistry( IN PREGISTRY_STATUS_ROUTINE StatusRoutine OPTIONAL, IN PFONTSUBSTSETTINGS SubstSettings OPTIONAL);
+#ifdef __cplusplus +} +#endif + /* EOF */ diff --git a/base/setup/lib/setuplib.rc b/base/setup/lib/setuplib.rc new file mode 100644 index 00000000000..072bfd23722 --- /dev/null +++ b/base/setup/lib/setuplib.rc @@ -0,0 +1,16 @@ +/* + * PROJECT: ReactOS Setup Library + * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) + * PURPOSE: Resources + * COPYRIGHT: Copyright 2003-2024 ReactOS Team + */ + +#include <windef.h> + +LANGUAGE LANG_NEUTRAL, SUBLANG_NEUTRAL + +#define REACTOS_VERSION_DLL +#define REACTOS_STR_FILE_DESCRIPTION "ReactOS Setup Library DLL" +#define REACTOS_STR_INTERNAL_NAME "setuplib" +#define REACTOS_STR_ORIGINAL_FILENAME "setuplib.dll" +#include <reactos/version.rc> diff --git a/base/setup/lib/setuplib.spec b/base/setup/lib/setuplib.spec new file mode 100644 index 00000000000..a2809a06b0a --- /dev/null +++ b/base/setup/lib/setuplib.spec @@ -0,0 +1,83 @@ + +@ extern IsUnattendedSetup +@ extern MbrPartitionTypes +@ extern GptPartitionTypes + +;; fileqsup and infsupp function pointers to be initialized by the user of this library +;;@ extern SpFileExports +;;@ extern SpInfExports + +;; infsupp +@ cdecl INF_GetDataField(ptr long ptr) ## -private + + +;; filesup +@ cdecl ConcatPathsV(ptr long long ptr) +@ cdecl CombinePathsV(ptr long long ptr) +@ varargs ConcatPaths(ptr long long) +@ varargs CombinePaths(ptr long long) +@ cdecl SetupCopyFile(wstr wstr long) +@ cdecl SetupDeleteFile(wstr long) +@ cdecl SetupMoveFile(wstr wstr long) + +;; genlist +@ stdcall CreateGenericList() +@ stdcall DestroyGenericList(ptr long) +@ stdcall GetCurrentListEntry(ptr) +@ stdcall GetFirstListEntry(ptr) +@ stdcall GetNextListEntry(ptr) +@ stdcall GetListEntryData(ptr) +@ stdcall GetNumberOfListEntries(ptr) +@ stdcall SetCurrentListEntry(ptr ptr) + +;; partlist +@ stdcall CreatePartitionList() +@ stdcall CreatePartition(ptr ptr int64 ptr) +@ stdcall DeletePartition(ptr ptr ptr) +@ stdcall DestroyPartitionList(ptr) +@ stdcall GetNextPartition(ptr ptr) +@ stdcall GetPrevPartition(ptr ptr) +@ stdcall GetAdjUnpartitionedEntry(ptr long) +@ stdcall PartitionCreateChecks(ptr int64 ptr) +@ cdecl -ret64 RoundingDivide(int64 int64) +;;;; +@ cdecl IsPartitionActive(ptr) ## -private +@ cdecl SelectPartition(ptr long long) + +;; osdetect +@ stdcall CreateNTOSInstallationsList(ptr) +@ stdcall FindSubStrI(wstr wstr) + +;; settings +@ cdecl CreateComputerTypeList(ptr) +@ cdecl CreateDisplayDriverList(ptr) +@ cdecl CreateKeyboardDriverList(ptr) +@ cdecl CreateKeyboardLayoutList(ptr wstr ptr) +@ cdecl CreateLanguageList(ptr ptr) +@ cdecl GetDefaultLanguageIndex() + +;; mui +@ cdecl MUIDefaultKeyboardLayout(wstr) +@ cdecl MUIGetOEMCodePage(wstr) ## -private + +;; setuplib +@ stdcall CheckUnattendedSetup(ptr) +@ stdcall FinishSetup(ptr) +@ stdcall IsValidInstallDirectory(wstr) +@ stdcall InitDestinationPaths(ptr wstr ptr) +@ stdcall InitializeSetup(ptr ptr ptr ptr) +@ stdcall InitSystemPartition(ptr ptr ptr ptr ptr) +@ stdcall InstallSetupInfFile(ptr) +@ stdcall UpdateRegistry(ptr long ptr long wstr ptr ptr) + +;; fsutil +@ stdcall FsVolCommitOpsQueue(ptr ptr ptr ptr ptr) +@ stdcall GetRegisteredFileSystems(long ptr) + +;; install +@ stdcall PrepareFileCopy(ptr ptr) +@ stdcall DoFileCopy(ptr ptr ptr) + +;; bootsup +@ stdcall InstallBootManagerAndBootEntries(long ptr ptr ptr ptr) +@ stdcall InstallBootcodeToRemovable(long ptr ptr ptr) diff --git a/base/setup/lib/spapisup/fileqsup.c b/base/setup/lib/spapisup/fileqsup.c index 34da6a45803..6b095fc5960 100644 --- a/base/setup/lib/spapisup/fileqsup.c +++ b/base/setup/lib/spapisup/fileqsup.c @@ -20,15 +20,6 @@ * These externs should be defined by the user of this library. * They are kept there for reference and ease of usage. */ -#if 0 - -pSpFileQueueOpen SpFileQueueOpen = NULL; -pSpFileQueueClose SpFileQueueClose = NULL; -pSpFileQueueCopy SpFileQueueCopy = NULL; -pSpFileQueueDelete SpFileQueueDelete = NULL; -pSpFileQueueRename SpFileQueueRename = NULL; -pSpFileQueueCommit SpFileQueueCommit = NULL; - -#endif +SPFILE_EXPORTS SpFileExports = {NULL};
/* EOF */ diff --git a/base/setup/lib/spapisup/fileqsup.h b/base/setup/lib/spapisup/fileqsup.h index c6cc3839c1b..ffa41efe334 100644 --- a/base/setup/lib/spapisup/fileqsup.h +++ b/base/setup/lib/spapisup/fileqsup.h @@ -51,7 +51,7 @@ #define FILEOP_NEWPATH 4
-/* TYPES ********************************************************************/ +/* TYPES *********************************************************************/
typedef PVOID HSPFILEQ;
@@ -72,21 +72,17 @@ typedef UINT (CALLBACK* PSP_FILE_CALLBACK_W)( #endif
-/* FUNCTIONS ****************************************************************/ +/* FUNCTIONS *****************************************************************/
// #define SetupOpenFileQueue typedef HSPFILEQ (WINAPI* pSpFileQueueOpen)(VOID);
-extern pSpFileQueueOpen SpFileQueueOpen; - // #define SetupCloseFileQueue typedef BOOL (WINAPI* pSpFileQueueClose)( IN HSPFILEQ QueueHandle);
-extern pSpFileQueueClose SpFileQueueClose; - // #define SetupQueueCopyW typedef BOOL (WINAPI* pSpFileQueueCopy)( @@ -101,8 +97,6 @@ typedef BOOL IN PCWSTR TargetFileName OPTIONAL, IN ULONG CopyStyle);
-extern pSpFileQueueCopy SpFileQueueCopy; - // #define SetupQueueDeleteW typedef BOOL (WINAPI* pSpFileQueueDelete)( @@ -110,8 +104,6 @@ typedef BOOL IN PCWSTR PathPart1, IN PCWSTR PathPart2 OPTIONAL);
-extern pSpFileQueueDelete SpFileQueueDelete; - // #define SetupQueueRenameW typedef BOOL (WINAPI* pSpFileQueueRename)( @@ -121,8 +113,6 @@ typedef BOOL IN PCWSTR TargetPath OPTIONAL, IN PCWSTR TargetFileName);
-extern pSpFileQueueRename SpFileQueueRename; - // #define SetupCommitFileQueueW typedef BOOL (WINAPI* pSpFileQueueCommit)( @@ -131,6 +121,23 @@ typedef BOOL IN PSP_FILE_CALLBACK_W MsgHandler, IN PVOID Context OPTIONAL);
-extern pSpFileQueueCommit SpFileQueueCommit; +typedef struct _SPFILE_EXPORTS +{ + pSpFileQueueOpen SpFileQueueOpen; + pSpFileQueueClose SpFileQueueClose; + pSpFileQueueCopy SpFileQueueCopy; + pSpFileQueueDelete SpFileQueueDelete; + pSpFileQueueRename SpFileQueueRename; + pSpFileQueueCommit SpFileQueueCommit; +} SPFILE_EXPORTS, *PSPFILE_EXPORTS; + +extern /*SPLIBAPI*/ SPFILE_EXPORTS SpFileExports; + +#define SpFileQueueOpen (SpFileExports.SpFileQueueOpen) +#define SpFileQueueClose (SpFileExports.SpFileQueueClose) +#define SpFileQueueCopy (SpFileExports.SpFileQueueCopy) +#define SpFileQueueDelete (SpFileExports.SpFileQueueDelete) +#define SpFileQueueRename (SpFileExports.SpFileQueueRename) +#define SpFileQueueCommit (SpFileExports.SpFileQueueCommit)
/* EOF */ diff --git a/base/setup/lib/spapisup/infsupp.c b/base/setup/lib/spapisup/infsupp.c index 65bdbd23e6a..fdf3e35664f 100644 --- a/base/setup/lib/spapisup/infsupp.c +++ b/base/setup/lib/spapisup/infsupp.c @@ -21,20 +21,7 @@ * These externs should be defined by the user of this library. * They are kept there for reference and ease of usage. */ -#if 0 - -pSpInfCloseInfFile SpInfCloseInfFile = NULL; -pSpInfFindFirstLine SpInfFindFirstLine = NULL; -pSpInfFindNextLine SpInfFindNextLine = NULL; -pSpInfGetFieldCount SpInfGetFieldCount = NULL; -pSpInfGetBinaryField SpInfGetBinaryField = NULL; -pSpInfGetIntField SpInfGetIntField = NULL; -pSpInfGetMultiSzField SpInfGetMultiSzField = NULL; -pSpInfGetStringField SpInfGetStringField = NULL; -pSpInfGetField SpInfGetField = NULL; -pSpInfOpenInfFile SpInfOpenInfFile = NULL; - -#endif +SPINF_EXPORTS SpInfExports = {NULL};
/* HELPER FUNCTIONS **********************************************************/
diff --git a/base/setup/lib/spapisup/infsupp.h b/base/setup/lib/spapisup/infsupp.h index c5d24ca6b71..0a2adef49fd 100644 --- a/base/setup/lib/spapisup/infsupp.h +++ b/base/setup/lib/spapisup/infsupp.h @@ -29,6 +29,8 @@ typedef struct _INFCONTEXT
#endif
+C_ASSERT(sizeof(INFCONTEXT) == 2 * sizeof(HINF) + 2 * sizeof(UINT)); + /* Lower the MAX_INF_STRING_LENGTH value in order to avoid too much stack usage */ #undef MAX_INF_STRING_LENGTH #define MAX_INF_STRING_LENGTH 1024 // Still larger than in infcommon.h @@ -41,27 +43,14 @@ typedef struct _INFCONTEXT #define INF_STYLE_WIN4 0x00000002 #endif
-#if 0 -typedef PVOID HINF; -typedef struct _INFCONTEXT -{ - HINF Inf; - HINF CurrentInf; - UINT Section; - UINT Line; -} INFCONTEXT, *PINFCONTEXT; -#endif - -C_ASSERT(sizeof(INFCONTEXT) == 2 * sizeof(HINF) + 2 * sizeof(UINT));
+/* FUNCTIONS *****************************************************************/
// #define SetupCloseInfFile InfCloseFile typedef VOID (WINAPI* pSpInfCloseInfFile)( IN HINF InfHandle);
-extern pSpInfCloseInfFile SpInfCloseInfFile; - // #define SetupFindFirstLineW InfpFindFirstLineW typedef BOOL (WINAPI* pSpInfFindFirstLine)( @@ -70,23 +59,17 @@ typedef BOOL IN PCWSTR Key, IN OUT PINFCONTEXT Context);
-extern pSpInfFindFirstLine SpInfFindFirstLine; - // #define SetupFindNextLine InfFindNextLine typedef BOOL (WINAPI* pSpInfFindNextLine)( IN PINFCONTEXT ContextIn, OUT PINFCONTEXT ContextOut);
-extern pSpInfFindNextLine SpInfFindNextLine; - // #define SetupGetFieldCount InfGetFieldCount typedef ULONG (WINAPI* pSpInfGetFieldCount)( IN PINFCONTEXT Context);
-extern pSpInfGetFieldCount SpInfGetFieldCount; - // #define SetupGetBinaryField InfGetBinaryField typedef BOOL (WINAPI* pSpInfGetBinaryField)( @@ -96,8 +79,6 @@ typedef BOOL IN ULONG ReturnBufferSize, OUT PULONG RequiredSize);
-extern pSpInfGetBinaryField SpInfGetBinaryField; - // #define SetupGetIntField InfGetIntField typedef BOOL (WINAPI* pSpInfGetIntField)( @@ -105,8 +86,6 @@ typedef BOOL IN ULONG FieldIndex, OUT INT *IntegerValue); // PINT
-extern pSpInfGetIntField SpInfGetIntField; - // #define SetupGetMultiSzFieldW InfGetMultiSzField typedef BOOL (WINAPI* pSpInfGetMultiSzField)( @@ -116,8 +95,6 @@ typedef BOOL IN ULONG ReturnBufferSize, OUT PULONG RequiredSize);
-extern pSpInfGetMultiSzField SpInfGetMultiSzField; - // #define SetupGetStringFieldW InfGetStringField typedef BOOL (WINAPI* pSpInfGetStringField)( @@ -127,16 +104,12 @@ typedef BOOL IN ULONG ReturnBufferSize, OUT PULONG RequiredSize);
-extern pSpInfGetStringField SpInfGetStringField; - // #define pSetupGetField typedef PCWSTR (WINAPI* pSpInfGetField)( IN PINFCONTEXT Context, IN ULONG FieldIndex);
-extern pSpInfGetField SpInfGetField; - /* A version of SetupOpenInfFileW with support for a user-provided LCID */ // #define SetupOpenInfFileExW InfpOpenInfFileW typedef HINF @@ -147,8 +120,32 @@ typedef HINF IN LCID LocaleId, OUT PUINT ErrorLine);
-extern pSpInfOpenInfFile SpInfOpenInfFile; - +typedef struct _SPINF_EXPORTS +{ + pSpInfCloseInfFile SpInfCloseInfFile; + pSpInfFindFirstLine SpInfFindFirstLine; + pSpInfFindNextLine SpInfFindNextLine; + pSpInfGetFieldCount SpInfGetFieldCount; + pSpInfGetBinaryField SpInfGetBinaryField; + pSpInfGetIntField SpInfGetIntField; + pSpInfGetMultiSzField SpInfGetMultiSzField; + pSpInfGetStringField SpInfGetStringField; + pSpInfGetField SpInfGetField; + pSpInfOpenInfFile SpInfOpenInfFile; +} SPINF_EXPORTS, *PSPINF_EXPORTS; + +extern /*SPLIBAPI*/ SPINF_EXPORTS SpInfExports; + +#define SpInfCloseInfFile (SpInfExports.SpInfCloseInfFile) +#define SpInfFindFirstLine (SpInfExports.SpInfFindFirstLine) +#define SpInfFindNextLine (SpInfExports.SpInfFindNextLine) +#define SpInfGetFieldCount (SpInfExports.SpInfGetFieldCount) +#define SpInfGetBinaryField (SpInfExports.SpInfGetBinaryField) +#define SpInfGetIntField (SpInfExports.SpInfGetIntField) +#define SpInfGetMultiSzField (SpInfExports.SpInfGetMultiSzField) +#define SpInfGetStringField (SpInfExports.SpInfGetStringField) +#define SpInfGetField (SpInfExports.SpInfGetField) +#define SpInfOpenInfFile (SpInfExports.SpInfOpenInfFile)
/* HELPER FUNCTIONS **********************************************************/
diff --git a/base/setup/lib/utils/genlist.c b/base/setup/lib/utils/genlist.c index 79a4895075c..c204eacb1bf 100644 --- a/base/setup/lib/utils/genlist.c +++ b/base/setup/lib/utils/genlist.c @@ -17,6 +17,7 @@ /* FUNCTIONS ****************************************************************/
PGENERIC_LIST +NTAPI CreateGenericList(VOID) { PGENERIC_LIST List; @@ -33,6 +34,7 @@ CreateGenericList(VOID) }
VOID +NTAPI DestroyGenericList( IN OUT PGENERIC_LIST List, IN BOOLEAN FreeData) @@ -59,6 +61,7 @@ DestroyGenericList( }
BOOLEAN +NTAPI AppendGenericListEntry( IN OUT PGENERIC_LIST List, IN PVOID Data, @@ -84,6 +87,7 @@ AppendGenericListEntry( }
VOID +NTAPI SetCurrentListEntry( IN PGENERIC_LIST List, IN PGENERIC_LIST_ENTRY Entry) @@ -94,6 +98,7 @@ SetCurrentListEntry( }
PGENERIC_LIST_ENTRY +NTAPI GetCurrentListEntry( IN PGENERIC_LIST List) { @@ -101,6 +106,7 @@ GetCurrentListEntry( }
PGENERIC_LIST_ENTRY +NTAPI GetFirstListEntry( IN PGENERIC_LIST List) { @@ -111,6 +117,7 @@ GetFirstListEntry( }
PGENERIC_LIST_ENTRY +NTAPI GetNextListEntry( IN PGENERIC_LIST_ENTRY Entry) { @@ -123,6 +130,7 @@ GetNextListEntry( }
PVOID +NTAPI GetListEntryData( IN PGENERIC_LIST_ENTRY Entry) { @@ -137,6 +145,7 @@ GetListEntryUiData( }
ULONG +NTAPI GetNumberOfListEntries( IN PGENERIC_LIST List) { diff --git a/base/setup/lib/utils/genlist.h b/base/setup/lib/utils/genlist.h index ea3c7473bf5..d8ccd64260e 100644 --- a/base/setup/lib/utils/genlist.h +++ b/base/setup/lib/utils/genlist.h @@ -24,37 +24,45 @@ typedef struct _GENERIC_LIST
PGENERIC_LIST +NTAPI CreateGenericList(VOID);
VOID +NTAPI DestroyGenericList( IN OUT PGENERIC_LIST List, IN BOOLEAN FreeData);
BOOLEAN +NTAPI AppendGenericListEntry( IN OUT PGENERIC_LIST List, IN PVOID Data, IN BOOLEAN Current);
VOID +NTAPI SetCurrentListEntry( IN PGENERIC_LIST List, IN PGENERIC_LIST_ENTRY Entry);
PGENERIC_LIST_ENTRY +NTAPI GetCurrentListEntry( IN PGENERIC_LIST List);
PGENERIC_LIST_ENTRY +NTAPI GetFirstListEntry( IN PGENERIC_LIST List);
PGENERIC_LIST_ENTRY +NTAPI GetNextListEntry( IN PGENERIC_LIST_ENTRY Entry);
PVOID +NTAPI GetListEntryData( IN PGENERIC_LIST_ENTRY Entry);
@@ -63,6 +71,7 @@ GetListEntryUiData( IN PGENERIC_LIST_ENTRY Entry);
ULONG +NTAPI GetNumberOfListEntries( IN PGENERIC_LIST List);
diff --git a/base/setup/lib/utils/osdetect.c b/base/setup/lib/utils/osdetect.c index 1a4abd5cb6a..2d5ccd4a2dc 100644 --- a/base/setup/lib/utils/osdetect.c +++ b/base/setup/lib/utils/osdetect.c @@ -209,12 +209,16 @@ EnumerateInstallations( return STATUS_SUCCESS; }
-/* - * FindSubStrI(PCWSTR str, PCWSTR strSearch) : - * Searches for a sub-string 'strSearch' inside 'str', similarly to what - * wcsstr(str, strSearch) does, but ignores the case during the comparisons. - */ -PCWSTR FindSubStrI(PCWSTR str, PCWSTR strSearch) +/** + * @brief + * Finds the first occurrence of a sub-string 'strSearch' inside 'str', + * using case-insensitive comparisons. + **/ +PCWSTR +NTAPI +FindSubStrI( + _In_ PCWSTR str, + _In_ PCWSTR strSearch) { PCWSTR cp = str; PCWSTR s1, s2; @@ -760,6 +764,7 @@ FindNTOSInstallations( **/ // EnumerateNTOSInstallations PGENERIC_LIST +NTAPI CreateNTOSInstallationsList( _In_ PPARTLIST PartList) { diff --git a/base/setup/lib/utils/osdetect.h b/base/setup/lib/utils/osdetect.h index e329b1c32f4..fc30aec6367 100644 --- a/base/setup/lib/utils/osdetect.h +++ b/base/setup/lib/utils/osdetect.h @@ -30,14 +30,14 @@ typedef struct _NTOS_INSTALLATION
// EnumerateNTOSInstallations PGENERIC_LIST +NTAPI CreateNTOSInstallationsList( _In_ PPARTLIST PartList);
-/* - * FindSubStrI(PCWSTR str, PCWSTR strSearch) : - * Searches for a sub-string 'strSearch' inside 'str', similarly to what - * wcsstr(str, strSearch) does, but ignores the case during the comparisons. - */ -PCWSTR FindSubStrI(PCWSTR str, PCWSTR strSearch); +PCWSTR +NTAPI +FindSubStrI( + _In_ PCWSTR str, + _In_ PCWSTR strSearch);
/* EOF */ diff --git a/base/setup/lib/utils/partinfo.h b/base/setup/lib/utils/partinfo.h index d1c598aba90..dac7af89158 100644 --- a/base/setup/lib/utils/partinfo.h +++ b/base/setup/lib/utils/partinfo.h @@ -16,7 +16,7 @@ typedef struct _MBR_PARTITION_TYPE } MBR_PARTITION_TYPE, *PMBR_PARTITION_TYPE;
#define NUM_MBR_PARTITION_TYPES 153 -extern const MBR_PARTITION_TYPE MbrPartitionTypes[NUM_MBR_PARTITION_TYPES]; +extern SPLIBAPI const MBR_PARTITION_TYPE MbrPartitionTypes[NUM_MBR_PARTITION_TYPES];
/* GPT PARTITION TYPES ******************************************************/
@@ -27,6 +27,6 @@ typedef struct _GPT_PARTITION_TYPE } GPT_PARTITION_TYPE, *PGPT_PARTITION_TYPE;
#define NUM_GPT_PARTITION_TYPES 177 -extern const GPT_PARTITION_TYPE GptPartitionTypes[NUM_GPT_PARTITION_TYPES]; +extern SPLIBAPI const GPT_PARTITION_TYPE GptPartitionTypes[NUM_GPT_PARTITION_TYPES];
/* EOF */ diff --git a/base/setup/lib/utils/partlist.c b/base/setup/lib/utils/partlist.c index b138c9abdca..59ad7843c46 100644 --- a/base/setup/lib/utils/partlist.c +++ b/base/setup/lib/utils/partlist.c @@ -1984,6 +1984,7 @@ GetActiveDiskPartition( }
PPARTLIST +NTAPI CreatePartitionList(VOID) { PPARTLIST List; @@ -2069,6 +2070,7 @@ CreatePartitionList(VOID) }
VOID +NTAPI DestroyPartitionList( IN PPARTLIST List) { @@ -2288,6 +2290,7 @@ SelectPartition( }
PPARTENTRY +NTAPI GetNextPartition( IN PPARTLIST List, IN PPARTENTRY CurrentPart OPTIONAL) @@ -2380,6 +2383,7 @@ GetNextPartition( }
PPARTENTRY +NTAPI GetPrevPartition( IN PPARTLIST List, IN PPARTENTRY CurrentPart OPTIONAL) @@ -2783,6 +2787,7 @@ UpdateDiskLayout( * @return The adjacent unpartitioned region, if it exists, or NULL. **/ PPARTENTRY +NTAPI GetAdjUnpartitionedEntry( _In_ PPARTENTRY PartEntry, _In_ BOOLEAN Direction) @@ -2872,6 +2877,7 @@ MBRPartitionCreateChecks( }
ERROR_NUMBER +NTAPI PartitionCreateChecks( _In_ PPARTENTRY PartEntry, _In_opt_ ULONGLONG SizeBytes, @@ -2900,6 +2906,7 @@ PartitionCreateChecks( // (see VDS::CREATE_PARTITION_PARAMETERS and PPARTITION_INFORMATION_MBR/GPT for example) // So far we only use it as the optional type of the partition to create. BOOLEAN +NTAPI CreatePartition( _In_ PPARTLIST List, _Inout_ PPARTENTRY PartEntry, @@ -2990,6 +2997,7 @@ DismountPartition( }
BOOLEAN +NTAPI DeletePartition( _In_ PPARTLIST List, _In_ PPARTENTRY PartEntry, diff --git a/base/setup/lib/utils/partlist.h b/base/setup/lib/utils/partlist.h index 71b9eb99181..03d7999d11b 100644 --- a/base/setup/lib/utils/partlist.h +++ b/base/setup/lib/utils/partlist.h @@ -283,9 +283,11 @@ IsPartitionActive( IN PPARTENTRY PartEntry);
PPARTLIST +NTAPI CreatePartitionList(VOID);
VOID +NTAPI DestroyPartitionList( IN PPARTLIST List);
@@ -323,27 +325,32 @@ SelectPartition( _In_ ULONG PartitionNumber);
PPARTENTRY +NTAPI GetNextPartition( IN PPARTLIST List, IN PPARTENTRY CurrentPart OPTIONAL);
PPARTENTRY +NTAPI GetPrevPartition( IN PPARTLIST List, IN PPARTENTRY CurrentPart OPTIONAL);
PPARTENTRY +NTAPI GetAdjUnpartitionedEntry( _In_ PPARTENTRY PartEntry, _In_ BOOLEAN Direction);
ERROR_NUMBER +NTAPI PartitionCreateChecks( _In_ PPARTENTRY PartEntry, _In_opt_ ULONGLONG SizeBytes, _In_opt_ ULONG_PTR PartitionInfo);
BOOLEAN +NTAPI CreatePartition( _In_ PPARTLIST List, _Inout_ PPARTENTRY PartEntry, @@ -351,6 +358,7 @@ CreatePartition( _In_opt_ ULONG_PTR PartitionInfo);
BOOLEAN +NTAPI DeletePartition( _In_ PPARTLIST List, _In_ PPARTENTRY PartEntry, diff --git a/base/setup/reactos/CMakeLists.txt b/base/setup/reactos/CMakeLists.txt index 23adc345081..92dbb317904 100644 --- a/base/setup/reactos/CMakeLists.txt +++ b/base/setup/reactos/CMakeLists.txt @@ -17,9 +17,10 @@ list(APPEND SOURCE file(GLOB reactos_rc_deps res/*.*) add_rc_deps(reactos.rc ${reactos_rc_deps}) add_executable(reactos ${SOURCE} reactos.rc) + add_pch(reactos reactos.h SOURCE) set_module_type(reactos win32gui UNICODE) -target_link_libraries(reactos uuid setuplib ext2lib vfatlib btrfslib ${PSEH_LIB}) +target_link_libraries(reactos uuid) target_link_libraries(reactos zlib_solo) ## We use USETUP's cabinet implementation -add_importlibs(reactos advapi32 gdi32 user32 comctl32 shlwapi setupapi msvcrt kernel32 ntdll) +add_importlibs(reactos advapi32 gdi32 user32 comctl32 shlwapi setupapi setuplib msvcrt kernel32 ntdll) add_cd_file(TARGET reactos DESTINATION reactos NO_CAB FOR bootcd) diff --git a/base/setup/reactos/reactos.c b/base/setup/reactos/reactos.c index 3909f104e20..fe169bd3ee9 100644 --- a/base/setup/reactos/reactos.c +++ b/base/setup/reactos/reactos.c @@ -38,7 +38,6 @@ /* GLOBALS ******************************************************************/
HANDLE ProcessHeap; -BOOLEAN IsUnattendedSetup = FALSE; SETUPDATA SetupData;
/* The partition where to perform the installation */ @@ -2826,11 +2825,9 @@ _tWinMain(HINSTANCE hInst, /* Initialize the NT to Win32 path prefix mapping list */ InitNtToWin32PathMappingList(&SetupData.MappingList);
- /* Initialize Setup, phase 0 */ - InitializeSetup(&SetupData.USetupData, 0); - - /* Initialize Setup, phase 1 */ - Error = InitializeSetup(&SetupData.USetupData, 1); + /* Initialize Setup */ + Error = InitializeSetup(&SetupData.USetupData, NULL, + &SpFileExports, &SpInfExports); if (Error != ERROR_SUCCESS) { // diff --git a/base/setup/reactos/reactos.h b/base/setup/reactos/reactos.h index 8beffc4c86a..286a303034c 100644 --- a/base/setup/reactos/reactos.h +++ b/base/setup/reactos/reactos.h @@ -163,8 +163,6 @@ typedef struct _SETUPDATA } SETUPDATA, *PSETUPDATA;
extern HANDLE ProcessHeap; -extern BOOLEAN IsUnattendedSetup; - extern SETUPDATA SetupData;
extern PPARTENTRY InstallPartition; diff --git a/base/setup/reactos/spapisup/fileqsup.c b/base/setup/reactos/spapisup/fileqsup.c index 95c789de085..334290a17be 100644 --- a/base/setup/reactos/spapisup/fileqsup.c +++ b/base/setup/reactos/spapisup/fileqsup.c @@ -173,12 +173,15 @@ SpFileQueueRename_NtToWin32(
/* GLOBALS *******************************************************************/
-pSpFileQueueOpen SpFileQueueOpen = SetupOpenFileQueue; -pSpFileQueueClose SpFileQueueClose = SetupCloseFileQueue; -pSpFileQueueCopy SpFileQueueCopy = SpFileQueueCopy_NtToWin32; -pSpFileQueueDelete SpFileQueueDelete = SpFileQueueDelete_NtToWin32; -pSpFileQueueRename SpFileQueueRename = SpFileQueueRename_NtToWin32; -pSpFileQueueCommit SpFileQueueCommit = SetupCommitFileQueueW; +SPFILE_EXPORTS SpFileExports = +{ + SetupOpenFileQueue, + SetupCloseFileQueue, + SpFileQueueCopy_NtToWin32, + SpFileQueueDelete_NtToWin32, + SpFileQueueRename_NtToWin32, + SetupCommitFileQueueW +};
#endif
diff --git a/base/setup/reactos/spapisup/infsupp.c b/base/setup/reactos/spapisup/infsupp.c index 98e48b07bb3..0e2586e1ee1 100644 --- a/base/setup/reactos/spapisup/infsupp.c +++ b/base/setup/reactos/spapisup/infsupp.c @@ -83,16 +83,19 @@ SetupOpenInfFileExW(
/* GLOBALS *******************************************************************/
-pSpInfCloseInfFile SpInfCloseInfFile = SetupCloseInfFile; -pSpInfFindFirstLine SpInfFindFirstLine = SetupFindFirstLineW; -pSpInfFindNextLine SpInfFindNextLine = SetupFindNextLine; -pSpInfGetFieldCount SpInfGetFieldCount = SetupGetFieldCount; -pSpInfGetBinaryField SpInfGetBinaryField = SetupGetBinaryField; -pSpInfGetIntField SpInfGetIntField = SetupGetIntField; -pSpInfGetMultiSzField SpInfGetMultiSzField = SetupGetMultiSzFieldW; -pSpInfGetStringField SpInfGetStringField = SetupGetStringFieldW; -pSpInfGetField SpInfGetField = pSetupGetField; -pSpInfOpenInfFile SpInfOpenInfFile = SetupOpenInfFileExW; +SPINF_EXPORTS SpInfExports = +{ + SetupCloseInfFile, + SetupFindFirstLineW, + SetupFindNextLine, + SetupGetFieldCount, + SetupGetBinaryField, + SetupGetIntField, + SetupGetMultiSzFieldW, + SetupGetStringFieldW, + pSetupGetField, + SetupOpenInfFileExW +};
/* HELPER FUNCTIONS **********************************************************/ diff --git a/base/setup/usetup/CMakeLists.txt b/base/setup/usetup/CMakeLists.txt index 21d2bdcf646..ba47a8a9822 100644 --- a/base/setup/usetup/CMakeLists.txt +++ b/base/setup/usetup/CMakeLists.txt @@ -35,6 +35,6 @@ endif()
add_pch(usetup usetup.h SOURCE) set_module_type(usetup nativecui) -target_link_libraries(usetup inflib setuplib zlib_solo ext2lib vfatlib btrfslib chkstk ${PSEH_LIB}) -add_importlibs(usetup ntdll) +target_link_libraries(usetup inflib zlib_solo chkstk) +add_importlibs(usetup setuplib ntdll) add_cd_file(TARGET usetup DESTINATION reactos/system32 NO_CAB NAME_ON_CD smss.exe FOR bootcd regtest) diff --git a/base/setup/usetup/spapisup/fileqsup.c b/base/setup/usetup/spapisup/fileqsup.c index 10fc5d335ac..77a14e531c9 100644 --- a/base/setup/usetup/spapisup/fileqsup.c +++ b/base/setup/usetup/spapisup/fileqsup.c @@ -997,11 +997,14 @@ Quit:
/* GLOBALS *******************************************************************/
-pSpFileQueueOpen SpFileQueueOpen = SetupOpenFileQueue; -pSpFileQueueClose SpFileQueueClose = SetupCloseFileQueue; -pSpFileQueueCopy SpFileQueueCopy = SetupQueueCopyWithCab; -pSpFileQueueDelete SpFileQueueDelete = SetupQueueDeleteW; -pSpFileQueueRename SpFileQueueRename = SetupQueueRenameW; -pSpFileQueueCommit SpFileQueueCommit = SetupCommitFileQueueW; +SPFILE_EXPORTS SpFileExports = +{ + SetupOpenFileQueue, + SetupCloseFileQueue, + SetupQueueCopyWithCab, + SetupQueueDeleteW, + SetupQueueRenameW, + SetupCommitFileQueueW +};
/* EOF */ diff --git a/base/setup/usetup/spapisup/infsupp.c b/base/setup/usetup/spapisup/infsupp.c index 012b29b9e63..4a4ace20ac0 100644 --- a/base/setup/usetup/spapisup/infsupp.c +++ b/base/setup/usetup/spapisup/infsupp.c @@ -228,16 +228,19 @@ SetupOpenInfFileExW(
/* GLOBALS *******************************************************************/
-pSpInfCloseInfFile SpInfCloseInfFile = SetupCloseInfFile; -pSpInfFindFirstLine SpInfFindFirstLine = SetupFindFirstLineW; -pSpInfFindNextLine SpInfFindNextLine = SetupFindNextLine; -pSpInfGetFieldCount SpInfGetFieldCount = SetupGetFieldCount; -pSpInfGetBinaryField SpInfGetBinaryField = SetupGetBinaryField; -pSpInfGetIntField SpInfGetIntField = SetupGetIntField; -pSpInfGetMultiSzField SpInfGetMultiSzField = SetupGetMultiSzFieldW; -pSpInfGetStringField SpInfGetStringField = SetupGetStringFieldW; -pSpInfGetField SpInfGetField = pSetupGetField; -pSpInfOpenInfFile SpInfOpenInfFile = SetupOpenInfFileExW; +SPINF_EXPORTS SpInfExports = +{ + SetupCloseInfFile, + SetupFindFirstLineW, + SetupFindNextLine, + SetupGetFieldCount, + SetupGetBinaryField, + SetupGetIntField, + SetupGetMultiSzFieldW, + SetupGetStringFieldW, + pSetupGetField, + SetupOpenInfFileExW +};
/* HELPER FUNCTIONS **********************************************************/ diff --git a/base/setup/usetup/usetup.c b/base/setup/usetup/usetup.c index 10d133eb560..13b4238aa46 100644 --- a/base/setup/usetup/usetup.c +++ b/base/setup/usetup/usetup.c @@ -40,7 +40,6 @@ /* GLOBALS & LOCALS *********************************************************/
HANDLE ProcessHeap; -BOOLEAN IsUnattendedSetup = FALSE;
static USETUP_DATA USetupData;
@@ -534,6 +533,28 @@ GetNTOSInstallationName( }
+// PSETUP_ERROR_ROUTINE +static VOID +__cdecl +USetupErrorRoutine( + IN PUSETUP_DATA pSetupData, + ...) +{ + INPUT_RECORD Ir; + va_list arg_ptr; + + va_start(arg_ptr, pSetupData); + + if (pSetupData->LastErrorNumber >= ERROR_SUCCESS && + pSetupData->LastErrorNumber < ERROR_LAST_ERROR_CODE) + { + // Note: the "POPUP_WAIT_ENTER" actually depends on the LastErrorNumber... + MUIDisplayErrorV(pSetupData->LastErrorNumber, &Ir, POPUP_WAIT_ENTER, arg_ptr); + } + + va_end(arg_ptr); +} + /* * Start page * @@ -566,8 +587,9 @@ SetupStartPage(PINPUT_RECORD Ir)
MUIDisplayPage(SETUP_INIT_PAGE);
- /* Initialize Setup, phase 1 */ - Error = InitializeSetup(&USetupData, 1); + /* Initialize Setup */ + Error = InitializeSetup(&USetupData, USetupErrorRoutine, + &SpFileExports, &SpInfExports); if (Error != ERROR_SUCCESS) { MUIDisplayError(Error, Ir, POPUP_WAIT_ENTER); @@ -3037,28 +3059,6 @@ InitInstallDir: }
-// PSETUP_ERROR_ROUTINE -static VOID -__cdecl -USetupErrorRoutine( - IN PUSETUP_DATA pSetupData, - ...) -{ - INPUT_RECORD Ir; - va_list arg_ptr; - - va_start(arg_ptr, pSetupData); - - if (pSetupData->LastErrorNumber >= ERROR_SUCCESS && - pSetupData->LastErrorNumber < ERROR_LAST_ERROR_CODE) - { - // Note: the "POPUP_WAIT_ENTER" actually depends on the LastErrorNumber... - MUIDisplayErrorV(pSetupData->LastErrorNumber, &Ir, POPUP_WAIT_ENTER, arg_ptr); - } - - va_end(arg_ptr); -} - /* * Displays the PrepareCopyPage. * @@ -4034,10 +4034,6 @@ RunUSetup(VOID) return STATUS_APP_INIT_FAILURE; }
- /* Initialize Setup, phase 0 */ - InitializeSetup(&USetupData, 0); - USetupData.ErrorRoutine = USetupErrorRoutine; - /* Hide the cursor and clear the screen and keyboard buffer */ CONSOLE_SetCursorType(TRUE, FALSE); CONSOLE_ClearScreen(); diff --git a/base/setup/usetup/usetup.h b/base/setup/usetup/usetup.h index 91b94ac9699..e0d7f7face8 100644 --- a/base/setup/usetup/usetup.h +++ b/base/setup/usetup/usetup.h @@ -70,7 +70,6 @@
extern HANDLE ProcessHeap; -extern BOOLEAN IsUnattendedSetup; extern PCWSTR SelectedLanguageId;
typedef enum _PAGE_NUMBER diff --git a/modules/rostests/unittests/setuplib/CMakeLists.txt b/modules/rostests/unittests/setuplib/CMakeLists.txt index de454898e67..8ac15930dad 100644 --- a/modules/rostests/unittests/setuplib/CMakeLists.txt +++ b/modules/rostests/unittests/setuplib/CMakeLists.txt @@ -11,8 +11,7 @@ list(APPEND SOURCE precomp.h)
add_executable(setuplib_unittest ${SOURCE}) -target_link_libraries(setuplib_unittest setuplib ${PSEH_LIB}) set_module_type(setuplib_unittest win32cui) -add_importlibs(setuplib_unittest msvcrt kernel32 ntdll) +add_importlibs(setuplib_unittest setuplib msvcrt kernel32) #add_pch(setuplib_unittest precomp.h SOURCE) add_rostests_file(TARGET setuplib_unittest) diff --git a/modules/rostests/unittests/setuplib/IsValidInstallDirectory.c b/modules/rostests/unittests/setuplib/IsValidInstallDirectory.c index c7f3a599271..9456ab16a7e 100644 --- a/modules/rostests/unittests/setuplib/IsValidInstallDirectory.c +++ b/modules/rostests/unittests/setuplib/IsValidInstallDirectory.c @@ -7,24 +7,8 @@
#include "precomp.h"
-// -// FIXME: Temporary symbols defined to make linking work. -// They will be defined to something once INF file testing is implemented. -// -pSpInfCloseInfFile SpInfCloseInfFile = NULL; -pSpInfFindFirstLine SpInfFindFirstLine = NULL; -pSpInfFindNextLine SpInfFindNextLine = NULL; -pSpInfGetFieldCount SpInfGetFieldCount = NULL; -pSpInfGetBinaryField SpInfGetBinaryField = NULL; -pSpInfGetIntField SpInfGetIntField = NULL; -pSpInfGetMultiSzField SpInfGetMultiSzField = NULL; -pSpInfGetStringField SpInfGetStringField = NULL; -pSpInfGetField SpInfGetField = NULL; -pSpInfOpenInfFile SpInfOpenInfFile = NULL; - -BOOLEAN IsUnattendedSetup = FALSE; -HANDLE ProcessHeap; - +// SPFILE_EXPORTS SpFileExports = {NULL}; +// SPINF_EXPORTS SpInfExports = {NULL};
START_TEST(IsValidInstallDirectory) { @@ -93,8 +77,6 @@ START_TEST(IsValidInstallDirectory)
#define BOOL_TO_STR(b) ((b) ? "TRUE" : "FALSE")
- ProcessHeap = GetProcessHeap(); - UINT i; for (i = 0; i < _countof(tests); ++i) {
-
Hermès Bélusca-Maïto