kaspar/RIOT
1
0
Fork 0
mirror of https://github.com/RIOT-OS/RIOT.git synced 2025-12-16 10:03:50 +01:00
Code Releases Activity

tests/core/msg_queue_print: fully initialize msg for sending

Browse Source 
On 8-bit and 16-bit platforms `uint32_t` is wider than `void *`, as
pointers are (typically) only 16 bit in size. This causes output like:

    Message queue of thread 2
        size: 8 (avail: 8)
        * 0: sender: 2, type: 0x0000, content: 2701197312 (0)
        * 1: sender: 2, type: 0x0001, content: 2701197313 (0x1)

As seen here, the leading two bytes of `msg.content.value` contain
"random" bits, as those bytes are not explicitly initialized.

This fixes the issue by explicitly initializing the whole `msg_t` via
an initializer list.
This commit is contained in:
Marian Buschsieweke 2025-11-16 23:06:55 +01:00
parent dcf4aa5627
commit 2d475b3dcd
No known key found for this signature in database
GPG Key ID: 77AA882EC78084E6
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tests/core/msg_queue_print/main.c
View File

@ -39,8 +39,10 @@ int main(void)
/* fill message queue */ /* fill message queue */
for (uintptr_t i = 0; i < QUEUE_SIZE; i++) { for (uintptr_t i = 0; i < QUEUE_SIZE; i++) {
msg.type = i; msg = (msg_t) {
msg.content.ptr = (void *) i; .type = i,
.content.ptr = (void *)i,
};
msg_send_to_self(&msg); msg_send_to_self(&msg);
} }
@ -55,8 +57,10 @@ int main(void)
/* fill up message queue again */ /* fill up message queue again */
for (uintptr_t i = QUEUE_SIZE; i < QUEUE_SIZE + QUEUE_SIZE/2; i++) { for (uintptr_t i = QUEUE_SIZE; i < QUEUE_SIZE + QUEUE_SIZE/2; i++) {
msg.type = i; msg = (msg_t) {
msg.content.ptr = (void *) i; .type = i,
.content.ptr = (void *)i,
};
msg_send_to_self(&msg); msg_send_to_self(&msg);
} }