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

Remove, in do_sys_renameat(), wrong KASSERTs that check for non NULL

Browse Source 
from/to arguments. Such values are correctly handled by later
pathbuf_maybe_copyin() calls, that will fail with EFAULT.

ok from dholland@.
This commit is contained in:
njoly 2013-07-20 15:55:57 +00:00
parent 1ded926c3d
commit 8d578817f8
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
sys/kern/vfs_syscalls.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: vfs_syscalls.c,v 1.466 2013/07/18 14:06:27 matt Exp $ */
/* $NetBSD: vfs_syscalls.c,v 1.467 2013/07/20 15:55:57 njoly Exp $ */
/*-
* Copyright (c) 2008, 2009 The NetBSD Foundation, Inc.
@ -70,7 +70,7 @@
*/
#include <sys/cdefs.h>
__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls.c,v 1.466 2013/07/18 14:06:27 matt Exp $");
__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls.c,v 1.467 2013/07/20 15:55:57 njoly Exp $");
#ifdef _KERNEL_OPT
#include "opt_fileassoc.h"
@ -4169,8 +4169,6 @@ do_sys_renameat(struct lwp *l, int fromfd, const char *from, int tofd,
int error;
KASSERT(l != NULL || (fromfd == AT_FDCWD && tofd == AT_FDCWD));
KASSERT(from != NULL);
KASSERT(to != NULL);
error = pathbuf_maybe_copyin(from, seg, &fpb);
if (error)