[ 
https://issues.apache.org/jira/browse/HADOOP-11494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14297231#comment-14297231
 ] 

Benoy Antony commented on HADOOP-11494:
---------------------------------------

+1 .
I'll commit this tomorrow if there are no further comments. 

> Lock acquisition on WrappedInputStream#unwrappedRpcBuffer may race with 
> another thread
> --------------------------------------------------------------------------------------
>
>                 Key: HADOOP-11494
>                 URL: https://issues.apache.org/jira/browse/HADOOP-11494
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>            Priority: Minor
>         Attachments: hadoop-11494-001.patch, hadoop-11494-002.patch
>
>
> In SaslRpcClient, starting at line 576:
> {code}
>     public int read(byte[] buf, int off, int len) throws IOException {
>       synchronized(unwrappedRpcBuffer) {
>         // fill the buffer with the next RPC message
>         if (unwrappedRpcBuffer.remaining() == 0) {
>           readNextRpcPacket();
>         }
> {code}
> readNextRpcPacket() may assign another ByteBuffer to unwrappedRpcBuffer, 
> making the lock on previous ByteBuffer not useful.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to