Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Migrate some security-related changes from rolling to humble #2576

Closed
wants to merge 2 commits into from
Closed

Migrate some security-related changes from rolling to humble #2576

wants to merge 2 commits into from

Commits on Jul 8, 2024

  1. Fix subscription.is_serialized() for callbacks with message info (ros… 

    …2#1950)

* Fix subscription.is_serialized() for callbacks with message info argument

Signed-off-by: Ivan Santiago Paunovic <[email protected]>

* Add tests + please linters

Signed-off-by: Ivan Santiago Paunovic <[email protected]>
    @ivanpauno
    ivanpauno authored and root committed Jul 8, 2024
    Configuration menu
    Copy the full SHA
    4404c47 View commit details
    Browse the repository at this point in the history

  2. Declare rclcpp callbacks before the rcl entities (ros2#2024) 

    This is to ensure callbacks are destroyed last
on entities destruction, avoiding the gap in time
in which rmw entities hold a reference to a
destroyed function.

Signed-off-by: Mauro Passerino <[email protected]>
Co-authored-by: Mauro Passerino <[email protected]>
    @mauropasse
    2 people authored and root committed Jul 8, 2024
    Configuration menu
    Copy the full SHA
    913070c View commit details
    Browse the repository at this point in the history