Author: delphij
Date: Wed Jan 27 07:28:55 2016
New Revision: 294901
URL: https://svnweb.freebsd.org/changeset/base/294901

Log:
  MFC r294900:
  
  Implement AT_SECURE properly.
  
  AT_SECURE auxv entry has been added to the Linux 2.5 kernel to pass a
  boolean flag indicating whether secure mode should be enabled. 1 means
  that the program has changes its credentials during the execution.
  Being exported AT_SECURE used by glibc issetugid() call.
  
  Submitted by: imp, dchagin
  Security:     FreeBSD-SA-16:10.linux
  Security:     CVE-2016-1883

Modified:
  stable/10/sys/amd64/linux/linux_sysvec.c
  stable/10/sys/amd64/linux32/linux32_sysvec.c
  stable/10/sys/i386/linux/linux_sysvec.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/amd64/linux/linux_sysvec.c
==============================================================================
--- stable/10/sys/amd64/linux/linux_sysvec.c    Wed Jan 27 07:20:55 2016        
(r294900)
+++ stable/10/sys/amd64/linux/linux_sysvec.c    Wed Jan 27 07:28:55 2016        
(r294901)
@@ -271,6 +271,7 @@ elf_linux_fixup(register_t **stack_base,
        Elf_Addr *pos;
        struct ps_strings *arginfo;
        struct proc *p;
+       int issetugid;
 
        p = imgp->proc;
        arginfo = (struct ps_strings *)p->p_sysent->sv_psstrings;
@@ -281,6 +282,7 @@ elf_linux_fixup(register_t **stack_base,
        args = (Elf64_Auxargs *)imgp->auxargs;
        pos = base + (imgp->args->argc + imgp->args->envc + 2);
 
+       issetugid = p->p_flag & P_SUGID ? 1 : 0;
        AUXARGS_ENTRY(pos, LINUX_AT_SYSINFO_EHDR,
            imgp->proc->p_sysent->sv_shared_page_base);
        AUXARGS_ENTRY(pos, LINUX_AT_HWCAP, cpu_feature);
@@ -296,7 +298,7 @@ elf_linux_fixup(register_t **stack_base,
        AUXARGS_ENTRY(pos, AT_EUID, imgp->proc->p_ucred->cr_svuid);
        AUXARGS_ENTRY(pos, AT_GID, imgp->proc->p_ucred->cr_rgid);
        AUXARGS_ENTRY(pos, AT_EGID, imgp->proc->p_ucred->cr_svgid);
-       AUXARGS_ENTRY(pos, LINUX_AT_SECURE, 0);
+       AUXARGS_ENTRY(pos, LINUX_AT_SECURE, issetugid);
        AUXARGS_ENTRY(pos, LINUX_AT_PLATFORM, PTROUT(linux_platform));
        AUXARGS_ENTRY(pos, LINUX_AT_RANDOM, imgp->canary);
        if (imgp->execpathp != 0)

Modified: stable/10/sys/amd64/linux32/linux32_sysvec.c
==============================================================================
--- stable/10/sys/amd64/linux32/linux32_sysvec.c        Wed Jan 27 07:20:55 
2016        (r294900)
+++ stable/10/sys/amd64/linux32/linux32_sysvec.c        Wed Jan 27 07:28:55 
2016        (r294901)
@@ -230,6 +230,7 @@ elf_linux_fixup(register_t **stack_base,
        Elf32_Addr *base;
        Elf32_Addr *pos;
        struct linux32_ps_strings *arginfo;
+       int issetugid;
 
        arginfo = (struct linux32_ps_strings *)LINUX32_PS_STRINGS;
 
@@ -239,6 +240,7 @@ elf_linux_fixup(register_t **stack_base,
        args = (Elf32_Auxargs *)imgp->auxargs;
        pos = base + (imgp->args->argc + imgp->args->envc + 2);
 
+       issetugid = imgp->proc->p_flag & P_SUGID ? 1 : 0;
        AUXARGS_ENTRY_32(pos, LINUX_AT_SYSINFO_EHDR,
            imgp->proc->p_sysent->sv_shared_page_base);
        AUXARGS_ENTRY_32(pos, LINUX_AT_SYSINFO, linux32_vsyscall);
@@ -261,7 +263,7 @@ elf_linux_fixup(register_t **stack_base,
        AUXARGS_ENTRY_32(pos, AT_FLAGS, args->flags);
        AUXARGS_ENTRY_32(pos, AT_ENTRY, args->entry);
        AUXARGS_ENTRY_32(pos, AT_BASE, args->base);
-       AUXARGS_ENTRY_32(pos, LINUX_AT_SECURE, 0);
+       AUXARGS_ENTRY_32(pos, LINUX_AT_SECURE, issetugid);
        AUXARGS_ENTRY_32(pos, AT_UID, imgp->proc->p_ucred->cr_ruid);
        AUXARGS_ENTRY_32(pos, AT_EUID, imgp->proc->p_ucred->cr_svuid);
        AUXARGS_ENTRY_32(pos, AT_GID, imgp->proc->p_ucred->cr_rgid);

Modified: stable/10/sys/i386/linux/linux_sysvec.c
==============================================================================
--- stable/10/sys/i386/linux/linux_sysvec.c     Wed Jan 27 07:20:55 2016        
(r294900)
+++ stable/10/sys/i386/linux/linux_sysvec.c     Wed Jan 27 07:28:55 2016        
(r294901)
@@ -238,11 +238,13 @@ elf_linux_fixup(register_t **stack_base,
        Elf32_Addr *uplatform;
        struct ps_strings *arginfo;
        register_t *pos;
+       int issetugid;
 
        KASSERT(curthread->td_proc == imgp->proc,
            ("unsafe elf_linux_fixup(), should be curproc"));
 
        p = imgp->proc;
+       issetugid = imgp->proc->p_flag & P_SUGID ? 1 : 0;
        arginfo = (struct ps_strings *)p->p_sysent->sv_psstrings;
        uplatform = (Elf32_Addr *)((caddr_t)arginfo - linux_szplatform);
        args = (Elf32_Auxargs *)imgp->auxargs;
@@ -270,7 +272,7 @@ elf_linux_fixup(register_t **stack_base,
        AUXARGS_ENTRY(pos, AT_FLAGS, args->flags);
        AUXARGS_ENTRY(pos, AT_ENTRY, args->entry);
        AUXARGS_ENTRY(pos, AT_BASE, args->base);
-       AUXARGS_ENTRY(pos, LINUX_AT_SECURE, 0);
+       AUXARGS_ENTRY(pos, LINUX_AT_SECURE, issetugid);
        AUXARGS_ENTRY(pos, AT_UID, imgp->proc->p_ucred->cr_ruid);
        AUXARGS_ENTRY(pos, AT_EUID, imgp->proc->p_ucred->cr_svuid);
        AUXARGS_ENTRY(pos, AT_GID, imgp->proc->p_ucred->cr_rgid);
_______________________________________________
svn-src-stable-10@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
To unsubscribe, send any mail to "svn-src-stable-10-unsubscr...@freebsd.org"

Reply via email to