Andrew, Patch from Erez Zadok wrote: > From: Erez Zadok <ezk@xxxxxxxxxxxxx> > > Signed-off-by: Erez Zadok <ezk@xxxxxxxxxxxxx> > Cc: Dustin Kirkland <kirkland@xxxxxxxxxxxxx> > Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > Cc: Al Viro <viro@xxxxxxxxxxxxxxxxxx> > Signed-off-by: Tyler Hicks <tyhicks@xxxxxxxxxxxxxxxxxx> Sorry for all those crappy messages! Please discard all messages you received today from linuxtv-commits. Due to an error at the post-receive hook I wrote, linuxtv server tried to send a message for every commit from 2.6.32-rc5 to 2.6.33, when I merged back from your tree. After noticing the problem, I've stopped the exim and dropped all such messages from the server queue, but several messages were already sent. I've disabled the hook for now, until I find a way to properly handle merged patches on it. Sorry for the inconvenience. Mauro. > > fs/ecryptfs/inode.c | 13 ++++++++++++- > 1 files changed, 12 insertions(+), 1 deletions(-) > > --- > > http://git.linuxtv.org/v4l-dvb.git?a=commitdiff;h=0d132f7364694da8f7cafd49e2fc2721b73e96e4 > > diff --git a/fs/ecryptfs/inode.c b/fs/ecryptfs/inode.c > index 5726d7a..764dc77 100644 > --- a/fs/ecryptfs/inode.c > +++ b/fs/ecryptfs/inode.c > @@ -614,6 +614,7 @@ ecryptfs_rename(struct inode *old_dir, struct dentry *old_dentry, > struct dentry *lower_new_dentry; > struct dentry *lower_old_dir_dentry; > struct dentry *lower_new_dir_dentry; > + struct dentry *trap = NULL; > > lower_old_dentry = ecryptfs_dentry_to_lower(old_dentry); > lower_new_dentry = ecryptfs_dentry_to_lower(new_dentry); > @@ -621,7 +622,17 @@ ecryptfs_rename(struct inode *old_dir, struct dentry *old_dentry, > dget(lower_new_dentry); > lower_old_dir_dentry = dget_parent(lower_old_dentry); > lower_new_dir_dentry = dget_parent(lower_new_dentry); > - lock_rename(lower_old_dir_dentry, lower_new_dir_dentry); > + trap = lock_rename(lower_old_dir_dentry, lower_new_dir_dentry); > + /* source should not be ancestor of target */ > + if (trap == lower_old_dentry) { > + rc = -EINVAL; > + goto out_lock; > + } > + /* target should not be ancestor of source */ > + if (trap == lower_new_dentry) { > + rc = -ENOTEMPTY; > + goto out_lock; > + } > rc = vfs_rename(lower_old_dir_dentry->d_inode, lower_old_dentry, > lower_new_dir_dentry->d_inode, lower_new_dentry); > if (rc) > > _______________________________________________ > linuxtv-commits mailing list > linuxtv-commits@xxxxxxxxxxx > http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits -- Cheers, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html