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

Restore IgnMonitor behavior after bug in Ignition::transport is fixed #760

Open
francocipollone opened this issue Mar 5, 2021 · 2 comments
Open

Restore IgnMonitor behavior after bug in Ignition::transport is fixed #760

francocipollone opened this issue Mar 5, 2021 · 2 comments

Comments

@francocipollone
Copy link
Collaborator

Ignition-transport7
When you subscribe to a topic it doesn't guarantee that the callback method you provide is going to be executed while
the instance of the class that holds that callback method is still alive.

In particular in IgnMonitor class, it caused SEGFAULTS because in certain situations the callbacks were executed after that IgnMonitor instance class had been destroyed. See: #751

A workaround was provided in this PR: #759

However, the implementation/functioning of IgnMonitor is expected to be restored once this bug in Ignition-transport is fixed.
This was referenced Mar 5, 2021
Merged
Closed
Closed
@agalbachicar
Copy link
Collaborator

agalbachicar commented Mar 16, 2021 •
edited by francocipollone
Loading

  • Requires an upstream report

@francocipollone
Copy link
Collaborator Author

  • Requires an upstream report

gazebosim/gz-transport#228

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@agalbachicar @francocipollone