2010-11-03 10:12:29

by Roberto Sassu

[permalink] [raw]
Subject: [PATCH 0/5] ecryptfs fixes

This patch set applies to the latest kernel 2.6.37-rc1.


Roberto Sassu (5):
ecryptfs: moved ECRYPTFS_SUPER_MAGIC definition to linux/magic.h
ecryptfs: missing initialization of the superblock 'magic' field
ecryptfs: test lower_file pointer when lower_file_mutex is locked
ecryptfs: fixed testing of file descriptor flags
ecryptfs: open lower files with kthread credentials

fs/ecryptfs/ecryptfs_kernel.h | 4 +---
fs/ecryptfs/file.c | 20 +++++++++-----------
fs/ecryptfs/inode.c | 32 ++++++++++++++------------------
fs/ecryptfs/kthread.c | 17 +++++++++++++----
fs/ecryptfs/main.c | 5 +++--
include/linux/magic.h | 1 +
6 files changed, 41 insertions(+), 38 deletions(-)

--
1.7.2.3


Attachments:
smime.p7s (4.60 kB)

2010-11-09 22:01:53

by Tyler Hicks

[permalink] [raw]
Subject: Re: [PATCH 0/5] ecryptfs fixes

On Wed Nov 03, 2010 at 11:11:06AM +0100, Roberto Sassu <[email protected]> wrote:
> This patch set applies to the latest kernel 2.6.37-rc1.
>
>
> Roberto Sassu (5):
> ecryptfs: moved ECRYPTFS_SUPER_MAGIC definition to linux/magic.h
> ecryptfs: missing initialization of the superblock 'magic' field
> ecryptfs: test lower_file pointer when lower_file_mutex is locked
> ecryptfs: fixed testing of file descriptor flags
> ecryptfs: open lower files with kthread credentials

Hey Roberto - I pushed these patches to
git://git.kernel.org/pub/scm/linux/kernel/git/ecryptfs/ecryptfs-2.6.git#next

Thanks,
Tyler

>
> fs/ecryptfs/ecryptfs_kernel.h | 4 +---
> fs/ecryptfs/file.c | 20 +++++++++-----------
> fs/ecryptfs/inode.c | 32 ++++++++++++++------------------
> fs/ecryptfs/kthread.c | 17 +++++++++++++----
> fs/ecryptfs/main.c | 5 +++--
> include/linux/magic.h | 1 +
> 6 files changed, 41 insertions(+), 38 deletions(-)
>
> --
> 1.7.2.3

2010-11-10 09:33:53

by Roberto Sassu

[permalink] [raw]
Subject: Re: [PATCH 0/5] ecryptfs fixes

Hi Tyler

thanks for accepting my patches.
There is another O_RDONLY test in the function ecryptfs_privileged_open(), in the file fs/ecryptfs/kthread.c,
that is not actually fixed, because i need some explanations on the code.
In this function the dentry_open() is called first time with flags (O_LARGEFILE | O_RDWR) or
(O_LARGEFILE | O_RDONLY) depending on the options passed during the mount of the lower
filesystem.
If the dentry_open() fails, a lower_file pointer can be obtained from the function ecryptfs_threadfn()
executed by the kernel kthread service. However, this option is available only if the lower filesystem
is mounted in read-write mode.
Can you explain me this?
Thanks.

Roberto


On Tuesday, November 09, 2010 11:01:44 pm Tyler Hicks wrote:
> On Wed Nov 03, 2010 at 11:11:06AM +0100, Roberto Sassu <[email protected]> wrote:
> > This patch set applies to the latest kernel 2.6.37-rc1.
> >
> >
> > Roberto Sassu (5):
> > ecryptfs: moved ECRYPTFS_SUPER_MAGIC definition to linux/magic.h
> > ecryptfs: missing initialization of the superblock 'magic' field
> > ecryptfs: test lower_file pointer when lower_file_mutex is locked
> > ecryptfs: fixed testing of file descriptor flags
> > ecryptfs: open lower files with kthread credentials
>
> Hey Roberto - I pushed these patches to
> git://git.kernel.org/pub/scm/linux/kernel/git/ecryptfs/ecryptfs-2.6.git#next
>
> Thanks,
> Tyler
>
> >
> > fs/ecryptfs/ecryptfs_kernel.h | 4 +---
> > fs/ecryptfs/file.c | 20 +++++++++-----------
> > fs/ecryptfs/inode.c | 32 ++++++++++++++------------------
> > fs/ecryptfs/kthread.c | 17 +++++++++++++----
> > fs/ecryptfs/main.c | 5 +++--
> > include/linux/magic.h | 1 +
> > 6 files changed, 41 insertions(+), 38 deletions(-)
> >
>
>
>