[ 
https://issues.apache.org/jira/browse/ARTEMIS-5099?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrey Slepykh updated ARTEMIS-5099:
------------------------------------
    Description: 
There is an imposiible cast from LargeServerMessageImpl to AMQPLargeMessage in 
[https://github.com/apache/activemq-artemis/blob/4c6cded2e79cf10d2323de1b5fa7418b289ab034/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPLargeMessage.java#L638]

I suppose there is a mistake in the check above 
([here|https://github.com/apache/activemq-artemis/blob/4c6cded2e79cf10d2323de1b5fa7418b289ab034/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPLargeMessage.java#L637])
 and original should be instance of 
AMQPLargeMessage, but not LargeServerMessageImpl.

Found by Linux Verification Center (portal.linuxtesting.ru) with SVACE.
Author A.Burke.

  was:
There is an imposiible cast from LargeServerMessageImpl to AMQPLargeMessage in 
[https://github.com/apache/activemq-artemis/blob/4c6cded2e79cf10d2323de1b5fa7418b289ab034/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPLargeMessage.java#L638]

I suppose there is a mistake in the check above 
([here|https://github.com/apache/activemq-artemis/blob/4c6cded2e79cf10d2323de1b5fa7418b289ab034/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPLargeMessage.java#L637])
 and original should be instance of 
AMQPLargeMessage, but not LargeServerMessageImpl.


> Probable impossible cast in AMQPLargeMessage
> --------------------------------------------
>
>                 Key: ARTEMIS-5099
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-5099
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>            Reporter: Andrey Slepykh
>            Priority: Minor
>
> There is an imposiible cast from LargeServerMessageImpl to AMQPLargeMessage 
> in 
> [https://github.com/apache/activemq-artemis/blob/4c6cded2e79cf10d2323de1b5fa7418b289ab034/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPLargeMessage.java#L638]
> I suppose there is a mistake in the check above 
> ([here|https://github.com/apache/activemq-artemis/blob/4c6cded2e79cf10d2323de1b5fa7418b289ab034/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPLargeMessage.java#L637])
>  and original should be instance of 
> AMQPLargeMessage, but not LargeServerMessageImpl.
> Found by Linux Verification Center (portal.linuxtesting.ru) with SVACE.
> Author A.Burke.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact


Reply via email to