Skip to content

Commit d77c06e

Browse files
authored
Clean memory in test_process.cpp (backport ros2#495) (ros2#496)
* Clean memory in test_process.cpp (ros2#495) Signed-off-by: Alejandro Hernandez Cordero <[email protected]> (cherry picked from commit 072ba18)
1 parent b4c01b6 commit d77c06e

File tree

2 files changed

+12
-0
lines changed

2 files changed

+12
-0
lines changed

test/test_join.cpp

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,4 +89,12 @@ TEST(test_join, join) {
8989
EXPECT_EQ(
9090
RCUTILS_RET_OK,
9191
rcutils_string_array_fini(&tokens0));
92+
93+
EXPECT_EQ(
94+
RCUTILS_RET_OK,
95+
rcutils_string_array_fini(&tokens1));
96+
97+
EXPECT_EQ(
98+
RCUTILS_RET_OK,
99+
rcutils_string_array_fini(&tokens2));
92100
}

test/test_process.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,8 @@ TEST(TestProcess, test_process_creation) {
8383

8484
rcutils_process_close(process);
8585

86+
ret = rcutils_string_array_fini(&args);
87+
8688
// cmake -E cat "file with space.txt" (returns 0)
8789
ret = rcutils_string_array_resize(&args, 4);
8890
ASSERT_EQ(RCUTILS_RET_OK, ret);
@@ -100,6 +102,8 @@ TEST(TestProcess, test_process_creation) {
100102

101103
rcutils_process_close(process);
102104

105+
ret = rcutils_string_array_fini(&args);
106+
103107
// cmake -E false (returns 1)
104108
ret = rcutils_string_array_resize(&args, 3);
105109
ASSERT_EQ(RCUTILS_RET_OK, ret);

0 commit comments

Comments
 (0)