[RTL/PATH]
- Update path.c code to Wine 1.3.14. Fixes a buffer overwrite happening in RtlDosSearchPath_U() when invoked by "shell32_winetest.exe shlexec".
- RtlDosPathNameToNtPathName_U remains unsynced.
- Author names added to the header of the file.
See issue #5964 for more details.

svn path=/trunk/; revision=50954

[USER32]
Go back to use of SetWindowLongW instead of GetWindowLongPtrW for GWL_STYLE to reduce diff to wine code.

svn path=/trunk/; revision=50953

[USER32]
Revert a previous change that shouldn't have been comitted yet.

svn path=/trunk/; revision=50952

[FORMATTING]
user32/edit.c: Sync formatting changes from wine head, removes some unneccessary casts, no code actual change.

svn path=/trunk/; revision=50951

[GDI32_APITESTS]
  - Test behaviour of SetDIBits for 1bpp bitmaps.
  - Add small test to GetPixel just to verify that SetDIBits doesn't say BS.

svn path=/trunk/; revision=50950

[WIN32K]
  - in 1bpp bitmaps, 0 means white. Take that into in SetDIBits
  - fix a fixme in XLATEOBJ implementation
  - remove useless field from ROS_DCINFO
Fixes fox audio player GUI :-) Enjoy!

svn path=/trunk/; revision=50949

[WIN32K]
  - only lock DC if asked to in NtGdiSetDIBits.

svn path=/trunk/; revision=50948

[[GDI32_APITEST]
  - actually make a test, with == instead of =, relevant bits sets, good test value, etc.

svn path=/trunk/; revision=50947

[GDI32_APITEST]
  - add a basic test for SetDIBits

svn path=/trunk/; revision=50946

[WIN32K]
  - Dismiss alpha channel when creating a solid brush.
Finally fixes VLC icons and numerous winetests.

svn path=/trunk/; revision=50945

[WIN32K]
Silence some DPRINTS

svn path=/trunk/; revision=50944

[USER32_APITEST]
Patch by Thomas Faber: some tests for GetMessage and PeekMessage

svn path=/trunk/; revision=50943

[WIN32K]
Do raster operation on 4 bytes instead of only 3. Fixes ...
maybe noone has noticed yet ;-)

svn path=/trunk/; revision=50942

[WIN32K]
  - Raster operations in user mode are on higher bytes, whereas they are on lower bytes for drivers. Try to clarify this situation.
  - Add sanity check about what was said previously.
  - Implement masking in EngBitBlt
  - Rewrite NtGdiMaskBlt accordingly
  - Realize the palette when selecting it into a device DC.
  - When applying raster operation, do so only on 24 bits, we don't support alpha channel in win32k
This fixes VLC pink icons, Timo's MaskBlt tests and probably a lot of other things.
[SHELL32]
  - Use correct (?) raster operations for drawing sjortcuts.
Also note that now NtGdiMaskBlt locks the device contexts : this should avoid some race conditions, such as icons drawn on top of windows and the like.
Win32k sucks less.
So does reactos.
Dedicated to Timo. Sleep well, mate.

svn path=/trunk/; revision=50941

[FORMATTING]
fix dibobj.c, to please pigglesworth

svn path=/trunk/; revision=50940

[FORMATTING]
  fix formatting. No code change

svn path=/trunk/; revision=50939

[GDI32_APITEST]
Fix a bug

svn path=/trunk/; revision=50938

[GDI32_APITEST]
More tests for MaskBlt

svn path=/trunk/; revision=50937

[Win32k|User32]
- Use User32 GetWindowRgnBox, still passing both wine msg SetWindowRgn and win winregion tests, see bug 5959.

svn path=/trunk/; revision=50936

[WIN32K]
Fix calculating of ScanLines and source point in NtGdiGetDIBitsInternal for top-down bitmaps to more closely mimic win xp behavior. (Timo - high five for your test app)
See issue #5524 for more details.

svn path=/trunk/; revision=50935

[Win32k|User32]
- Use User32 GetWindowRgnBox, still passing both wine msg SetWindowRgn and win winregion tests, see bug 5959.

svn path=/trunk/; revision=50934

