Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 12 Aug 2014 15:29:14 -0500
From:      Pedro Giffuni <pfg@freebsd.org>
To:        Hajimu UMEMOTO <ume@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r269873 - head/lib/libc/nameser
Message-ID:  <7608F6C6-3E7C-4D0B-B71A-BBFED422C7B6@freebsd.org>
In-Reply-To: <53ea120c.6165.1de1e179@svn.freebsd.org>
References:  <53ea120c.6165.1de1e179@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

Il giorno 12/ago/2014, alle ore 08:09, Hajimu UMEMOTO <ume@FreeBSD.org> =
ha scritto:

> Author: ume
> Date: Tue Aug 12 13:09:32 2014
> New Revision: 269873
> URL: http://svnweb.freebsd.org/changeset/base/269873
>=20
> Log:
>  Fix broken pointer overflow check ns_name_unpack()
>=20
>  Many compilers may optimize away the overflow check `msg + l < msg',
>  where `msg' is a pointer and `l' is an integer, because pointer
>  overflow is undefined behavior in C.
>=20
>  Use a safe precondition test `l >=3D eom - msg' instead.
>=20
>  Reference:
>  https://android-review.googlesource.com/#/c/50570/
>=20
>  Requested by:	pfg
>  Obtained from:	NetBSD (CVS rev. 1.10)
>=20

Thanks!

Pedro.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?7608F6C6-3E7C-4D0B-B71A-BBFED422C7B6>