posix: cond: check whether condattr is initialized in attr fns

Check whenther a pthread_condattr_t has been initialized in
pthread_condattr_*() functions.

Signed-off-by: Chris Friedt <cfriedt@tenstorrent.com>

Author: cfriedt

lib/posix/options/cond.c

@@ -227,11 +227,18 @@ static int pthread_cond_pool_init(void)
 
 int pthread_condattr_init(pthread_condattr_t *att)
 {
-	__ASSERT_NO_MSG(att != NULL);
-
 	struct posix_condattr *const attr = (struct posix_condattr *)att;
 
+	if (att == NULL) {
+		return EINVAL;
+	}
+	if (attr->initialized) {
+		LOG_DBG("%s %s initialized", "attribute", "already");
+		return EINVAL;
+	}
+
 	attr->clock = CLOCK_MONOTONIC;
+	attr->initialized = true;
 
 	return 0;
 }
@@ -240,7 +247,8 @@ int pthread_condattr_destroy(pthread_condattr_t *att)
 {
 	struct posix_condattr *const attr = (struct posix_condattr *)att;
 
-	if (attr == NULL) {
+	if ((attr == NULL) || !attr->initialized) {
+		LOG_DBG("%s %s initialized", "attribute", "not");
 		return EINVAL;
 	}
 
@@ -254,6 +262,11 @@ int pthread_condattr_getclock(const pthread_condattr_t *ZRESTRICT att,
 {
 	struct posix_condattr *const attr = (struct posix_condattr *)att;
 
+	if ((attr == NULL) || !attr->initialized) {
+		LOG_DBG("%s %s initialized", "attribute", "not");
+		return EINVAL;
+	}
+
 	*clock_id = attr->clock;
 
 	return 0;
@@ -267,6 +280,11 @@ int pthread_condattr_setclock(pthread_condattr_t *att, clockid_t clock_id)
 		return -EINVAL;
 	}
 
+	if ((attr == NULL) || !attr->initialized) {
+		LOG_DBG("%s %s initialized", "attribute", "not");
+		return EINVAL;
+	}
+
 	attr->clock = clock_id;
 
 	return 0;