[Win32k|User32]
- Clarify the meaning and the use of 0, 1 and 2 for user window regions, wine never did. Noticing patches not corresponding in correctness and understanding, leaving potential crashes in the DCE code, which depend on these types.
- Patch based on Rafal Harabien patch, fixes wine msg SetWindowRgn test, see also bug 5959.

svn path=/trunk/; revision=50933

[WIN32K]
addendum to r50928: 24 and 32bpp BI_RGB bitmaps are BGR

svn path=/trunk/; revision=50932

[GDI32]
Revert r50519 and r50519

svn path=/trunk/; revision=50931

[SERVICES]
- Protect whole contents of ScmStartService() with a critical section instead of just a few parts of ScmStartUserModeService(). Seems to fix the comment #8 issue of bug #5924.

svn path=/trunk/; revision=50930

[GDI32_APITEST]
More testcases for SelectObject and GetDIBits

svn path=/trunk/; revision=50929

[WIN32K]
  - remove duplicate prototype
  - use RGB macro where possible
  - we create BGR palette for RGB DIB sections, let's do the other way around
  - simplify overcomplicated IntGetDIBColorTable
  - Add a first implementation of IntRealizePalette
No, it's not the VLC icons bugfix

svn path=/trunk/; revision=50928

[RTL/HEAP]
- A few comments, and a new structure definition (currently unused). Absolutely no changes to the actual code.

svn path=/trunk/; revision=50927

[SERVICES]
Protect the service start and sending of control packages by critical sections. This way, services will be started one by one and control packages will also be sent one by one.

Please test if this fixes bug #5924.

svn path=/trunk/; revision=50926

- Promote service start failures from TRACE to ERR.
See issue #5924 for more details.

svn path=/trunk/; revision=50925

[NTOSKRNL]
Fix aborting of backtraces

svn path=/trunk/; revision=50924

[imagehlp]
-partial winesync

svn path=/trunk/; revision=50923

[imagehlp_winetest]
-add imagehlp_winetest

svn path=/trunk/; revision=50922

[WIN32K]
- mark surface as not selected in DC_vSelectSurface when its released from a DC
NtGdiSelectBitmap:
- Don't allow selecting a bitmap into a dc that is already selected!
- Use DC_vSelectSurface instead of manually selecting
- set the DC size based on bitmap size

svn path=/trunk/; revision=50921

[WIN32K]
- In NtGdiGetDIBitsInternal use a shared lock for the bitmaps and provide background colors when initializing the XLATEOBJ. This fixes mono bitmaps passed to GetDIBits. (no it does not fix pink icons in VLC)
- In BuildDIBPalette don't handle 15bpp, its not valid. and 16 bpp is 555, this is documented in MSDN.

svn path=/trunk/; revision=50920

Fix build

svn path=/trunk/; revision=50919

[GDI32]
In GetDIBits convert a BITMAPCOREHEADER to a BITMAPINFOHEADER before calling win32k, like done in other places. Convert it back after. This allows to simplify NtGdiGetDIBitsInternal (not done yet)

svn path=/trunk/; revision=50918

[NTOSKRNL]
Fix a serious bug, where NtAllocateVirtualMemory could allocate memory user mode memory above MmHighestUserAddress up to MmSystemRangeStart - 1, which is a no-access area.

svn path=/trunk/; revision=50917

[URLCACHE]
Do not initialize the URL-Cache for a system process.
Patch will be submitted to WINE.
See issue #5372 for more details.

svn path=/trunk/; revision=50916

[SETUPAPI]
Rename functions according to the new naming scheme:
  CaptureAndConvertAnsiArg --> pSetupCaptureAndConvertAnsiArg
  CenterWindowRelativeToParent --> pSetupCenterWindowRelativeToParent
  ConcatenatePaths --> pSetupConcatenatePaths
  DuplicateString --> pSetupDuplicateString
  EnablePrivilege --> pSetupEnablePrivilege
  GetVersionInfoFromImage --> pSetupGetVersionInfoFromImage
  IsUserAdmin --> pSetupIsUserAdmin
  MultiByteToUnicode --> pSetupMultiByteToUnicode
  MyGetFileTitle --> pSetupGetFileTitle
  OpenAndMapForRead --> pSetupOpenAndMapForRead
  RegistryDelnode --> pSetupRegistryDelnode
  UnicodeToMultiByte --> pSetupUnicodeToMultiByte
  UnmapAndCloseFile --> pSetupUnmapAndCloseFile

