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

Fix race condition #22

Open
wants to merge 2 commits into
base: kinetic-devel
Choose a base branch
from
Open

Fix race condition #22

wants to merge 2 commits into from

Commits on Sep 12, 2016

  1. Fix race condition by starting subscriber as last thing 

    As soon as a subscriber is started, it is able to receive messages and call the registered callback. However unlikely, this could cause the callback to access parameters and publishers which have not been inited yet. This race condition can cause obscure segmentation faults.
    @marcoesposito1988
    marcoesposito1988 committed Sep 12, 2016
    Configuration menu
    Copy the full SHA
    645eae3 View commit details
    Browse the repository at this point in the history

  2. Removed leftover dynamic_reconfigure file from aruco 

    Resides now in aruco_ros
    @marcoesposito1988
    marcoesposito1988 committed Sep 12, 2016
    Configuration menu
    Copy the full SHA
    61ea357 View commit details
    Browse the repository at this point in the history