[PATCH 1/2] server: Add mechanism to retrieve the connect time of a socket (try 2).

Dmitry Timoshkov dmitry at baikal.ru
Wed Apr 11 02:18:17 CDT 2012


"Erich E. Hoover" <ehoover at mines.edu> wrote:

> +/* Get information about a socket */
> +DECL_HANDLER(get_socket_info)
> +{
> +    struct sock *sock;
> +
> +    sock=(struct sock *)get_handle_obj( current->process, req->handle, FILE_WRITE_ATTRIBUTES, &sock_ops );

Shouldn't the required access be FILE_READ_ATTRIBUTES or FILE_READ_PROPERTIES?
Why write access is requied in order to read the socket connect time?

Also please avoid including auto generated code in the patch.

-- 
Dmitry.



More information about the wine-devel mailing list