[WINESYNC] msvcrt: Don't fail when _aligned_offset_malloc is used with size=0.

wine commit id 7771552f247660acc472b4862f32c4f8426be180 by Piotr Caban <piotr@codeweavers.com>

[WINESYNC] msvcrt: Fixed strncpy_s behavior when count equals 0.

wine commit id bdf9a9f9bd56354f70a7431a9682183b84353211 by Piotr Caban <piotr@codeweavers.com>

[CRT] Add a number of functions from wine

This adds _aligned_msize, _recalloc, wmemmove_s, memcpy_s, wmemcpy_s

[CRT] Reduce diff

- use msvcrt_set_errno
- MSVCRT_CHECK_PMT already sets the error.
- NULL check is already in HeapFree

[CRT] Use the original wine heap functions

[CRT] reduce diff to wine of heap code

No functional changes intended.

[CRT] Get back MSVCRT prefix in wine heap code

[NTUSER] Check IME-like after WNDS_DESTROYED check (#5005)

The guilty commit a2c6af0 enabled the IMM mode. So we have to take care of the default IME windows.
Using IS_WND_IMELIKE against WNDS_DESTROYED window was the cause of BSoD. CORE-18777

[USER32_WINETEST] Add test for WM_CONTEXTMENU from grandchild window

Cherry-picked from https://github.com/wine-mirror/wine/commit/3af8415ca9dc50e6c394c1001aad97db5f514069

[NTUSER][USER32] Set WM_CONTEXTMENU's wParam to the child window's handle

Cherry-picked from https://github.com/wine-mirror/wine/commit/3af8415ca9dc50e6c394c1001aad97db5f514069

CORE-18801

[OLEAUT32] OaBuildVersion: Add Windows Server 2003 case (#5004)

Fixes debug print in Visual Studio 6 PRO installation. CORE-18419

[ATL] CImage::Load(): Return E_FAIL without ATLASSERT on failure (#4993)

Application won't be interrupted by ATLASSERT when image load failure happens,
just like it was done in MS ATL.

CORE-18589

[WIN32K:NTUSER] ValidateHwndNoErr: Fix window handle validation (#4981)

Also check window state. Addendum to 4d48b88b.

[NTOS:PNP] Avoid recursion when walking the device tree.

[PCI] Fix I/O range size of the resource descriptors

Fixes incorrect PCI IDE resources in VPC 2007
CORE-17256

[COMPBATT] Add inf entry. CORE-18167

[NTDLL_APITEST] Add test for a section where PointerToRawData = 0 while SizeOfRawData != 0. CORE-18797

[NTOS:MM] Ignore sections either PointerToRawData or SizeOfRawData is zero

VC1 Linker fills BSS.SizeOfRawData with data in IMAGE_OPTIONAL_HEADER.SizeOfUninitializedData but keeps PointerToRawData to zero.

CORE-18797

[SHELL32] Add 3 accelerators for IDM_DRAGFILE MENU de-DE.rc (#5001)

They got lost when the menu was translated to german during 0.4.9-dev'ing.

[NTGDI] Check if face->charmap is not zero before accessing its encoding (#4390)

This allows DevExpress Ribbon Notepad sample to start.

Also add a fat DPRINT1 suggested by Timo Kreuzer,
because there is a lot wrong in the code.

CORE-18091 CORE-18558

[NTOS:MM] Use image prefix in MmLoadSystemImage

MmLoadSystemImage has a PUNICODE_STRING NamePrefix parameter which is
currently unused in ReactOS. When the kernel loads the crash dump
storage stack drivers, the drivers will be loaded with MmLoadSystemImage
with a "dump_" or "hiber_" (for hibernation, which uses crash dump
stack too) prefix. This change adds in the prefix support, and is
supposed to push crash dump support forward.

CORE-376

[NTOS:IO] Handle missing device instance in IopInitializeBuiltinDriver. CORE-18793

[HIVESYS] Move ndis to the "NDIS Wrapper" group. CORE-18790

Miniport drivers import from NDIS, but NDIS does not support being
loaded as a dependent driver (it does not have DllInitialize).
Instead, NDIS needs to load before all possible miniport drivers,
even boot-start ones. We achieve this by placing it in its own service
order group, which loads before the NDIS group.

All our miniport drivers are demand-start, so would automatically start
later. The ndisprot driver from the ticket is likely the first boot-start
miniport we've encountered. Since DriverEntry did not run,
AdapterListHead was NULL, resulting in the crash.

[WIN32SS:NTUSER] restore snapped window to normal size by dragging window caption

CORE-15638

[THEMES][DOC] "Open Sans" -> "Trebuchet MS" (#4983)

[THEMES] Eliminates the last references to "Open Sans" font.

This is an addendum to 0.4.13-dev-765-g 04a361d091095a96bb3af376e46daf4c5fe81c63
"Open Sans" has been replaced by "Trebuchet MS" back then.

[DOC] Mention trebuc*.ttf as being based on 'Open Sans Fonts'

[NTOS:KD] Revisit KdSendPacket() and KdReceivePacket() for DBGKD_DEBUG_IO. (#4914)

- Use SAL2 annotations.
- KdSendPacket(): Validate DEBUG_IO API call.
- KdReceivePacket(): Take the LengthOfStringRead into account; use
  KdbpReadCommand() to read the input, so that correct line edition
  is available (backspace, etc.)

[NTOS:KDBG] Fix and improve KdbpReadCommand(). (#4914)

- Don't read anything and return immediately if the buffer size is zero.

- Allow the controlling keys (up/down arrows, backspace) even if the
  buffer is full! (especially backspace if too much has been written).

- Return the number of characters stored, not counting the NULL terminator.

[KERNEL32][NTOS:PS][RTL] Cleanup some DbgPrompt() calls.

[BOOTDATA] Use the standard "Helv" -> "MS Sans Serif" (#4982)

This is what Windows does. And we forgot this location here.
This is an addendum to
0.4.15-dev-5365-g 9bb5627df60392b90039266f277ff668b574b4cd [SETUP][INF] Use the standard "Helv" -> "MS Sans Serif" substitution... (#4864)

[RAPPS] Add tooltip for "Refresh" and "Update Database" (#4975)

[LIBWINE] Fix GCC build warning. (#4938)

In file included from ../sdk/lib/3rdparty/libwine/debug_ros.c:9:
../sdk/lib/3rdparty/libwine/debug.c: In function 'get_temp_buffer':
../sdk/lib/3rdparty/libwine/debug.c:343:33: warning: passing argument 1 of '_InterlockedExchangeAdd' from incompatible pointer type [-Wincompatible-pointer-types]
     idx = interlocked_xchg_add( &pos, 1 ) % (sizeof(list)/sizeof(list[0]));
                                 ^~~~
In file included from ../sdk/include/crt/mingw32/intrin.h:98,
                 from ../sdk/include/crt/intrin.h:1017,
                 from sdk/include/psdk/winnt.h:48,
                 from ../sdk/include/psdk/windef.h:202,
                 from ../sdk/include/reactos/wine/debug.h:26,
                 from ../sdk/lib/3rdparty/libwine/debug.c:32,
                 from ../sdk/lib/3rdparty/libwine/debug_ros.c:9:
../sdk/include/crt/mingw32/intrin_x86.h:215:70: note: expected 'volatile long int *' but argument is of type 'int *'
 __INTRIN_INLINE long __cdecl _InterlockedExchangeAdd(volatile long * Addend, long Value)
                                                      ~~~~~~~~~~~~~~~~^~~~~~

[WIN32K] Add some NULL checks for THREADINFO::rpdesk

rpdesk should probably never be NULL, but it happens in rare circumstances, like csrss being terminated.

[WIN32K] Properly reference owner window

[NTOS] Fix interrupt flag handling in KiPageFault

This fixes a crash in kmtest:ZwAllocateVirtualMemory

[NTOS] Fix KiGeneralProtectionFault

[BEEPMIDI] Make DeviceInfo structure 64 bit compatible

Fixes winmm_winetest midi on x64

[WINMM] Sync mci.c with wine

Date: 2022-02-22
Author: Eric Pouech
Commit: "winmm: Enable compilation with long types."
Hash 71189f62604c2cb8208c85372a0e027dd2b79055

[SHELL32] Check for valid listview in CDefView::_PidlByItem and CDefView::RemoveObject

This fixes failing ATLASSERTs in shell32_winetest:shlview

[ATL] Remove an assert that doesn't exist in the native ATL

This fixes an ATL assert in shell32_winetest:assoc and shell32_winetest:shfldr_special

[SHELL32] Add check to CMenuToolbarBase::ShowDW

This fixes an ATL assert in shell32_apitest:menu

[ADVAPI32] Fix a buffer overflow in RegQueryValueExA

The code was trying to check whether the output string was already NULL terminated by RtlUnicodeToMultiByteN before NULL terminating it by checking DataStr[*count - 1] for a NULL terminator. But since RtlUnicodeToMultiByteSize always returns the size without the NULL terminator, DataStr[*count - 1] would always be the last actual character, never an optional NULL terminator.
For 0 sized strings this would actually lead to accessing the output buffer at position -1 (on 32 bit)  or 0xFFFFFFFF (on 64 bit).
Fix this by removing the check. This fixes a crash in advapi32_winetest:registry on x64.

[PSDK] Fix definition of DRVCALLBACK for 64 bit

[KERNEL32] Silence 2 annoying DPRINTs

[TRANSLATIONS] Email update (#4974)

[NTOS:CM] Cleanup the hive in case linking it to master fails (#4969)

Currently the failure code path doesn't do any kind of cleanup against
the hive that was being linked to master. The cleanup is pretty
straightforward as you just simply close the hive file handles and free
the registry kernel structures.

CORE-5772
CORE-17263
CORE-13559

🎊 🍾 🥳 Happy New Year 2023 to the ReactOS Community! 🎆 ⚛️ ☢️

.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:
 __,  ,__)            __,  ,__)       __, ,__)        ░▄████▄░▄███▄░▄████▄░▄█▀▀█▄░
(--|__| _ ,_ ,_      (--|\ | _       (--\ | _  _ ,_   ░▀▀░▄██░██░██░▀▀░▄██░░░░▄█▀░
  _|  |(_||_)|_)(_|    _| \|(/_(_|_)     \|(/_(_||    ░░▄██▀░░██░██░░▄██▀░░▄░▀▀█▄░
 (        |  |  ,_|   (                (__|           ░██████░▀███▀░██████░▀█▄▄█▀░
.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:*~*:._.:

[NTOBJSHEX] Return early from GetInfoFromPidl when pcidl is null in LPCITEMIDLIST (#4946)

This fixes a crash when attempting to drag-n-drop items from NTObject Namespace and System Registry.
CORE-18480, CORE-18481

[MSPAINT] Don't allow to set image as wallpaper if it has the wrong format (#4924)

CORE-18661

Our Paint allows user to try to set a .ico file as a wallpaper, which isn't possible. Different Windows versions have different behaviour, so it was decided that the simplest fix would be to just grey out "Set as wallpaper" buttons as in 2K3 (See the Jira ticket).

[SDK][CMLIB] Properly check for failure if hive free list creation fails

HvpCreateHiveFreeCellList returns a NTSTATUS code yet the way the code path checks
checks for failure is just wrong. This was spotted whilst working on #4571 PR.

[WIN32NT_APITEST] Fix entirely broken win32knt_apitest NtGdiDdQueryDirectDrawObject

[DBGHELP_APITEST] Comment out rsym test for anything other than x86 for now

[GDI32_APITEST] Fix test results for SetWorldTransform on Windows 2003 x64

[GDI32_APITEST] Fix test results for CombineTransform on Windows 2003 x64

[XMLLITE_WINETEST] Fix stack corruption on 64 bit

[WS2_32_WINETEST] Fix use of uninitialized variable

[NTDLL_APITEST] Fix LdrEnumResources test for x64

[NDK] Fix definition of RTL_DEBUG_INFORMATION

Fixes ntdll_aiptest RtlDebugInformation on x64

[NDK] Fix definition of LDR_ENUM_RESOURCE_INFO

This fixes ntdll_apitest LdrEnumResources on x64 Windows

[LOG2LINES] Compile the tool for MSVC builds

[NTOS] Set SwapBusy properly for i386

Co-authored-by: Hermès BÉLUSCA - MAÏTO <hermes.belusca-maito@reactos.org>

[NTOS] Increment ActiveProcessors accurately

[NTOS] fix timer lock data and hardcod

[NTOS] Swap MAXIMUM_PROCESSORS with NUMBER_POOL_LOOKASIDE_LISTS

[INF] More deduplication and style harmonization (#4952)

- deduplication of manufacturer strings
- at some places harmonizes the different length of separator lines within the same file, centers the words (as requested by hbelusca during review), harmonizes those lines to 74characters length each
- moving some strings that are not to be localized into the non-localization string section
- other minor formatting preferences coauthored by StasM

[SYSSETUP] Fix *.rc combobox for the timezone is too small CORE-18587

Use the same width for COMBOBOX IDC_TIMEZONELIST that we use in timedate.cpl to
prevent strings from getting cut off for some timezones with long names.
To still make it look appealing afterwards, enlarge and move a bit to the right also IDC_TIMEPICKER.

IDC_AUTODAYLIGHT used different widths in several translations, we can harmonize
for all languages to the biggest one that was used: 230 (taken from eu-ES.rc).

And also fix a few additional whitespace-glitches in zh-CN, zh-HK, zh-TW and en-GB.

After this commit the whole dialog IDD_DATETIMEPAGE uses the exact same dimension for
all of its controls for all languages.

[TIMEDATE] Limit the expanded combobox height by using CBS_NOINTEGRALHEIGHT CORE-18665

It is good practice to not make the expanded combobox larger than the dialog itself.
CBS_NOINTEGRALHEIGHT will allow to set a fixed amount of pixels for the v6 combobox,
and therefore allows us to hide the problem for this specific dialog.

The problem was unhidden by 0.4.12-dev-882-g e3e173ffaafbccb0e9b52926d41626d5a4dc38cd

Even after this patch I think Wines comctl32 can be improved for comboboxes with very
many elements that do not use this small trick yet. I will create a follow-up-ticket.

[REG] Fix mis-converted Yes-No-All keypress shortcuts strings.

The PO \"#msgctx\" control strings were not stripped out.

[NTOS:CM] Enable verbose debug output of registry lazy flushing component

Log debug output of the lazy flusher as much information as possible so that we can examine how does the lazy flusher behave, since it didn't work for almost a decade. Verbose debugging will be disabled once we're confident enough the registry implementation in ReactOS is rock solid.

[NTOS:CM] Don't lazy flush the registry during unlocking operation

Whenever ReactOS finishes its operations onto the registry and unlocks it, a lazy flush is invoked to do an eventual flushing of the registry to the backing storage of the system. Except that... lazy flushing never comes into place.

This is because whenever CmpLazyFlush is called that sets up a timer which needs to expire in order to trigger the lazy flusher engine worker. However, registry locking/unlocking is a frequent occurrence, mainly when on desktop. Therefore as a matter of fact, CmpLazyFlush keeps removing and inserting the timer and the lazy flusher will never kick in that way.

Ironically the lazy flusher actually does the flushing when on USETUP installation phase because during text-mode setup installation in ReactOS the frequency of registry operations is actually less so the timer has the opportunity to expire and fire up the flusher.

In addition to that, we must queue a lazy flush when marking cells as dirty because such dirty data has to be flushed down to the media storage of the system. Of course, the real place where lazy flushing operation is done should be in a subset helper like HvMarkDirty that marks parts of a hive as dirty but since we do not have that, we'll be lazy flushing the registry during cells dirty marking instead for now.

CORE-18303

Translate favorites menu

Improve translation for Favorites menu

[TRANSLATION] Don't translate Documents and Settings folder

[SHELL32][USER32] Improve pt-PT translation

[TRANSLATION] Hungarian translation update for sysdm

[DXDIAG] Fix Portuguese (pt-BR) translation (#4953)

Fix minor translation error.

[NTUSER] Fix zombie window created by CTRL+ALT+DEL (#4935)

Based on patch by I_Kill_Bugs. CORE-18258
See also commit 58b0558f9.

[WIN32K:NTGDI] PATH_PathToRegion(): Update documentation (#4941)

Addendum to 215e0d9 (r64177).

[NTOBJSHEX] Fix displayed registry value type and icon (#4949)

- RegistryTypeNames[]: Remove an unwanted space

- CRegistryFolderExtractIcon::GetIconLocation():
  Add the REG_ENTRY_VALUE_WITH_CONTENT case

- CRegistryFolder::GetDetailsOf(): Fix copypasta

- CRegistryFolder::FormatValueData(): Fix magic constant

CORE-18747

[NTOBJSHEX] Fix sorting elements by different columns (#4947)

- Add ability to sort NT objects by symlinks
- Add ability to sort registry entries by type/value

CORE-18761 CORE-18762

[SHELL32] Read the label for a CD from autorun.inf (#4945)

Some CDs may set a custom label using autorun.inf.
Read the label from the file, and if it succeeds, show it to the user.

CORE-18567

[NTOBJSHEX] Formatting only

[SHELL32] Fix detailed list views in Control Panel and Registry Folder

- Add enums for Control Panel and Registry Folder columns
- Fix iColumn values in GetDetailsOf() and fix checks in
  GetDefaultColumnState()

Current CControlPanelFolder::CompareIDs was using wrong lparam column
index. Actual column index should be 1 instead of 4. Because of this
the comment column next to name in details view was not being displayed
correctly. The same fixes for CRegFolder class functions.

PR #4944. CORE-18743 CORE-18501

[RPCRT4_WINETEST] Remove obsolete todo_wine

Addendum to 00ccbd2f

[RPCRT4] RpcStringBindingParseA/W must fail, if a colon is missing in the string binding

This fixes a bug in the rpc winetest.
This patch will be sent upstream.

[RPCRT4] Call WSAStartup in rpcrt4_ip_tcp_get_top_of_tower

This fixes the crash in the rpc winetest.

[RPCRT4] RpcStringFreeA/W must set the pointer to NULL

Add a matching wine test.

This change will be sent upstream.

[DESK] Update german translation de-DE.rc (#4943)

[NTOS:IO] Replace an outdated E-Mail Address

The old Address will be gone soon.

[SYSSETUP] Fix missing comma in german translation de-DE.rc

The old string lacked a *mandatory* comma before the word "übereinstimmen"
as the part "die Sie nutzen wollen" is a relative clause,
relative to the noun "Anwendungen".
But we can fix it without having to add the comma by some shuffling.
That also improves the reading flow.

[WINLOGON] Update german translation de-DE.rc

Addendum to 0.4.15-dev-2358-g 5f033392397a101b6a9113b2d0de5c09e0a71737
to avoid regressing the german translation compared to releases/0.4.14

[WIN32K] Workaround for broken/retarded (on non-x86) FLOATOBJ_GetFloat

The official definition in winddi.h always returns the result in an ULONG. This works to assign to a FLOATL on x86, which is also an ULONG, but it doesn't work on any other architecture, where FLOATL is actually a FLOAT, so the assignment would convert the ULONG encoding of the FLOAT to an actual float, which gives broken results. Work around this by simply replacing the definition with a sane one in win32k.

[WIN32K] Fix memory leak on failure in REGION_SetPolyPolygonRgn

[WIN32K] Fix a bug in PATH_PathToRegion

[WIN32K] Fix missing NULL check in NtGdiPathToRegion

[WIN32K] Fix uninitialized variable in IntExtTextOutW

[NTOS] Fix saving of XMM registers on some trap handlers

[KMTEST] Add a NULL check

[KMTEST] Handle failure in KmtLoadDriver and KmtOpenDriver

[NTOS:IO] Bring back the NDEBUG definition

Addendum to 947f60b20716e1581b0785360d2d71363eff3657

[NTOS:IO] Allow REG_SZ type for ImagePath of a driver

Fixes the load of the Sysinternals FileMon driver.

CORE-18725

[BTRFS] btrfs.inf: Comment out mkbtrfs.exe (#4937)

It is not imported into ReactOS.

Addendum to 0.4.13-dev-788-g c8875c182ac638a9aed627ecaf39e30ab22daf40 and 0.4.14-dev-1535-g 194ea909fdff60294ce2d5434cbded63f5377754