Tweaks to client.c and subscription.c for cleaner init/fini

Signed-off-by: Stephen Brawner <brawner@gmail.com>
ros2 · Jul 29, 2020 · 53052d9 · 53052d9
1 parent af438bc
commit 53052d9
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 0 deletions.
diff --git a/rcl/src/rcl/client.c b/rcl/src/rcl/client.c
@@ -223,6 +223,7 @@ rcl_client_fini(rcl_client_t * client, rcl_node_t * node)
       result = RCL_RET_ERROR;
     }
     allocator.deallocate(client->impl, allocator.state);
+    client->impl = NULL;
   }
   RCUTILS_LOG_DEBUG_NAMED(ROS_PACKAGE_NAME, "Client finalized");
   return result;

diff --git a/rcl/src/rcl/publisher.c b/rcl/src/rcl/publisher.c
@@ -236,6 +236,7 @@ rcl_publisher_fini(rcl_publisher_t * publisher, rcl_node_t * node)
       result = RCL_RET_ERROR;
     }
     allocator.deallocate(publisher->impl, allocator.state);
+    publisher->impl = NULL;
   }
   RCUTILS_LOG_DEBUG_NAMED(ROS_PACKAGE_NAME, "Publisher finalized");
   return result;

diff --git a/rcl/src/rcl/service.c b/rcl/src/rcl/service.c
@@ -228,6 +228,7 @@ rcl_service_fini(rcl_service_t * service, rcl_node_t * node)
       result = RCL_RET_ERROR;
     }
     allocator.deallocate(service->impl, allocator.state);
+    service->impl = NULL;
   }
   RCUTILS_LOG_DEBUG_NAMED(ROS_PACKAGE_NAME, "Service finalized");
   return result;

diff --git a/rcl/src/rcl/subscription.c b/rcl/src/rcl/subscription.c
@@ -194,6 +194,7 @@ rcl_subscription_init(
 fail:
   if (subscription->impl) {
     allocator->deallocate(subscription->impl, allocator->state);
+    subscription->impl = NULL;
   }
   ret = fail_ret;
   // Fall through to cleanup
@@ -229,6 +230,7 @@ rcl_subscription_fini(rcl_subscription_t * subscription, rcl_node_t * node)
       result = RCL_RET_ERROR;
     }
     allocator.deallocate(subscription->impl, allocator.state);
+    subscription->impl = NULL;
   }
   RCUTILS_LOG_DEBUG_NAMED(ROS_PACKAGE_NAME, "Subscription finalized");
   return result;