Please review: Add EACCES check to the return value of fcntl()
Alan.Bateman at oracle.com
Wed Feb 15 00:06:26 PST 2012
On 15/02/2012 07:11, Charles Lee wrote:
> Hi guys,
> Patch @ http://cr.openjdk.java.net/~littlee/OJDK-327/webrev.00/
> The patch is mainly about add the EACCES when check the errno of fcntl
> with command F_SETLK.
> From the man page:
> /If a conflicting lock is held by another process, this call returns
> -1 and sets errno to EACCES or EAGAIN.
> /I propose to add the EACCES check. Would anyone help to review the patch?
Looks okay to me although I'm surprised we haven't run into this before
(the file locking code is there since jdk1.4). Are you seeing this only
on AIX or have you seen it on Linux/Solaris too?
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the nio-dev