search

ros2 / rmw_dds_common

Apache License 2.0
11 stars 20 forks source link

Fix memory leaks in test_graph_cache.cpp. #40

Closed clalancette closed 3 years ago

clalancette commented 3 years ago

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

clalancette commented 3 years ago

CI:

y-okumura-isp commented 3 years ago

@clalancette Oops, we did similar things. According to the ASAN report, we may need to finalize more variables. I'm happy if you would check https://github.com/ros2/rmw_dds_common/pull/42.

Thank you.

clalancette commented 3 years ago

@clalancette Oops, we did similar things. According to the ASAN report, we may need to finalize more variables. I'm happy if you would check #42.

Oh, thanks. I'm going to close this one out in favor of that one.