Please review: Add EACCES check to the return value of fcntl()

[Charles Lee]

On 02/17/2012 05:22 PM, Alan Bateman wrote:
> On 17/02/2012 08:21, Charles Lee wrote:
>> :
>>
>> Do I have your thumb up for this patch? Is it ok for me to commit it?
>>
>> By the way, I have create a sunbug @ 
>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7146506 . Should I 
>> wait for this bug coming online?
> Sorry I wasn't clear, it was a thumbs up from me.
>
> I've moved 7146506 to the right place as:
>
> 7146506: (fc) Add EACCES check to the return of fcntl native method
>
> I'll marked it as fixed once I see your change-set go in.
>
> -Alan.
>
>
>
>
>
>
>
Here it is:

Changeset: de7f6d5841b6
Author:    littlee
Date:      2012-02-20 11:24 +0800
URL:http://hg.openjdk.java.net/jdk8/tl/jdk/rev/de7f6d5841b6

7146506: (fc) Add EACCES check to the return of fcntl native method
Summary: Add EACCES check according to the spec of fcntl
Reviewed-by: alanb

! src/solaris/native/sun/nio/ch/FileDispatcherImpl.c


Thank you very much !

-- 
Yours Charles

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20120220/8314bcd1/attachment.html