ros2 / rmw_implementation

CMake infrastructure and dependencies for rmw implementations
Apache License 2.0
21 stars 48 forks source link

Set the return value of rmw_ret_t before entering the loop. #171

Closed clalancette closed 3 years ago

clalancette commented 3 years ago

clang static analysis pointed out that if we never did any iterations of the loop in SLEEP_AND_RETRY_UNTIL, then we would be comparing an uninitialized value of rmw_ret_t with RMW_RET_OK. Just set it to RMW_RET_ERROR in this unlikely case which will cause us to fail the test.

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

clalancette commented 3 years ago

CI: