From 00aaea733070db83f03b206199a4a57bc23d1f26 Mon Sep 17 00:00:00 2001 From: Jorge Perez <jjperez@ekumenlabs.com> Date: Tue, 7 Apr 2020 16:50:03 -0300 Subject: [PATCH] Add missing allocator check for NULL (#606) Signed-off-by: Jorge Perez <jjperez@ekumenlabs.com> --- rcl/src/rcl/time.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/rcl/src/rcl/time.c b/rcl/src/rcl/time.c index 9172050e0..a050a008c 100644 --- a/rcl/src/rcl/time.c +++ b/rcl/src/rcl/time.c @@ -146,6 +146,11 @@ rcl_ros_clock_init( RCL_CHECK_ARGUMENT_FOR_NULL(allocator, RCL_RET_INVALID_ARGUMENT); rcl_init_generic_clock(clock); clock->data = allocator->allocate(sizeof(rcl_ros_clock_storage_t), allocator->state); + if (NULL == clock->data) { + RCL_SET_ERROR_MSG("allocating memory failed"); + return RCL_RET_BAD_ALLOC; + } + rcl_ros_clock_storage_t * storage = (rcl_ros_clock_storage_t *)clock->data; // 0 is a special value meaning time has not been set atomic_init(&(storage->current_time), 0);