Fix memory leak in test_publisher, test msg not released (#469)

y-okumura-isp · y-okumura-isp · commit 9435ea96a11d · 2020-02-03T17:31:26.000+09:00
Signed-off-by: y-okumura-isp &lt;y-okumura@isp.co.jp&gt;
diff --git a/rcl/test/rcl/test_publisher.cpp b/rcl/test/rcl/test_publisher.cpp
@@ -168,6 +168,7 @@ TEST_F(CLASSNAME(TestPublisherFixture, RMW_IMPLEMENTATION), test_publishers_diff
   test_msgs__msg__Strings__init(&msg_string);
   ASSERT_TRUE(rosidl_generator_c__String__assign(&msg_string.string_value, "testing"));
   ret = rcl_publish(&publisher_in_namespace, &msg_string, nullptr);
+  test_msgs__msg__Strings__fini(&msg_string);
   ASSERT_EQ(RCL_RET_OK, ret) << rcl_get_error_string().str;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -168,6 +168,7 @@ TEST_F(CLASSNAME(TestPublisherFixture, RMW_IMPLEMENTATION), test_publishers_diff`
`168`	`168`	`test_msgs__msg__Strings__init(&msg_string);`
`169`	`169`	`ASSERT_TRUE(rosidl_generator_c__String__assign(&msg_string.string_value, "testing"));`
`170`	`170`	`ret = rcl_publish(&publisher_in_namespace, &msg_string, nullptr);`
	`171`	`+ test_msgs__msg__Strings__fini(&msg_string);`
`171`	`172`	`ASSERT_EQ(RCL_RET_OK, ret) << rcl_get_error_string().str;`
`172`	`173`	`}`
`173`	`174`