From 96e68c393f65046c0cf69d29f89efcfee949e5de Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dag-Erling=20Sm=C3=B8rgrav?= Date: Fri, 13 Jan 2023 15:57:19 +0100 Subject: [PATCH] tzcode: Avoid memory leak if pthread_setspecific() fails. Reported by: Coverity (CID 1018472, 1018474) MFC after: 1 week Sponsored by: Klara, Inc. Differential Revision: https://reviews.freebsd.org/D38036 --- contrib/tzcode/localtime.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/contrib/tzcode/localtime.c b/contrib/tzcode/localtime.c index 30ae18a965e8..1b52e26a5faf 100644 --- a/contrib/tzcode/localtime.c +++ b/contrib/tzcode/localtime.c @@ -1761,7 +1761,10 @@ localtime(const time_t *timep) if ((p_tm = malloc(sizeof(*p_tm))) == NULL) { return (NULL); } - _pthread_setspecific(localtime_key, p_tm); + if (_pthread_setspecific(localtime_key, p_tm) != 0) { + free(p_tm); + return (NULL); + } } } return localtime_tzset(timep, p_tm, true); @@ -1829,7 +1832,10 @@ gmtime(const time_t *timep) if ((p_tm = malloc(sizeof(*p_tm))) == NULL) { return (NULL); } - _pthread_setspecific(gmtime_key, p_tm); + if (_pthread_setspecific(gmtime_key, p_tm) != 0) { + free(p_tm); + return (NULL); + } } } return gmtime_r(timep, p_tm);