[User32]
- Fix bug that was uncovered by r72143. See CORE-11810.
- Move System Commands to server side. Leaving Alt-Tab and Scrolls.

svn path=/trunk/; revision=72155

[INPUT]
- Implement list of keyboard layouts on Settings Page

svn path=/trunk/; revision=72154

[LSASRV]
- Partital implementation of LsarRemoveAccountRights.
- Simplify LsapLookupAccountRightValue. Return ACCESS_MASK instead of NTSTATUS.
- LsarAddAccountRights: Check parameters before referncing the policy handle.

svn path=/trunk/; revision=72153

[ADVAPI32]: Addendum to r72151: It's incorrect to compare the variable of BOOL type with TRUE. Any non-zero value is considered to be "true". Caught by Thomas.

svn path=/trunk/; revision=72152

[ADVAPI32]: Don't directly redirect IsTextUnicode to RtlIsTextUnicode, but use a helper function, so that the return value is correctly set to TRUE or FALSE. Indeed the former function returns a BOOL (long) while the latter returns a BOOLEAN (byte), and the high bytes in the return value could not be correctly set otherwise.

Adapted from a patch by 'andy-123'.

CORE-11803 #resolve #comment Fixesd, thanks!

svn path=/trunk/; revision=72151

[SDK][RTL][NDK]: Update RtlIsTextUnicode prototype.

svn path=/trunk/; revision=72150

[ADVAPI32]: Cleanup file headers and whitespace.

svn path=/trunk/; revision=72149

[APITESTS]: Add a test for advapi32!IsTextUnicode.
CORE-11803

svn path=/trunk/; revision=72148

[FASTFAT]
- Update FCB hashes after changing the path name
CORE-11377

svn path=/trunk/; revision=72147

[INPUT]
- Create an empty project for re-working keyboard layouts management

svn path=/trunk/; revision=72146

[FASTFAT]
After renaming a directory, rename any children FCB that would still exist in the FSD. This will allow next directory opening to properly work and open correct data.
This defeats corruption on directory renaming.
Unfortunately, this solution is not ideal and our driver should be reworked so that it quits using a hash table and it quits storing the whole path in the FCBs.

Deep thanks to Wim Hueskes and Thomas Faber for their help debugging the issue!

CORE-11377

svn path=/trunk/; revision=72145

[NTOS:CC]
- Define CcPfEnablePrefetcher
CORE-11799

svn path=/trunk/; revision=72144

[KBSWITCH]
- Use strsafe functions
- Use Shell Hooks in dll for detecting keyboard layout switching
- Fix bug in loading kbsdll

* Detection switch keyboard layout is now working in Windovs, but does not work in ReactOS (WH_SHELL not work in win32k)

svn path=/trunk/; revision=72143

[WIN32K]
- Detect Alt+Shift hotkeys and send input language change request message to window

* Switching the keyboard layout works now. The code needs to be improved

svn path=/trunk/; revision=72142

[LSASRV]
Move ServiceInit() into a separate file.

svn path=/trunk/; revision=72141

[EVENTVWR]
Don't hardcode a fixed size for the memory buffer holding the event details data to be copied into the clipboard, but instead compute it using the real lengths of the different strings. Use also StringCbPrintfW instead.

svn path=/trunk/; revision=72140

[LSASRV]
Implement LsarAddAccountRigths and fix some bugs along the way:
- LsarGetSystemAccessAccount: Fix an uninitialized varable.
- LsarOpenAccount, LsarCreateAccount: Make these functions wrappers around the real implementation in LsarpCreateAccount and LsarpOpenAccount.
- LsarEnumerateAccountRights: Fix the use of the wrong array index.
- LsarAddAccountRights: Implement it.
- LsarpLookupPrivilegeValue: Return a pointer to an LUID instead of an NTSTATUS. It is much earier to use it this way.
- LsapLookupAccountRightValue: Implement it.

