diff options
| author | Shehjar Tikoo <shehjart@zresearch.com> | 2009-05-11 18:24:43 +0530 |
|---|---|---|
| committer | Anand V. Avati <avati@amp.gluster.com> | 2009-05-18 19:13:18 +0530 |
| commit | 35fc174ea3a37546d7c90be8fa6076b8f9416d73 (patch) | |
| tree | 8e391d5f94d9709cdb4470e60476d2f1d9f0e6ab | |
| parent | 95ce0a87f5fbb139f7002360d63a005bfb9c097d (diff) | |
libglusterfsclient: Comply with EEXIST rule on rename
During a rename, if the new file exists, the old name needs to
over-write the new name. We're returning EEXIST, which is wrong
behaviour.
Signed-off-by: Anand V. Avati <avati@amp.gluster.com>
| -rwxr-xr-x | libglusterfsclient/src/libglusterfsclient.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/libglusterfsclient/src/libglusterfsclient.c b/libglusterfsclient/src/libglusterfsclient.c index 91f1057765c..7e850164416 100755 --- a/libglusterfsclient/src/libglusterfsclient.c +++ b/libglusterfsclient/src/libglusterfsclient.c @@ -5342,14 +5342,6 @@ glusterfs_glh_rename (glusterfs_handle_t handle, const char *oldpath, goto out; op_ret = libgf_client_path_lookup (&newloc, ctx, 1); - if (op_ret == 0) { - gf_log ("libglusterfsclient", GF_LOG_ERROR, - "newpath (%s) already exists, returning" - " EEXIST", newloc.path); - errno = EEXIST; - op_ret = -1; - goto out; - } oldname = strdup (oldloc.path); op_ret = libgf_client_loc_fill (&oldloc, ctx, 0, oldloc.parent->ino, |