From da06bd767b0db6cea8b03b0057eee3ce9fd8c6ad Mon Sep 17 00:00:00 2001 From: Qingwei Zhang Date: Fri, 4 Jul 2025 19:57:23 -0400 Subject: [PATCH] fix(osx): consolidate error checks in alt_clock_gettime for clarity --- src/clock.cpp | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/src/clock.cpp b/src/clock.cpp index f0a29ad01e..2a74a99c7d 100644 --- a/src/clock.cpp +++ b/src/clock.cpp @@ -37,9 +37,17 @@ int alt_clock_gettime (int clock_id, timespec *ts) { clock_serv_t cclock; mach_timespec_t mts; - host_get_clock_service (mach_host_self (), clock_id, &cclock); - clock_get_time (cclock, &mts); - mach_port_deallocate (mach_task_self (), cclock); + kern_return_t kr; + + kr = host_get_clock_service (mach_host_self (), clock_id, &cclock); + if (kr == KERN_SUCCESS) { + kr = clock_get_time (cclock, &mts); + mach_port_deallocate (mach_task_self (), cclock); + } + + if (kr != KERN_SUCCESS) + return -1; + ts->tv_sec = mts.tv_sec; ts->tv_nsec = mts.tv_nsec; return 0;