svn path=/trunk/; revision=72139

[NTOS:MM][USER32]
- Correctly handle/avoid C99-style inline semantics
CORE-11794

svn path=/trunk/; revision=72138

[WS2_32_NEW]
- Fix always true conditions in ComputeQuerySetSize
CORE-11799

svn path=/trunk/; revision=72137

[FASTFAT]
Properly check for opened handles in subdirectories when we're about to rename a directory.
This helps reducing corruption where some renaming operations were allowed whereas they shouldn't have been.

CORE-11377
CORE-11426

svn path=/trunk/; revision=72136

[SYSDM]
- Less ambiguously check the return value of DialogBoxParam
CORE-11794

svn path=/trunk/; revision=72135

[WIN32K:NTUSER]
- Add some missing VOIDs
CORE-11799

svn path=/trunk/; revision=72134

[INCLUDE/CRT] Correct _wctime and _wctime_s when _USE_32BIT_TIME_T is defined. CORE-11799

svn path=/trunk/; revision=72133

[CRT] Correct some types. CORE-11799

svn path=/trunk/; revision=72132

[WIN32DLLS] We don't need mingwex for these modules anymore. CORE-11799

svn path=/trunk/; revision=72131

[NTOS:MM] Add some missing VOIDs. CORE-11799

svn path=/trunk/; revision=72130

[DIRECTX/WINE] We don't need mingwex for these anymore. CORE-11799

svn path=/trunk/; revision=72129

[NTOS:PNPMGR] Mark IopReportTargetDeviceChangeAsyncWorker as stdcall and remove the PWORKER_THREAD_ROUTINE cast. CORE-11799

svn path=/trunk/; revision=72128

[NTOS:MM]
- Correctly check for overflow in PeFmtCreateSection
CORE-11794

svn path=/trunk/; revision=72127

[FASTFAT]
Don't mess with parentless FCB.
Dedicated to Thomas :-).

CORE-11377
CORE-11426

svn path=/trunk/; revision=72126

[FASTFAT]
Addendum to r72124

svn path=/trunk/; revision=72125

[FASTFAT]
Track child FCB in parent FCB to allow browsing them in case it's needed.
This allows fixing a FIXME and offering better performances when renaming a directory.

CORE-11377
CORE-11426

svn path=/trunk/; revision=72124

[CALC] We don't need mingwex anymore. CORE-11799

svn path=/trunk/; revision=72123

[NTDLL] Add missing VOID. CORE-11799

svn path=/trunk/; revision=72122

[ARP]
- Do not return failure after successful operations
CORE-11794

svn path=/trunk/; revision=72121

[CMD][UXTHEME]
- Clarify misleading indentation
CORE-11794

svn path=/trunk/; revision=72120

[CRT]
- Apply Wine commit b7f6362d (msvcrt: Use the correct object type for __non_rtti_object exceptions.) by Alexandre Julliard
CORE-11794

svn path=/trunk/; revision=72119

[CRT]
- Remove pointless inline
CORE-11794

svn path=/trunk/; revision=72118

[COMCTL32]: Addendum to r72116: Add the __REACTOS__ ifdefs to ease wine syncing, and update the patch in accordance.
CORE-11532

svn path=/trunk/; revision=72117

[COMCTL32]
- rebar: actually pass the hittest flag to the NM_NCHITTEST notification.
CORE-11532

svn path=/trunk/; revision=72116

[FASTFAT_NEW]: Fix add_pch command usage.

svn path=/trunk/; revision=72115

[NET/USE]
Always delete remembered connections on /delete

ROSAPPS-303

svn path=/trunk/; revision=72114

[FASTFAT]
Refactor vfatMakeFCBFromDirEntry() and vfatUpdateFCB() so that they share code. It will help avoiding wild FCB modifications on renaming.

CORE-11377

svn path=/trunk/; revision=72113

[EXPLORER]
- Fix subclassing the tasks toolbar which was broken in r65274.
CORE-11532

svn path=/trunk/; revision=72111

[ATL][ATL_APITEST] Add implementation + tests for most functions from CPoint, CSize, CRect. CORE-11747 #resolve
Not all functions are implemented, the ones that are not implemented are left in a comment in the header.

svn path=/trunk/; revision=72110

[INCLUDE/WINE] Update test.h with debug functions for POINT and SIZE.

svn path=/trunk/; revision=72109

[MPR]
Check for the right capability.
Already sent upstream.

CORE-11757

svn path=/trunk/; revision=72108

[MPR]
Add a missing break

svn path=/trunk/; revision=72107

[SHELL32]: Fix copypasta bug.

svn path=/trunk/; revision=72105

[BROWSEUI] set min width for addressband to 100. patch by Katayama Hirofumi MZ CORE-11766

svn path=/trunk/; revision=72103

[NtUser]
- Fix regression from r66350. See CORE-11324 for more details.

svn path=/trunk/; revision=72102

[NTOS:SE]
- Remove unnecessary null checks & fix up cleanup in SepCreateToken and SepDuplicateToken. Based on a patch by Víctor Martínez Calvo
CORE-11768 #resolve

svn path=/trunk/; revision=72101

Fix gcc build

svn path=/trunk/; revision=72100

[SHELL32]
- CDesktopBrowser: Convert to an ATL CWindow class, remove unused fields and cleanup.

svn path=/trunk/; revision=72099

[NET/USE]
In case an automatic drive letter assignment was asked (net use * REMOTE), print the mapping that was performed on success.

ROSAPPS-303

svn path=/trunk/; revision=72098

[MPR]
Return the correct local name in case of automatic redirection.
Alreaedy sent upstream.

CORE-11757

svn path=/trunk/; revision=72097

[FASTFAT]
Fixes suggested by Thomas:
- Don't dereference cached data after unpinning
- Pin big enough data to read direntries

CORE-11377

svn path=/trunk/; revision=72096

[WIN32K]
- Deduplicate NtUserScrollWindowEx / IntScrollWindowEx by making NtUserScrollWindowEx properly call IntScrollWindowEx instead.
- Fix potential memory leaks in failure paths in IntScrollWindowEx.

svn path=/trunk/; revision=72093

[BROWSEUI] pass a full pidl to ILGetDisplayNameEx CORE-11458

svn path=/trunk/; revision=72092

[MPR]
Properly set local name in case of redirection for *A function calls.
Alreaedy sent upstream.

CORE-11757

svn path=/trunk/; revision=72091

[MPR]
Misc fixes.
Already sent upstream (d'oh!)

CORE-11757
ROSAPPS-303

svn path=/trunk/; revision=72090

[HACKSSIGN]
Drop hackssign app. It's now obsolete.
You served us well :-).

ROSAPPS-303

svn path=/trunk/; revision=72089

[NET]
Implement the "net use" usage of the net command.
This allows connecting remote resources to local system with assigning them a local name, to enumerate such connected resources and to delete them.

This obsoletes the hackssign application.

The implementation is a bit... raw. It is mainly here to demonstrate what's doable in ReactOS now. And to help using features we were lacking up to now.
For instance, you can make use of 'net use * \\vmware-host\Shared Folders\{YOURSHARE}' to assign a local letter to your VMware shared folders.

ROSAPPS-303

svn path=/trunk/; revision=72088

[MPR]
Implement connected resources enumeration
To be sent upstream.

Patch 6/6

CORE-11757
ROSAPPS-303

svn path=/trunk/; revision=72087

[MPR]
Set the CONNECT_LOCALDRIVE on successful local redirection
To be sent upstream.

Patch 5/6

CORE-11757
ROSAPPS-303

svn path=/trunk/; revision=72086

[MPR]
Implement WNetCancelConnectionA(), WNetCancelConnectionW(), WNetCancelConnection2A()
Already sent upstream.

