<br>
<div class="gmail_quote">2010/10/27 Alexandre Julliard <span dir="ltr"><<a href="mailto:julliard@winehq.org">julliard@winehq.org</a>></span><br>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">
<div class="im"><br>Eric Pouech <<a href="mailto:eric.pouech@orange.fr">eric.pouech@orange.fr</a>> writes:<br><br>> + if (access & GENERIC_READ) access |= FILE_GENERIC_READ;<br>> + /* FIXME: we add FILE_READ_PROPERTIES access even in write-only mode.<br>
> + * As tested on Windows, this is not correct (ie GetConsoleMode() on with GENERIC_WRITE only<br>> + * access fails).<br>> + * But, this is currently needed as part of kernel32/console.c client side code relies on the<br>
> + * fact that it can read properties: lots of 'set' functions like WriteConsole,<br>> + * ScrollConsoleScreenBuffer, SetConsoleCursorPosition (and many others) do call 'get'<br>> + * functions.<br>
> + */<br><br></div>These should be fixed then.<br><font color="#888888"><br></font></blockquote>
<div>this will require moving more than half a dozen of operations done client side to the server so that it does read the attributes</div>
<div>that's both:</div>
<div>1/ a tremendous change</div>
<div>2/ will either move most of console handling into the server, or more the access control on the client side. and I'd bet none of those two options you'll like</div>
<div>A+</div></div><br><br clear="all"><br>-- <br>-- <br>Eric Pouech<br>