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.

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

Subscription segfaults on take #549

Closed
hidmic opened this issue Apr 27, 2020 · 2 comments · Fixed by #550
Closed

Subscription segfaults on take #549

hidmic opened this issue Apr 27, 2020 · 2 comments · Fixed by #550
Labels
bug Something isn't working

Comments

@hidmic
Copy link
Contributor

hidmic commented Apr 27, 2020

Bug report

Required Info:

  • Operating System:
    • Ubuntu 20.04
  • Installation type:
    • From source
  • Version or commit hash:
  • DDS implementation:
    • ADLINK CycloneDDS, RTI Connext
  • Client library (if applicable):
    • rclpy

Steps to reproduce issue

  1. Start up a listener, e.g.
ros2 run demo_nodes_py listener
  1. Start up a talker, e.g.
ros2 run demo_nodes_py talker
  1. Once they are communicating, Ctrl-C the talker.

Expected behavior

Talker exits and listener stops receiving messages.

Actual behavior

Talker exits and listener segfaults.

Additional information

Limited backtrace:

#0  0x00000000005a8c1d in PyTuple_Pack ()                                                  
#1  0x00007ffff6fd185c in rclpy_take ()                                                    
   from /home/michel/Workspaces/focal_ros2_ws/install/rclpy/lib/python3.8/site-packages/rclpy/_rclpy.cpython-38-x86_64-linux-gnu.so                                                    
#2  0x00000000005f268a in PyCFunction_Call ()                                              
#3  0x00000000005f31fe in _PyObject_MakeTpCall ()
#4  0x000000000056c50e in _PyEval_EvalFrameDefault ()
#5  0x0000000000507a8e in ?? ()

The issue does not occur when using eProsima Fast-RTPS.

@hidmic hidmic added the bug Something isn't working label Apr 27, 2020
@hidmic hidmic changed the title Subscription segfaults on take when using CycloneDDS Subscription segfaults on take Apr 27, 2020
@hidmic
Copy link
Contributor Author

hidmic commented Apr 27, 2020

Well, it's a regression introduced by #542. I'll make a patch.

@wjwwood
Copy link
Member

wjwwood commented Apr 28, 2020

Thanks for fixing it @hidmic, @iluetkeb FYI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants