Skip to content

Fixes issue #1015: UDP server crash #1265

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
Open
tarunkartik wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fixes issue #1015: UDP server crash #1265

tarunkartik wants to merge 1 commit into from
+14 −1

Conversation

@tarunkartik
Copy link

@tarunkartik tarunkartik commented Jan 14, 2026

Summary

Fixes issue #1015 where PlotJuggler crashes when trying to start a UDP server.

  • Was not able to recreate the exact scenario that reporter experienced but the stack trace in the video shows that the error was an std::out_of_range exception within UDP::start().
  • The only possible location that can happen is when the at() function is called on the parserFactories()

Changes

  • Replaced the call to at() with a safer lookup by calling find() inside the lambda for onComboChanged()
  • Provided a fallback in case the selected_protocol does not exist

Testing

  • Built in docker container
  • Manually tested UDP streaming
@facontidavide
Copy link
Owner

facontidavide commented Jan 21, 2026

nice ,but I don't thing that we should default to another one. we should just close the connection, IMO
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@tarunkartik @facontidavide