[SHELL32] close hKey in error case

svn path=/trunk/; revision=71983

[NTOS:IO]
- Avoid infinite loop in IopMountVolume. Brought to you by Vadim Galyant.
CORE-11663 #resolve

svn path=/trunk/; revision=71982

[SHELL32] Addendum to r71980, RegSetValueEx expects a size in bytes, not chars. CORE-10439

svn path=/trunk/; revision=71981

[SHELL32] cache entries of "New" menu
CORE-10439

svn path=/trunk/; revision=71980

[SHELL32]
- CDefView: Add the skeleton check that should be used to implement moving items in the CDefView. Nothing more is implemented because GetKeyState is completely unreliable.

svn path=/trunk/; revision=71979

[ADVPACK]: Addendum to r71977: Add the resource file into the CMakeList.
CORE-7645

svn path=/trunk/; revision=71978

[ADVPACK]: Add file version information resource.
CORE-7645 #comment Please retest the installation of Visual Basic 6 runtime with revision >= 71977.

svn path=/trunk/; revision=71977

[USETUP]
Keyboard Modifier LEDs not working in usetup.
Patch by Carlo Bramini.
CORE-3368 #resolve #comment Thanks a lot!

svn path=/trunk/; revision=71976

[ACPPAGE] Add windows ME to the Win98 selection item. Patch by Lee Schroeder. CORE-11606 #resolve #comment Thanks!

svn path=/trunk/; revision=71975

[SERIAL]
- Properly stub SerialPower
CORE-11648

svn path=/trunk/; revision=71974

[PARPORT]
- Properly stub FdoPower and PdoPower
CORE-11648

svn path=/trunk/; revision=71973

[PCIIDEX]
- Add a stub for PciIdeXPowerDispatch
CORE-11648

svn path=/trunk/; revision=71972

[SERVICES]
Service logon page:
- Handle the interactive service type flag.
- Change the service configuration in the apply function.

svn path=/trunk/; revision=71971

[KERNEL32_WINETEST] Sync thread tests with Wine Staging 1.9.14 now that Thomas fixed CORE-11511.

svn path=/trunk/; revision=71970

[NTOS:KE]
- Load LDT on context switch if required.
CORE-11511 #resolve

svn path=/trunk/; revision=71967

[REGEDIT]
- Minor whitespace fixes in main.c
- Use a correct info msgbox title when showing regedit command-line usage. Something to think about: translate the cmdline usage! :)

svn path=/trunk/; revision=71966

[SERVMAN]
Service logon property page:
- Fix initialization and display bugs.
- Start work on the 'Apply' function. It doe not apply changes yet.
- Add password check messages.

svn path=/trunk/; revision=71965

