You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@activemq.apache.org by "Andrey Slepykh (Jira)" <ji...@apache.org> on 2024/02/29 06:47:00 UTC
[jira] [Updated] (ARTEMIS-4660) BAD_COPY_PASTE in VersionedStompFrameHandler.java
[ https://issues.apache.org/jira/browse/ARTEMIS-4660?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Andrey Slepykh updated ARTEMIS-4660:
------------------------------------
Description:
PR: [https://github.com/apache/activemq-artemis/pull/4835|https://github.com/apache/activemq-artemis/pull/4831]
In 1st branch (line 274) of the {{if()}} statement, the {{Boolean.parseBoolean()}} method accepts the value {{{}frame.getHeader(Stomp.Headers.Subscribe.NO_LOCAL){}}}, which is used in line 273.
In 2nd branch (line 276), the {{Boolean.parseBoolean()}} method accepts the value {{{}frame.getHeader(Stomp.Headers.Subscribe.NO_LOCAL){}}}, although the other value {{frame.hasHeader(Stomp.Headers.Subscribe.ACTIVEMQ_NO_LOCAL)}} is used.
Found by Linux Verification Center (portal.linuxtesting.ru) with SVACE.
Author A. Slepykh.
was:
PR: https://github.com/apache/activemq-artemis/pull/4831
In 1st branch (line 274) of the {{if()}} statement, the {{Boolean.parseBoolean()}} method accepts the value {{{}frame.getHeader(Stomp.Headers.Subscribe.NO_LOCAL){}}}, which is used in line 273.
In 2nd branch (line 276), the {{Boolean.parseBoolean()}} method accepts the value {{{}frame.getHeader(Stomp.Headers.Subscribe.NO_LOCAL){}}}, although the other value {{frame.hasHeader(Stomp.Headers.Subscribe.ACTIVEMQ_NO_LOCAL)}} is used.
Found by Linux Verification Center (portal.linuxtesting.ru) with SVACE.
Author A. Slepykh.
> BAD_COPY_PASTE in VersionedStompFrameHandler.java
> -------------------------------------------------
>
> Key: ARTEMIS-4660
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4660
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Components: ActiveMQ-Artemis-Native
> Affects Versions: 2.32.0
> Reporter: Andrey Slepykh
> Assignee: Clebert Suconic
> Priority: Minor
> Labels: easyfix, pull-request-available
>
> PR: [https://github.com/apache/activemq-artemis/pull/4835|https://github.com/apache/activemq-artemis/pull/4831]
> In 1st branch (line 274) of the {{if()}} statement, the {{Boolean.parseBoolean()}} method accepts the value {{{}frame.getHeader(Stomp.Headers.Subscribe.NO_LOCAL){}}}, which is used in line 273.
> In 2nd branch (line 276), the {{Boolean.parseBoolean()}} method accepts the value {{{}frame.getHeader(Stomp.Headers.Subscribe.NO_LOCAL){}}}, although the other value {{frame.hasHeader(Stomp.Headers.Subscribe.ACTIVEMQ_NO_LOCAL)}} is used.
> Found by Linux Verification Center (portal.linuxtesting.ru) with SVACE.
> Author A. Slepykh.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)