On Wed, Apr 11, 2012 at 1:18 AM, Dmitry Timoshkov <dmi...@baikal.ru> wrote:
> "Erich E. Hoover" <ehoo...@mines.edu> wrote:
>> ...
>> +    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?

Yes, I apologize for that - apparently I wasn't paying enough
attention when I checked that over.

> Also please avoid including auto generated code in the patch.

Sorry about that, I thought I was supposed to include all of the files
checked into the repository.

Erich Hoover
ehoo...@mines.edu


Reply via email to