[
https://issues.apache.org/jira/browse/ARTEMIS-5147?focusedWorklogId=952803&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-952803
]
ASF GitHub Bot logged work on ARTEMIS-5147:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 16/Jan/25 20:31
Start Date: 16/Jan/25 20:31
Worklog Time Spent: 10m
Work Description: jbertram opened a new pull request, #5454:
URL: https://github.com/apache/activemq-artemis/pull/5454
The `transportConnection` variable will always be `null` so there's no use
in checking it.
Issue Time Tracking
-------------------
Worklog Id: (was: 952803)
Remaining Estimate: 0h
Time Spent: 10m
> Unreachable code in ClientSessionFactoryImpl.java
> -------------------------------------------------
>
> Key: ARTEMIS-5147
> URL: https://issues.apache.org/jira/browse/ARTEMIS-5147
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Reporter: Alex
> Assignee: Justin Bertram
> Priority: Minor
> Time Spent: 10m
> Remaining Estimate: 0h
>
> The createTransportConnection() function (line 1288) uses the try{}...catch{}
> block. In this block, the transportConnection variable is created, which,
> under any conditions of creation, is checked for the value != NULL. There is
> no security threat, as this is just an unnecessary check.
> https://github.com/apache/activemq-artemis/blob/935690a0b40d2887a765ca962e769215a680e1d6/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientSessionFactoryImpl.java#L1288-L1293
--
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
