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

ecal udp sender "bad file descriptor" on shutdown #1141

Closed
rex-schilasky opened this issue Jul 17, 2023 · 0 comments · Fixed by #1142
Closed

ecal udp sender "bad file descriptor" on shutdown #1141

rex-schilasky opened this issue Jul 17, 2023 · 0 comments · Fixed by #1142
Assignees
@rex-schilasky
Labels
bug Something isn't working
Milestone
eCAL 5.12

Comments

@rex-schilasky
Copy link
Contributor

Problem Description

In the eCAL shutdown sequence (eCAL::Finalize) there is a sporadic segmentation fault cause by using an udp socket that is closing in another thread.

Screenshot 2023-07-17 122638

How to reproduce

Run eCAL google test pubsub_test on Ubuntu 22.04.

How did you get eCAL?

Custom Build / Built from source

Environment

  • eCAL: current master
  • OS: Ubuntu 22.04

eCAL System Information

No response
@rex-schilasky rex-schilasky self-assigned this Jul 17, 2023
@rex-schilasky rex-schilasky added the bug Something isn't working label Jul 17, 2023
@rex-schilasky rex-schilasky added this to the eCAL 5.12 milestone Jul 17, 2023
rex-schilasky added a commit that referenced this issue Jul 17, 2023
@rex-schilasky
stop cyclic registration thread before sending last (un)registration …
dd3f1e8 
…message and udp socket shutdown (fixing #1141)
@rex-schilasky rex-schilasky mentioned this issue Jul 17, 2023
Merged
2 tasks
rex-schilasky added a commit that referenced this issue Jul 17, 2023
@rex-schilasky
core: stop cyclic registration thread before sending last (un)registr…
56f8751 
…ation message and udp socket shutdown (fixing #1141)
FlorianReimold pushed a commit that referenced this issue Jul 20, 2023
@rex-schilasky @FlorianReimold
core: stop cyclic registration thread before sending last (un)registr…
23bcbc8 
…ation message and udp socket shutdown (fixing #1141)
FlorianReimold pushed a commit that referenced this issue Jul 20, 2023
@rex-schilasky @FlorianReimold
core: stop cyclic registration thread before sending last (un)registr…
1674121 
…ation message and udp socket shutdown (fixing #1141)
FlorianReimold pushed a commit that referenced this issue Jul 20, 2023
@rex-schilasky @FlorianReimold
core: stop cyclic registration thread before sending last (un)registr…
d84d705 
…ation message and udp socket shutdown (fixing #1141)
FlorianReimold pushed a commit that referenced this issue Jul 24, 2023
@rex-schilasky @FlorianReimold
core: stop cyclic registration thread before sending last (un)registr…
db56f0a 
…ation message and udp socket shutdown (fixing #1141)
FlorianReimold pushed a commit that referenced this issue Jul 24, 2023
@rex-schilasky @FlorianReimold
core: stop cyclic registration thread before sending last (un)registr…
4ca9651 
…ation message and udp socket shutdown (fixing #1141)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rex-schilasky rex-schilasky

Labels
bug Something isn't working
Projects
None yet
Milestone
eCAL 5.12
Development

Successfully merging a pull request may close this issue.

hotfix/ecal-udp-sender-shutdown eclipse-ecal/ecal
1 participant
@rex-schilasky