[SCM] Samba Shared Repository - branch master updated

David Disseldorp ddiss at samba.org
Sat Apr 27 12:45:02 MDT 2013


The branch, master has been updated
       via  251767c Fix bug #9822 - Samba crashing during Win8 sync.
      from  b7765fe torture: Only test leases if supported

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 251767cde9a146d8122d76e257ab232c05ad452a
Author: Jeremy Allison <jra at samba.org>
Date:   Fri Apr 26 10:47:41 2013 -0700

    Fix bug #9822 - Samba crashing during Win8 sync.
    
    When refactoring the dptr desctructor in the
    fix for bug:
    
    9778 (Samba directory code uses dirfd() without vectoring through a VFS call)
    
    I removed the code to NULL out the struct smb_Dir *
    pointer inside the fsp struct by mistake.
    
    Re-add the NULLing out of that pointer when
    closing a directory pointer associated with
    an open file.
    
    Reporter confirms it fixes the crash.
    
    Signed-off-by: Jeremy Allison <jra at samba.org>
    Reviewed-by: David Disseldorp <ddiss at samba.org>
    
    Autobuild-User(master): David Disseldorp <ddiss at samba.org>
    Autobuild-Date(master): Sat Apr 27 20:44:55 CEST 2013 on sn-devel-104

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

Summary of changes:
 source3/smbd/dir.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/dir.c b/source3/smbd/dir.c
index 0c40a50..7dd959f 100644
--- a/source3/smbd/dir.c
+++ b/source3/smbd/dir.c
@@ -685,6 +685,7 @@ void dptr_CloseDir(files_struct *fsp)
 		 * all resource deallocation.
 		 */
 		dptr_close_internal(fsp->dptr);
+		fsp->dptr = NULL;
 	}
 }
 


-- 
Samba Shared Repository


More information about the samba-cvs mailing list