[CONSRV]
"Reply Already Sent" and not already done !

svn path=/branches/ros-csrss/; revision=57818

[KERNEL32]
- Simplify IntGetConsoleInput and IntReadConsole.

[CONSRV]
- Implement event waiting for reading console input buffer with CSR wait blocks. This replaces the event-based waiting which, after being moved from client-side (kernel32) to server-side (see r57721), made all consoles hang when one of them was waiting for new input.

TODO: Dereference all the waits in Console->ReadWaitQueue.

svn path=/branches/ros-csrss/; revision=57817

[CSRSRV/BASESRV/CONSRV/WINSRV]
- Add a useful CHECK_API_MSG_SIZE macro to check whether a server message structure can hold in a CSR_API_MESSAGE structure. These checks are required because LPC will use the generic CSR_API_MESSAGE structure for communicating all the different servers' messages, and thus we avoid possible buffer overflows with this method.
- Effectively use this macro for all the server message structures.
- Remove a hack regarding the maximum data size we can pass through the CSR LPC port.
- Remove the now unused CSRSS_HEADER_SIZE symbol.

svn path=/branches/ros-csrss/; revision=57810

[NTDLL/CSRSRV]
Re-fix part of the capture-buffer offset setting, based on what I understood from all my previous investigations. It reverts a little part of r57673 but fixing some existing bugs there.

svn path=/branches/ros-csrss/; revision=57809

[NTDLL]
Correct some comments.

svn path=/branches/ros-csrss/; revision=57808

[CSRSRV]
- Comment on the size of some members of the CSR_WAIT_BLOCK structure.
- Initialize the WaitBlock member of CSR_THREAD to a valid value when creating a wait block, and NULLify it when we release a wait block.
- ALWAYS USE offsets in CSR_CAPTURE_BUFFER structure, instead of real pointers !! It is needed when their base address change (eg. during a CSR wait, their base address, corresponding to the address of an CSR API message, change) (found when testing CSR waits with the console).

svn path=/branches/ros-csrss/; revision=57806

[CSRSRV]
Add the CSRSRV status file from trunk/reactos/subsystems/csr/csrsrv/ and update it.

svn path=/branches/ros-csrss/; revision=57804

Well, "Types".

svn path=/branches/ros-csrss/; revision=57802

[CSRSRV]
- Correct a misspelling 'CsrThreadAltertable' -> 'CsrThreadAlertable'.
- Introduce CSR_REPLY_CODEs instead of using hardcoded values, and use them with CSR_API_ROUTINE-type functions. They correspond to which decision CSRSRV should take after a server function is called: answer to the client or not, and perform according tasks.

[BASESRV]
Use CSR_REPLY_CODEs.

svn path=/branches/ros-csrss/; revision=57801

Synchronize up to trunk's revision r57784.

svn path=/branches/ros-csrss/; revision=57785

[LSASRV]
- Fix object deletion.
- Add a registry wrapper function that enables us to delete registry keys which have already been opened.
- Make LsarDelete call LsarDeleteObject because otherwise object deletion would not be possible.

svn path=/trunk/; revision=57784

[KDCOM]
Fixes COM port debugging output with Virtual PC 2007. See r57781 for more details.

CORE-4247 #comment Fixed also in r57782.

svn path=/trunk/; revision=57782

[KDDLL]
- Use cportlib for COM port facilities. Fixes COM port debugging output with Virtual PC 2007.
- Remove now unneeded header file.

CORE-4247 #comment Fixed in r57776, r57777, r57780 and r57781. #resolve

svn path=/trunk/; revision=57781

[CPORTLIB/FREELDR]
GetByte  --> Wait for data (with timeout) and get it if available.
PollByte --> Check for data, get it if available, and return immediately.

svn path=/trunk/; revision=57780

[Win32k]
- Fix crash when the exception is thrown.

svn path=/trunk/; revision=57779

[FREELDR]
Remove unuseful defines, now.

svn path=/trunk/; revision=57778

[FREELDR]
- Make use of the previously committed cportlib code. This works great on VPC 2007 :)
- Comment the unused Rs232PortPollByte function.
- In hardware.c, add useful comments and two macros holding the maximum number of COM and LPT ports.

svn path=/trunk/; revision=57777