[BOOTDATA]: Disable automatic debugger startup on application crash. This allows getting back the error popup as we used to have, and also warn the user there was indeed a problem, instead of silently start the default debugger (that I've set to an unexistant "drwtsn.exe") and fail silently, since that default debugger doesn't exist.
CORE-11508

svn path=/trunk/; revision=71964

[USER32_WINETEST] Sync with Wine Staging 1.9.14.

svn path=/trunk/; revision=71963

[USER32] Add missing lprc check in SetRect().

svn path=/trunk/; revision=71962

[WS2_32_WINETEST] Sync with Wine Staging 1.9.14. Made possible thanks to Peter Hater's changes. CORE-11368 ROSTESTS-233

svn path=/trunk/; revision=71961

[SHELL32] Add AVI file 151. By Jared Smudde. CORE-11638

svn path=/trunk/; revision=71960

[User32]
- Fix log out shutdown dialog window input. See CORE-11633 #resolve

svn path=/trunk/; revision=71959

[EVENTVWR]
Commit "Part 1" of my Improvements for the Event Log Viewer. See CORE-11637 description for more details (too long to stay there).

svn path=/trunk/; revision=71958

[EXPLORER] Fix the Explore all users/open all users menu items. Patch by Jared Smudde. CORE-11146 #resolve #comment Thanks!

svn path=/trunk/; revision=71956

Ooops! These were not meant to be committed!

svn path=/trunk/; revision=71955

[SERVICES]
Check for NULL pointers before derreferecing them in ScmIsSameServiceAccount().
Patch by Victor Martinez Calvo.
CORE-11599 #resolve #comment Thanks a lot!

svn path=/trunk/; revision=71954

[ADVAPI32]
Remove unneeded NULL checks at CreateServiceA/W and ChangeServiceConfigW/A().
Patch by Victor Martinez Calvo.
CORE-11610 #resolve #comment Thanks a lot!

svn path=/trunk/; revision=71953

[LSASRV]
Fix cleanup in LsarRetrievePrivateData().
Modified patch by Victor Martinez Calvo.
CORE-11611 #resolve #comment Thanks a lot!

svn path=/trunk/; revision=71952

[WS2_32]
- Fix a few ws2_32:getaddrinfo test failures. Based on a patch by Peter Hater.
CORE-11368

svn path=/trunk/; revision=71951

[KERNEL32]
- Fix GetComputerNameEx behavior with regard to NULL/non-NULL buffers and size calculation
CORE-11368

svn path=/trunk/; revision=71950

[EXPLORER] HACK: ignore unimplemented SetBandSiteInfo. Patch by Wim Hueskes. CORE-9809 #resolve #comment Thanks!

svn path=/trunk/; revision=71949

[PSDK]: Add two missing flags concerning custom-drawn controls.

svn path=/trunk/; revision=71948

[KERNEL32] Check the return buffer for NULL before initializing the unicode string buffer with it. By Peter Hater. CORE-11368

svn path=/trunk/; revision=71947

[NtGdi]
- Make SetBkColor work the same as client side.

svn path=/trunk/; revision=71946

[EVENTVWR]: Fix transparency problems for some icons.

svn path=/trunk/; revision=71944

[BOOTLIB]
Fix checksum computation

CID #1363451

svn path=/trunk/; revision=71941

[BOOTLIB]
Fix out of bounds read.

CID #1363549

svn path=/trunk/; revision=71940

[BOOTMGR]
Don't use uninit var.

CID #1363650

svn path=/trunk/; revision=71939

[BOOTLIB]
Don't deal with uninit ptr.

CID #1363698

svn path=/trunk/; revision=71938

[NTOSKRNL]
Assert that the allocated buffers are not leaked.

CID #1248416

svn path=/trunk/; revision=71937

[MUP]
Don't deal with uninit ptr.

CID #1322222

svn path=/trunk/; revision=71936

[ACPPAGE] Remove unused var. CID 1363705

svn path=/trunk/; revision=71934

[RAPPS_NEW] Use the actual buffer size instead of hardcoding a random number. CID 1363551

svn path=/trunk/; revision=71933

[ACLAYER] Use the return of StringCbPrintfW instead of ignoring it. CID 1363460-63, 65-70, 76, 78, 85, 88, 90

svn path=/trunk/; revision=71932

[APPHELP] Dereference after null check (default_dir). CID 1363509

svn path=/trunk/; revision=71931

[NTOSKRNL_VISTA]
- Avoid ugly hacks
CORE-11596 #resolve

svn path=/trunk/; revision=71930

[NET]
No need to check for whether 'pBuffer' is NULL to call NetApiBufferFree, because:
- 'pBuffer' was used above this call without any particular checks;
- 'pBuffer' can be only null when NetUserEnum fails with an error different from NERR_Success or ERROR_MORE_DATA. But this case was actually checked for before using 'pBuffer'.
By Victor Martinez aka. Mr.Coverity ^^ :)
CID 1363629
CORE-11598 #resolve

svn path=/trunk/; revision=71929

[NTOSKRNL_VISTA lib]
Initialize utf8_ch_len to zero before using it (at each turn of the for-loop). As I couldn't find this code in Wine, I couldn't see whether they already fixed it or not. Caught by Victor.
CORE-11596

svn path=/trunk/; revision=71928

[SETUP]
- Fix leaking 'pSetupData' in case of failure. CID 1363604
- Since we know that in the remaining of the code, 'pSetupData' is a valid pointer, we can free it at the very end without rechecking whether it was NULL or not.
CORE-11591 #resolve

svn path=/trunk/; revision=71927

[MAGNIFY]
Missing break. CID 1363532
CORE-11593 #resolve

svn path=/trunk/; revision=71926

[MKHIVE]
- Fix usage of the 'Status' variable. CID 1363689
- Fix some DPRINTs.
CORE-11595 #resolve

svn path=/trunk/; revision=71925

[CHARMAP]
Don't overrun FillGrid() when accessing the last elements. Rewrite the code using 'for' loops, in the same style as what is done elsewhere in the code. CID 1363552
CORE-11597 #resolve

