Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 16 Mar 2016 22:35:56 +0000 (UTC)
From:      Gleb Smirnoff <glebius@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org
Subject:   svn commit: r296957 - stable/10/sys/amd64/amd64
Message-ID:  <201603162235.u2GMZuku046155@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: glebius
Date: Wed Mar 16 22:35:55 2016
New Revision: 296957
URL: https://svnweb.freebsd.org/changeset/base/296957

Log:
  Merge r296956:
  
    Due to invalid use of a signed intermediate value in the bounds checking
    during argument validity verification, unbound zero'ing of the process LDT
    and adjacent memory can be initiated from usermode.
  
  Submitted by:   CORE Security
  Patch by:       kib
  Security:       SA-16:15

Modified:
  stable/10/sys/amd64/amd64/sys_machdep.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/amd64/amd64/sys_machdep.c
==============================================================================
--- stable/10/sys/amd64/amd64/sys_machdep.c	Wed Mar 16 22:33:12 2016	(r296956)
+++ stable/10/sys/amd64/amd64/sys_machdep.c	Wed Mar 16 22:35:55 2016	(r296957)
@@ -591,8 +591,8 @@ amd64_set_ldt(td, uap, descs)
 	struct i386_ldt_args *uap;
 	struct user_segment_descriptor *descs;
 {
-	int error = 0, i;
-	int largest_ld;
+	int error = 0;
+	unsigned int largest_ld, i;
 	struct mdproc *mdp = &td->td_proc->p_md;
 	struct proc_ldt *pldt;
 	struct user_segment_descriptor *dp;



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