[CPORTLIB]
- Do a synthesis of all the code I've found concerning initializing COM ports (in kdcom / kddll / kdbg in the kernel and in freeldr). The "// Windows" comments concern the original code for testing the existence of COM ports, inside cportlib, whereas "// ReactOS" comments concern the equivalent existing in our kdcom/kddll/kdbg etc...
- CORRECT THE VIRTUAL PC COM PORT INITIALIZATION !! The problem was, that the current COM port existence test fails on VPC whereas it works well in all other virtual machines. Therefore :
  * I keep the existing testing code in a function called ComPortTest1, this works everywhere but on VPC ;
  * I add a very basic COM port existence test function called ComPortTest2, which basically looks for the scratch register, this works everywhere including on VPC.
  * The COM port existence test consists in the two tests above.

svn path=/trunk/; revision=57776

[SERIAL]
Comment the different defines.

svn path=/trunk/; revision=57775

[SERIAL]
The SER_...(x) macros return UCHAR pointers, i.e. PUCHAR.

svn path=/trunk/; revision=57774

[REGEDIT]
- Allow canceling import operations. Patch by Lee Schroeder.
CORE-6766 #resolve

svn path=/trunk/; revision=57773

[REGEDIT]
- Add a line break to import prompt for improved readability. Suggested by Lee Schroeder.
CORE-6766

svn path=/trunk/; revision=57772

[LSASRV]
Do not store a key name in an LSA object. Fixes CORE-6792.

svn path=/trunk/; revision=57771

[WineTests]
- Sync to 1.5.18

svn path=/trunk/; revision=57770

[LSASRV]
- Implement LsarDeleteObject.
- Store the object name in the database object in order to be able to delete the objects key later.

svn path=/trunk/; revision=57769

CORE-6650 #resolve #comment Add missing files on bootcd. Patch By Lee Schroeder.

svn path=/trunk/; revision=57768

[LSASRV]
- Add enumeration of user rights to LsarEnumerateAccountRights.
- Use RPC_UNICODE_STRING instead of UNICODE_STRING in the privilege lookup code.

svn path=/trunk/; revision=57767

[LSASRV]
Implement LsarGetQuotasForAccount and LsarSetQuotasForAccount.

svn path=/trunk/; revision=57766

[WineTests]
- Sync to 1.5.18

svn path=/trunk/; revision=57765

[LSASRV]
- Add new registry API.
- Implement LsarEnumerateAccounts.

svn path=/trunk/; revision=57764

[NTOSKRNL]
Code formatting only

svn path=/trunk/; revision=57763

[NTOSKRNL]
Code formatting only

svn path=/trunk/; revision=57762

[KD64]
Use \t instead of hardcoding the tab character.

svn path=/trunk/; revision=57761

[CPORTLIB]
Code formatting only.

svn path=/trunk/; revision=57759

[CONSRV]
Correct a debug message.

svn path=/branches/ros-csrss/; revision=57758

Synchronize up to trunk's revision r57756.

svn path=/branches/ros-csrss/; revision=57757

[WIN32CSR]
Correct the initialization of text-mode console.

svn path=/trunk/; revision=57756

Install the vga fonts needed for the Blue driver.

svn path=/trunk/; revision=57755

[SERVICES]
- (Re-)implement acquiring/releasing driver-loading and unloading privilege (Note: the two privileges have the same name SE_LOAD_DRIVER_NAME).
- Rework the SCM initialization code. In particular, acquiring the starting lock MUST BE done BEFORE starting the RPC server, otherwise, a program can try to start a service (and during this operation, the global start lock is acquired by the SCM) DURING the initialization of SCM, and in particular between the call to ScmStartRpcServer and ScmAcquireServiceStartLock. The SCM would then see an already-acquired start lock at this point and would fail to initialize. ==> This commit corrects this behaviour.

svn path=/trunk/; revision=57754

[LSASRV]
- LsarSetInformationPolicy: Add stubs for all missing case.
- Implement LsarSetAuditLog and LsarSetDefaultQuota.

svn path=/trunk/; revision=57753

[FREELDR]
When filling LoadOptions, strip the '/' commutator symbol in front of each option. Now we fully comply with Windows when holding LoaderBlock->LoadOptions, and the content of the registry value SystemStartOptions under HKLM\CurrentControlSet\Control display the options without their / symbol.

[WIN32K]
Now, the SystemStartOptions value doesn't contain / anymore.

svn path=/trunk/; revision=57752

[NTOSKRNL]
SystemBootDevice should be the ARC-formatted boot device (see what Windows does).

svn path=/trunk/; revision=57751

The console boot switch is really named CONSOLE, not CMDCONS, since r29948.

svn path=/trunk/; revision=57750

[KERNEL32]
Deactivate the code of GetConsoleInputWaitHandle, because it calls a function that doesn't seem to appear in the console server apis listed in http://j00ru.vexillium.org/csrss_list/api_list.html (and that I haven't included in the console server apis list), until I find a better solution. It fixes compilation.

[CONSRV]
- Code reorganization
- Introduce the CONSOLE_CONNECTION_INFO structure needed for initializing consoles when launching a program (see http://svn.reactos.org/svn/reactos/trunk/reactos/lib/kernel32/misc/dllmain.c?r1=16826&r2=16825&pathrev=16826), instead of calling AllocConsole. Not used at this time.

Now the branch compiles.

svn path=/branches/ros-csrss/; revision=57749

Revert 57718

svn path=/trunk/; revision=57748

Fix Unicode output on the console, broken in 33866, fixed in 34042, broken again in 38185, fixed again in 51058 and finally broken yet another time since 54651...
I think this history deserves champagne and forceful warning comments, which have been added.
Simple solution: Don't blindly sync Wine code!

#resolve #CORE-6495

svn path=/trunk/; revision=57747

[LSASRV]
Move set functions to the end of the file. No code changes!

svn path=/trunk/; revision=57746

[LSASRV]
LsarSetInformationPolicy: Add labels for unimplemented cases.

svn path=/trunk/; revision=57745

[CSRSRV]
Remove the unneeded CsrpHandleConnectionRequest function since CsrApiHandleConnectionRequest does already the job.

svn path=/branches/ros-csrss/; revision=57744

[NTDLL]
When calling CsrClientConnectToServer with a valid connection info user buffer, after capturing the buffer and calling CsrClientCallServer, do not forget to copy back to the user buffer the updated connection info and to free the capture buffer !

svn path=/branches/ros-csrss/; revision=57743

[LSASRV]
- Create and set a security descriptor for newly created secrets.

svn path=/trunk/; revision=57742

[CSRSRV]
- Remove unused functions CallHardError and CsrHandleHardError (in fact calling the hard error callback is already done in the listening thread).
- Add an annotation and correct a typographical error; remove old comments.
- Remove the now unneeded csrplugin.h header (a relic of the old callback system of the old csrss/csrsrv).

svn path=/branches/ros-csrss/; revision=57741

[KERNEL32/CONSRV]
- Merge IntWriteConsoleOutputCharacter and WriteConsoleOutputAttribute functions into IntWriteConsoleOutputCode helper functions, which is used inside WriteConsoleOutputAttribute and WriteConsoleOutputCharacterW/A.
- In server-side, merge CsrWriteConsoleOutputChar and CsrWriteConsoleOutputAttrib into the server API SrvWriteConsoleOutputString.
- The structures CSRSS_WRITE_CONSOLE_OUTPUT_CHAR and CSRSS_WRITE_CONSOLE_OUTPUT_ATTRIB are merged into CSRSS_WRITE_CONSOLE_OUTPUT_CODE.

[CONSRV]
- Add a CsrValidateMessageBuffer usage.

svn path=/branches/ros-csrss/; revision=57740

Forgot this part.

svn path=/branches/ros-csrss/; revision=57739

[KERNEL32/CONSRV]
- Modify the helper function IntWriteConsole to make it use of capture buffers for passing things to write.
- Now, the loop disappears.
- I moved the pause-event wait (introduced in revision r47359) inside consrv, and (concerning the server-api-part SrvWriteConsole), especially in the ConioWriteConsole function, to be sure that all functions calling ConioWriteConsole become sensible to pause event. This REALLY NEEDS testing (and of course the code needs cleaning of the introduced comments, when I'm sure it works).

svn path=/branches/ros-csrss/; revision=57738

[LSASRV]
- Move LsapCreatePolicySd into a separate file.
- Create and set a security descriptor to newly created accounts.

svn path=/trunk/; revision=57737

[INCLUDE]
Add generic rights for account and secret objects.

svn path=/trunk/; revision=57736

[LSASRV]
Implement LsarQuerySecurityObject.

svn path=/trunk/; revision=57735

[KERNEL32/CONSRV]
- Merge IntFillConsoleOutputCharacter and FillConsoleOutputAttribute functions into IntFillConsoleOutputCode helper and make it used by FillConsoleOutputAttribute/CharacterW/A.
- In server-side, CsrFillOutputChar and CsrFillOutputAttrib are merged into SrvFillConsoleOutput.
- Merge CSRSS_FILL_OUTPUT and CSRSS_FILL_OUTPUT_ATTRIB structures into CSRSS_FILL_OUTPUT.

svn path=/branches/ros-csrss/; revision=57734

[LSASRV]
Create a security descriptor for the policy object and store as attribute "SecDesc".

svn path=/trunk/; revision=57733

[CONSRV]
CsrSetTextAttrib --> SrvSetConsoleTextAttribute and activate it in the list of APIs indices.

svn path=/branches/ros-csrss/; revision=57732

[CONSRV]
- Use the ConsoleGetPerProcessData macro (--> CONSOLE_PROCESS_DATA structure).
- Do not forget to leave a critical section if we fail.

svn path=/branches/ros-csrss/; revision=57731

[CONSRV]
Use the ConsoleGetPerProcessData macro (--> CONSOLE_PROCESS_DATA structure).

svn path=/branches/ros-csrss/; revision=57730

[KERNEL32]
Use new structures and api indices namings.

[CONSRV]
Use CsrValidateMessageBuffer instead of Win32CsrValidateBuffer.

svn path=/branches/ros-csrss/; revision=57729

[CONSRV]
Fix a DPRINT.

svn path=/branches/ros-csrss/; revision=57728

[KERNEL32/CONSRV]
- Use new structures and api indices namings (as usual).

[CONSRV]
- Merge CSRSS_SET_TITLE and CSRSS_GET_TITLE into CSRSS_CONSOLE_TITLE.
- Merge CSRSS_GET_CONSOLE_(OUTPUT_)CP and CSRSS_SET_CONSOLE_(OUTPUT_)CP into CSRSS_CONSOLE_CP.
- NtDuplicateObject(GetCurrentProcess(), ... ---> NtDuplicateObject(NtCurrentProcess(), ...
- Use CsrValidateMessageBuffer instead of Win32CsrValidateBuffer.
- Use CONSOLE_PROCESS_DATA structure.

TODO: Remove a DPRINT1 checkpoint in SrvAllocConsole after future tests are OK.

svn path=/branches/ros-csrss/; revision=57727

[KERNEL32]
- Add some NULL checks.
- Use new structures and api indices namings.

svn path=/branches/ros-csrss/; revision=57726

[KERNEL32]
- Use new structures and api indices namings.

[CONSRV]
- CSRSS_SET_CURSOR ==> CSRSS_SET_CURSOR_POSITION for SrvSetConsoleCursorPosition.
- Merge CSRSS_GET_CURSOR_INFO and CSRSS_SET_CURSOR_INFO into CSRSS_CURSOR_INFO.

svn path=/branches/ros-csrss/; revision=57725

[KERNEL32]
- Make some functions compliant with the new structures and server apis indices names.

[KERNEL32/CONSRV]
- Merge CSRSS_SET_CONSOLE_MODE and CSRSS_GET_CONSOLE_MODE into CSRSS_CONSOLE_MODE.
- Rename CSRSS_SETGET_CONSOLE_HW_STATE into CSRSS_CONSOLE_HW_STATE.

svn path=/branches/ros-csrss/; revision=57724

[KERNEL32/CONSRV]
- Make OpenConsole call only one server api, namely SrvOpenConsole, which was known before under the name CsrGetHandle (this latter function was removed in r57689 by error).
- Merge CSRSS_GET_INPUT_HANDLE and CSRSS_GET_OUTPUT_HANDLE structures into CSRSS_OPEN_CONSOLE (used by the functions named above).

[CONSRV]
- Add SrvOpenConsole (copied from CsrGetHandle), and add DPRINT1s to it.
- Load the localized string "Command Prompt" using the Server DLL's instance handle instead of calling GetModuleHandleW.
- Make use of CONSOLE_PROCESS_DATA structures.

svn path=/branches/ros-csrss/; revision=57723

[CONSRV]
Reorganize the data members according to their purposes (increases the lisibility only). No code changes.

svn path=/branches/ros-csrss/; revision=57722

[KERNEL32/CONSRV]
- Move the waiting loop of IntReadConsole into SrvReadConsole instead.
- Merge IntPeekConsoleInput and IntReadConsoleInput helpers into a single IntGetConsoleInput helper function
  and make PeekConsoleInputW/A and ReadConsoleInputW/A use it.
  Indeed, the "only" difference between these two functions, is that the first one keeps the data read inside
  the console input buffer, but on the contrary the second removes the data and does not return until at least
  one input record has been read.
  In server-side, CsrReadInputEvent and CsrPeekConsoleInput are merged into SrvGetConsoleInput. Merge the corresponding
  structures.
- Use shorter variable names in IntReadConsoleOutput and use CONSOLE_PROCESS_DATA structure in SrvReadConsoleOutput.
- Merge IntReadConsoleOutputCharacter and ReadConsoleOutputAttribute functions into a single IntReadConsoleOutputCode
  helper function, and make ReadConsoleOutputAttribute and ReadConsoleOutputCharacterW/A use it.
  In server-side, CsrReadConsoleOutputChar and CsrReadConsoleOutputAttrib are merged into SrvReadConsoleOutputString.
  (Remark: the SrvReadConsoleOutputString name comes from http://j00ru.vexillium.org/csrss_list/api_list.html and I
  checked that it was indeed the server function that was called by both ReadConsoleOutputAttribute and ReadConsoleOutputCharacter).
  The idea behind merging these two functions is, that an attribute or a character are instances of the same entity, namely
  a "code" (what would be called an (ANSI) escape sequence). Therefore I encode them inside the same CSRSS_READ_CONSOLE_OUTPUT_CODE
  structure and I use it with the previous functions.

[CONSRV]
- Make use of the CONSOLE_PROCESS_DATA structure (introduced in r57685).
- Use CsrValidateMessageBuffer instead of Win32CsrValidateBuffer.
- Reorganize conmsg.h a little bit.

svn path=/branches/ros-csrss/; revision=57721

[CONSRV]
Use CsrValidateMessageBuffer instead of Win32CsrValidateBuffer.

svn path=/branches/ros-csrss/; revision=57720

- Fix build
- Partly revert 57718

svn path=/trunk/; revision=57719

- Don't compile apps with Unicode as ReactOS console screenbuffer is not supporting this yet
- fixes #CORE-6495, #CORE-6786

svn path=/trunk/; revision=57718

[CONSRV]
Code reorganization only, no code changes.

svn path=/branches/ros-csrss/; revision=57717

[PSDK|User32]
- Fix error return for future wine Win.c test_map_points tests.
- Add missing error code.

svn path=/trunk/; revision=57716

[KERNEL32/CONSRV]
Fix compilation.

svn path=/branches/ros-csrss/; revision=57715

[KERNEL32/CONSRV]
- Use a standard win32 thread function for the ConsoleControlDispatcher thread, instead of a custom PCONTROLDISPATCHER one.
- Deactivate debug outputs in console.c, which were activated by error in r57706.

svn path=/branches/ros-csrss/; revision=57714

[KERNEL32]
- Move console-related Vista+ functions from client/vista.c to a dedicated client/console/vista.c file, and add it to compilation.
- Move some helper functions from console.c to history.c.
- Use the new api index names in history.c.

[CONSRV]
- Merge CSRSS_GET_HISTORY_INFO and PCSRSS_SET_HISTORY_INFO structures declarations into CSRSS_HISTORY_INFO, and therefore use only one member of it inside CONSOLE_API_MESSAGE structure.
- WORD --> UINT for few structure members related to history information.

svn path=/branches/ros-csrss/; revision=57713

[KERNEL32/CONSRV]
- Use capture buffers to pass (variable-length) strings to the console server.
- Merge the CSRSS_ADD_CONSOLE_ALIAS and CSRSS_GET_CONSOLE_ALIAS structures into a unique structure CSRSS_CONSOLE_ALIAS.

TODO: Check what happens to the "/ sizeof(WCHAR);" in GetConsoleAliasesW.

svn path=/branches/ros-csrss/; revision=57712

[KERNEL32]
- Move all the read/write related functions to an appropriate source file and add it to compilation.
- Fix credits for alias.c

svn path=/branches/ros-csrss/; revision=57711

[KERNEL32]
Compile alias.c and history.c.

svn path=/branches/ros-csrss/; revision=57710

[KERNEL32]
- Move history-functions to a dedicated source file. The functions were created in r47580 and therefore update the description header of the file.

svn path=/branches/ros-csrss/; revision=57709

Fix header (no code changes).

svn path=/branches/ros-csrss/; revision=57708

[KERNEL32]
- Move alias-functions to a dedicated file. The names of the programmers were retrieved via svn information (see r32093 and r32118).

svn path=/branches/ros-csrss/; revision=57707

[KERNEL32]
Fix some comments formatting.

svn path=/branches/ros-csrss/; revision=57706

[KERNEL32]
- Place console.c file in a dedicated directory because I will after split it into logical units (eg. a file for Alias-functions, a file for Read-write functions, etc...) to facilitate maintenance.

svn path=/branches/ros-csrss/; revision=57705

[Win32k]
- Rewrite message events to use posting to queues instead of sending them. See CORE-6551.

svn path=/trunk/; revision=57704

[CSRSRV]
- Comments formatting.
- Correct a nasty initialization bug which unveiled itself as, eg., critical sections being already initialized in consoles, whereas we were creating new ones. It was because we didn't copy correctly per-process server data from parent process to child process. Now consoles initialize finely.

Now, only modifications to the console subsystem remain to make it totally conform to the new messaging structures of csrss (I mean, using the capture buffers).

svn path=/branches/ros-csrss/; revision=57703

[CONSRV]
- Initialize the per-process structure size : fixes console initialization.

svn path=/branches/ros-csrss/; revision=57702

[CSRSRV]
- Use a variable ServerDll instead of using each time CsrLoadedServerDll[i], as it is done in some other places.
- Clean the code (remove extra-parentheses).
- Zero-out the memory buffer allocated for CsrSrvSharedStaticServerData.

svn path=/branches/ros-csrss/; revision=57701

[NETAPI32]
NetLocalGroupEnum: Add code to return information about a single alias to the caller.
Now, the user account control panel enumerates the built-in groups. ;-)

svn path=/trunk/; revision=57700

[Win32k]
- Implement Gdi32 counterpart SetStretchBltMode for Win32k.

svn path=/trunk/; revision=57699

[Win32k]
- Fix api test crash.

svn path=/trunk/; revision=57698

[MSGINA]
Resize the dialogs to fit the ReactOS image, and add the logo to the "Please wait..." dialog
(which is displayed when a user is logging-on or quitting his session and the "Save your parameters..."
message is displayed). Now ReactOS looks perfect.

svn path=/trunk/; revision=57697

[MSGINA]
Introduce a new logon image.
Resizing the dialog boxes remains to do.

svn path=/trunk/; revision=57696

[WIN32K]
- Implement usage of alpha cursors (does not work yet due to a bug in windres)
- Directly use bitmaps passed to NtUserSetCursorIconData instead of copying them

svn path=/trunk/; revision=57695

[BASESRV]
- Remove the unneeded CallProcessCreated function since the same functionality is programmed and executed in CSRSRV when CsrInsertProcess is called via a call of CsrCreateProcess.
- Remove some hacks in process creation.
- In BaseSrvExitProcess, use the 'Reply = 2' value instead of the old hack 'ApiMessage->ApiNumber = 0xBABE'.

[USER32/WINSRV]
- Make SetLogonNotifyWindow call our 'RosSetLogonNotifyWindow' hack and explicitely mark it as a hack.
- CsrSetLogonNotifyWindow --> RosSetLogonNotifyWindow to emphasize that it is a ReactOS-specific hack.

svn path=/branches/ros-csrss/; revision=57694

[GDI32_APITEST]
- Add test showing that a bitmap created using CreateDIBItmap can be selected into a DC despite a different bits depth

svn path=/trunk/; revision=57693