logging: Initialize 'chosen' pointer in z_log_msg_claim_oldest to NULL

Jacob McClellan · mmahadevan108 · commit 39042dc9cccf · 2024-10-30T14:28:52.000-05:00
Fixes a compiler warning 'chosen' may be uninitialized when later
dereferenced

Signed-off-by: Jacob McClellan &lt;mcclellanjacob@meta.com&gt;
diff --git a/subsys/logging/log_core.c b/subsys/logging/log_core.c
@@ -705,7 +705,7 @@ union log_msg_generic *z_log_msg_local_claim(void)
 union log_msg_generic *z_log_msg_claim_oldest(k_timeout_t *backoff)
 {
 	union log_msg_generic *msg = NULL;
-	struct log_msg_ptr *chosen;
+	struct log_msg_ptr *chosen = NULL;
 	log_timestamp_t t_min = sizeof(log_timestamp_t) > sizeof(uint32_t) ?
 				UINT64_MAX : UINT32_MAX;
 	int i = 0;

Original file line number	Diff line number	Diff line change
`@@ -705,7 +705,7 @@ union log_msg_generic *z_log_msg_local_claim(void)`
`705`	`705`	`union log_msg_generic z_log_msg_claim_oldest(k_timeout_t backoff)`
`706`	`706`	`{`
`707`	`707`	`union log_msg_generic *msg = NULL;`
`708`		`- struct log_msg_ptr *chosen;`
	`708`	`+ struct log_msg_ptr *chosen = NULL;`
`709`	`709`	`log_timestamp_t t_min = sizeof(log_timestamp_t) > sizeof(uint32_t) ?`
`710`	`710`	`UINT64_MAX : UINT32_MAX;`
`711`	`711`	`int i = 0;`