svn commit: samba r13749 - in trunk/source/smbd: .

jra at samba.org jra at samba.org
Tue Feb 28 06:41:10 GMT 2006


Author: jra
Date: 2006-02-28 06:41:09 +0000 (Tue, 28 Feb 2006)
New Revision: 13749

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=13749

Log:
Don't reference memory after we just freed it (Doh!).
Thanks to tridge's changes to the directory delete on close tests
for catching this.
Jeremy.

Modified:
   trunk/source/smbd/close.c


Changeset:
Modified: trunk/source/smbd/close.c
===================================================================
--- trunk/source/smbd/close.c	2006-02-28 06:41:07 UTC (rev 13748)
+++ trunk/source/smbd/close.c	2006-02-28 06:41:09 UTC (rev 13749)
@@ -363,8 +363,6 @@
 
 	delete_dir = (lck->delete_on_close | lck->initial_delete_on_close);
 
-	TALLOC_FREE(lck);
-
 	if ((close_type == NORMAL_CLOSE || close_type == SHUTDOWN_CLOSE) &&
 				delete_dir &&
 				lck->delete_token) {
@@ -382,6 +380,8 @@
 				lck->delete_token->groups,
 				NULL);
 
+		TALLOC_FREE(lck);
+
 		ok = rmdir_internals(fsp->conn, fsp->fsp_name);
 
 		DEBUG(5,("close_directory: %s. Delete on close was set - deleting directory %s.\n",
@@ -402,6 +402,7 @@
 		}
 		process_pending_change_notify_queue((time_t)0);
 	} else {
+		TALLOC_FREE(lck);
 		remove_pending_change_notify_requests_by_fid(fsp, NT_STATUS_CANCELLED);
 	}
 



More information about the samba-cvs mailing list