Skip to content

Commit c1558f7

Browse files
author
Fangyi Zhou
committed
ipc: Use memmove instead of memcpy
Memcpy may trigger undefined behaviour when there is an overlap between two memory regions. Using memmove is safer.
1 parent e557afa commit c1558f7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

core/system/src/ipc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ static int ipc_deliver(uvisor_ipc_t * send_ipc, uvisor_ipc_t * recv_ipc,
142142
}
143143

144144
size_t len = send_desc->len;
145-
memcpy(recv_io->msg, send_io->msg, len);
145+
memmove(recv_io->msg, send_io->msg, len);
146146
send_ipc->completed_tokens |= send_desc->token;
147147

148148
recv_desc->box_id = send_box_id;

0 commit comments

Comments
 (0)