Skip to content

fix: Fix data race in CallbackGroup::size() (backport #3056)#3059

Merged
ahcorde merged 1 commit intokiltedfrom
mergify/bp/kilted/pr-3056
Feb 13, 2026
Merged

fix: Fix data race in CallbackGroup::size() (backport #3056)#3059
ahcorde merged 1 commit intokiltedfrom
mergify/bp/kilted/pr-3056

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Feb 11, 2026

The computation of the size was not protected by a mutex, leading to possible data races.

Did you use Generative AI?

No

Additional Information

Discovered using helgrind


This is an automatic backport of pull request #3056 done by Mergify.

The computation of the size was not protected by a mutex, leading
to possible data races.

Signed-off-by: Janosch Machowinski <J.Machowinski@cellumation.com>
Co-authored-by: Janosch Machowinski <J.Machowinski@cellumation.com>
(cherry picked from commit 4d950ba)
@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented Feb 11, 2026

Pulls: #3059
Gist: https://gist.githubusercontent.com/ahcorde/65c39f1da2e75cb1219b9df4c65dca75/raw/0cee5dbbc6fd8618661bc833fee3e7ca7efaddeb/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/18174

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

@ahcorde ahcorde merged commit c5c0034 into kilted Feb 13, 2026
2 of 3 checks passed
@ahcorde ahcorde deleted the mergify/bp/kilted/pr-3056 branch February 13, 2026 08:48
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