cvs commit: src/sys/sys vnode.h src/sys/gnu/ext2fs ext2_vnops.c
src/sys/nfsclient nfs_vnops.c src/sys/ufs/ufs ufs_vnops.c
marcel at xcllnt.net
Sun Dec 5 15:14:10 PST 2004
On Dec 5, 2004, at 2:37 PM, Poul-Henning Kamp wrote:
> In message <200412052230.iB5MUTZs021927 at repoman.freebsd.org>, Marcel
>> marcel 2004-12-05 22:30:29 UTC
>> FreeBSD src repository
>> Modified files:
>> sys/sys vnode.h
>> sys/gnu/ext2fs ext2_vnops.c
>> sys/nfsclient nfs_vnops.c
>> sys/ufs/ufs ufs_vnops.c
>> Fix null-pointer indirect function calls introduced in the previous
>> commit. In the new world order, the transitive closure on the vector
>> operations is not precomputed. As such, it's unsafe to actually use
>> any of the function pointers in an indirect function call.
> Uhm not really.
> You'll notice that these three cases vector through a fifo_*specop*.
> The "specop" as opposed to "vnodeop", means that you're supposed to
> know what you're doing.
> Did you actually get a zero pointer deref on this ?
Yes, on vop_write(). See my posting to current@
if you want to trigger the bug. The null-pointer dereference happens
on all platforms.
Marcel Moolenaar USPA: A-39004 marcel at xcllnt.net
More information about the cvs-all