Skip to content

node_handle must be destroyed after client_handle to prevent memory leak#1562

Merged
fujitatomoya merged 1 commit intoros2:masterfrom
fujitatomoya:bugfix-20210225-memory-leak-1545
Feb 25, 2021
Merged

node_handle must be destroyed after client_handle to prevent memory leak#1562
fujitatomoya merged 1 commit intoros2:masterfrom
fujitatomoya:bugfix-20210225-memory-leak-1545

Conversation

@fujitatomoya
Copy link
Copy Markdown
Collaborator

address #1545

Signed-off-by: Tomoya.Fujita [email protected]

@fujitatomoya
Copy link
Copy Markdown
Collaborator Author

@ivanpauno if you got time, could you take a look at this? thanks in advance.

@fujitatomoya
Copy link
Copy Markdown
Collaborator Author

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@fujitatomoya fujitatomoya merged commit dc7e95a into ros2:master Feb 25, 2021
@fujitatomoya
Copy link
Copy Markdown
Collaborator Author

the problem originally found in ros:foxy, backport is required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants