[NTOS:CC] Avoid some magic numbers.

[SHELL32] Fix PIDL leak in CMenuSFToolbar::FillToolbar. CORE-13194

[DHCPCSVC] Fix a memory leak in bind_lease introduced in r14645 (!). CORE-14353

[KMTESTS:FSRTL] Add a few more tests matching MS FastFAT.

CORE-11819

[IPHLPAPI] Fix memory leak in getNumRoutes. CORE-14353

[KMTESTS:FSRTL] Avoid writing to string literals. ROSTESTS-298

[VFATLIB] Fix FAT partitions formatting in a non clean fashion.

(So the fun begins)
In spite of what VFATLIB headers pretend, there's not magic in FAT boot sector.
The 3 first bytes are just the jump instruction (to the boot code). No jump, no boot.
Also, some (many?) FAT implementations rely on the jump code to help detecting that
a FAT volume is really a FAT volume. Like MS FastFAT. Or our own FAT recognizer in FS_REC.
The story is that, up to that commit, we zeroed the 3 first bytes; leading to broken
FAT volumes.
This got hidden in most cases by the fact that during setup, when we install boot
loader, we erase parts of the boot sector, including the jump instruction, making the
volume valid again. But that wouldn't fix secondary volumes where the boot loader isn't
installed.
And, also, imagine a scenario where you want to install ReactOS on a newly formatted volume
with MS FastFAT instead of our own implementation... That would simply not work to
the fact that the driver wouldn't recognize the fresh formatted volume!

(So the non fashion begins)
Fix this by putting a not that valid jump into the boot sector when formatting our
partitions. That way, our volume is always regarding a FAT view point. But, instead of
putting values that mean (nearly) nothing. We should also put a dummy bootloader
displaying the user and error message, as done by dosfstools.

