Aren/NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Range checks are not optional. The omission of this one, allows the user

Browse Source 
to corrupt the heap and/or crash the kernel (Christer Oeberg).
This commit is contained in:
christos 2005-09-12 22:17:49 +00:00
parent 6a2a989266
commit 49110e63f0
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
sys/compat/freebsd/freebsd_misc.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: freebsd_misc.c,v 1.21 2005/02/26 23:10:18 perry Exp $ */
/* $NetBSD: freebsd_misc.c,v 1.22 2005/09/12 22:17:49 christos Exp $ */
/*
* Copyright (c) 1995 Frank van der Linden
@ -36,7 +36,7 @@
*/
#include <sys/cdefs.h>
__KERNEL_RCSID(0, "$NetBSD: freebsd_misc.c,v 1.21 2005/02/26 23:10:18 perry Exp $");
__KERNEL_RCSID(0, "$NetBSD: freebsd_misc.c,v 1.22 2005/09/12 22:17:49 christos Exp $");
#if defined(_KERNEL_OPT)
#include "opt_ntp.h"
@ -232,11 +232,15 @@ freebsd_sys_utrace(l, v, retval)
struct proc *p = l->l_proc;
if (KTRPOINT(p, KTR_USER))
ktruser(p, "FreeBSD utrace", SCARG(uap, addr), SCARG(uap, len),
0);
return 0;
return (0);
if (SCARG(uap, len) > KTR_USER_MAXLEN)
return EINVAL;
ktruser(p, "FreeBSD utrace", SCARG(uap, addr), SCARG(uap, len), 0);
return 0;
#else
return (ENOSYS);
return ENOSYS;
#endif
}