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).

3 files changed, 4 insertions, 4 deletions

diff --git a/render/allocator/drm_dumb.c b/render/allocator/drm_dumb.c
index 03ff8e3f..6de44f40 100644
--- a/render/allocator/drm_dumb.c
+++ b/render/allocator/drm_dumb.c
@@ -129,7 +129,7 @@ static void drm_dumb_buffer_end_data_ptr_access(struct wlr_buffer *wlr_buffer) {
 static bool buffer_get_dmabuf(struct wlr_buffer *wlr_buffer,
 		struct wlr_dmabuf_attributes *attribs) {
 	struct wlr_drm_dumb_buffer *buf = drm_dumb_buffer_from_buffer(wlr_buffer);
-	memcpy(attribs, &buf->dmabuf, sizeof(buf->dmabuf));
+	*attribs = buf->dmabuf;
 	return true;
 }
 
diff --git a/render/allocator/gbm.c b/render/allocator/gbm.c
index 32636d30..25ddbbae 100644
--- a/render/allocator/gbm.c
+++ b/render/allocator/gbm.c
@@ -79,7 +79,7 @@ static bool export_gbm_bo(struct gbm_bo *bo,
 		attribs.stride[i] = gbm_bo_get_stride_for_plane(bo, i);
 	}
 
-	memcpy(out, &attribs, sizeof(attribs));
+	*out = attribs;
 	return true;
 
 error_fd:
@@ -168,7 +168,7 @@ static bool buffer_get_dmabuf(struct wlr_buffer *wlr_buffer,
 		struct wlr_dmabuf_attributes *attribs) {
 	struct wlr_gbm_buffer *buffer =
 		get_gbm_buffer_from_buffer(wlr_buffer);
-	memcpy(attribs, &buffer->dmabuf, sizeof(buffer->dmabuf));
+	*attribs = buffer->dmabuf;
 	return true;
 }
 
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;
 }