[NETAPI32] Implement NetGroupGetUsers

[FREETYPE] Fix Regression "Monospaced font in LibreOffice" CORE-15018

LibreOffice started to show GUI in a mono-spaced font after 0.4.10-dev-466-g
35f62fc5ba0b69e7335ff41400cb3b45660f4557

We can fix this by partial revert
as proposed by Katayama Hirofumi MZ, who was author of those changes as well.
We revert the changes in penalty system here.

[USBEHCI_NEW] Bring-in the USB EHCI miniport driver created by Vadim Galyant. (#301)

[SERVICES] Only services starting under the LocalSystem account can run interactively. Otherwise ignore the interactive flag.

[SERVICES] Re-implement RChangeServiceConfigA() around RChangeServiceConfigW(), much as what is done for RCreateServiceA().

[SERVICES] RChangeServiceConfigW: Check valid usage of lpdwTagId when lpLoadOrderGroup is not specified.

[NET] Update Russian translation (#849)

[TRANSLATION] Updated Polish translation in USETUP (#848)

[USETUP] Italian translation update

[USETUP] Romanian translation update

[BIDITEXT] Add more demo code. (#842)

Add code that demonstrates the use of the lpOrder and lpCaretPos struct members of GCP_RESULTS w/ and  w/o GCP_REORDER flag.
Changed demo string literals to arrays containing character unicode values.

[LPK] Make GetCharacterPlacement caret positions respect bidi. (#842)

- Try to make use of ScriptStringCPtoX when trying to fill the lpCaretPos struct member;
  if USP10 error happens fallback to the unreorder method.
- Remove now incorrect debug print.

[WIN32SS][COMCTL32] Fix button text Y positioning (#766)

Fixes the vertical position of button text.
CORE-14906

[NETAPI32] Implement NetGroupAddUser and NetGroupDelUser

[SETUP][REACTOS] Added Vietnamese translation. (#716)

[USETUP] Update French translation.

[USETUP] Implement the shutdown timeout progress bar. (#833)

- Add the MUI ID resource.
- Implement the shutdown timeout progress bar function and the TimeOut string handler.
- Add the resource string to other resource languages.

[NTOSKRNL] Move data mapping implementation to an internel helper

[NETAPI32] Implement NetGroupSetInfo

[NETAPI32] Implement NetGroupAdd and NetGroupDel

[USBHUB_NEW][USBPORT] Fix buffer size calculation for multi-sz values in IRP_MN_QUERY_ID.

Also fix some copypasta'd DPRINTs.

[NETAPI] Implement NetGroupGetInfo

[PSDK] Add missing struct _GROUP_INFO_3 to lmaccess.h

[NETAPI] Implement NetGroupEnum

[SAMSRV] Fix SamrQueryInformationGroup

- Use the registry value AdminComment for admin comments instead of Description. Fix SamrSetInformationGroup as well.
- Add some TRACEs.
- Do not fail if the Members value does not exist.

[KMTESTS:CC] Fix two failing tests on WHS bot

[NET] Implement the group command

[SDK] Make gen_baseaddress.py compatible with Python 3.

[USBUHCI_NEW] Add license headers.

[USBOHCI_NEW] Add license headers.

[DDK] Add NT6+ implementation for Rx*PrefixTableLock macros

[USBOHCI_NEW][USBUHCI_NEW] Avoid unnecessary/incorrect status defines.

[KMTESTS:FLTMGR] Hide FltMgrLoad and FltMgrReg

And add "// TODO: WIP/untested/crashes."

ROSTESTS-299

[CPL][DESK] Update monitor resource (#816)

[DESK.CPL] Update monitor resource

[LIBTIRPC] Fix CVE-2018-14622 by backporting its fix

CORE-15005

[NTOSKRNL] Don't raise a status when parameters are invalid on file mapping

[NTOSKRNL] When allocating a new BCB, save it in a list

This list is stored in the shared map. Later, this will allow
reusing BCB when appropriate

[SDK] Generate msvc baseaddresses from an RTC build

[CMLIB] Fix compiling as NT6+ (#840)

Addendum to r70582 (734bc291).
CORE-12596

[MSV1_0] LsaApLogonUser: Update the logon time/count and bad password time/count

[MSV1_0] Use the account expiration time and check account expiration on logon

[IDL] Implement parts of SamrSetInformationUser.UserInternal2Information

[IDL] Add some Flags SamrSetInformationUser.UserInternal2Information

[FONTEXT] Add Italian translation (#831)

[ACCESS] Fix some truncated strings in French resource (#838)

[SAMSRV] Implement SamrQueryInformationUser.UserInternal2Information

[MSGINA] Update russian translation (#834)

Addendum to 6f4d70b.

[IDL] Add the  undocumented struct _SAMPR_USER_INTERNAL2_INFORMATION to sam.idl

[KMTESTS:CC] Add tests showing our CcMapData/CcPinRead raise invalid status

[WIN32K:NTUSER] co_UserDestroyWindow(): Simplify the destruction of the owned windows.

[WIN32K:NTUSER] Optimize IntWinListOwnedPopups() a little bit. Improve a trace.

[WIN32K:NTUSER] Code formatting only.

[KMTESTS:CC] Add tests for BCB extension

[KMTESTS:CC] Add more tests crossing CcPinRead and CcMapData

[MSGINA] Display the 'account expired' message when a user tries to log on using an expired account.

[USBOHCI_NEW] Avoid storing pointers as ULONG and physical addresses as pointers.

[USBHUB_NEW] Fix x64 warnings.

[USBUHCI_NEW] Bring-in the USB UHCI miniport driver created by Vadim Galyant. (#245)

[KERNEL32] fix spec file for -version=0x600+

CORE-12596

[GDI32_APITEST:GetTextFace] Replace tabs by spaces.

[GDI32_APITEST] Add tests for GetTextFaceAliasW(), based on Katayama Hirofumi MZ's tests from CORE-14926.

CORE-14995

[KERNEL32] Shhhh!

[KMTESTS:CC] Add tests for PIN_EXCLUSIVE

[KMTESTS:CC] Add a test for pinning without access

[NTOSKRNL] Fail on pinning when there's no pin access set

Instead of assert, now, CcPinRead will just fail. This is
not consistent without Windows behavior, but still better
than asserting while testing!

[KMTESTS:CC] Add tests for PIN_IF_BCB flag

[KMTESTS:CC] Duplicate tests for CcPinRead

To be extended.

[KMTESTS:CC] Add missing FsRtlEnter/ExitFileSystem() calls

Spotted by Thomas

[KMTESTS:CC] That wasn't meant to be committed

[COMCTL32_WINETEST] treeview.c: Skip test_right_click()

until CORE-14975 is fixed upstream and WINESYNC.

CORE-14975

[KMTESTS:CC] Add a test to show we map our buffers wrong in CC

That is expected (cf. CORE-14259)

[KMTESTS:CC] Remove dummy leftover

[KMTESTS:CC] Extended previous test to show an overlap is enough

[KMTESTS:CC] Add a test that shows that our CC implementation duplicates BCB

It should instead reuse BCBs when mapping something already mapped.

[NTOSKRNL] Fix a typo

[NTOSKRNL] Warn about unimplemented feature in CcMapData() (in all callers)

Currently, our CcMapData() behavior (same goes for CcPinRead()) is broken
and is the total opposite of what Windows kernel does. By default, the later
will let you map a view in memory without even attempting to bring its
data in memory. On first access, there will be a fault and memory will
be read from the hardware and brought to memory. If you want to force read
on mapping/pinning, you have to set the MAP_NO_READ (or PIN_NO_READ) flag
where kernel will fault on your behalf (hence the need for MAP_WAIT/PIN_WAIT).

On ReactOS, by default, on mapping (and thus pinning), we will force a view
read so that data is in memory. The way our cache memory is managed at the
moment seems not to allow to fault on invalid access and if we don't force
read, the memory content will just be zeroed.
So trying to match Windows behavior, by default, now CcMapData() will enforce
the MAP_NO_READ flag and warn once about this behavior change.

[KMTESTS] Re-enable kmtest_.exe installation (#389)

ExPools and other tests don't break Windows Server 2003 anymore.

Revert r73535, so KmTests run on "Test WHS" bot.

CORE-12657

[MODULES] Order the cmake inclusions alphabetically as we're used to everywhere else.

[REACTOS] Introduce a "DEBUGFORMAT" environment variable that allows to select different debug trace formats.
CORE-12671

- The default format is used when no format name is specified: this is
  the one we use so far in ReactOS:

    <debug_class>:(<file>:<line>) <message>

  with "debug_class" being "trace", "warn", "err".

- The "wine" format is the one used by Wine. It can be used when trying
  to diff-compare traces for a module with the corresponding one
  obtained from a Wine run. It can also be useful because the logging of
  Wine-synced code assumes that the function names are automatically
  added by the helper macros "FIXME()", "TRACE()", "WARN()" or "ERR()",
  and not manually inside the logging string given to these macros:
  for example:

    FIXME("(%params) message\n", params);

  displays:

    fixme:<module>:SomeFunc(params) message

- The "extended" (or "ext") format is very noisy and tries to output a
  lot of information; it is a hybrid of the previous two formats:

    <debug_class>:(<file>:<line>):<channel>:SomeFunc <message>

Support for displaying the current process ID is added in
addition to the already existing support for thread ID.

[WIN32SS][FONT] Fix GetTextFace function and related (#829)

Google Chrome with -no-sandbox parameter in ReactOS wouldn't display the web page because first-chance exception raised.
CORE-14926

- Remove FullName, Style, and FaceName members from TEXTOBJ structure.
- Add TextFace member into TEXTOBJ structure.
- Add MatchFontName() and MatchFontNames() helper functions.
- Fix GetTextFace() and related.

[FONT][SETUP] Add 'Tms Rmn' and 'Times' font substitutes (#830)

Add "Tms Rmn" and "Times" font substitutes (same as "Times New Roman"). Windows has these substitutes, hence we also need those to remain compatible.

[CMAKE] Update baseaddress files to match latest changes.

[STORAHCI] Fix MSVC "uninitialized local variable" error (C4700) (#828)

See also PR #821.

[SDK] Duplicate English messages as Polish Messages (#826)

[MC] Fix VS2017 build.

[NET] Initialize a new users primary group ID in order to fix logon failures

[NET] Fix indentation

[CDFS] Revert 9d0596a

The appropriate fix was the previous one, where the macro
for checking legal chars have been fixed.

Thanks to MS for their review :-).

CORE-14067

[SDK] Fix the implementation of the FsRtlIsAnsiCharacter* macros.

These macros should be implemented using the FsRtlTestAnsiCharacter() macro
that performs extended tests, especially for buffer overruns.
This fixes FsRtlIsAnsiCharacterWild(), FsRtlIsAnsiCharacterLegalFat(),
FsRtlIsAnsiCharacterLegalHpfs(), FsRtlIsAnsiCharacterLegalNtfs().

CORE-14067

[SHELL32] Fix typo in comment

[TCPSVCS] Remove hardcoded C drive letter (#800)

CORE-13228, CORE-13235

For USETUP, add a comment about why the code uses it when adding a boot-sector boot entry to NTLDR. CORE-13226

[CDFS] Properly check for legal names in CdIsLegalName()

Up to now, it was working by chance. Indeed, due to the invalid
ASCII check performed before calling FsRtlIsAnsiCharacterLegalHpfs(), the
macro is improperly called and overruns the FsRtlLegalAnsiCharacterArray
buffer. Fortunately, up to now, right after that buffer in kernel binary
there are strings which are more or less consistent with the flags that
are expected by the macro, causing a decent behavior of
FsRtlIsAnsiCharacterLegalHpfs() even for extended ASCII characters
(whereas FsRtlIsAnsiCharacterLegalHpfs() is only designed for ASCII
characters). But this is a totally out of control and wrong behavior.
A single change in the way the kernel was built could have caused the
CDFS driver not to work as previously.

I have made the choice to allow any extended ASCII character as done
for the unicode characters. This is a good compromise to avoid drastic
regressions for users having extended ASCII characters in their CD
file names.

This imports proposed upstream commit 1b6b625641dffb49951e60398e1a9c672318ea71
See pull request https://github.com/Microsoft/Windows-driver-samples/pull/278

CORE-14067

[USETUP] Progress-bar: minor improvements.

- Simplify the usage of the PUPDATE_PROGRESS callback.

- Add the possibility of specifying an initial non-zero StepCount when
  creating the progress-bar (using the -Ex version), so that it can be
  initially drawn with the expected initial count.
  Of course ProgressSetStepCount() can continue to be used.

[LDR] Load kernel32 in LdrpInitializeProcess.
This allows kernel32 to register itself for manifest probing when loading dlls
CORE-9219

[NTDLL_APITEST] Various small fixes
- Add a missing file
- Add convenience targets for MSVC_IDE
- Remove whitespace at the end of the lines

[COMCTL32] Unregister some classes that were registered.
This should help fix some crashes.

[MAGNIFY] Fix the magnification ratio bug for CORE-14946 (#823)

This bug was caused by leaving the magnify window unrefreshed when updating the "Magnification Level" ComboBox.
CORE-14946

[SHELL32] Adjust the IDC_PICKICON_LIST control's height in the IDD_PICK_ICON dialog resource. (#824)

CORE-14974

[SHELL32] Check the result from PathUnExpandEnvStringsW

[SHELL32] Remove some mostly dead code from CShellLink