PERFORCE change 77201 for review

Tom Rhodes trhodes at FreeBSD.org
Fri May 20 14:47:01 GMT 2005


On Fri, 20 May 2005 00:17:48 GMT
"Christian S.J. Peron" <csjp at freebsd.org> wrote:

> http://perforce.freebsd.org/chv.cgi?CH=77201
> 
> Change 77201 by csjp at csjp_xor on 2005/05/20 00:17:30
> 
> 	
> 	-replace a size_t for int in the checksum calculation routines
> 	-declare temp variable to stuff the size of the extended attribute in.
> 	 We have to do this because for some reason the data type representing
> 	 extended attribute size between vn_extattr_get and VOP_GETEXTATTR conflict.
> 	 This makes the mac_chkexec build happy on AMD64

Requested by:	trhodes

Thanks!!!

[SNIP]

>  		return (error);
> +	/* XXX why are'nt extended attribute size specification types between
> +	 * VOP_GETEXTATTR and vn_extattr_get the same?
> +	 */
> +	alen = ealen;
>  	depends = malloc(alen + 1, M_CHKEXEC, M_WAITOK | M_ZERO);
>  	error = vn_extattr_get(vp, IO_NODELOCKED, MAC_CHKEXEC_ATTRN,
>  	    MAC_CHKEXEC_DEP, &alen, depends, curthread);
> 

Good question.

-- 
Tom Rhodes
To Unsubscribe: send mail to majordomo at trustedbsd.org
with "unsubscribe trustedbsd-cvs" in the body of the message



More information about the trustedbsd-cvs mailing list