svn path=/trunk/; revision=50915

[ntoskrnl] Fix compilation with _WINKD_=1

svn path=/trunk/; revision=50914

[KERNEL32]
Start unifying kernel32 Find* functions.
All calls end in same methods that are working only in unicode now.

Next step would be to unify stream & files using only one struct.

svn path=/trunk/; revision=50913

[KERNEL32]
- Fix buffer overwrite in GetModuleFileName(). Spotted by DPH.
See issue #5964 for more details.

svn path=/trunk/; revision=50912

[ROSTESTS]
- Skip test_apm to prevent the oleaut_winetest:olepicture crash.
See issue #5000 for more details.

svn path=/trunk/; revision=50911

[NTDLL/LDR]
- Don't try to free a pointer allocated on stack.
See issue #4751 for more details.

svn path=/trunk/; revision=50910

[USRMGR]
Fix build
Blame rbuild ~

svn path=/trunk/; revision=50909

- Don't free an invalid handle. Suggested by Johannes. Test failure remains in the log, so it's easy to spot the function misbehaves.
- P.S. Not much of an issue if it's lost during the next tests sync, as it would be a clearly visible change.
See issue #5771 for more details.

svn path=/trunk/; revision=50908

[KERNEL32]
Remove unused .def file. We're using .pspec here

svn path=/trunk/; revision=50907

[USRMGR]
French translation by Jason Toscano <JasonGenie at gmail dot com>
Corrections/adjustements by me
See issue #5921 for more details.

svn path=/trunk/; revision=50906

[ROSTESTS]
Bug 5778:
Disable part of rpcrt4:rpc test,
it needs unimplemented functionality in current rpcrt4 version.

svn path=/trunk/; revision=50905

[NTOSKNRL]
For both FsRtlIsNameInExpression() & FsRtlIsDbcsInExpression():
As both UNICODE_STRING & ANSI_STRING might not be NULL-termined, don't attempt to read null char.
This fixes potential buffer overruns.
Then it fixes some (all?) 'TempPte.u.Long != 0' assertion failure.
See issue #5923 for more details.

svn path=/trunk/; revision=50904

[WININET]
Revert part of r47411. The revision was a wine sync, but no wine code was reverted, only the use of zlib is disabled. Fixes downloading audiograbber with rapps.
See issue #5663 for more details.

svn path=/trunk/; revision=50903

[WIN32K]
- In BITMAP_CopyBitmap, also "copy" the palette of the source bitmap, instead of assuming the one created by GreCreateBitmapEx matches the bitmap bits we have copied.
Fixes broken colors on the firefox firstrun page
See issue #5902 for more details.

svn path=/trunk/; revision=50902

[ROSTESTS]
Bug 5957
Disable test_shell_window() in user32:win test.

Allow to run previously disabled tests in WINE_INTERACTIVE mode.

svn path=/trunk/; revision=50901

[WDMAUD.DRV]
- Check if the provided waveformat is at least size of WAVEFORMAT
- Compute extra details such as the size or wBitsPerSample from the provided members
- Backup i/o completion handle before commiting the audio buffer as the struct might have already been released
- Fixes playback of mmsys cpl
- Found by DPH!

svn path=/trunk/; revision=50900

[SHELL32]
- Check the correct pidl instead of the free one
- Found by the amazing DPH!

svn path=/trunk/; revision=50899

[RTL/DPH]
- Fix wrong pointer comparison in RtlpDebugPageHeapFree(). Should check for NULL pointer freeing, instead of checking for NULL heap pointer.

svn path=/trunk/; revision=50898

[MMEBUDDY]
- Store the internal state if an audio stream has been stopped
- Construct a thread when WODM_RESTART is received. This thread then triggers performs a callback to the wave thread, which finally restarts playback
- Only start the wave thread when playback is active (not paused by WODM_PAUSE)
- Fixes playback in Winamp 5.601 despite graphical issues
- Tested in Vbox 4.0 + 512MB RAM + Winamp 5.601

svn path=/trunk/; revision=50897