svn path=/trunk/; revision=71924

[BOOTLIB]
- BiOpenKey is being feed with the wrong ElementHandle, it should be feed with ElementsHandle one. CID 1363670 . By Victor Martinez Calvo. CORE-11600 #resolve
[BOOTMGR]
- Fix an Assign vs Compare issue. CID 1363558 .  By Victor Martinez Calvo. CORE-11592 #resolve

svn path=/trunk/; revision=71923

[BOOTLIB] Remove stray semicolon from a condition in ConsoleCreateLocalInputConsole(). Patch by Victor for CID 1363701. CORE-11588

svn path=/trunk/; revision=71919

[USER32_WINETEST] Sync with Wine Staging 1.9.11 Part 4. CORE-11368

svn path=/trunk/; revision=71918

[User32]
- Sync/port up to Wine Staging 1.9.11 or current. See CORE-11368.

svn path=/trunk/; revision=71917

[NtUser]
- Fix crash in User32:win:test_winproc_handles. See CORE-11368.

svn path=/trunk/; revision=71916

[User32]
- Fix regression in API:RealGetWindowClass test due to wine sync/port.
- ReactOS use of DIALOG_get_info is an enhanced fork of the same wine function.

svn path=/trunk/; revision=71915

[CONCFG]: Add needed inclusion for StrSafe functions.

svn path=/trunk/; revision=71914

[CONSRV]
- Fix potential Out-of-bounds access during string copy/concatenation. CID 1322098.
- Fix check for NULL after potential dereference. CID 1322175.

svn path=/trunk/; revision=71913

[SHELL32]
- Use StringCbCopyW

[PROGMAN]
- Use StringCbCopyW: CID #1363712.
- Don't read registry values in a registry key if we failed to open it. CID #514350.

svn path=/trunk/; revision=71912

[USETUP]
- Don't hardcode buffer string size in GetPartTypeStringFromPartitionType calls.
- Possibly check whether the first character of the PartTypeString string is NULL before printing a generic partition information in case the partition type is unknown. We might check instead for the STRING_FORMATUNKNOWN string...
- Don't check for PartTypeString being NULL since it's not a pointer. CID #1363481 and CID #1363494.

svn path=/trunk/; revision=71911

[KERNEL32_WINETEST] Sync with Wine Staging 1.9.14 except thread tests. CORE-11511

svn path=/trunk/; revision=71910

[NTDLL_WINETEST] Partial sync with Wine Staging 1.9.14. CORE-11368 holds some remaining tests.

svn path=/trunk/; revision=71909

[USER32_WINETEST] Sync with Wine Staging 1.9.11 Part 3. CORE-11368

svn path=/trunk/; revision=71907

[Win32SS]
- Update (Sync/Port) User32:Text to Wine Staging 1.9.11, see CORE-11368.

svn path=/trunk/; revision=71906

[Win32SS]
- Cleanup error codes for Classes and Menus.
- Update (Sync/Port) User32:Dialog to Wine Staging 1.9.11, see CORE-11368.
- Now pass exactly the same tests as windows, except for the two ToDos lines 1596 & 1617. The other test failures are on wine for not being compatible!

svn path=/trunk/; revision=71904

[SCHTASKS] Import from Wine Staging 1.9.14. CORE-11576

svn path=/trunk/; revision=71903

[UXTHEME_WINETEST] Sync with Wine Staging 1.9.14.

svn path=/trunk/; revision=71902

[UXTHEME] Apply Wine commit 997e204 by Louis Lenders: Fix CloseThemeData with invalid handle value. CORE-11368

svn path=/trunk/; revision=71901

[SHELL32_WINETEST] Sync with Wine Staging 1.9.14.

svn path=/trunk/; revision=71900

[PSDK] Add some missing ASSOCF_* enum elements.

svn path=/trunk/; revision=71899

[ADVAPI32_WINETEST] Sync with Wine Staging 1.9.14.

svn path=/trunk/; revision=71898

[TRANSLATION] Russian translations by Nikolay Burshtyn (amber). CORE-11567 #resolve #comment Thanks!

svn path=/trunk/; revision=71896

[INCLUDES/WINE] Update test.h.

svn path=/trunk/; revision=71895

[GDI32_WINETEST] Sync with Wine Staging 1.9.14.

svn path=/trunk/; revision=71894

Fix usage of the first parameter for TranslateAccelerator: this should be the handle of the window that will receive the accelerator commands (not the window from which the message originates).

