[SCM] Samba Shared Repository - branch v3-3-test updated - release-3-2-0pre2-4198-g48c0dff

Jeremy Allison jra at samba.org
Tue Sep 30 20:23:40 GMT 2008


The branch, v3-3-test has been updated
       via  48c0dff0c5e36313619fb39d2bfb57d49ed2141e (commit)
      from  f0f2fbd4800fb4a1df26f03f37b9972101657fdd (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-3-test


- Log -----------------------------------------------------------------
commit 48c0dff0c5e36313619fb39d2bfb57d49ed2141e
Author: Jeremy Allison <jra at samba.org>
Date:   Tue Sep 30 13:22:44 2008 -0700

    Fix the make test problem Karolin reported. Now rename_open_files actually works correctly we must emit the change notify before we change the name, not before.
    Jeremy

-----------------------------------------------------------------------

Summary of changes:
 source/smbd/reply.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/smbd/reply.c b/source/smbd/reply.c
index 3ddfad0..fbda57d 100644
--- a/source/smbd/reply.c
+++ b/source/smbd/reply.c
@@ -5586,10 +5586,10 @@ NTSTATUS rename_internals_fsp(connection_struct *conn,
 		DEBUG(3,("rename_internals_fsp: succeeded doing rename on %s -> %s\n",
 			fsp->fsp_name,newname));
 
-		rename_open_files(conn, lck, newname);
-
 		notify_rename(conn, fsp->is_directory, fsp->fsp_name, newname);
 
+		rename_open_files(conn, lck, newname);
+
 		/*
 		 * A rename acts as a new file create w.r.t. allowing an initial delete
 		 * on close, probably because in Windows there is a new handle to the


-- 
Samba Shared Repository


More information about the samba-cvs mailing list