Re: [patch 03/10] apparmor: use task path helpers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 18, 2010 at 04:37:32AM +1000, Nick Piggin wrote:
> apparmor: use task path helpers
> 
> Signed-off-by: Nick Piggin <npiggin@xxxxxxxxx>
> 
> ---
> BTW, argh! dcache_lock! Why does everything have to invent its own reverse
> path lookup crud and hide it away in its own code? Also, admire the rest of
> this beautiful function.
> 
> ---
>  security/apparmor/path.c |    9 ++-------
>  1 file changed, 2 insertions(+), 7 deletions(-)
> 
> Index: linux-2.6/security/apparmor/path.c
> ===================================================================
> --- linux-2.6.orig/security/apparmor/path.c	2010-08-18 04:04:02.000000000 +1000
> +++ linux-2.6/security/apparmor/path.c	2010-08-18 04:04:29.000000000 +1000
> @@ -62,19 +62,14 @@ static int d_namespace_path(struct path
>  	int deleted, connected;
>  	int error = 0;
>  
> -	/* Get the root we want to resolve too */
> +	/* Get the root we want to resolve too, released below */
>  	if (flags & PATH_CHROOT_REL) {
>  		/* resolve paths relative to chroot */
> -		read_lock(&current->fs->lock);
> -		root = current->fs->root;
> -		/* released below */
> -		path_get(&root);
> -		read_unlock(&current->fs->lock);
> +		get_fs_root(current->fs, &root);
>  	} else {
>  		/* resolve paths relative to namespace */
>  		root.mnt = current->nsproxy->mnt_ns->root;
>  		root.dentry = root.mnt->mnt_root;
> -		/* released below */
>  		path_get(&root);
>  	}

Gosh, maybe this VFS stuff isn't as hard as I thought!  This one's
easy to review too. :)

Looks good.

Reviewed-by: Valerie Aurora <vaurora@xxxxxxxxxx>

-VAL

--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux