[PATCH 4/5] [kernel] Support opening of console for read (try 2)

Alexandre Julliard julliard at winehq.org
Tue Nov 20 06:31:11 CST 2012


Eric Pouech <eric.pouech at orange.fr> writes:

>> That doesn't look right. There's no guarantee that the handle is for the
>> current process console.
> in fact, it's guaranted, because we check before (in the function
> calling this helper) that the handle is really a console handler and
> that it's in bare mode
> one could make it clearer by adding a comment to the helper function
> saying that those two conditions are met

That doesn't guarantee that it's the process console, console handles
can be passed across processes.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list