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

ASF subversion and git services commented on QPID-6784:
-------------------------------------------------------

Commit 1708945 from [~lorenz.quack] in branch 'java/trunk'
[ https://svn.apache.org/r1708945 ]

QPID-6784: [Java Broker] Make _unexpectedByteBufferSizeReported volatile to 
avoid publication issue.

Addresses review comments from Keith Wall.

> [Java Broker, Java Client] Client sends too large frames and broker does 
> wrong frame size validation on 0-9
> -----------------------------------------------------------------------------------------------------------
>
>                 Key: QPID-6784
>                 URL: https://issues.apache.org/jira/browse/QPID-6784
>             Project: Qpid
>          Issue Type: Bug
>          Components: Java Broker, Java Client
>    Affects Versions: 0.17, 0.32
>            Reporter: Lorenz Quack
>            Assignee: Keith Wall
>            Priority: Blocker
>             Fix For: qpid-java-6.0
>
>
> The Client does not include the frame attributes (except for 1 byte) in its 
> calculation of the AMQP Frame size thus creating frames which are 7 bytes too 
> large. This happens in 
> {{org.apache.qpid.client.BasicMessageProducer_0_8#calculateContentBodyFrameCount}}
>  and maybe other places.
> The broker on the other end does an incorrect validation (in 
> {{AMQDecoder#decodable}} at least) where it does not take any of the 
> attributes into account thus accepting too large frames.
> Besides not being compliant with the spec this causes problems in 
> {{NonBlockingConnectionPlainDelegate#processData}} where the frame does not 
> fit into a single networkBuffer.
> I did not investigate other protocol versions.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to