"Erich E. Hoover" <ehoo...@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.


Reply via email to