]> asedeno.scripts.mit.edu Git - linux.git/commit
ecryptfs: use vfs_get_link()
authorMiklos Szeredi <mszeredi@redhat.com>
Fri, 9 Dec 2016 15:45:03 +0000 (16:45 +0100)
committerMiklos Szeredi <mszeredi@redhat.com>
Fri, 9 Dec 2016 15:45:03 +0000 (16:45 +0100)
commit6c988f575915ab5162332d61f91dca99dcc17335
treec17f5c57209779aca2b1f9845750fa73fa69e0d2
parent3f9ca75516a7e581ff803f751a869c1da5ae5fa5
ecryptfs: use vfs_get_link()

Here again we are copying form one buffer to another, while jumping through
hoops to make kernel memory look like userspace memory.

For no good reason, since vfs_get_link() provides exactly what is needed.

As a bonus, now the security hook for readlink is also called on the
underlying inode.

Note: this can be called from link-following context.  But this is okay:

 - not in RCU mode

 - commit e54ad7f1ee26 ("proc: prevent stacking filesystems on top")

 - ecryptfs is *reading* the underlying symlink not following it, so the
   right security hook is being called

Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
Cc: Tyler Hicks <tyhicks@canonical.com>
fs/ecryptfs/inode.c