summaryrefslogtreecommitdiffstats
path: root/fs/cifs
diff options
context:
space:
mode:
authorJeff Layton <jlayton@redhat.com>2008-10-22 13:57:01 -0400
committerSteve French <sfrench@us.ibm.com>2008-10-23 04:50:17 +0000
commit8d281efb67463fe8aac8f6e10b31492fc218bf2b (patch)
tree72b5642a48725656f2d1edac506ab5d2152de648 /fs/cifs
parent84210e9120a8c01a14379ba1f9a9b0f963641d94 (diff)
downloadop-kernel-dev-8d281efb67463fe8aac8f6e10b31492fc218bf2b.zip
op-kernel-dev-8d281efb67463fe8aac8f6e10b31492fc218bf2b.tar.gz
cifs: fix unlinking of rename target when server doesn't support open file renames
cifs: fix unlinking of rename target when server doesn't support open file renames The patch to make cifs_rename undoable broke renaming one file on top of another when the server doesn't support busy file renames. Remove the code that uses busy file renames to unlink the target file, and just have it call cifs_unlink. If the rename of the source file fails, then the unlink won't be undoable, but hopefully that's rare enough that it won't be a problem. Signed-off-by: Jeff Layton <jlayton@redhat.com> Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs')
-rw-r--r--fs/cifs/inode.c87
1 files changed, 8 insertions, 79 deletions
diff --git a/fs/cifs/inode.c b/fs/cifs/inode.c
index 8ac4eab..d54fa8a 100644
--- a/fs/cifs/inode.c
+++ b/fs/cifs/inode.c
@@ -1293,12 +1293,9 @@ int cifs_rename(struct inode *source_dir, struct dentry *source_dentry,
struct cifs_sb_info *cifs_sb_source;
struct cifs_sb_info *cifs_sb_target;
struct cifsTconInfo *tcon;
- struct cifsInodeInfo *target_cinode;
FILE_UNIX_BASIC_INFO *info_buf_source = NULL;
FILE_UNIX_BASIC_INFO *info_buf_target;
- __u16 dstfid;
- int xid, rc, tmprc, oplock = 0;
- bool delete_already_pending;
+ int xid, rc, tmprc;
cifs_sb_target = CIFS_SB(target_dir->i_sb);
cifs_sb_source = CIFS_SB(source_dir->i_sb);
@@ -1348,104 +1345,36 @@ int cifs_rename(struct inode *source_dir, struct dentry *source_dentry,
}
info_buf_target = info_buf_source + 1;
- rc = CIFSSMBUnixQPathInfo(xid, tcon, fromName,
+ tmprc = CIFSSMBUnixQPathInfo(xid, tcon, fromName,
info_buf_source,
cifs_sb_source->local_nls,
cifs_sb_source->mnt_cifs_flags &
CIFS_MOUNT_MAP_SPECIAL_CHR);
- if (rc != 0)
+ if (tmprc != 0)
goto unlink_target;
- rc = CIFSSMBUnixQPathInfo(xid, tcon,
+ tmprc = CIFSSMBUnixQPathInfo(xid, tcon,
toName, info_buf_target,
cifs_sb_target->local_nls,
/* remap based on source sb */
cifs_sb_source->mnt_cifs_flags &
CIFS_MOUNT_MAP_SPECIAL_CHR);
- if (rc == 0 && (info_buf_source->UniqueId ==
- info_buf_target->UniqueId))
+ if (tmprc == 0 && (info_buf_source->UniqueId ==
+ info_buf_target->UniqueId))
/* same file, POSIX says that this is a noop */
goto cifs_rename_exit;
-
- rc = -EEXIST;
} /* else ... BB we could add the same check for Windows by
checking the UniqueId via FILE_INTERNAL_INFO */
- if ((rc == -EACCES) || (rc == -EEXIST)) {
unlink_target:
- /* don't bother if this is a negative dentry */
- if (!target_dentry->d_inode)
- goto cifs_rename_exit;
-
- target_cinode = CIFS_I(target_dentry->d_inode);
-
- /* try to move the target out of the way */
- tmprc = CIFSSMBOpen(xid, tcon, toName, FILE_OPEN, DELETE,
- CREATE_NOT_DIR, &dstfid, &oplock, NULL,
- cifs_sb_target->local_nls,
- cifs_sb_target->mnt_cifs_flags &
- CIFS_MOUNT_MAP_SPECIAL_CHR);
+ if ((rc == -EACCES) || (rc == -EEXIST)) {
+ tmprc = cifs_unlink(target_dir, target_dentry);
if (tmprc)
goto cifs_rename_exit;
- /* rename the file to random name */
- tmprc = CIFSSMBRenameOpenFile(xid, tcon, dstfid, NULL,
- cifs_sb_target->local_nls,
- cifs_sb_target->mnt_cifs_flags &
- CIFS_MOUNT_MAP_SPECIAL_CHR);
-
- if (tmprc)
- goto close_target;
-
- delete_already_pending = target_cinode->delete_pending;
-
- if (!delete_already_pending) {
- /* set delete on close */
- tmprc = CIFSSMBSetFileDisposition(xid, tcon,
- true, dstfid,
- current->tgid);
- /*
- * This hack is for broken samba servers, remove this
- * once more fixed ones are in the field.
- */
- if (tmprc == -ENOENT)
- delete_already_pending = false;
- else if (tmprc)
- goto undo_target_rename;
-
- target_cinode->delete_pending = true;
- }
-
-
rc = cifs_do_rename(xid, source_dentry, fromName,
target_dentry, toName);
-
- if (rc == 0)
- goto close_target;
-
- /*
- * after this point, we can't bother with error handling on
- * the undo's. This is best effort since we can't do anything
- * about failures here.
- */
- if (!delete_already_pending) {
- tmprc = CIFSSMBSetFileDisposition(xid, tcon,
- false, dstfid,
- current->tgid);
- if (tmprc == 0)
- target_cinode->delete_pending = false;
- }
-
-undo_target_rename:
- /* rename failed: undo target rename */
- CIFSSMBRenameOpenFile(xid, tcon, dstfid,
- target_dentry->d_name.name,
- cifs_sb_target->local_nls,
- cifs_sb_target->mnt_cifs_flags &
- CIFS_MOUNT_MAP_SPECIAL_CHR);
-close_target:
- CIFSSMBClose(xid, tcon, dstfid);
}
cifs_rename_exit:
OpenPOWER on IntegriCloud