[[WIN32K]
Fix biCompression filed in the DIBSECTION structure, returned by BITMAP_GetObject. Fixes K-Meleon toobar. Kudos for the fix go to Rafał Harabień.

See issue #5940 for more details.

svn path=/trunk/; revision=50896

[SECUR32]
- Don't leave packageinfo uninitialized. Even though it's not the way it should work, wine's rpcrt4 doesn't really expect that function to return no packages at all. Should prevent freeing an unitialized pointer.
See issue #5778 for more details.

svn path=/trunk/; revision=50895

[WDMAUD.DRV]
- Set wave in pins into run state
- Fixes hang of ReactOS sndrec32, though it crashes then instantly. Needs more investigation

svn path=/trunk/; revision=50894

[WIN32K]
Fix possible lock leak in failure case of NtGdiSetBoundsRect. Fixes a failed assertion spotted by r3ddr4g0n.

svn path=/trunk/; revision=50893

[RTL/DPH]
- Implement ValidateHeap API for DPH. Fixes attempts to validate DPH as a normal heap.

svn path=/trunk/; revision=50892

[PING]
- Add Ukrainian translation by Eugene Sakara.

svn path=/trunk/; revision=50891

[COMCTL32]
- Fix include order of resources.
- remove comctl_Cn.rc, it contains and outdated simplified chineese translation, that wine has in comctl_Zh.rc

svn path=/trunk/; revision=50890

[RTL/DPH]
- Implement user flags/values and HeapSize APIs in debug page heap (nearly the same code in all three functions). With these done, DPH core features are complete. Some debugging is still needed.

svn path=/trunk/; revision=50888

[COMCTL32]
- Sync with wine 1.3.14

svn path=/trunk/; revision=50887

[PSDK]
- Improve some definitions.

svn path=/trunk/; revision=50886

[WIN32K]
Patch by Rafał Harabień :
- don't transform cordinates twice in GreExtTextOutW
- Fixes AbiWord toolbar

See issue #4811 for more details.

svn path=/trunk/; revision=50885

[NTDLL/LDR]
- DPH could be selectively enabled for a specific application by means of Image File Execution options. Commit a rewrite of LdrQueryImageFileExecution* APIs based on a quite old patch by Alex Ionescu along with my fixes.
- This is a forced measure to commit first step of an ntdll/ldr rewrite. This particular commit should not introduce any regressions, because previously that code part just barely worked.

svn path=/trunk/; revision=50884

[PSDK]
- Improve commctrl.h
[CRT]
- Update _mingw_mac.h from mingw-w64 and add _mingw_unicode.h

svn path=/trunk/; revision=50883

[USER32]
Patch by Rafał Harabień :
- remove a hack in DrawMenuBar

See issue #5949 for more details.

svn path=/trunk/; revision=50882

[WIN32K]
Patch by Rafał Harabień :
- Create a new region covering the full desktop in UserRedrawDesktop instead of using Window->hrgnUpdate. Fixes "IntGdiCombineRgn requires hSrc2  != NULL" messages and redrawing issues on mode change.
See issue #5949 for more details.

svn path=/trunk/; revision=50881

[WIN32K]
Small readability improvement.

svn path=/trunk/; revision=50880

[WIN32K]
add code to register pre/post systemcall hooks. Its #if 0'ed to avoid unneccessary overhead. Can be enabled for debugging puposes.

svn path=/trunk/; revision=50879

- Fix build

svn path=/trunk/; revision=50878

[PORTCLS]
- Re-add the hack for gcc

svn path=/trunk/; revision=50877

[kernel32]
- Add some debug prints
- Fix some cases where we didn't check for the return value of CsrAllocateCaptureBuffer

svn path=/trunk/; revision=50876

[NTOSKRNL]
Apply r50873 on FsRtlIsNameInExpression()

svn path=/trunk/; revision=50875

[NTOSKRNL]
Reverted changes commited to largemcb.c
They weren't intended to be commited

svn path=/trunk/; revision=50874

[NTOSKRNL]
One-line fix for FsRtlIs*InExpression(). This fixes failing tests, but not overrun.
Added some breaks to reach more quickly the end

svn path=/trunk/; revision=50873

[RTL/DPH]
- Implement DPH version of ReAllocateHeap. Scenario of falling back to normal heap is stubbed everywhere now.
- Fix a bug in POINTER_ADD_BIAS which noone noticed.

svn path=/trunk/; revision=50872

[MPLAY32]
- Hackfix playback

See issue #5867 for more details.

svn path=/trunk/; revision=50871

[KMTEST]
Adding more tests for FsRtlIs*InExpression() functions.
All are failing (overrun / wrong result).
Based on Thomas Faber's C program.
See bug #5923

svn path=/trunk/; revision=50870

[KERNEL32]
Don't make CopyFileExA() rely on Wine's strings conversions functions

svn path=/trunk/; revision=50869

[KERNEL32]
Don't make FindFirstChangeNotificationA() rely on Wine's strings conversions functions

svn path=/trunk/; revision=50868

[KERNEL32]
Don't make GetBinaryTypeA() rely on Wine's strings conversions functions

svn path=/trunk/; revision=50867

[CRT]
Fix special behaviour of wsprintf: when used with # flag, the "0x" prefix does not count into the field width, so we have to add additional 2 characters here. Fixes Opera installer.
See issue #5900 for more details.

svn path=/trunk/; revision=50866

[NOTEPAD]
Patch by EDIJS <terminedijs at yahoo dot com>: Improve window caption string.
By me: use StringCchCat instead of _tcscat to prevent possible buffer overruns.

See issue #5871 for more details.

svn path=/trunk/; revision=50864

Update rapps database.

svn path=/trunk/; revision=50862

[CRT]
Improve _getdrive and _tchdir. Patch by devpao (devotip at tiscali dot it), small formatting change by me.
See issue #5848 for more details.

svn path=/trunk/; revision=50861

[RTL/DPH]
- Implement heap free operation using already implemented busy/free/available/unused lists support and other base routines.
- Implement missing place to free list and remove from busy list routines.
- Implement find busy block routine (using AVL tree).
- Fix a bug in RtlpDphCoalesceNodeIntoAvailable() which resulted in unwanted attempt to merge the only node with itself (which failed anyway).
- Fix a bug in RtlpDphCoalesceNodeIntoAvailable() which incorrectly removed a node from available list (which is implemented as a standard NT double-linked list, compared to unused and free lists which are implemented as single-linked custom lists and busy list which is an AVL tree). Result of that bug was an infinite loop at the next attempt to traverse the list of available nodes.
- In RtlpDphCoalesceFreeIntoAvailable() break when FreeAllocations becomes less than LeaveOnFreeList (before it would break 1 size too early).
- Fix list traversal in RtlpDphSearchAvailableMemoryListForBestFit(). If it couldn't find a node, it would return the last node in the list instead of NULL.
- In RtlpDphFindAvailableMemory(), a new smaller size should be 4 times smaller, not just 2.
- Add a #if0-ed section in RtlpDphRemoveFromAvailableList which checks if a request to remove node not in the list performed. Used only for debugging.
- Add a trace dprint to every type of list insert/removal operation for easier tracking.
- Add break on NULL ptr freeing support.
- RtlpDphSetProtectionAfterUse() is stubbed and protection is set directly in RtlpDphHeapFree(). To be moved into this function.

svn path=/trunk/; revision=50860

[USER32]
Fix a typo. Fixes "Calling invalid routine number 0x10 in NtUserCallOneParam()"

svn path=/trunk/; revision=50858

[RTL/AVL]
- Zero out contents of a newly allocated AVL table node.

svn path=/trunk/; revision=50855

[RTL]
- Silence this dprint. It's flooding the debug log.

svn path=/trunk/; revision=50854

[Win32k]
- Fix crash with win wine test. Win test works on my tree here and hardware as well.

svn path=/trunk/; revision=50853

[KERNEL32]
Forgotten changes to .pspec.
Why is there still a .def?

svn path=/trunk/; revision=50852

[MMEBUDDY]
- Check if there is a reset request in progress while the wave thread is active. In that case exit the wave buffer commiting activity
- In the reset routine, perform an alertable wait in order to make the apc io completion complete
[PORTCLS]
- Only perform copying audio bytes to dma buffer while the adapter is reset state is KSREST_END
- Set the reset state indicator before canceling the audio buffers
- Should fix the audio hang experienced in Winamp when skipping audio bytes

svn path=/trunk/; revision=50851

[PORTCLS]
- Remove the oldest hack in portcls
- Please report any regression(s)

svn path=/trunk/; revision=50850

[KERNEL32]
Don't mix tab index and memory length...
This fixes several bugs in GetTempFileNameW() and prevents memory corruption ~
Winetest regression is gone :)

svn path=/trunk/; revision=50849