diff options
author | Simon Ser <contact@emersion.fr> | 2023-08-03 12:07:27 +0200 |
---|---|---|
committer | Alexander Orzechowski <alex@ozal.ski> | 2023-08-03 14:40:28 +0000 |
commit | c74f89d4f84bfed0284d3908aee5d207698c70c5 (patch) | |
tree | 6ded2e7ac9010daf75437ffd6ad45f0659412bbf /render/allocator/shm.c | |
parent | 77dc1c28aa551616521b60f1a8727a25a45f82e1 (diff) |
Avoid using memcpy() to copy structs
We can just use a regular assignment instead. This is more
type-safe since there is no need to provide the struct size.
The remaining memcpy() calls perform array copies or copies from
void pointers (which may be unaligned).
Diffstat (limited to 'render/allocator/shm.c')
-rw-r--r-- | render/allocator/shm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/render/allocator/shm.c b/render/allocator/shm.c index b6d3138c..2622f99a 100644 --- a/render/allocator/shm.c +++ b/render/allocator/shm.c @@ -31,7 +31,7 @@ static void buffer_destroy(struct wlr_buffer *wlr_buffer) { static bool buffer_get_shm(struct wlr_buffer *wlr_buffer, struct wlr_shm_attributes *shm) { struct wlr_shm_buffer *buffer = shm_buffer_from_buffer(wlr_buffer); - memcpy(shm, &buffer->shm, sizeof(*shm)); + *shm = buffer->shm; return true; } |