Created attachment 83567 [details] [review]
user: Really don't leak login_history

Created attachment 83568 [details] [review]
daemon: Don't leak struct UserPreviousLogin

Attachment 83567 [details] pushed as d440985 - user: Really don't leak login_history

Thanks, this patch looks good.

Comment on attachment 83568 [details] [review]
daemon: Don't leak struct UserPreviousLogin

Review of attachment 83568 [details] [review]:
-----------------------------------------------------------------

Would prefer something like this instead:

- g_list_free (accounting->previous_logins);
+ g_list_free_full (accounting->previous_logins, user_previous_login_free);

Are you able to make this change? If not let me know.

Created attachment 83840 [details] [review]
daemon: Don't leak struct UserPreviousLogin v2

with g_list_free_full

Thanks. Pushed the second patch.