[
https://issues.apache.org/jira/browse/GEODE-9200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17334979#comment-17334979
]
ASF GitHub Bot commented on GEODE-9200:
---------------------------------------
lgtm-com[bot] commented on pull request #793:
URL: https://github.com/apache/geode-native/pull/793#issuecomment-828743251
This pull request **introduces 1 alert** when merging
42f79a127a85d72440582653acf1b68039451a79 into
0f40b8d8b5d530c5f650593526c01c0185d5efba - [view on
LGTM.com](https://lgtm.com/projects/g/apache/geode-native/rev/pr-20cf132a5d1054aef5f477c77fcd57c896561cc6)
**new alerts:**
* 1 for Catching by value
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Leaked message used to signal "all endpoints disconnected"
> ----------------------------------------------------------
>
> Key: GEODE-9200
> URL: https://issues.apache.org/jira/browse/GEODE-9200
> Project: Geode
> Issue Type: Bug
> Components: native client
> Reporter: Blake Bender
> Assignee: Blake Bender
> Priority: Major
> Labels: pull-request-available
>
> Geode native uses a "blank" reply message to signal to regions that all
> endpoints have disconnected, i.e. they can shut down. This message is
> allocated on the heap into a static pointer in the function
> `TcrMessage::getAllEPDisMess`, but is never deleted. This should be a
> dynamically allocated message, so it can be cleaned up via `delete` in the
> region like all other messages.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
