[HAL] Add APIC hals to setup and livecd

[HAL:APIC] Fix interrupt assignment

This is probably not how it is supposed to work, but the results are somewhat similar to what Windows has.

[HAL:APIC] Remove obsolete hacks

These hacks were used to work around broken VBox. VBox works fine without the hacks these days.

[HAL:APIC] Code fixes

* Fix some broken code
* Add some ASSERTs
* Use ApicWriteIORedirectionEntry where appropriate
* Use KeQueryInterruptHandler/KeRegisterInterruptHandler to save/restore the old handler instead of saving the KIDTENTRY
* Move HalpProfileInterruptHandler to apictimer.c and implement it
* Use READ/WRITE_REGISTER macros
* Add some symbolic names

[MINIHAL] Compile less stuff

[HAL] Rename apic.h to apicp.h and move it back to the apic folder

[NTOS:IOPNP] Both PNP_PROPERTY_REMOVAL_POLICY_OVERRIDE and PNP_PROPERTY_LOCATION_PATHS are unimplemented, return such status.

Fixes CID 1441387: Uninitialized scalar variable (UNINIT),
in the case of PNP_PROPERTY_LOCATION_PATHS.

[NTOS:IO] Comment out (and explain why it is there) an unused LdrEntry assignment in IopInitializeBootDrivers(). CID 1237114

[NTOS:IO] Revert part of commit a82ff90b2: don't do direct ULONG data access: it might be unaligned.

Requested by Victor.

[NTOS:IO] Fix driverName.Buffer leak in some failure paths in IopGetDriverNames().

driverName.Buffer leaked when the "(!NT_SUCCESS(status) || ServiceName != NULL)"
case is taken because ServiceName != NULL, and some of the functions fail.

[NTOS:IO] Fix basicInfo handling in IopGetDriverNames().

- Fix CID 1477246: Uninitialized pointer read (UNINIT) (happens in
  the last ExFreePoolWithTag(basicInfo, TAG_IO) call when the
  "(!NT_SUCCESS(status) || ServiceName != NULL)" case is not taken).

- Centralize all the ExFreePoolWithTag(basicInfo, TAG_IO) cleanups
  at the end of the function.

- Both cases "(driverName.Buffer == NULL)" and "(ServiceName != NULL)"
  can only be taken when basicInfo != NULL, so assert on this fact.

[NTOS:IO] Fix similar bugs in IopGetDriverNames(), IopInitializeBuiltinDriver() and IopLoadDriver().

- As they are initialized, some registry string values must be NULL-terminated.
- When retrieving REG_DWORD registry values, validate their length.

[NTOS:IO] Minor code style changes...

- Fix/add comments;
- Reduce indentation level;
- Direct copy for registry integer values;
- Use for-loops for linked lists;
- Use ULONG when the API uses it (sizes for Ob, or REG_DWORD data in registry).

[LDR] Restore Logging when images overlap CORE-17635

This reverts
0.4.13-dev-284-g 74be88ac473425fc99e9a363ba8cd604702262bf

The logging is helpful to alert us when we should really
recalculate the base addresses for performance reasons.
Spams a bit sometimes, sure, but it's worth it!

[NTOS:MM] Fix placing entries into MmNonPagedPoolFreeListHead

When freeing pages, free page entries with pages num == 3 were
incorrectly treated as entries with pages num >= 4 and thus
their re-insertion was not triggered. That lead to non paged pool
fragmentation (can be triggered by kmtest:ExPools, for example)

Also, altered the index acquisition code for MmNonPagedPoolFreeList
entries so it looks more clear

[COMCTL32] Strip EOL whitespace within ifdef __REACTOS__ section

no functional change.
Make that look beautiful before porting it back further.

Addendum to 0.4.15-dev-534-g 887764e6075fdde7fb81d2a54d5255164c046511
(CORE-17062)

[MSI] Enumerate only type DRIVE_FIXED CORE-17623 (& CORE-14432)

