Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 23 May 2017 09:41:36 +0100
From:      Roger Pau =?iso-8859-1?Q?Monn=E9?= <royger@FreeBSD.org>
To:        Nick Hibma <n_hibma@FreeBSD.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org
Subject:   Re: svn commit: r318630 - stable/11/sbin/dhclient
Message-ID:  <20170523084136.m4apjlyyvpaoqpwt@dhcp-3-128.uk.xensource.com>
In-Reply-To: <201705221028.v4MASHNe041436@repo.freebsd.org>
References:  <201705221028.v4MASHNe041436@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, May 22, 2017 at 10:28:17AM +0000, Nick Hibma wrote:
> Author: n_hibma
> Date: Mon May 22 10:28:17 2017
> New Revision: 318630
> URL: https://svnweb.freebsd.org/changeset/base/318630
> 
> Log:
>   MFC:
>   
>   	------------------------------------------------------------------------
>   	r317923 | n_hibma | 2017-05-07 23:11:28 +0200 (Sun, 07 May 2017) | 8 lines
>   
>   	Fix the output of very large rebind, renew and lease time options in
>   	lease file.
>   
>   	Some routers set very large values for rebind time (Netgear) and these
>   	are erroneously reported as negative in the leasefile. This was due to a
>   	wrong printf format specification of %ld for an unsigned long on 32-bit
>   	platforms.
>   
>   	------------------------------------------------------------------------
>   	r317915 | n_hibma | 2017-05-07 21:59:37 +0200 (Sun, 07 May 2017) | 16 lines
>   
>   	Fix handling of large DHCP expiry values.
>   
>   	They would overflow a signed 32-bit time_t on 32 bit architectures. This
>   	was taken care of, but a compiler optimisation makes this behave
>   	erratically. This could be resolved by adding a -fwrapv flag, but
>   	instead we can check the value before adding the current timestamp to
>   	it.
>   
>   	In the lease file values are still wrong though:
>   
>   		option dhcp-rebinding-time -644245096;
>   
>   PR:		218980
> 
> Modified:
>   stable/11/sbin/dhclient/dhclient.c
>   stable/11/sbin/dhclient/options.c

This is missing the mergeinfo in stable/11.

Roger.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20170523084136.m4apjlyyvpaoqpwt>