Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/netbsd-7-0]: src/sys/kern Pull up following revision(s) (requested by ma...



details:   https://anonhg.NetBSD.org/src/rev/e8c68287dbc3
branches:  netbsd-7-0
changeset: 446623:e8c68287dbc3
user:      martin <martin%NetBSD.org@localhost>
date:      Wed Dec 12 11:37:42 2018 +0000

description:
Pull up following revision(s) (requested by maxv in ticket #1664):

        sys/kern/sys_sig.c: revision 1.47

Fix kernel info leak, 4 bytes of padding in struct _ksiginfo. Maybe we
should just set _pad to zero on LP64?

        + Possible info leak: [len=40, leaked=4]
        | #0 0xffffffff80baf397 in kleak_copyout
        | #1 0xffffffff80bda817 in sigtimedwait1
        | #2 0xffffffff80bdab95 in sys_____sigtimedwait50
        | #3 0xffffffff80259c42 in syscall

diffstat:

 sys/kern/sys_sig.c |  6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diffs (27 lines):

diff -r 75ec8a20fc83 -r e8c68287dbc3 sys/kern/sys_sig.c
--- a/sys/kern/sys_sig.c        Tue Dec 04 19:18:44 2018 +0000
+++ b/sys/kern/sys_sig.c        Wed Dec 12 11:37:42 2018 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: sys_sig.c,v 1.42 2014/02/14 16:35:11 christos Exp $    */
+/*     $NetBSD: sys_sig.c,v 1.42.6.1 2018/12/12 11:37:42 martin Exp $  */
 
 /*-
  * Copyright (c) 2006, 2007, 2008 The NetBSD Foundation, Inc.
@@ -66,7 +66,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: sys_sig.c,v 1.42 2014/02/14 16:35:11 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: sys_sig.c,v 1.42.6.1 2018/12/12 11:37:42 martin Exp $");
 
 #include <sys/param.h>
 #include <sys/kernel.h>
@@ -759,6 +759,8 @@
         */
        sigminusset(&sigcantmask, &l->l_sigwaitset);
 
+       memset(&ksi.ksi_info, 0, sizeof(ksi.ksi_info));
+
        mutex_enter(p->p_lock);
 
        /* Check for pending signals in the process, if no - then in LWP. */



Home | Main Index | Thread Index | Old Index