svn path=/trunk/; revision=71893

[SERVMGR]
Service recovery page:
- Dissect the 'run process' command line and display it.
- Implement the 'Browse' button.

svn path=/trunk/; revision=71892

[NEWDEV]
Fix words in newdev. (Italian translation)
Patch by decodp.
CORE-11437 #resolve #comment Thanks a lot!

svn path=/trunk/; revision=71891

[FASTFAT]
CcMapData 4th arg isn't a boolean but a flag. Fix it accordingly
CcPinRead 4th arg isn't a boolean but a flag. Fix it accordingly

svn path=/trunk/; revision=71890

[CDFS]
CcMapData 4th arg isn't a boolean but a flag. Fix it accordingly

svn path=/trunk/; revision=71889

[FASTFAT][CDFS]
Call CcMapData(), CcPinRead() inside a SEH block
This allows returning more significant error codes on failure

[NTOSKRNL]
Enable Thomas code for raising exceptions

CORE-9848

svn path=/trunk/; revision=71888

[SDK] Add the first appcompat database, containing references to version lie shims. CORE-11302 #resolve

svn path=/trunk/; revision=71887

[SDK][XML2SDB] Introducing the xml2sdb host tool, used to convert xml files to appcompat Sdb files. CORE-11302
Uses tinyxml2 to parse the xml.
Thanks to Thomas for reviewing / suggesting fixes!

svn path=/trunk/; revision=71886

[APPHELP] Prepare sdbwrite related api for a new hosttool. CORE-11302
- Adding an extra argument to SdbReAlloc
- Do not rely on platform wcs* functions.

svn path=/trunk/; revision=71885

[USETUP]
Add CAPS-LOCK support to the keyboard layout.
Patch by Carlo Bramini.
CORE-3368 #comment Thanks a lot!

svn path=/trunk/; revision=71884

[NTOS:CC]
- Raise exceptions on failure in CcMapData if RAISE_FROM_CC_MAP_DATA is defined. This is how the function should behave, and should be enabled once all our file system drivers correctly handle this.
CORE-9848

svn path=/trunk/; revision=71883

[WIN32SS] Do not try to call DevModes when it is NULL (mirror drivers). Patch by Victor Matovykh. CORE-11441 #resolve #comment Thanks!

svn path=/trunk/; revision=71882

[RAPPS] Fix some translated / misspelled tags.

svn path=/trunk/; revision=71881

[SHELL32]
- Fix the last parameter of a StringCchCopyEx call; fix a misspelling.
- Do not exclusively use GetFileAttributesEx to retrieve file attributes (for the file properties dialog). Indeed, it happens that this API (as well as GetFileAttributes), and equivalently, NtQueryFullAttributesFile (as well as NtQueryAttributesFile), can fail on *locked system* files (on Win2k/Win2k3/Win7, and on ROS), such as C:\pagefile.sys . See for example http://stackoverflow.com/questions/16772931/getfileattributes-on-locked-system-file and my comment in CORE-10757 . Therefore to retrieve file attributes also for these files we do a trick: we call FindFirstFile on the full path to said file. It happens that FindFirstFile can work on locked system files, and thus we are able to retrieve their attributes (+ size and file dates) as well!
See the code for more details.
CORE-10757 #resolve

svn path=/trunk/; revision=71880

[KERNEL32]
- Do not use a "magic number" for the return value 0xFFFFFFFF from GetFileAttributes.
- Use a meaningful variable name for retrieving the result of GetFileAttributes.

svn path=/trunk/; revision=71879

[SERVMAN]
- Fix MSVC build

svn path=/trunk/; revision=71878

[LIBXML2]
- Fix MSVC build (already fixed upstream)

svn path=/trunk/; revision=71877

[LIBXSLT] Update to v1.1.29. CORE-11557

svn path=/trunk/; revision=71876

[LIBXML2] Update to v2.9.4. CORE-11556

svn path=/trunk/; revision=71875

[CMAKE] Update baseaddress.cmake for the GCC build. CORE-11382

svn path=/trunk/; revision=71874

[FONTSUB] Import from Wine Staging 1.9.13. CORE-11219

svn path=/trunk/; revision=71873

[SERVMAN]
Add logon and recovery property pages. Still work in progress.

svn path=/trunk/; revision=71872

[FLTMGR]
Fix GCC build

svn path=/trunk/; revision=71871