diff options
| author | Tyler Hicks | 2009-03-20 08:23:57 +0100 |
|---|---|---|
| committer | Linus Torvalds | 2009-03-22 19:20:43 +0100 |
| commit | 2aac0cf88681bfa092f731553bc7fbd23516be73 (patch) | |
| tree | b723cbe9c67b0cafa9081690d03b4ecec038d9f6 /fs/ufs/super.c | |
| parent | eCryptfs: Allocate a variable number of pages for file headers (diff) | |
| download | kernel-qcow2-linux-2aac0cf88681bfa092f731553bc7fbd23516be73.tar.gz kernel-qcow2-linux-2aac0cf88681bfa092f731553bc7fbd23516be73.tar.xz kernel-qcow2-linux-2aac0cf88681bfa092f731553bc7fbd23516be73.zip | |
eCryptfs: NULL crypt_stat dereference during lookup
If ecryptfs_encrypted_view or ecryptfs_xattr_metadata were being
specified as mount options, a NULL pointer dereference of crypt_stat
was possible during lookup.
This patch moves the crypt_stat assignment into
ecryptfs_lookup_and_interpose_lower(), ensuring that crypt_stat
will not be NULL before we attempt to dereference it.
Thanks to Dan Carpenter and his static analysis tool, smatch, for
finding this bug.
Signed-off-by: Tyler Hicks <tyhicks@linux.vnet.ibm.com>
Acked-by: Dustin Kirkland <kirkland@canonical.com>
Cc: Dan Carpenter <error27@gmail.com>
Cc: Serge Hallyn <serue@us.ibm.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/ufs/super.c')
0 files changed, 0 insertions, 0 deletions