MSI on 2k3sp2 hides empty CD-ROM, and hides CD-ROM with iso mounted.
MSI on 2k3sp2 hides VBox shares, both: writable and read-only-ones.
MSI on 2k3sp2 hides floppy drives with a floppy img mounted.
MSI on XPSP3 hides a mounted writable USB stick.

Thanks to the patches author Doug Lyons.

This patch supersedes
0.4.15-dev-2651-g 864e20b881130cbc9668b8e253a31b8c325e9b99 (CORE-14432)
and we are also better than Wines
https://source.winehq.org/git/wine.git/commit/0dd0d879a1c487dcefedd44762d4ea0fcebd98fe

[GITIGNORE] Add compile_commands.json and .cache folder for clangd cache

[WIN32K] Add x64/arm exports

Checked against Windows XP x64 and Windows 8 ARM.

[FORMATTING][NTOS][HAL][DRIVERS][SDK] Fix trailing spaces

[NTOS:IO] Modify when 'PartitionBuffer' and how 'DriveLayout' are freed in IopCreateArcNamesDisk().

- Manage the lifetime of the temporary 'PartitionBuffer' buffer where
  it is locally used only, and free it as soon as possible, just after
  calculating the sector checksum. No need to then free it outside of
  the main for-loop.

- When the 'DriveLayout' buffer is freed, ensure the pointer is NULL-ed
  (and assert this at the top of the main for-loop), since it can also
  be freed at cleanup outside this for-loop, and in this case a NULL
  check is performed.
  This will avoid the scenario of possibly double-freeing a pointer,
  in the case the 'DriveLayout' was previously freed (when e.g. reading
  the sector for checksum calculation failed), then the for-loop goes to
  the next disk and stops early.

[NTOS:IO/FSTUB] Fix the determination of 'SingleDisk' in IoGetBootDiskInformation().

The purpose of 'SingleDisk' is the same as in the IopCreateArcNames()
function. It is an optimization for that when looking up the
firmware-recognized ARC disks list, in order to match one of these with
the current NT disk being analysed (see e.g. also in IopCreateArcNamesDisk()),
we avoid a possible IopVerifyDiskSignature() call and directly build a
corresponding ARC name NT symbolic link for it.

'SingleDisk' will actually be TRUE, whether the DiskSignatureListHead
list is empty or contains only one element: Indeed in only both these
cases, 'DiskSignatureListHead.Flink->Flink' will refer to the list head.
(If the list is empty but 'SingleDisk' is TRUE, this does not matter,
because the DiskSignatureListHead looking-up loop never starts.)

[NTOS:IO] Avoid hardcoding variable types in sizeofs when possible.

[NTOS:IO] Minor code style fixes (typos; improve comments/DPRINT; IN vs. OUT parameter).

