[systemd-devel] [PATCH] sd-dhcp-client: prevent timer related memory leaks
Umut Tezduyar Lindskog
umut.tezduyar at axis.com
Thu Feb 20 23:48:20 PST 2014
> -----Original Message-----
> From: Lennart Poettering [mailto:lennart at poettering.net]
> Sent: den 21 februari 2014 03:22
> To: Umut Tezduyar Lindskog
> Cc: systemd-devel at lists.freedesktop.org; Umut Tezduyar Lindskog
> Subject: Re: [systemd-devel] [PATCH] sd-dhcp-client: prevent timer related
> memory leaks
>
> On Thu, 20.02.14 21:04, Umut Tezduyar Lindskog (umut.tezduyar at axis.com)
> wrote:
>
> Tom commited this. Thanks!
>
> Tom, please always do a quick reply on the ML so that it is easy to see what is
> commited and what is not!
>
> Thanks!
Hi, actually he did but maybe there is an oversight. I kept track of it too :)
Umut
>
> > ---
> > src/libsystemd-dhcp/sd-dhcp-client.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/src/libsystemd-dhcp/sd-dhcp-client.c
> > b/src/libsystemd-dhcp/sd-dhcp-client.c
> > index ec2b53f..53abe22 100644
> > --- a/src/libsystemd-dhcp/sd-dhcp-client.c
> > +++ b/src/libsystemd-dhcp/sd-dhcp-client.c
> > @@ -392,6 +392,8 @@ static int client_timeout_resend(sd_event_source
> > *s, uint64_t usec,
> >
> > next_timeout += (random_u32() & 0x1fffff);
> >
> > + client->timeout_resend =
> > + sd_event_source_unref(client->timeout_resend);
> > +
> > r = sd_event_add_monotonic(client->event,
> > &client->timeout_resend,
> > next_timeout, @@ -477,6 +479,8
> > @@ static int client_initialize_events(sd_dhcp_client *client,
> > if (r < 0)
> > goto error;
> >
> > + client->timeout_resend =
> > + sd_event_source_unref(client->timeout_resend);
> > +
> > r = sd_event_add_monotonic(client->event,
> > &client->timeout_resend,
> > usec, 0,
>
>
> Lennart
>
> --
> Lennart Poettering, Red Hat
More information about the systemd-devel
mailing list