Package: dnsmasq
Version: 2.85-1

Hi,

This isn't a new topic at all, but I wonder what else I could do to raise
awareness.

It was initially submitted and discussed:
 https://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/2020q3/014341.html
I pinged about the case whenever I touched dnsmasq in Ubuntu:
 https://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/2020q3/014387.html
 https://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/2021q1/014639.html

The discussion was abit back and forth but I thought that eventually the
change suggested by Dominik dl6er at dl6er.de seemed to be simple and working.
Since then Ubuntu carries this as:

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;

Any chance to apply that upstream and/or in Debian?
Or did I miss a part of the dsicussion that identified this as being wrong?

-- 
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd

Reply via email to