Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 2 Dec 2004 23:35:14 +0000 (GMT)
From:      Robert Watson <rwatson@FreeBSD.org>
To:        Mark Santcroos <marks@ripe.net>
Cc:        current@FreeBSD.org
Subject:   Re: VOP_* changes causes NULL ptr dereference for acl_get_file(3)
Message-ID:  <Pine.NEB.3.96L.1041202233327.34029M-100000@fledge.watson.org>
In-Reply-To: <20041202215123.GA1512@laptop.6bone.nl>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 2 Dec 2004, Mark Santcroos wrote:

> I need at least the following patch to keep my system alive.  A simple
> call to acl_get_file(3) is enough to crash the machine. 
> 
> You will have to think a bit more about how you want to initialize stuff
> or about better defaults for cases like ACLs. 

Should probably be using VOP_EOPNOTSUPP rather than VOP_EINVAL, I think.
Also, it's likely that if you need a vop_getacl entry here, you probably
also want a vop_setacl and a vop_checkacl entry.

Robert N M Watson             FreeBSD Core Team, TrustedBSD Projects
robert@fledge.watson.org      Principal Research Scientist, McAfee Research


> 
> Mark
> 
> ==== //depot/projects/custom/kern/vfs_default.c#35 - /home/marks/proj/perforce/projects/custom/kern/vfs_default.c ====
> @@ -95,6 +95,7 @@
>  	.vop_revoke =		VOP_PANIC,
>  	.vop_strategy =		vop_nostrategy,
>  	.vop_unlock =		vop_stdunlock,
> +	.vop_getacl =		VOP_EINVAL,
>  };
>  
>  /*
> 
> -- 
> RIPE NCC - Delft University of Technology - The FreeBSD Project
> marks@ripe.net - m.a.santcroos@ewi.tudelft.nl - marks@freebsd.org
> _______________________________________________
> freebsd-current@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"
> 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.NEB.3.96L.1041202233327.34029M-100000>