Do not deref msg ptr for rmw_{publish,return}_loaned_message*() (#240)

Signed-off-by: Christophe Bedard <christophe.bedard@apex.ai>
ros2 · Aug 27, 2024 · b103174 · b103174
1 parent e01f783
commit b103174
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/test_rmw_implementation/test/test_publisher.cpp b/test_rmw_implementation/test/test_publisher.cpp
@@ -537,11 +537,11 @@ class TestPublisherUseLoan : public TestPublisherUse
       EXPECT_EQ(RMW_RET_UNSUPPORTED, ret) << rmw_get_error_string().str;
       rmw_reset_error();
       EXPECT_EQ(nullptr, msg_pointer);
-      ret = rmw_return_loaned_message_from_publisher(pub, &msg_pointer);
+      ret = rmw_return_loaned_message_from_publisher(pub, msg_pointer);
       EXPECT_EQ(RMW_RET_UNSUPPORTED, ret) << rmw_get_error_string().str;
       rmw_reset_error();
       EXPECT_EQ(nullptr, msg_pointer);
-      ret = rmw_publish_loaned_message(pub, &msg_pointer, null_allocation);
+      ret = rmw_publish_loaned_message(pub, msg_pointer, null_allocation);
       EXPECT_EQ(RMW_RET_UNSUPPORTED, ret) << rmw_get_error_string().str;
       rmw_reset_error();
       EXPECT_EQ(nullptr, msg_pointer);