Patch 4/6

CORE-11757
ROSAPPS-303

svn path=/trunk/; revision=72085

[MPR]
Implement WNetCancelConnection2W()
Already sent upstream.

Patch 3/6

CORE-11757
ROSAPPS-303

svn path=/trunk/; revision=72084

[MPR]
Implement provider automatic selection for WNetUseConnection. That means you don't need anylonger to provider the network provider handling the remote name to establish a connection to the resource.
Already sent upstream.

Patch 2/6

CORE-11757
ROSAPPS-303

svn path=/trunk/; revision=72083

[MPR]
Time to react (again!).

Implement automatic local name redirection for new connection. This implements the behavior of the CONNECT_REDIRECT flag of WNetUseConnection.
Already sent upstream.

Patch 1/6

CORE-11757
ROSAPPS-303

svn path=/trunk/; revision=72082

[TRANSLATIONS] Russian translation update by Sergey Stopkin and co, with some fixes by me. CORE-11752

svn path=/trunk/; revision=72081

[NtUser]
- Fix crash while mouse cursor is moving over stress test window (RegEdit). This fixes VLC too. Tested SeaMonkey and it did not crash. See CORE-9169 and CORE-9836.

svn path=/trunk/; revision=72080

[EVENTVWR]
- Reorganize the header inclusions; add a PCH.
- Transform the event details dialog into a control that can be embedded:
  * in a resizable dialog,
  * or in a pane below the window listing the events.
CORE-11637

svn path=/trunk/; revision=72079

[NOTEPAD]: Fix danish menu shortcuts.

svn path=/trunk/; revision=72078

[KDGDB]
 - Turn this thing into something useful
   - Pretend we are only one process to GDB (making it a kernel-only debugger...) until the GDB team sorts this out
   - This is a kernel debugger stub you know. You didn't seriously pretend that recursive functions would be fine, did you ?
[NTOS:KD64]
 - Fix build
Pix or it din't happen : https://jira.reactos.org/secure/attachment/36322/source_level_debugging_ros.png

svn path=/trunk/; revision=72077

[NtUser]
- Fix a recursion issue while processing posted messages. Related to CORE-10471. Dedicated to Thomas Faber for Free before Use testing.
- Pass extra information along with posted messages.
- Trigger an event when cursor position has changed.

svn path=/trunk/; revision=72076

- Remove trace.

svn path=/trunk/; revision=72075

[User32]
- Use correct value for win:test_mdi. Use post instead of send, fix message sequence.
- Minor code changes.

svn path=/trunk/; revision=72074

[SHELL32] remove obsolete FIXME. was fixed in r68015

svn path=/trunk/; revision=72073

[USERENV]
- Create separate function for setup session manager variables
- Fixed a regression in msi package winetest after r72066

svn path=/trunk/; revision=72072

[VFATLIB]
Don't leak memory

svn path=/trunk/; revision=72071

[SHELL32]
- Fixed memory leak

[DNSLIB]
- Fixed uninitialized variable

[USERENV]
- Fixed typo

svn path=/trunk/; revision=72070

[NTFS]
Don't attempt to read beyond stream size.
Patch by Trevor Thompson.

CORE-11707

svn path=/trunk/; revision=72067

[USERENV]
- Implement GetUserSid() (based on GetUserSidFromToken())
- Remove GetUserSidFromToken() and use new exported function
- Move SID functions to sid.c
- SetUserEnvironmentVariable should not fail, if can't get the short path. Instead, a long path
- Setup environment variables from Session Manager

* This fixes all winetests for userenv.dll (+20 passed tests)

svn path=/trunk/; revision=72066

[NtUser]
- Remove unused data.

svn path=/trunk/; revision=72065

[BROWSEUI] Correct the 'Is this legal' url as suggested by Ken Rochard. Also rewrite the function to remove todo's, and make it localizable. CORE-11721 #resolve #comment Thanks!

