From 7c43dd3e09ad7453fd14aae04f43a7505077948b Mon Sep 17 00:00:00 2001 From: delphij Date: Mon, 8 Dec 2014 06:10:47 +0000 Subject: Use calloc() instead of malloc() + bzero(). This also gets rid of a warning because bzero is defined by strings.h which is not included in thread_pool.c. MFC after: 2 weeks --- cddl/compat/opensolaris/misc/thread_pool.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'cddl/compat/opensolaris') diff --git a/cddl/compat/opensolaris/misc/thread_pool.c b/cddl/compat/opensolaris/misc/thread_pool.c index a6a834f..6ae0055 100644 --- a/cddl/compat/opensolaris/misc/thread_pool.c +++ b/cddl/compat/opensolaris/misc/thread_pool.c @@ -233,12 +233,11 @@ tpool_create(uint_t min_threads, uint_t max_threads, uint_t linger, return (NULL); } - tpool = malloc(sizeof (*tpool)); + tpool = calloc(1, sizeof (*tpool)); if (tpool == NULL) { errno = ENOMEM; return (NULL); } - bzero(tpool, sizeof(*tpool)); (void) pthread_mutex_init(&tpool->tp_mutex, NULL); (void) pthread_cond_init(&tpool->tp_busycv, NULL); (void) pthread_cond_init(&tpool->tp_workcv, NULL); @@ -267,9 +266,8 @@ tpool_dispatch(tpool_t *tpool, void (*func)(void *), void *arg) { tpool_job_t *job; - if ((job = malloc(sizeof (*job))) == NULL) + if ((job = calloc(1, sizeof (*job))) == NULL) return (-1); - bzero(job, sizeof(*job)); job->tpj_next = NULL; job->tpj_func = func; job->tpj_arg = arg; -- cgit v1.1