On Mon, Jan 25, 2021 at 4:28 PM Christian Ehrhardt
<christian.ehrha...@canonical.com> wrote:
>
> On Thu, Sep 17, 2020 at 8:57 AM Christian Ehrhardt
> <christian.ehrha...@canonical.com> wrote:
> >
> > Hi,
> > since nothing happened on this topic (e.g. committed to git / further
> > discussion) I wondered if you waited for a confirmation.
> >
> > The first patch by Iain was ignored or not seen, I'd expect both might work.
> > Nevertheless in case there was any (hidden) reason I picked the second
> > patch suggested at:
> > http://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/2020q3/014346.html
> >
> > I built dnsmasq 2.82 with it and ran the various Ubuntu tests against it.
> > I wanted to state that this made it fully work again from my POV and
> > hope that it can be considered to be applied.
> >
>
> @Simon - Re-ping

And re-ping again we are at 2.85 and Ubuntu still has to carry this
patch that came up in the original discussion:

diff --git a/src/radv.c b/src/radv.c
index 3255904b..d10f2b6b 100644
--- a/src/radv.c
+++ b/src/radv.c
@@ -628,8 +628,9 @@ static int add_prefixes(struct in6_addr *local,  int prefix,

  /* find floor time, don't reduce below 3 * RA interval.
     If the lease time has been left as default, don't
-    use that as a floor. */
- if ((context->flags & CONTEXT_SETLEASE) &&
+    use that as a floor.
+    Always set lease time if requested to do so. */
+ if ((context->flags & CONTEXT_SETLEASE) ||
      time > context->lease_time)
    {
      time = context->lease_time;

History on this
Submitted and Discussed:
 https://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/2020q3/014341.html
Pinged:
 https://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/2020q3/014387.html
 https://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/2021q1/014639.html



> We now have had 2.83 (released for CVEs) so some activity is happening
> again I guess.
> So I thought it might be worth to ping on this "old issue with known
> fix" once more.
>
> > P.S. I wasn't subscribed before so I can't reply directly, due to that
> > it might be broken up in the tree view of the ML archive, sorry for
> > that
> >
> > --
> > Christian Ehrhardt
> > Staff Engineer, Ubuntu Server
> > Canonical Ltd
>
>
>
> --
> Christian Ehrhardt
> Staff Engineer, Ubuntu Server
> Canonical Ltd



--
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd

_______________________________________________
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
https://lists.thekelleys.org.uk/cgi-bin/mailman/listinfo/dnsmasq-discuss

Reply via email to