diff options
| author | Julia Lawall <julia@diku.dk> | 2010-01-16 17:00:26 +0100 |
|---|---|---|
| committer | Tyler Hicks <tyhicks@linux.vnet.ibm.com> | 2010-01-19 22:36:05 -0600 |
| commit | 4aa25bcb7dac2d583f1557e2be2d0b598581da54 (patch) | |
| tree | 6627afceaed30b0897298e83ceb4ecd580eeb9f0 | |
| parent | fe0fc013cd8bbd2f4737c1b2694b37dd7fe459cb (diff) | |
| download | kernel_goldelico_gta04-4aa25bcb7dac2d583f1557e2be2d0b598581da54.zip kernel_goldelico_gta04-4aa25bcb7dac2d583f1557e2be2d0b598581da54.tar.gz kernel_goldelico_gta04-4aa25bcb7dac2d583f1557e2be2d0b598581da54.tar.bz2 | |
ecryptfs: Eliminate useless code
The variable lower_dentry is initialized twice to the same (side effect-free)
expression. Drop one initialization.
A simplified version of the semantic match that finds this problem is:
(http://coccinelle.lip6.fr/)
// <smpl>
@forall@
idexpression *x;
identifier f!=ERR_PTR;
@@
x = f(...)
... when != x
(
x = f(...,<+...x...+>,...)
|
* x = f(...)
)
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Tyler Hicks <tyhicks@linux.vnet.ibm.com>
| -rw-r--r-- | fs/ecryptfs/file.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ecryptfs/file.c b/fs/ecryptfs/file.c index dde7bb4..678172b 100644 --- a/fs/ecryptfs/file.c +++ b/fs/ecryptfs/file.c @@ -158,7 +158,7 @@ static int ecryptfs_open(struct inode *inode, struct file *file) struct dentry *ecryptfs_dentry = file->f_path.dentry; /* Private value of ecryptfs_dentry allocated in * ecryptfs_lookup() */ - struct dentry *lower_dentry = ecryptfs_dentry_to_lower(ecryptfs_dentry); + struct dentry *lower_dentry; struct ecryptfs_file_info *file_info; mount_crypt_stat = &ecryptfs_superblock_to_private( |