[ 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)