Skip to content

Fix: crash on interface listing, build log paths correctly#1

Open
russkel wants to merge 3 commits intoadvanced-navigation:mainfrom
Greenroom-Robotics:main
Open

Fix: crash on interface listing, build log paths correctly#1
russkel wants to merge 3 commits intoadvanced-navigation:mainfrom
Greenroom-Robotics:main

Conversation

@russkel
Copy link

@russkel russkel commented Nov 10, 2025

Hello,

This should be included as part of advanced-navigation/ros2-driver#17.

Fixes:

  • The driver would crash before connecting due to the presence of some different types of interfaces.
  • Only disconnect if connected to prevent exception during deconstruction
  • Log paths are now concatted using std::filesystem::path and now build correctly if they do not have a trailing /.

Thanks,

Russ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant