cvs commit: src/lib/libc/posix1e acl_delete_entry.c

Jacques A. Vidrine nectar at FreeBSD.org
Tue Jan 6 12:31:08 PST 2004


On Tue, Jan 06, 2004 at 09:18:04PM +0100, Stefan Farfeleder wrote:
> @@ -61,9 +61,9 @@ acl_delete_entry(acl_t acl, acl_entry_t 
>  		if ((acl->ats_acl.acl_entry[i].ae_tag == entry_d->ae_tag) &&
>  		    (acl->ats_acl.acl_entry[i].ae_id == entry_d->ae_id)) {
>  			/* ...shift the remaining entries... */
> -			while (i < acl->ats_acl.acl_cnt - 1)
> +			for (; i < acl->ats_acl.acl_cnt - 1; ++i)
>  				acl->ats_acl.acl_entry[i] =
> -				    acl->ats_acl.acl_entry[++i];
> +				    acl->ats_acl.acl_entry[i];
> 
> I think that last assignment should read:
> 
> acl->ats_acl.acl_entry[i] = acl->ats_acl.acl_entry[i + 1];

*blush*  Um, thanks!!!
-- 
Jacques Vidrine   NTT/Verio SME      FreeBSD UNIX       Heimdal
nectar at celabo.org jvidrine at verio.net nectar at freebsd.org nectar at kth.se


More information about the cvs-all mailing list