[SHELL32] CCopyToMenu/CMoveToMenu: Check if this_ is NULL

[USER32] Follow-up of #4262 (a47590c) (#4312)

Display icons/cursors correctly without hungup. CORE-16287

[SHELL32] Fix assertion failure in CMenuToolbars

CORE-17505

[NTOS:KE] Remove all checks for x87 not being present

[NTOS:KE] Add a check for unsupported CPU features on i586

Currently, these features are vital for the kernel:
- CPUID instruction
- CMPXCHG8B instruction
- TSC aka Time Stamp Counter

All of that have to be present on i586

[NTOS:KE] Move CPU features detection to a separate function on i586

[SHELL32] Enable _DEBUG and fix assertion failures (#4307)

Improve debuggability. Fix "WorkerW" window class conflict.
- Enable _DEBUG macro if debugging, in order to enable ATLASSERT macro.
- Fix assertion failures.
- Add SHCreateDefaultWorkerWindow helper function and CWorkerTraits, then use them.
- Don't create the broker and server windows by themselves, but subclass.
CORE-17505, CORE-13950

[RAPPS] Enable _DEBUG and fix assertion failures

CORE-17505

[EXPLORER] Enable _DEBUG and fix assertion failures (#4306)

- Enable _DEBUG if debugging.
- Fix assertion failures.
CORE-18013

[RAPPS] Update outdated README regarding size CORE-18018

[RAPPS] Addendum to 0.4.15-dev-3724-g3a7c0d8 CORE-18018

For a moment I thought this member was obsolete entirely,
and intended to delete it entirely.
(as its size is just ULONG which is too small for big downloads > 4GB size).
It apparently works also without it to display the size,
as we actually use a different member that stores the size as a string.

But there actually still *is* a usage for that ULONG representation.
In (the rare) corner-case the http download does not allow to query the size,
then the ULONG representation m_SizeBytes is used as a fallback for the
download-progress-bar to avoid 'marquee mode'.

The 'fallback of the fallback' so to say.
We should enlarge this to be a 64bit unsigned value some day.

[RAPPS] Fallback to legacy "size" entry is not longer needed CORE-18018

on 2021-10-03 Mark Jansen deleted all old size entries from the rapps database via
https://git.reactos.org/?p=rapps-db.git;a=commit;h=551e8cb95716f46e05c3be08996467bd93b1936f
So it makes no sense to keep that fallback-code in RAPPS available.cpp RetrieveSize() func.

[NTOS:MM] Implement partial virtual region releases. CORE-17938

Fixes boot with MS videoprt.sys (and some apitests).

[ATL][MSPAINT] Add ATLTRACE and fix assertion failures (#4299)

- Implement ATLTRACE and AtlTrace in atlbase.h.
- Fix assertion failures in mspaint.
CORE-17969, CORE-18012

[SFCFILES] Add sfcfiles dll and header file

Sfcfiles.dll contains only a short list of protected files (WIP).

[CRT] Fix MSVC version of call_handler

[NTUSER] Improve NtUserGetThreadState (#4300)

- Improve enum ThreadStateRoutines.
- Improve NtUserGetThreadState function.
- Improve CLIENTIMC structure.
- Improve imm32 code.
CORE-17732

[TRANSLATION] Cumulative update for Russian (ru-RU) translation

[SYSDM] Addendum to 5a2911d7, e58b4a49, ea2ca8fa.
[SHELL32] Addendum to 16f64a95, 44bb99cc, 2ae526a0, 34b0f3e4, fd39a292.
[SYSSETUP] Addendum to 556cfbf5, 6f1a57a9, 3f053db3.
[SAMSRV] Addendum to 6f1a57a9.
[BOOTDATA] Addendum to 63cd5d54.
[INF] Addendum to fb500b88.

[NTUSER] Initialize correctly CaretWidth value. (#4297) CORE-13984 CORE-14359 CORE-16624

Fixes:
CORE-13984 LibreOffice Writer shows no blinking caret
CORE-14359 Apache Open Office 4.1.2 SWriter - no blinking Caret
CORE-16624 KVIRC, no blinking caret in configuration edit controls

[SHELL32] List CD-ROM drives in the "Hardware" tab of disk properties

CORE-17397

[SAMSRV] Fix typo in IDS_ALIAS_BACKUP_OPS_COMMENT (en-US and others)

Addendum to f1e768c0.

[USETUP] Default to en-US output codepage when initializing the console. Addendum to commit 9f201d462.

Reported by julenuri.

[WIN32KNT_APITEST] Strengthen NtUserGetThreadState testcase (#4291)

Add some paramters to test on. CORE-17732

[SVCHOST][SECLOGON][MSWSOCK] Minimal headers cleanup.

- SVCHOST, SECLOGON: Add missing NTOS_MODE_USER.
- MSWSOCK: Remove some useless windef.h includes.

[INF] Fix Mizu theme display problems with CJK font (#4284)

Add font substitute from "Source Sans Pro" to "Droid Sans Fallback" if the user is CJK.
CORE-17997

[SHELL32] Implement Recycler Bin icons (#4221)

CORE-3938

Icon extraction from the recycle bin was not possible due to the partial absence of IExtractIcon interface support.
Adding it inside the RecycleBinFile's QueryInterface private implementation, and then calling it from CRecycleBin, now allows retrieving and displaying the files' icons.

[BOOTDATA] Make Secondary Logon service name and description translatable

Addendum to 43fa9548.

[reactos] Add missing prototypes for svchost callbacks

Addendum to 43fa9548d32a10967f9d53c73c1d4484c8927ee8

[dhcpsvc][srvsvc][wkssvc] Servcies run by svchost must be shared process services

[NEWDEV] Disable NewFolder btn and Show browser title when selecting a directory (#4288)

* [NEWDEV] Add browser title to rc files

* [NEWDEV] Show browser title when selecting a directory

* Update dll/win32/newdev/lang/pl-PL.rc

Co-authored-by: Adam Stachowicz <saibamenppl@gmail.com>
* Update dll/win32/newdev/lang/bg-BG.rc

Co-authored-by: Hermès BÉLUSCA - MAÏTO <hermes.belusca-maito@reactos.org>
* Update dll/win32/newdev/wizard.c

Co-authored-by: Stanislav Motylkov <x86corez@gmail.com>
* Update dll/win32/newdev/lang/pl-PL.rc

Co-authored-by: Adam Stachowicz <saibamenppl@gmail.com>
* Update dll/win32/newdev/lang/pt-PT.rc

Co-authored-by: Stanislav Motylkov <x86corez@gmail.com>
* Remove obvious comment

* Add de-DE and ru-RU translations

* Add French translation

Co-authored-by: Adam Stachowicz <saibamenppl@gmail.com>
Co-authored-by: Hermès BÉLUSCA - MAÏTO <hermes.belusca-maito@reactos.org>
Co-authored-by: Stanislav Motylkov <x86corez@gmail.com>

[services] Add the secondary logon service

[IDL] Add idl file for the secondary logon service

[SYSDM] Update pl translation (#4289)

Addendum to https://github.com/reactos/reactos/commit/ecfd01eefc430876a95cbaca595d2f2e6f52a215 and other

[SYSDM] pt-PT.rc Fix compiler warning RC4206 CORE-17827

Thanks to Twitter user @denzilferreira for that shortened string.

[SYSDM] sq-AL.rc Fix compiler warning RC4206 CORE-17827

Thanks to Twitter users @illyrian598 and @jan_shkreli for that shortened string.

[SYSDM] sk-SK.rc Fix compiler warning RC4206 CORE-17827

Vic Marcal reached out for help on Twitter for that shortened string.

[SYSDM] el-GR.rc Fix compiler warning RC4206

Vic Marcal reached out for help on Twitter
for that shortened string.

[NTUSER] Implement NtUserBuildHimcList (#4286)

- Add UserBuildHimcList helper function.
- Implement NtUserBuildHimcList function.
CORE-11700

[USER32] Adjust the threshold in CascadeWindows (#4283)

Improve the threshold of CascadeWindows function not to make the windows too small.

[PSDK] Add prototypes CreateProcessWithLogonW and CreateProcessWithTokenW

[CONSRV] Trivially implement TuiSetCodePage(). CORE-17601

[BLUE][CONSRV][USETUP] Completely remove support for CAB font extraction from driver, and move it to CONSRV, much like what was done for USETUP.

Addendum to aaa416d3 and 2391e31c.

[CONSRV] Stub out support for terminal-level changing of output code page.
CORE-17601

[W32TIME] Add Russian (ru-RU) translation (#4287)

Addendum to 3d8d88ea.

[SYSDM] Add missing changes from e58b4a49 to zh-HK resource file

[SYSDM] Add mnemonic/access keys for environment variable editor

Addendum to e58b4a49. CORE-11733

[SYSDM] Make file browse filter string translatable

Addendum to e58b4a49. CORE-11733

[NTDLL:LDR] Don't hardcode process-execute flag values. (#4111)

[SDK:NDK] Add the NX_SUPPORT_* flags for the SharedUserData.NXSupportPolicy member. (#4111)

And move those in the XDK to a better place, adding versioning as well.

[TRANSLATION] Update Portuguese (pt-PT) translation (#4273)

- [DEVMGR] Improve pt-PT translation
- [NEWDEV] Small improvements to pt-PT translation
- [SHELL32] Update pt-PT translation

[TRANSLATION] Update Chinese Traditional (zh-TW) translation (#4154)

- Added translation for:
  - [W32TIME]
  - [MODE]
  - [TIMEOUT]
  - [MODEMUI]
  - [ARPING]
  - [FRAGINATOR]
  - and more
- Updated existing translation
- [WINNLS] Update cht (zh-TW) and zhh (zh-HK)
- [DOC] Update rules in Chinese translation notes.txt
- Wine translation sync

Some of the punctuation use in WIne has been converted to half-width,
so the punctuation will no longer synced.

[MSGINA] Adjust Status Window if Asian

The Asian logo and bar positions are different from English or Latin positions, due to Asian font.
Add positional adjustment to IDD_STATUS if Asian.
Follow-up of #4280 ef4f0d0.

[SYSSETUP] Adjust Status Window if Asian (#4280)

The Asian logo and bar positions are different from English or Latin positions, due to Asian font.
Add positional adjustment to IDD_STATUSWINDOW_DLG if Asian.

[TRANSLATION] Update Simplified Chinese (zh-CN) translation (#4240)

Reviewed-by: Chan Chilung <eason066@gmail.com>
Reviewed-by: He Yang <1160386205@qq.com>

[NET] Improve displaying status message (#4096)

- Simplify conditions
- Add application error translation function

[NET][MC] Add missing Russian (ru-RU) translation

[SAMSRV] Return correct status for existing groups and users

[NETAPI32] NetpNtStatusToApiStatus: Update accordingly

This fixes net command output in PR #4096.

[USERSRV][TRANSLATION] Improve Japanese (ja-JP.rc)

[POWERCFG][TRANSLATION] Improve Japanese (ja-JP.rc)

[WINLOGON][TRANSLATION] Improve Japanese translation (ja-JP.rc)

[NTUSER] Follow-up of #4277 (ea1353e)

According to ThFabba, NtUserSetThreadLayoutHandles returns zero.
CORE-11700

[NTUSER] Implement NtUserSetThreadLayoutHandles (#4277)

- Modify NtUserSetThreadLayoutHandles prototype.
- Implement NtUserSetThreadLayoutHandles function.
CORE-11700

[NTUSER] Follow-up of #4271 (36740ca)

- Delete the excessive comments.
- Delete the debugging leftover.
CORE-11700

[NTUSER] Implement NtUserSetImeInfoEx (#4276)

- Modify NtUserSetImeInfoEx prototype.
- Add UserSetImeInfoEx helper function.
- Implement NtUserSetImeInfoEx function.
CORE-11700

[NTUSER][IMM32] Implement NtUserGetImeInfoEx (#4271)

- Add UserGetImeInfoEx helper function.
- Implement NtUserGetImeInfoEx function by using UserGetImeInfoEx.
- Fix imm32.ImmGetImeInfoEx.
- Modify enum IMEINFOEXCLASS.
CORE-11700

[USER32] Improve icon extraction to handle RIFF formats and Fix crash (#4262)

- Add handling for 'RIFF' formats (for animated cursors). This makes explorer able to display animated cursors.
- Fix crash in extracting icons/cursors.
CORE-16287

[DESK] Update available settings when selecting another display

[DESK] Factorize changing device description to OnDisplayDeviceChanged()

[SHELL32:WINE][PSDK] Addendum to 0bf768a1: Add missing FARPROC16 definition for CallCPLEntry16() prototype.

[SHELL32:WINE][PSDK] Fix the prototype of CallCPLEntry16().

[KERNEL32] Add missing NTOS_MODE_USER when including the NDK.

[CPORTLIB] Remove any usage of debug.h since this lib implements the support for said debugging output.

[CRT/arm64] Fix handling of intrinsics

Fix msvc14.1-arm64-Release build.

Follow-up to db41787 (0.4.15-dev-2914).
CORE-17518

[CRT/arm64] Build wine/except_arm64.c

Follow-up to 9efafd6 (0.4.14-dev-934).
CORE-17518

[BCRYPT] Add ECDSA P384 to known algorithms

Add missing structs and handler in BCryptOpenAlgorithmProvider().

Cherry-pick a part of Michael Müller's
https://source.winehq.org/git/wine.git/commit/76b6c360fa7f3d6a0a14ed935075f5eb10f2f719

Follow-up to e0a47b7 (0.4.15-dev-3603).
CORE-14198

[NTOS:PS] Use quota types on process quota querying

Quota limits on a block are enumerated on a per quota type basis thus we should use the values from PS_QUOTA_TYPE enumeration, not from POOL_TYPE.

[KMTESTS:PS] Write some tests for process quota management

[NTOS:OB] Charge/Return pool quotas of objects

As it currently stands the Object Manager doesn't charge any quotas when objects are created, nor it returns quotas when objects are de-allocated and freed from the objects namespace database. This alone can bring inconsistencies in the kernel as we simply don't know what is the amount charged in an object and thus we aren't keeping track of quotas flow.

Now with both PsReturnSharedPoolQuota and PsChargeSharedPoolQuota implemented, the Object Manager can now track the said flow of quotas every time an object is created or de-allocated, thus enforcing consistency with the use of quota resources.

[NTOS:PS] Dereference the quota block during process cleanup

Ensure that when we're cleaning up the EPROCESS object, that we are dereferencing the quota block the process in question was using. The routine will automatically request a quota block cleanup if the process that dereferenced the quota block was the last.

[NTOS:PS] Process Quota Overhaul

-- Rewrite PspChargeProcessQuotaSpecifiedPool and PspReturnProcessQuotaSpecifiedPool private kernel routines, with the goal to implement the algorithms necessary to manage the fields of quota blocks (Usage, Return, Limit and Peak).
-- Invoke the Mm if quota limit raising or returning has to be done
-- When destroying a quota block, make sure that we're giving back all the rest of non-returned quotas to Memory Mm
-- Crash the system with QUOTA_UNDERFLOW if someone is returning way too much quota than it was previously charged
-- When a process exits, ensure that it doesn't hold up any charged quotas in QuotaUsage field of the process object, that way we're enforcing proper kernel consistency
-- Implement PsChargeSharedPoolQuota and PsChargeProcessPageFileQuota functions, used exclusively by the Object Manager. These routines are used to charge or return amount of quotas of a newly created object.
-- On PspInheritQuota, when assigning to process the default quota block if no parent process is given, we must increment the reference counts as we're using it
-- Handle the ProcessCount reference field, as it wasn't used
-- Annotate the functions with SAL
-- Document the code

=== REMARKS ===
Windows LogOn (Winlogon) is responsible for setting up a different quota block for all the processes within an interactive session, which is what we don't do. What we're currently doing instead is we're using the default block, PspDefaultQuotaBlock, for all the processes
across the system. The default block contains the default limits of -1 (which would imply no limits). By definition, the kernel won't ever return STATUS_QUOTA_EXCEEDED as we literally don't set up a definite limit for regular processes. This situation has to be tackled
in the future.

=== TODO FOR FUTURE ===
Most of the code in PspChargeProcessQuotaSpecifiedPool and PspReturnProcessQuotaSpecifiedPool private routines must be refactored in order to reduce the usage of the quota spin lock, possibly wrapping such code in a loop and whatnot.

CORE-17784

[NTOS:MM] Implement Raise/Return pool quota functions

This implements both MmRaisePoolQuota and MmReturnPoolQuota functions, which serve exclusively for quota pool management. The process manager communicates with the memory manager in a call of need to charge or return pool quota limits.

[NTOS:OB] Add a system process quota block macro

OBP_SYSTEM_PROCESS_QUOTA is a macro that'll be used as a way to assign a dummy quota block to system processes, as we mustn't do anything to those in case the Object Manager is charging or returning pool quotas.

[NTOS:PS] Declare some prototypes and annotate the quota functions with SAL

Declare PsReturnSharedPoolQuota and PsChargeSharedPoolQuota prototypes and annotate the functions. Furthermore, add two definitions related to quota pool limits threshold -- PSP_NON_PAGED_POOL_QUOTA_THRESHOLD and PSP_PAGED_POOL_QUOTA_THRESHOLD. For further details, see the commit description of "[NTOS:MM] Add the pool quota prototypes and some definitions".

[NTOS:MM] Add the pool quota prototypes and some definitions

Declare the MmRaisePoolQuota and MmReturnPoolQuota prototypes in the header and add some definitions related to pool quotas, namely MmTotalNonPagedPoolQuota and MmTotalPagedPoolQuota. These variables are used internally by the kernel as sort of "containers" (for the lack of a better term)
which uphold the amount of quotas that the Process Manager is requesting the Memory Manager to raise or return the pool quota limit. In addition to that, add some definitions needed for both of these functions.

The definitions, MI_CHARGE_PAGED_POOL_QUOTA and MI_CHARGE_NON_PAGED_POOL_QUOTA respectively, bear some interesting aspect. Seemingly the 0x80000 and 0x10000 values (that would denote to 524288 and 65536 specifically) are used as quota "limits" or in other words, thresholds that the kernel
uses. So for example if one would want to raise the quota limit charge, MmRaisePoolQuota will raise it so based on this formula -- NewMaxQuota = CurrentQuota + LIMIT_VALUE. LIMIT_VALUE can be either MI_CHARGE_PAGED_POOL_QUOTA or MI_CHARGE_NON_PAGED_POOL_QUOTA, depending a per quota pool basis.

What's more interesting is that these values are pervasive in Process Manager even. This is when quotas are to be returned back and trim the limit of the quota block if needed, the kernel would either take the amount provided by the caller of quotas to return or the threshold (paged or not paged)
if the amount to return exceeds the said threshold in question.

[NTOS:FSRTL] Assign the buffer length to ThisBufferLength field

This fixes an issue where ReactOS would assert on QuotaUsage == 0 as the process was still taking up quotas during a quota block de-reference with root cause of ThisBufferLength member field being 0 which made process quota charging/returning flow unbalanced.
In addition to that, on FsRtlCancelNotify routine API all we must ensure that if PsChargePoolQuota or ExAllocatePoolWithTag function fails we have to handle the raised exceptions accordingly and return the charged quota back (if we actually charged quotas that is). With said, wrap that part of code with SEH.

=== DOCUMENTATION REMARKS ===
The cause of the assert is due to the fact ThisBufferLength was being handled wrongly ever since, until this commit. When FsRtl of the Executive has to filter reported changes (with logic algorithm implemented in FsRtlNotifyFilterReportChange function), the said function will charge the quota of a given process
with an amount that is represented as the buffer length whose size is expressed in bytes. This length buffer is preserved in a local variable called NumberOfBytes, which is initialized from BufferLength member field of notification structure or from the length from stack parameters pointed from an IRP.

As it currently stands, the code is implemented in such a way that FsRtlNotifyFilterReportChange will charge quotas to a process but it doesn't assign the buffer length to ThisBufferLength. On the first glimpse ThisBufferLength and BufferLength are very similar members that serve exact same purpose but in reality there's a subtle distinction between the two.

BufferLength is a member whose length size is given by FSDs (filesystem drivers) during a notification dispatching. Whenever FsRtl receives the notification structure packed with data from the filesystem, the length pointed by BufferLength gets passed to ThisBufferLength and from now on the kernel has to use this member for the whole time of its task to accomplish
whatever request it's been given by the filesystem. In other words, BufferLength is strictly used only to pass length size data to the kernel by initializing ThisBufferLength based on that length and unequivocally the kernel uses this member field. What we're doing is that ThisBufferLength never receives the length from BufferLength therefore whenever FsRtl component
has to return quotas back it'll return an amount of 0 (which means no amount to return) and that's a bug in the kernel.

[NTOS:EX] Manage quotas when allocating or freeing pool tables

This fixes an assertion where QuotaUsage == 0 is actually not 0 when a process is about to be destroyed.

[NTOSKRNL] Implement InterlockedExchangeSizeT macro

[SHELL32] Fix folders on recycle bin and adjust the column size (#4234)

CORE-11000

- Now, folders can be sent to recycle bin (fixed a bug inside the implementation).
- Adjust column size of the RecycleBin virtual folder in details mode.

[SHELL32] Fix recycle bin handle reference leaking (#4222)

CORE-13730

There was a handle reference leak in the recycler bin and the bin wasn't removing the copied file after restoring it.
Close the handle were the memory leak was and move the file when restoring it.

[MSPAINT] Some minor improvements (#4264)

- Fix Japanese IDS_MINIATURETITLE resource string.
- Fix Japanese IDM_VIEWSHOWMINIATURE menu item text.
- Fix DWORD as COLORREF.
- Improve some OnPaint's.
CORE-17969

[MSPAINT] Don't hide selection on zooming (#4266)

- Don't hide the selection window on zooming.
- Move the selection window by calling placeSelWin() when zoomed.
CORE-17969

[SDK:DDK][NTOS:PNP] Implement PnP arbiters initialization

[MSPAINT] Prepare for debugging (#4257)

- Add #include <debug.h> and add link to ntdll.dll for debugging.
- Add CopyDIBImage helper function.
CORE-17969

[SHELL32] Use the already existing g_pszShell32 instead of re-defining a new variable holding the very same string.

[SHELL32] Use wide char string literals.

Import parts of Wine commit b215536852dc5a132108db92c90a41c70b738e50
shell32: Use wide char string literals.
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

[SHELL32] recyclebin.h: Correctly balance the INTERFACE defines/undefs.

[SYSDM] Fix the "fancy" edit Environment dialog box (#4258)

Fix the issue of the changes of the environment values not being set sometimes through fancy dialog box:
Make sure the environment value buffer is always reallocated before taking the values.

[MSPAINT] Add ICC_BAR_CLASSES (#4261)

The Fonts toolbar uses toolbar and tooltips. Thus this flag is needed. CORE-17949

[SYSSETUP] Improve the Process Page

- The page will support multiple tasks (WIP)
- The background thread uses messages to update the page

[HDAUDBUS] HDA_PdoPnp(): Remove a redundant ASSERT()

Addendum to cf7fc81 (r68311).

[KS] KsServiceBusEnumPnpRequest(): Remove a redundant ASSERT()

Addendum to c6b26fa (r50531).