Skip to content

[kilted] Fix for memory leaks in the rclcpp::SerializedMessage (backport #2861)#2863

Merged
ahcorde merged 1 commit intokiltedfrom
mergify/bp/kilted/pr-2861
May 30, 2025
Merged

[kilted] Fix for memory leaks in the rclcpp::SerializedMessage (backport #2861)#2863
ahcorde merged 1 commit intokiltedfrom
mergify/bp/kilted/pr-2861

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented May 29, 2025

Description

This PR fixes memory leaks found in the rclcpp::SerializedMessage copy assignment operators

Fixes # (issue) N/A

Is this user-facing behavior change?

No

Did you use Generative AI?

No.

Additional Information

These memory leaks were found with memory sanitizer.

This is an automatic backport of pull request #2861 done by [Mergify](https://mergify.com).

@MichaelOrlov @kylemarcey @mergify
Fix for memory leaks in rclcpp::SerializedMessage (#2861)
93be802 
Signed-off-by: Michael Orlov <morlovmr@gmail.com>
Signed-off-by: Michael Orlov <michael.orlov@apex.ai>
Co-authored-by: kylemarcey <marcey.kyle@gmail.com>
(cherry picked from commit 8d44b95)
@mergify mergify bot mentioned this pull request May 29, 2025
Merged
@fujitatomoya
Copy link
Copy Markdown
Collaborator

fujitatomoya commented May 29, 2025 •
edited by ahcorde
Loading

Pulls: #2863
Gist: https://gist.githubusercontent.com/fujitatomoya/c7894cf5b6becd9b6ce5f3cd7c0a1c4f/raw/62219cba2de792e97b53210d182563351ef2eaf7/ros2.repos
BUILD args: --packages-above-and-dependencies rclcpp
TEST args: --packages-above rclcpp
ROS Distro: kilted
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/16113

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@MichaelOrlov MichaelOrlov changed the title Fix for memory leaks in the rclcpp::SerializedMessage (backport #2861) [kilted] Fix for memory leaks in the rclcpp::SerializedMessage (backport #2861) May 29, 2025
@ahcorde ahcorde merged commit 22939dd into kilted May 30, 2025
2 of 3 checks passed
@ahcorde ahcorde deleted the mergify/bp/kilted/pr-2861 branch May 30, 2025 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ahcorde ahcorde ahcorde approved these changes

@fujitatomoya fujitatomoya fujitatomoya approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fujitatomoya @ahcorde @MichaelOrlov