[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

[HAL]
Patch by Rafal Harabien, <rafalh1992 AT o2 DOT pl>:
- Fix Port numbers and ranges in HalpDefaultIoSpace (divide by 0x100)
By me:
- Fix DMA 1 port range and fix comment

See issue #5925 for more details.

svn path=/trunk/; revision=50848

[Win32k|User32]
- WM_MDICREATE originates from user space so treat it as it should, fixes bug 5903.

svn path=/trunk/; revision=50847

[Win32k]
- Restore original patch 50154.

svn path=/trunk/; revision=50846

[Win32k]
- Remove unused class list pointer and clear the class registered flag.

svn path=/trunk/; revision=50845

[SETUPAPI]
- Add 'pSetup'-Prefix to the string table functions according to the new naming convention.
- Remove the obsolete function StringTableTrim.

svn path=/trunk/; revision=50844

[WIN32CSR]
Silence CsrGetTempFile() debug

svn path=/trunk/; revision=50843

[CONSOLE]
French translation by Jason Toscano <JasonGenie@gmail.com>
Corrections/adjustements by me

svn path=/trunk/; revision=50842

[RUNONCE]
Fixed a typo in french translation

svn path=/trunk/; revision=50841

[SETUPAPI]
Implement CM_Query_And_Remove_SubTree[_Ex]A/W and CM_Request_Device_Eject[_Ex]A/W.

svn path=/trunk/; revision=50840

[KERNEL32]
- Get rid of Wine's implementation of GetTempFileNameA/W and reimplement them in the Windows 2003 way.
This fixes bug #5481

svn path=/trunk/; revision=50839

[KERNEL32]
- Get rid of old function Basep8BitStringToLiveUnicodeString() that doesn't exist under Windows, and that was even unused
- Rename Basep8BitStringToHeapUnicodeString() to Basep8BitStringToDynamicUnicodeString() and fix its implementation (and all the calls)
- Define BasepUnicodeStringTo8BitString()

svn path=/trunk/; revision=50838

[KERNEL32]
Properly implement GetLastError and export it.

svn path=/trunk/; revision=50837

[RTL]
- Fixed RtlMultiByteToUnicodeN & RtlAnsiCharToUnicodeChar prototype
- Added missing paged code marker to RtlAnsiCharToUnicodeChar
- Added a small hack to RtlAnsiCharToUnicodeChar. Indeed, when it's called during second stage, it's failing due to missing NLS table.
Probably usetup that doesn't define a registry entry. And then, FreeLdr just passes null pointer.

svn path=/trunk/; revision=50836

[WIN32CSR]
Add a new API GetTempFile() that returns an identifier to be used for creating temporary files.
This API should be in basesrv.dll

svn path=/trunk/; revision=50835

[USETUP]
- Get rid of indirection
See issue #5190 for more details.

svn path=/trunk/; revision=50834

- Add french translation for runonce utility
- Add svn:eol-style native property
- Fix language id (SUBLANG_DEFAULT -> SUBLANG_NEUTRAL)
See issue #5918 for more details.

svn path=/trunk/; revision=50833

[KERNEL32]
64 bit compatibility fix for IsWow64Process. Patch by Thomas Faber <thfabba at gmx dot de>

svn path=/trunk/; revision=50832

[NTOSKRNL]
Halfplement (win32 only) ProcessWow64Information case in NtQueryInformationProcess. Patch by Samuel Serapion.

See issue #5916 for more details.

svn path=/trunk/; revision=50831

[CHARMAP]
Patch by Carlo Bramini (carlo bramix at libero dot it)
- Make the text copy field a richedit control.
- Use WM_CHAR to copy the characters
- Fix definition of MSFTEDIT_CLASS for the resource compiler

See issue #5275 for more details.

svn path=/trunk/; revision=50830

[SETUPAPI]
Implement CM_Get_Hardware_Profile_Info[_Ex]A/W.

svn path=/trunk/; revision=50829

[RTL/DPH]
- Fix a bug with node count in RtlpDphAddNewPool().
- Unprotect needed amount of memory in RtlpDphSetProtectionBeforeUse() instead of always assuming PAGE_SIZE.
- Fix an incorrect virtual block pointer calculation in RtlpPageHeapAllocate().
- Silence RtlpDphShouldAllocateInPageHeap() debug print.
- Add helpful debug prints (disabled by default).
- Some code cleanup for better readibility.
- Heap create, destroy and allocating a block from the heap work now.

svn path=/trunk/; revision=50828

[WIN32K]
NtGdiSelectBitmap. Don't exclusively lock the old bitmap. This could cause pool corruptions, when the share count reachted 0, the object was deleted and unlocked after that. Use InterlockedCompareExchange to check and exchange the new bitmaps dc. This is commented out, because it causes a lot of drawing problems. Do the referencing manually instead of calling DC_vSelectSurface. Use a shared reference instead of an exclusive lock for the new bitmap. add code for proper handling of DEFAULT_BITMAP, currently hacked due to restrictions in other parts of win32k. Fixes bug 5498 and probably a lot of other problems.

svn path=/trunk/; revision=50827

[WIN32K]
NtGdiStretchDIBitsInternal: don't call NtGdiGetDCObject and NtGdiCreateCompatibleDC with a dc locked. These cases are not harmful, but generally Nt* and Gre* should never be called with an exclusive gdi lock held. Also don't return in failure case with the dc still locked.

svn path=/trunk/; revision=50826