[Dnsmasq-discuss] Make RA_INTERVAL configureable? Deprecate old prefixes?

[Uwe Schindler]

Hi Simon,

I did a "git pull" yesterday evening and problem was already fixed. Many thanks!

Uwe

-----
Uwe Schindler
H.-H.-Meier-Allee 63, D-28213 Bremen
http://www.thetaphi.de
eMail: uwe at thetaphi.de


> -----Original Message-----
> From: Simon Kelley [mailto:simon at thekelleys.org.uk]
> Sent: Monday, July 29, 2013 4:55 PM
> To: Uwe Schindler
> Cc: dnsmasq-discuss at lists.thekelleys.org.uk
> Subject: Re: [Dnsmasq-discuss] Make RA_INTERVAL configureable?
> Deprecate old prefixes?
> 
> On 27/07/13 19:21, Uwe Schindler wrote:
> >
> > Hi,
> >
> > I found the reason for this bug:
> >
> >> One small thing with your new code - maybe a bug, because the comment
> >> says something else: Although the configured (in my config)
> >> valid_lifetime of the prefix is now lower (30 mins on my machine),
> >> after deprecation it raises the valid_lifetime again up to 2 hrs.
> >
> > It misses to apply this code also to context->saved_lifetime in
> > radv.c, approx. line 484:
> >
> > /* configured time is ceiling */ if (!constructed || valid > time)
> > valid = time;
> >
> > So this 3 lines should be moved up, before the part where valid is
> > copied to saved_valid. Or the same code should also be applied to
> > saved_lifetime, if set.
> >
> 
> Got it. That's fixed in the git repo now.
> 
> 
> Simon.