lib/utmp.c: get_current_utmp(): Use simple assignment instead of memcpy(3)

memcpy(3) is overkill, and much more dangerous than simple assignment.
Simple assignment adds type safety, and removes any possibility of
buffer overflow due to accidentally specifying a wrong size.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

diff --git a/lib/utmp.c b/lib/utmp.c
index 00fac8ffa9b854d648e801dae4d075c0042d759a..678f9ac4f8197b60088db97c7f9d3dd1cac9a0fc 100644 (file)
--- a/lib/utmp.c
+++ b/lib/utmp.c
@@ -187,7 +187,7 @@ get_current_utmp(pid_t main_pid)
 
                ut_copy = malloc_T(1, struct utmpx);
                if (ut_copy != NULL)
-                       memcpy(ut_copy, ut, sizeof(*ut));
+                       *ut_copy = *ut;
                ut = ut_copy;
        }