On Fri, Mar 22, 2019 at 4:13 PM <marcandre.lureau@xxxxxxxxxx> wrote:
>
> From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
>
> According to sd_seat_get_active(2), you must free() the pointer.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> ---
> src/vdagentd/systemd-login.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/vdagentd/systemd-login.c b/src/vdagentd/systemd-login.c
> index a11b66d..0b2dd0a 100644
> --- a/src/vdagentd/systemd-login.c
> +++ b/src/vdagentd/systemd-login.c
> @@ -266,6 +266,7 @@ const char *session_info_get_active_session(struct session_info *si)
> int r;
> char *old_session = si->session;
>
> + free(si->session);
The session does get freed just a couple of lines below here.
Cheers,
Jakub
> si->session = NULL;
> r = sd_seat_get_active("seat0", &si->session, NULL);
> /* ENOENT happens when a seat is switching from one session to another */
> --
> 2.21.0.4.g36eb1cb9cf
>
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel