Date: Sat, 16 Jun 2012 14:27:19 -0400 From: Robert Simmons <rsimmons0@gmail.com> To: freebsd-security@freebsd.org Subject: Re: SA-12:04 commit on RELENG_8_1 incorrect? Message-ID: <CA%2BQLa9BKik76a3uX0UCrXiVoLz7ZJ6hExFR2HgkpKTmECinHQQ@mail.gmail.com> In-Reply-To: <4FDCC91A.7080403@pyro.eu.org> References: <4FDCC91A.7080403@pyro.eu.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Jun 16, 2012 at 1:57 PM, Steven Chamberlain <steven@pyro.eu.org> wr= ote: > Hi, > > This was the commit of SA-12:04.sysret to RELENG_7_4, which makes sense > to me: > > http://svnweb.freebsd.org/base/releng/7.4/sys/amd64/amd64/trap.c?r1=3D216= 618&r2=3D236953 > > But when it was applied to RELENG_8_1, it looks wrong, as if it was > applied in the wrong place. =A0The indentation is broken, and the code > inserted looks like it wouldn't be effective: > > http://svnweb.freebsd.org/base/releng/8.1/sys/amd64/amd64/trap.c?revision= =3D236953&view=3Dmarkup#l965 > > Please could someone take a look at this and let me know if it was > really intended. Looks like the if statement was inserted inside the if statement above it by accident. where in the other patch the new if statement occurs before. Certainly looks incorrect to me.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CA%2BQLa9BKik76a3uX0UCrXiVoLz7ZJ6hExFR2HgkpKTmECinHQQ>