Hi
On Sun, Mar 24, 2019 at 6:50 PM Jakub Janku <jjanku@xxxxxxxxxx> wrote:
>
> 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.
Right, my bad!
However, it should use free() and not g_free(). I'll update the patch.
thanks
>
> 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
--
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel