magazino / move_base_flex

Move Base Flex: a backwards-compatible replacement for move_base
https://uos.github.io/mbf_docs/
BSD 3-Clause "New" or "Revised" License
422 stars 154 forks source link

fix(mbf_abstract_nav): check for empty path to prevent segfault #314

Closed mbeutelspacher closed 8 months ago

mbeutelspacher commented 1 year ago

Otherwise https://github.com/magazino/move_base_flex/blob/master/mbf_abstract_nav/src/controller_action.cpp#L87 will segfault when the path is empty

corot commented 8 months ago

@mbeutelspacher, did you find out why MBF was crashing in your case? For the reasons we discussed above, I feel inclined to close this PR, but always thanks for the contribution!