From 09b850cc6f1b8885ad8e456ba7a7339cdf2092a2 Mon Sep 17 00:00:00 2001 From: Dirk Ziegelmeier Date: Wed, 10 Feb 2016 21:55:50 +0100 Subject: [PATCH] snmp_threadsync.c: Fix ignored return value of mutex/semaphore creation. Found by Coverity. --- src/apps/snmp/snmp_threadsync.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/apps/snmp/snmp_threadsync.c b/src/apps/snmp/snmp_threadsync.c index cc5c0322..9a41f677 100644 --- a/src/apps/snmp/snmp_threadsync.c +++ b/src/apps/snmp/snmp_threadsync.c @@ -199,8 +199,10 @@ snmp_threadsync_get_next_instance(const u32_t *root_oid, u8_t root_oid_len, stru /** Initializes thread synchronization instance */ void snmp_threadsync_init(struct snmp_threadsync_instance *instance, snmp_threadsync_synchronizer_fn sync_fn) { - sys_mutex_new(&instance->sem_usage_mutex); - sys_sem_new(&instance->sem, 0); + err_t err = sys_mutex_new(&instance->sem_usage_mutex); + LWIP_ASSERT("Failed to set up mutex", err == ERR_OK); + err = sys_sem_new(&instance->sem, 0); + LWIP_ASSERT("Failed to set up semaphore", err == ERR_OK); instance->sync_fn = sync_fn; }