michaelh/zephyr
1
0
Fork 0
Code Releases Activity

logging: Add fields initialization in z_log_msg_std_alloc

Browse source 
Added initialization of ids fields in the log message in
z_log_msg_std_alloc(). If user space is enabled then msg_free()
function is reading log level and previously it may have been
uninitialized.

Signed-off-by: Krzysztof Chruscinski <krzysztof.chruscinski@nordicsemi.no>
This commit is contained in:
Krzysztof Chruscinski 2020-07-01 07:48:45 +02:00 • committed by Carles Cufí
commit 2809cad50e
1 changed files with 8 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

8
include/logging/log_msg.h
View file

@ -7,6 +7,7 @@
#define ZEPHYR_INCLUDE_LOGGING_LOG_MSG_H_
#include <sys/atomic.h>
#include <sys/util.h>
#include <string.h>
#ifdef __cplusplus
@ -326,6 +327,13 @@ static inline struct log_msg *z_log_msg_std_alloc(void)
msg->hdr.ref_cnt = 1;
msg->hdr.params.raw = 0U;
msg->hdr.params.std.type = LOG_MSG_TYPE_STD;
if (IS_ENABLED(CONFIG_USERSPACE)) {
/* it may be used in msg_free() function. */
msg->hdr.ids.level = 0;
msg->hdr.ids.domain_id = 0;
msg->hdr.ids.source_id = 0;
}
}
return msg;