[SHELL32_APITEST] Improve ShellExecCmdLine testcase more (#3736)

- Simplify code.
- Loose the condition of success.
- Add some missing tests.
- Close newly opened windows gracefully.
CORE-17351, CORE-16898, CORE-17612

[CMAKE] host-tools: Define ROS_SAVED_* for MSVC_IDE only

Addendum to 18e95f5.
ROSBE-174

[SOUNDS] Add new Recycle Bin system sound
Add new sound for Recycle Bin sound event.
Since this sound event is now working properly after some fixes in shell32 code, it's now possible for user to set the sound for this sound event manually.
But don't enable it by default, since it is also disabled on Windows Server 2003.
CORE-15868

[RAPPS] Fix double free when closing rapps

[HIDPARSER] Don't include pshpack1.h before including other headers

NEVER DO THIS! It is guaranteed to be wrong. Instead always individually pack single structures that need packing.
This fixes USB mouse on 64 bit builds.

[MORE] Use positional elements to build the MORE prompt, whose state is controlled by flags.

[MORE] Update and complete MORE's help usage.

And add french translation.

[WIN32K:ENG] In EngLockDriverObj, properly return NULL when locking failed

CORE-15958

[MORE][CONUTILS] Implement missing features of the MORE command (#3658)

Implement missing features of the MORE command. Special thanks to @HBelusca. CORE-4019

[NTOS:MM] Handle PT references for page table faults in MmAccessFault

Idea from Timo Kreuzer

[NTOS:MM] Get rid of MiQueryPageTableReferences

[NTOS] Implement MiIsPageTablePresent as a replacement for the abused MiQueryPageTableReferences

[NTOS:MM] In MiMakePdeExistAndValid, do not overly check PDE validity

It's "okay" to not #ifdef, but it's useless and confusing.

[NTOS:MM] Implement proper refcounting of page tables on amd64

CORE-17552

[NTOS:MM] Make MiIsEntireRangeCommitted 64-bits aware

[NTOS:MM] Add some DPRINTs when incrementing or decrementing PFN share count

[NTOS:MM] Make MiCalculatePageCommitment 64-bits aware

[NTOS:MM] Add a few MiP*eToP*e helpers

And turn some of the existing ones into inline functions for the sake of type checking

[NTOS:MM] Properly initialize initial process address space on amd64

By applying the same kludge on the relevant MMPFNs than on x86

[NTOS:MM] Unconditionally delete the PFN in MiDeletePte when PTE is in transition

When we will have a modified page writer, it will have to know if the written
page is stale when being done, and act accordingly.

CORE-8552

[NTOS:MM] Remove some dead code from legacy Mm

[CMD] Use GetFullPathName instead of copying buffers when resolving executable names

This properly strips spaces & dots at the end of the file.

Fixes the infinite loop with CreateProcess calling cmd over and over with e.g. 'cmd /c "some_script.bat   "'
Uncovered by recent ShellExecuteEx tests
Dedicated to Katayama for the trigger & Hermès for the tests

[USER32_APITEST] Add SetFocus testcase (#3732)

Investigate about setting keyboard focus. CORE-17550

[HIVESYS] Use "Primary Disk" writing for this driver load order group

[NTOS:SE] Use IQS_NONE for the first class which is not existent and barely used

[NTOS:PS] Use IQS_NONE for classes that do not exist

[NTOSKRNL] Use CHAR for non existent classes

An alignment of 1 means no alignment required and the class doesn't exist. 0 shouldn't be used for alignment requirement in IQS_NONE!

[SHELL32_APITEST] Strengthen ShellExecuteEx testcase (#3731)

Investigate shell32!ShellExecuteEx function more. CORE-17351, CORE-16898, CORE-17612

[NTOS:MM] Fix remnants of CORE-17587

[NTOS:MM] Make MI_MAKE_HARDWARE_PTE_KERNEL actually respect the protection it is given

This one would set a page as writable & dirty even if MM_READONLY was passed
CORE-17587

[CPL][APPWIZ] Follow-up of #3726

Sorry, #3726 doesn't fix CORE-5737. CORE-17550

[USETUP] Polish translation update (#3715)

[VIDEOPRT] Fix monitor ID decoding issues

[NTOS:PS] Enable alignment probing for thread/process information classes

In addition to that, here are some stuff done in this commit whilst testing:

- ICIF_QUERY_SIZE_VARIABLE and friends were badly misused, they should be used only when an information class whose information length size is dyanmic and not fixed. By removing such flags from erroneous classes, this fixes the STATUS_INFO_LENGTH_MISMATCH testcases.

- Use CHAR instead of UCHAR for classes that do not need alignment probing, as every other class in the table do, for the sake of consistency.

- ProcessEnableAlignmentFaultFixup uses BOOLEAN as type size, not CHAR. This fixes a testcase failure on ROS.

- Check for information length size before proceeding further on querying the process' cookie information.

- ProcessHandleTracing wants an alignment of a ULONG, not CHAR.

- Move PROCESS_LDT_INFORMATION and PROCESS_LDT_SIZE outside of NTOS_MODE_USER macro case. This fixes a compilation issue when enabling the alignment probing. My mistake of having them inside NTOS_MODE_USER case, sorry.

- On functions like NtQueryInformationThread and the Process equivalent, complete probing is not done at the beginning of the function, complete probing including if the buffer is writable alongside with datatype misalignment check that is. Instead such check is done on each information class case basis. With that said, we have to explicitly tell DefaultQueryInfoBufferCheck if we want a complete probing or not initially.

[SHELL32_APITEST] ShellHook testcase Fix compiler error (addendum to #3289)

addendum to 0.4.15-dev-1123-g 0089291751837d977431c85b6d8d6b1232fbb67b

Fix a compiler error on RosBE2.1.6 GCC4.7.2
that I got when I merged this test back to 0.4.14RC:

C:/0414rls/reactos/modules/rostests/apitests/shell32/ShellHook.cpp: In function
'LRESULT WindowProc(HWND, UINT, WPARAM, LPARAM)':
C:/0414rls/reactos/modules/rostests/apitests/shell32/ShellHook.cpp:339:26: error
: comparison between signed and unsigned integer expressions [-Werror=sign-compa
re]
cc1plus.exe: all warnings being treated as errors
ninja: build stopped: subcommand failed.

[LSASRV] Notify all authentication packages of a session to be deleted

[SHELL32_APITEST] Follow-up of #3723

Strengthen FindExecutable testcase. CORE-17351, CORE-16898, CORE-17612

[HOST-TOOLS] Block sending telemetry when spawning a new shell

[SHELL32_APITEST] Add FindExecutable testcase (#3723)

Investigate the implementation of shell32!FindExecutable function.
CORE-17351, CORE-16898, CORE-17612

[VBE] Check if DDC is supported before trying to read EDID

Also set Edx to 0 when reading Edid, as per specification.

CORE-16695

[VIDEOPRT] Report monitor name instead of the generic 'Monitor' string

[VIDEOPRT] Give to each device its own entry in HKLM\SYSTEM\CurrentControlSet\Services

This is required if you have two graphic cards using the same driver.

[ATL_APITEST] Add some testcases for CString 'operator !='

[ATL] Implement CString 'operator !='

[SYSSETUP] Use the calculated full path

[MSI] setup should not enumerate read-only CDRoms as install-targets CORE-14432

We decided to check a bit different than Wine currently does.
Wine checks for
if (GetVolumeInformationW(ptr, NULL, 0, NULL, 0, &flags, NULL, 0) && flags & FILE_READ_ONLY_VOLUME)
to exclude read-only-media while we (for now) check for
if (GetDriveTypeW(ptr) == DRIVE_CDROM)

For now this gives us the benefit of covering also CDROM drives, that do not have
a CD put into it.
That does not work properly for us with the Wine patch yet, but it does for Wine!
So this maybe points to some other bug in ros, maybe in our CDROM(FS)-drivers?

In the future we should also test this on other read-only-media in Windows e.g.
a read-only USB stick to decide whether we want to stick with our current solution
or whether we want to switch to Wines solution or whether we want to logically OR
both conditions!

Thanks to Doug Lyons for providing what *currently* does work best for us!

VBox https://reactos.org/testman/compare.php?ids=77537,77546 (no change)
KVM https://reactos.org/testman/compare.php?ids=77531,77541 (no change)

[MPR] spec: Add stubs, document parameter types, stub-enable 2 functions (#3696)

* Add 20 stubs from mpr.dll v5.2.3790.3959.
* Document some parameter types from https://github.com/Microsoft/winfile/
* Change WNetGetDirectoryType[A|W] to no-op stubs

CORE-14376

[RAPPS] Implement support for architecture specific sections

See CORE-17616

[RTL][KMTEST] RtlPcToFileHeader doesn't handle user mode images in kernel mode

[RTL] Fix RtlWalkFrameChain and wrap it in SEH

[RTL] Simplify RtlRaiseException

[RTL] Implement RtplUnwindInternal and wrap RtlUnwindEx and RtlDispatchException around it

Based on the description in this blog article: http://www.nynaeve.net/?p=106

[RTL] Improve RtlVirtualUnwind

* Add support for version 2 unwind info
* Implement UnwindOpSlots() and use it (based on https://github.com/dotnet/coreclr/blob/master/src/unwinder/amd64/unwinder_amd64.cpp)
* Fix handling of UWOP_PUSH_MACHFRAME

[RTL] Move exception handling code to except.c

[RTL] Improve / fix the SList code

Don't push a register in the middle of a an asm function. Instead add a proper prolog and store rbx in a home param slot.

[CRT] Implement __C_specific_handler

[CPL][APPWIZ] Make gecko download cancellable by keyboard (#3726)

We shouldn't disable nor hide the control with focus. CORE-17550, CORE-5737

[WINETESTS] Workaround crashes in riched20:richole on VBox/KVM bots CORE-16799

The crashes began when those tests were synced to Wine Staging 4.18 in
0.4.14-dev-419-g 76cf09cfea15911082245a65547fb67512b710ce

WHS 2k3sp2 Before the revert implies the tests were actually ok:
https://reactos.org/testman/compare.php?ids=77492

Disabling some of the tests is very unsatisfying of course,
but at least allows us to observe again
the results of the other tests from that suite.

VBox https://reactos.org/testman/compare.php?ids=77504,77506
KVM https://reactos.org/testman/compare.php?ids=77502,77505

[NTOS:KE/x64] Handle shared interrupts

[EXPLORER] Check m_Position for -1 before using it as an index

This fixes a crash on x64.

[BROWSEUI] Fix broken code

[SHELL32_APITEST] ShellExecCmdLine: Don't copy Notepad.exe

On Windows 10, when I copyed Notepad.exe to the desktop and double-clicked it, it won't start Notepad. This is probably because there is a security protection mechanism (or some mechanism) at work. Tests that assume this mechanism will not work properly on ReactOS. Thus, I had to delete some tests. CORE-17351

[SHELL32_APITEST] ShellExecCmdLine: Close Notepad windows

By using newly added CloseNotepads function, close the Notepad window.
CORE-17351

[NTOS:PS/x64] Fix PspGetOrSetContextKernelRoutine

It now uses KiSetTrapContext to set the non-volatile context on the stack and the volatile context in the trap frame.

[NTOS:KE/x64] Implement KiSetTrapContext

KiSetTrapContext is an asm wrapper around RtlSetUnwindContext, which first stores an exception frame to assure that all non-volatile registers were put on the stack, then calls RtlSetUnwindContext to update their first saving positions on the stack and finally restore the exception frame to potentially load any updated registers, that haven't been saved elsewhere on the stack.

[RTL/x64] Implement RtlpCaptureNonVolatileContextPointers and RtlSetUnwindContext

RtlpCaptureNonVolatileContextPointers walks the stack and captures the addresses of all non-volatile registers on the stack, when they have been saved first. This is needed to be able to fix up the non-volatile on a system call, which doesn't capture non-volatiles, but relies on them to be restored by the callees.

Instead of only checking for the TargetFrame, also check for a mode change, i.e. RIP went from kernel to user, in which case the target frame was not reached yet, because it was too large, but processing can't continue here.

RtlSetUnwindContext uses RtlpCaptureNonVolatileContextPointers to set the non-volatile registers in the the stack. They will be picked up, when returning back or unwinding, e.g. to the system call handler.

[BOOT] CMakeLists.txt: Compare ARCH to arm64, not aarch64

This aarch64 was introduced first,
but arm64 was preferred everywhere else since then.

CORE-17518

[CMAKE] CMakeLists.txt: Fix 'omap-zoom2' typo

Addendum to fae2044 (r51783).
CORE-17604

[CRT][ARM] Fix uninitialized variables caught by RTC

CORE-17613

[EXPLORER] Polish translation update (#3704)

[BROWSEUI] Polish translation update (#3703)

[WINLOGON] Polish translation update (#3702)

[SHELL32] Polish translation update (#3701)

[NTOS:SE] Implement logon session deletion

And declare a prototype for SepCleanupLUIDDeviceMapDirectory and annotate it with SAL.

[CMAKE] CMakeLists.txt: Remove NO_UNDERSCORE_PREFIX leftovers

And switch to add_compile_definitions().

Addendum to 9efafd6.

[WIN32SS][NTUSER] ShowWindow.SW_MINIMIZE should show window (#3700)

- user32!ShowWindow.SW_MINIMIZE should show the window.
- Fix the return value of ShowWindow function on invalid parameter.
CORE-15669