(So the hope begins)
This opens the way for trying to install ReactOS with MS FastFAT (doesn't work yet).

CORE-11819
CORE-14362

[SC] Addendum to 4728157: Add missed values to query errors and usage output. (#387)

[SHELL32] CShellItem: Fix GetDisplayName and Compare
The two methods were completely broken because they passed a full pidl to the parent folder (and not desktop folder).
Compile SHGetNameFromIDList to use it in CShellItem but don't export it.

[SHELL32] CShellItem: Implement BindToHandler and improve GetAttributes based on wine
Fixes lots of tests.

[SHELL32] CShellItem: Fix return value of Compare method

[SHELL32] CDefaultContextMenu: Implement GetCommandString

[SHELL32] Fix a couple of tests

[SHELL32] shellpath: sync a part of _SHGetDefaultValue with wine
on 32bit builds CSIDL_PROGRAM_FILESX86 is treated as CSIDL_PROGRAM_FILES and on wow64 builds CSIDL_PROGRAM_FILES gets treated as CSIDL_PROGRAM_FILESX86

[SHELL32] Fix some test failures

[REACTOS] Fix a number of Get/SetWindowsLong*Ptr* issues and use GWLP_* instead of GWL_* where appropriate.

[WIN32K] Implement NtUserSetWindowLongPtr for 64 bit builds
Also fix return type of NtUserQueryWindow and type of WND::dwUserData and the value for bAnsi in SetWindowLongPtrA

[RTL][XDK][KERNERL32_WINETEST] Import wine actctx changes from 3.2-37c98396
This adds support for reading the trustInfo (requestedExecutionLevel),
and the compatibility (supportedOs) sections.

[RAPPS] Various fixes
- Ask the user if they want to continue on cab certificate problems
- Fix memory leaks in failure paths
- Properly disable caching

[SNDREC32] it-IT.rc: add a missing new line

[SHELL32] CFSFolder: Fix some tests for GetAttributesOf

[SERVICES] Update a drivers status properly

- ScmUnloadDriver: Handle failed unload properly.
- ScmGetDriverStatus: A driver is running when its driver object was found. Otherwise it has been stopped.
- ScmControlDriver: Check a drivers status before and after unloading.

CORE-14317

[COMCTL32] Export GetWindowSubclass by name.
CORE-14346

[NTOSKRNL] Fix mismatching spinlock release in CcPerformReadAhead()

[KMTESTS:CC] Addendum to 3fe0e28: use ok_irql.

CORE-14349

[KMTESTS:IO] Fix a Clang-Cl warning about NameLength

"warning: variable 'NameLength' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]"

CORE-14306

[KMTESTS:IO] Correctly free name in IoFilesystem's TestAllInformation.

Spotted by Serge Gautherie.

[KMTESTS:NPFS] Fix 2 Clang-Cl warnings about ShareAccess

"warning: variable 'ShareAccess' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]"

CORE-14306

[KMTESTS:CC] Add IRQL tests for CcCopyRead showing that a spinlock is held during Cc IO operations.

CORE-14349

[HAL] Eliminate tail calls from HalpHardwareInterruptLevel. CORE-14076

[HAL] Eliminate tail calls from HalpDispatchInterrupt[2]. CORE-14076

[HAL] Eliminate tail calls from HalEndSystemInterrupt. CORE-14076

[HAL] Create a macro for the HalpEndSoftwareInterrupt to allow reuse. CORE-14076

[HAL] Convert HalpEndSoftwareInterrupt2 to fastcall. CORE-14076

[RAPPS] Some fixes to our completely broken cert pinning attempt.
- We should not open a new connection to request a certificate.
- Update the issuer / subject for the LE certificate.
- User proper types.
- Require all fields that we check to be present.

Without checking the public key or the entire certificate it's still not secure, but we are a step closer.
Dedicated to Joachim Henze
CORE-14350

[NTOSKRNL] Don't make CcScheduleReadAhead() too noisy

[WIN32K:NTUSER] Safely handle allocation failure in IntCloneMenu. CORE-14222

[WIN32K:NTUSER] Avoid allocating an extra item in IntCloneMenuItems. CORE-14222

[WIN32K:NTUSER] Handle realloc failure in IntRemoveMenuItem. CORE-14222

[WIN32K:NTUSER] Handle allocation failure in PopupMenuWndProc. CORE-14222

[KMTESTS] Change license to LGPL and update to SPDX headers.

Permission for this change was given by all authors
(who are now mentioned in the copyright lines).

[NTSTATUS] Add some status messages.

[BOOTDATA] Add some informative registry values (Win2k3 compat).

- In HKLM\SYSTEM\CurrentControlSet\Control\Windows, add "Directory"
  and "SystemDirectory" values, giving the ReactOS installation
  directories (purely informative).
- Also add the "ErrorMode" value that specifies how the hard-error
  popups are displayed. This value will be used later.
- Fix event log sources ordering.

[BOOTDATA] Add "Application Popup" as a known source of events for the System event log.

[CMAKE] Increase KERNEL_VERSION_PATCH_LEVEL to 9 CORE-14345

To make 1st stage setup properly report "0.4.9"

[SC] Fix MSVC build.

[SHELL32] CFSDropTarget: style fixes

[SHELL32] CDefView: remove a useless print

[SHELL32] CFSDropTarget: Use copy depending on the source drive
Show the default action as the default menu item in the right click menu.

[SHELL32] CFSDropTarget: Initial implementation of right click drag menu

[SHELL32] CDefView: Implement several methods needed for drop targets
Set the view object as the site of the drop target.
Implement GetItemPosition, SelectAndPositionItems, IsDropOnSource, GetDragPoint.
Use DROPEFFECT_COPY only when the item supports it.

[SHELL32] Add menu resource which is used when dragging files with right click

[SC] Implement missing query/queryex options
- PrintService: Print optional display name.
- MainUsage: Print missing query options.
- Parse query options and use them to enumerate services.

[CMAKE] Call "git describe" with the --long parameter to always output the format we are used to.

[SHELL32_APITEST] Run the CUserNotification worker test function in a thread to copy with possible test timeouts. CORE-13177

[NTOSKRNL] Don't call OOM dumper in case we were trying to allocate more than 100 pages.
In such case, we'll consider the caller is bogus and we'll fail in silence.
Dedicated to Mark (again!)

[NTOSKRNL] Allow more chars in ExpTagAllowPrint().
Dedicated to Mark

[FASTFAT] Drop read ahead locking routines in favor of lazy writer routines.
This avoids code duplication while being consistent with MS FastFAT.

[SHELL32] Fix COM registration of IShellFolder, IShellLinkA/W, IQueryContinue and IUserNotification classes.

[EXPLORER] Update French translation

improve SHGetFileInfo (#118)

This patch reduces failures of SHGetFileInfo function. CORE-7159
* improve WideByteToWideChar calls
* fix bugs related to file attributes and SHGFI_EXETYPE
* SHGFI_USEFILEATTRIBUTES and SHGFI_ICON fix
* s/sizeof(temppsfi)/0/

[D3D9][ARPING] Add missing va_start() and va_end() (#371)

[COM_APITEST] Reenable two known interfaces (#367). Caught by Thomas.

[I8042PRT] Apply a workaround for Dell Latitude D610
CORE-14324

[SHELL32_APITEST] Add a test for the CUserNotification implementation. CORE-13177

[SHELL32] Implement the CUserNotification class, which implements the IUserNotification interface. CORE-13177

[COM_APITEST] Add tests for IUserNotification. CORE-13177

[PSDK] Add definitions for IUserNotification, IQueryContinue, and IUserNotification2 and IUserNotificationCallback (Vista+).
CORE-13177

[SC] Disable debug messages on user interface (#382)

[HAL] Implement APIC-based profiling for x64.

[IP] Silence "TCP operation failed" debug spam.

[SHELL32] CNewMenu: Don't use CWM_GETISHELLBROWSER.
CDefaultContextMenu: Set the site for context menu extensions
CNewMenu: Get SID_IFolderView service of the site to get the IShellView

[SHELL32] Fix the toolbar buttons in the file open dialog
CDefView: Make GetItemObject return an IContectMenu after setting itself as the site.
CDefViewBckgrndMenu: Make the inner context menu use the same site. Implement handling CMDSTR_VIEWLISTA and CMDSTR_VIEWDETAILSA.
CDefaultContextMenu: CMDSTR_VIEWLISTA and CMDSTR_VIEWDETAILSA shouldn't be handled in this class.
Fixes CORE-14327 and the two buttons that change the view mode

[SHELL32] Fix glitch in de-DE.rc

checkbox captions are too long.
issue was introduced by 3624c5d6fd2cf1c589c26f75ac2d6054f7a838a0

fixes CORE-14329 for de-DE

[SHELL32] CDefViewBckgrndMenu: Fix handling string verbs.
It shouldn't be necessary for QueryContextMenu to be called before InvokeCommand.
Move retrieving the IContextMenu of the folder to the initialization of the class.
CORE-12866

[SHLWAPI] Import wine commit 2c163225730e88 as addendum to the SHCreateWorkerWindowW/A fixes from commit 770bf93.

[TRANSLATION][INF] Update Simplified Chinese translation. (#378)

[COMCTL32] Import wine commit 71cf0a31d99feb to fix the name column width in rapps.

[SHLWAPI] Improvements for SHCreateWorkerWindowW/A() prototypes + fix x64-bit compatibility. Sent upstream.

[SHLWAPI] Refactor the SHCreateWorkerWindowW() prototype to match its ANSI SHCreateWorkerWindowA() counterpart.

The last parameter is really to be understood as an extra window data, and not a "message result" (as it would be the case for dialog window procedure).
That is why I also remove the mention of "DWLP_MSGRESULT" in the SetWindowLongPtrW() call.
SHCreateWorkerWindowA() had it OK but SHCreateWorkerWindowW() did not.

------------------

[SHLWAPI] Make SHCreateWorkerWindowA() and SHCreateWorkerWindowW() x64-compatible.

The first parameter of these functions is a pointer to a window procedure, having a definite prototype, so employ a correct typedef WNDPROC,
which ensures both correct pointer size and parameter type enforcement.
This also ensures that we use instead a correct pointer size, since otherwise LONG remains 32-bits for Windows compatibility on x64 platforms.
The wndProc parameter is thus casted to LONG_PTR to comply with the SetWindowLongPtrA/W calls.

In SHCreateWorkerWindowW(), the last "wnd_extra" parameter should also be LONG_PTR to be able to pass 64-bit data pointer on x64 platforms.
Therefore fix also setting the wc.cbWndExtra size. One should note that the ANSI SHCreateWorkerWindowA() function had everything OK already.

[CDFS_NEW] Fix a bug in Microsoft CDFS implementation where the top level IRP
is set before locking is actually attempted for Cc worker thread (lazy write/read ahead).
So in case locking fails, the top level IRP is set and never unset, and latter
call will hit the assert where it expects the top level IRP to be NULL.

Should be reported upstream (tm).

CORE-14315

[SERVICES] Set a complete service status for running boot start or system start drivers

[SERVICES] Fix the initial service status

- The initial dwWin32ExitCode for a disabled service is ERROR_SERVICE_DISABLED and ERROR_SRVICE_NEVER_STARTED for non-disabled services.
- The initial dwWaitHint is 2000 (2 seconds) for (user-mode) services and 0 for drivers.
- Move all driver-related code to driver.c.

[APPSHIM_APITEST] Fix 2 Clang-Cl warnings about ok2

"warning: expression result unused [-Wunused-value]"

CORE-14306

[EXPLORER] Fix a Clang-Cl warning about Icon->ProcessId

"warning: expression result unused [-Wunused-value]"
Regressed in be2bf9b.

CORE-14306

[EXPLORER] WatchList should be freed with delete[], not delete (#374)

WatchList is a set of array objects, initialized with "new[]", so it should be freed with "delete[]" to free all of its elements. Otherwise using only "delete" only frees the first variable but not its array. This would lead to an undefined behaviour.

[CONUTILS] Relax the enforcement of using FORMAT_MESSAGE_MAX_WIDTH_MASK in the ConMsg* and ConResMsg* functions.

[MAGNIFY] Call PostQuitMessage() only at the very end.

[NTOSKRNL] In debug builds, print more information about the top level IRP in the Cc worker thread

CORE-14315

[SDK] Fix NT6 building issues and improve SDK NT6 headers.

CORE-12596

[CRT:STARTUP] Fix a Clang-Cl warning about GS_ExceptionPointers

"warning: unused variable 'GS_ExceptionPointers' [-Wunused-const-variable]"
Missed in CORE-8626 r64704.

CORE-14306

[DXG] Fix a Clang-Cl warning about DDHAL_CB32_CANCREATESURFACE

"warning: use of logical '&&' with constant operand [-Wconstant-logical-operand]"

CORE-14306

[OPENGLCFG] Fix a Clang-Cl warning about IDC_DEBUG_OUTPUT

"warning: use of logical '||' with constant operand [-Wconstant-logical-operand]"

CORE-14306

[STRMBASE] Sync with Wine 3.0. CORE-14225

[SERVICES] Update the service status after unloading a driver

Update the service status after unloading the driver and pass the new status to the caller.

CORE-14269

[EXPLORER] Send balloon notifications to the callback message.
TODO: Send NIN_BALLOONUSERCLICK when the user clicks on the balloon content (not the X button).

[PSDK] Add missing shell notification message flags.

[SERVICES] Support the NoInteractiveServices registry value

[NTOSKRNL] Avoid private cache map allocation for the first handle
Standard shared cache map provides space for a private cache map, do the same
and make it available for the first handle. It avoids two allocations in a row.

[NTOSKRNL] Modified CcCanIWrite to handle Mm throttling limits

[NTOSKRNL] Define the throttling limits for Cc in Mm: MmThrottleTop, MmThrottleBottom.
Modified !defwrites to reflect that change