Fix bug #9822 - Samba crashing during Win8 sync.
authorJeremy Allison <jra@samba.org>
Fri, 26 Apr 2013 17:47:41 +0000 (10:47 -0700)
committerDavid Disseldorp <ddiss@samba.org>
Sat, 27 Apr 2013 18:44:55 +0000 (20:44 +0200)
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@samba.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
Autobuild-User(master): David Disseldorp <ddiss@samba.org>
Autobuild-Date(master): Sat Apr 27 20:44:55 CEST 2013 on sn-devel-104

source3/smbd/dir.c

index 0c40a509c42ad81bb56bea32e35d3da3b0c52434..7dd959f943ac5d3d0064358d7817d7513b971c94 100644 (file)
@@ -685,6 +685,7 @@ void dptr_CloseDir(files_struct *fsp)
                 * all resource deallocation.
                 */
                dptr_close_internal(fsp->dptr);
+               fsp->dptr = NULL;
        }
 }