Attachment 8762 Details for Bug 9778 – git-am fix for master and 4.0.next.

[patch] git-am fix for master and 4.0.next.

remove-dirfd.patchset (text/plain), 4.75 KB, created by Jeremy Allison on 2013-04-11 19:51:45 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Allison

Created: 2013-04-11 19:51:45 UTC

Size: 4.75 KB

patch

obsolete

>From 15c25476d94db1feda9774aeb17375eff65a6151 Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>Date: Wed, 10 Apr 2013 16:21:39 -0700
>Subject: [PATCH 1/4] Maintain a back-pointer to the fsp in struct smb_Dir when
> opening with FDOPENDIR.
>
>Signed-off-by: Jeremy Allison <jra@samba.org>
>---
> source3/smbd/dir.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
>diff --git a/source3/smbd/dir.c b/source3/smbd/dir.c
>index f1c177f..e623316 100644
>--- a/source3/smbd/dir.c
>+++ b/source3/smbd/dir.c
>@@ -50,6 +50,8 @@ struct smb_Dir {
> 	struct name_cache_entry *name_cache;
> 	unsigned int name_cache_index;
> 	unsigned int file_number;
>+	files_struct *fsp; /* Back pointer to containing fsp, only
>+			      set from OpenDir_fsp(). */
> };
> 
> struct dptr_struct {
>@@ -1540,7 +1542,9 @@ static struct smb_Dir *OpenDir_fsp(TALLOC_CTX *mem_ctx, connection_struct *conn,
> 
> 	if (fsp->is_directory && fsp->fh->fd != -1) {
> 		dirp->dir = SMB_VFS_FDOPENDIR(fsp, mask, attr);
>-		if (dirp->dir == NULL) {
>+		if (dirp->dir != NULL) {
>+			dirp->fsp = fsp;
>+		} else {
> 			DEBUG(10,("OpenDir_fsp: SMB_VFS_FDOPENDIR on %s returned "
> 				"NULL (%s)\n",
> 				dirp->dir_path,
>-- 
>1.8.1.3
>
>
>From 929a21434ba181e268d7abf2ee6163990f0443f8 Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>Date: Wed, 10 Apr 2013 16:24:15 -0700
>Subject: [PATCH 2/4] In the struct smb_Dir destructor, use the fsp back
> pointer to release resources.
>
>Removes one use of dirfd().
>
>Signed-off-by: Jeremy Allison <jra@samba.org>
>---
> source3/smbd/dir.c | 21 ++++++++++++---------
> 1 file changed, 12 insertions(+), 9 deletions(-)
>
>diff --git a/source3/smbd/dir.c b/source3/smbd/dir.c
>index e623316..a38f16f 100644
>--- a/source3/smbd/dir.c
>+++ b/source3/smbd/dir.c
>@@ -1448,17 +1448,20 @@ bool is_visible_file(connection_struct *conn, const char *dir_path,
> static int smb_Dir_destructor(struct smb_Dir *dirp)
> {
> 	if (dirp->dir) {
>-#ifdef HAVE_DIRFD
>-		if (dirp->conn->sconn) {
>-			files_struct *fsp = file_find_fd(dirp->conn->sconn,
>-						dirfd(dirp->dir));
>-			if (fsp) {
>-				/* The call below closes the underlying fd. */
>-				fsp->fh->fd = -1;
>+		SMB_VFS_CLOSEDIR(dirp->conn,dirp->dir);
>+		if (dirp->fsp) {
>+			/*
>+			 * The SMB_VFS_CLOSEDIR above
>+			 * closes the underlying fd inside
>+			 * dirp->fsp.
>+			 */
>+			dirp->fsp->fh->fd = -1;
>+			if (dirp->fsp->dptr) {
>+				SMB_ASSERT(dirp->fsp->dptr->dir_hnd == dirp);
>+				dirp->fsp->dptr->dir_hnd = NULL;
> 			}
>+			dirp->fsp = NULL;
> 		}
>-#endif
>-		SMB_VFS_CLOSEDIR(dirp->conn,dirp->dir);
> 	}
> 	if (dirp->conn->sconn && !dirp->conn->sconn->using_smb2) {
> 		dirp->conn->sconn->searches.dirhandles_open--;
>-- 
>1.8.1.3
>
>
>From 7ea68d22577c5d2ce19310a5400d71412341c5aa Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>Date: Wed, 10 Apr 2013 16:29:03 -0700
>Subject: [PATCH 3/4] Remove the "Ugly hack" that was the second use of
> dirfd().
>
>The destructor does all the resource deallocation needed.
>
>Signed-off-by: Jeremy Allison <jra@samba.org>
>---
> source3/smbd/dir.c | 18 +++++-------------
> 1 file changed, 5 insertions(+), 13 deletions(-)
>
>diff --git a/source3/smbd/dir.c b/source3/smbd/dir.c
>index a38f16f..3587abe 100644
>--- a/source3/smbd/dir.c
>+++ b/source3/smbd/dir.c
>@@ -679,20 +679,12 @@ done:
> void dptr_CloseDir(files_struct *fsp)
> {
> 	if (fsp->dptr) {
>-/*
>- * Ugly hack. We have defined fdopendir to return ENOSYS if dirfd also isn't
>- * present. I hate Solaris. JRA.
>- */
>-#ifdef HAVE_DIRFD
>-		if (fsp->fh->fd != -1 &&
>-				fsp->dptr->dir_hnd &&
>-				dirfd(fsp->dptr->dir_hnd->dir)) {
>-			/* The call below closes the underlying fd. */
>-			fsp->fh->fd = -1;
>-		}
>-#endif
>+		/*
>+		 * The destructor for the struct smb_Dir
>+		 * (fsp->dptr->dir_hnd) now handles
>+		 * all resource deallocation.
>+		 */
> 		dptr_close_internal(fsp->dptr);
>-		fsp->dptr = NULL;
> 	}
> }
> 
>-- 
>1.8.1.3
>
>
>From 64c63f2bb4f678a9b3ff19499535120baf84050b Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>Date: Wed, 10 Apr 2013 16:30:10 -0700
>Subject: [PATCH 4/4] Remove dependency on detection HAVE_DIRFD for use of
> fdopendir().
>
>Signed-off-by: Jeremy Allison <jra@samba.org>
>---
> source3/lib/system.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
>diff --git a/source3/lib/system.c b/source3/lib/system.c
>index d69f1c6..8dbf7dc 100644
>--- a/source3/lib/system.c
>+++ b/source3/lib/system.c
>@@ -634,13 +634,11 @@ void kernel_flock(int fd, uint32 share_mode, uint32 access_mask)
> 
> /*******************************************************************
>  An fdopendir wrapper.
>- Ugly hack - we need dirfd for this to work correctly in the
>- calling code.. JRA.
> ********************************************************************/
> 
> DIR *sys_fdopendir(int fd)
> {
>-#if defined(HAVE_FDOPENDIR) && defined(HAVE_DIRFD)
>+#if defined(HAVE_FDOPENDIR)
> 	return fdopendir(fd);
> #else
> 	errno = ENOSYS;
>-- 
>1.8.1.3
>

Flags:

asn: review-

Actions: View

Attachments on bug 9778: 8760 | 8762 | 8806 | 8822 | 8829 | 8942