svn path=/trunk/; revision=72063

[REISERFS]
Reduce verbosity

svn path=/trunk/; revision=72062

[ATL][ATL_APITEST] Implement / Improve CString, based upon the code that was already there. CORE-11579 #resolve
Add code + tests for:
Conversion of A->W and W->A, equality operators, MakeLower, MakeUpper, Find, FindOneOf, ReverseFind, Compare, Mid, Left, Right, Format, Replace, Trim, TrimLeft, TrimRight.

svn path=/trunk/; revision=72061

[SCSIPORT]: Port Dmitry's fix from r72059 to our scsiport driver too. I'm wondering how "difficult" it would be to have some kind of scsiport library to be shared between the driver and freeldr's.

svn path=/trunk/; revision=72060

[WHOAMI]
[ARP]
[TRACERT]
- Incorrect to compare the variable of BOOL type with TRUE. Any non-zero value is considered to be "true".

[FREELDR]
- Variable is assigned values twice
- The 'strlen' function was called multiple times inside the body of a loop
- It is inefficient to identify an empty string by using 'strlen(str) > 0' construct. A more efficient way is to check: str[0] != 0

[NTOBJSHEX]
[SLAYER]
[CMICONTROL]
- It is inefficient to identify an empty string by using 'strlen(str) > 0' construct. A more efficient way is to check: str[0] != 0

[SHELL32]
- There is no sense in testing the pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error
- Verifying that a pointer value is not NULL is not required. The 'if (ptr != NULL)' check can be removed
- Fix copy-paste error in CMenuFocusManager::PlaceHooks()

[SRCLIENT]
- Remove unneeded check. A part of conditional expression is always false.

[DISK]
[ATAPI]
- Variable is assigned values twice

* All bugs found by PVS-Studio

svn path=/trunk/; revision=72059

[SOLITAIRE]
Add Portuguese/Brazillian translation.
Patch by Lucas Raziel.
CORE-11665 #resolve #comment Thanks a lot!

svn path=/trunk/; revision=72058

[INF]
Update russian translation of inf strings.
Patch by amber.
CORE-11604 #resolve #comment Thanks a lot!

svn path=/trunk/; revision=72057

[FASTFAT]
Fix a FIXME

svn path=/trunk/; revision=72056

[BROWSEUI]
- Move handling of DISPID_NAVIGATECOMPLETE2 event from CAddressBand to CAddressEditBox as the todo comments say it should be.
- Remove the todo entries that are complete.
- Remove IDispatch from CAddressBand as it is not really needed. Now CAddressBand takes care only of positioning and handling the button while CAddressEditBox does all the parsing and interaction with the browser.

svn path=/trunk/; revision=72055

[NtUser]
- Fix hang while inputting mouse message. Fix CORE-9998.

svn path=/trunk/; revision=72054

[BROWSEUI]
- Use the _CreateInstance postfix for all functions that instantiate a class.
- Move the the CreateInstance functions of the exported classes to browseui.h where we can select uniformly which built in classes we use.

svn path=/trunk/; revision=72053

[USER32]
- Add missing NULL-pointer checkes in rect functions
- Small coding-style fixes

svn path=/trunk/; revision=72052

[PCIX]
Fix Device_SaveCurrentSettings: do not reference BarArray too soon if the index is out of bounds (in that case it means we are handling a ROM "BAR").
CID 716153, reported by Victor Martinez Calvo.
CORE-11215 #resolve

svn path=/trunk/; revision=72050

[XDK]
- Move the PCI_COMMON_HEADER_LAYOUT next to the structures that use it.
- Add IRP flags description comments.

svn path=/trunk/; revision=72049

Fix build.

svn path=/trunk/; revision=72048

[SHELL32]
- Factor CFSDropTarget out of CFSFolder.
- For now their code is mixed (no code was moved), until we decide where exactly is should be kept.

svn path=/trunk/; revision=72047