Re: CVS commit: src/sys/kern

2011-08-10 Thread YAMAMOTO Takashi
On Wed, Aug 10, 2011 at 03:10:13AM +, YAMAMOTO Takashi wrote: Log Message: Fail namei immediately if searchdir is unlinked / has been rmdir'd. Do this by checking if v_size == 0. Should fix PR 44658 (and PR 32661). why is this necessary? can't we just let VOP_LOOKUP fail?

Re: CVS commit: src

2011-08-10 Thread YAMAMOTO Takashi
hi, Module Name: src Committed By: manu Date: Wed Aug 3 04:11:17 UTC 2011 Modified Files: src/bin/cp: cp.c utils.c src/bin/mv: mv.c src/distrib/sets/lists/comp: mi src/lib/libc/gen: Makefile.inc extattr.3 extattr.c src/lib/libc/sys:

Re: CVS commit: src

2011-08-10 Thread Emmanuel Dreyfus
On Wed, Aug 10, 2011 at 08:59:48AM +, YAMAMOTO Takashi wrote: is extattr_namespace_access really necessary? uid-based priviledge check in userland is often a mistake. For now it duplicates the same simple access check as in kernel: system attributes are restricted to root. This is just a

Re: CVS commit: src/sys/arch

2011-08-10 Thread Cherry G. Mathew
On 10 August 2011 13:39, Cherry G. Mathew che...@netbsd.org wrote: Module Name:    src Committed By:   cherry Date:           Wed Aug 10 11:39:46 UTC 2011 Modified Files:        src/sys/arch/amd64/amd64: fpu.c machdep.c        src/sys/arch/i386/isa: npx.c        src/sys/arch/x86/x86:

Re: CVS commit: src

2011-08-10 Thread YAMAMOTO Takashi
hi, On Wed, Aug 10, 2011 at 08:59:48AM +, YAMAMOTO Takashi wrote: is extattr_namespace_access really necessary? uid-based priviledge check in userland is often a mistake. For now it duplicates the same simple access check as in kernel: system attributes are restricted to root. This is

Re: CVS commit: src/sys/kern

2011-08-10 Thread David Holland
On Wed, Aug 10, 2011 at 07:22:06AM +, YAMAMOTO Takashi wrote: why is this necessary? can't we just let VOP_LOOKUP fail? Not to fix PR 44658. it's better to fix the vn_isunder check instead of avoiding running it. IMO vn_isunder should return acutal error code (eg. ENOENT)

Re: CVS commit: src/sys/uvm

2011-08-10 Thread YAMAMOTO Takashi
[ moving to tech-kern ] hi, y...@mwd.biglobe.ne.jp (YAMAMOTO Takashi) wrote: Here is the updated patch after your changes: http://www.netbsd.org/~rmind/uvm_anon_freelst2.diff As you noted, uvm_anfree() can temporarily release the amap lock - that can happen in amap_copy().