Attachment 13697 Details for Bug 13086 – patch for 4.7

[patch] patch for 4.7

0001-vfs_fruit-Replace-closedir-by-SMB_VFS_CLOSEDIR.patch (text/plain), 1.56 KB, created by Anoop C S on 2017-10-17 05:41:31 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Anoop C S

Created: 2017-10-17 05:41:31 UTC

Size: 1.56 KB

patch

obsolete

>From 014ae0de9fc036848615d4143f3c2d3e8c72e9bb Mon Sep 17 00:00:00 2001
>From: Anoop C S <anoopcs@redhat.com>
>Date: Fri, 13 Oct 2017 20:38:31 +0530
>Subject: [PATCH] vfs_fruit: Replace closedir() by SMB_VFS_CLOSEDIR
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>Pointer to directory 'dh' inside fruit_rmdir() is obtained using
>SMB_VFS_OPENDIR. But this handle is closed directly by invoking
>closedir() rather than SMB_VFS_CLOSEDIR. This will result in a
>smbd crash if this handle was not obtained from local file system.
>Therefore use SMB_VFS_CLOSEDIR corresponding to SMB_VFS_OPENDIR
>to correctly close the directory handle.
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=13086
>
>Signed-off-by: Anoop C S <anoopcs@redhat.com>
>Reviewed-by: Ralph Boehme <slow@samba.org>
>Reviewed-by: Jeremy Allison <jra@samba.org>
>Reviewed-by: Guenther Deschner <gd@samba.org>
>
>Autobuild-User(master): GÃ¼nther Deschner <gd@samba.org>
>Autobuild-Date(master): Mon Oct 16 19:56:55 CEST 2017 on sn-devel-144
>
>(cherry picked from commit 7917f9721c9f8efdf9e7b7c50a9e5147250e36fe)
>---
> source3/modules/vfs_fruit.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/source3/modules/vfs_fruit.c b/source3/modules/vfs_fruit.c
>index 09e0fcdc3c9..4bed3b41d8a 100644
>--- a/source3/modules/vfs_fruit.c
>+++ b/source3/modules/vfs_fruit.c
>@@ -3435,7 +3435,7 @@ static int fruit_rmdir(struct vfs_handle_struct *handle,
> 
> exit_rmdir:
> 	if (dh) {
>-		closedir(dh);
>+		SMB_VFS_CLOSEDIR(handle->conn, dh);
> 	}
> 	return SMB_VFS_NEXT_RMDIR(handle, smb_fname);
> }
>-- 
>2.13.6
>

Flags:

slow: review+
gd: review+

Actions: View

Attachments on bug 13086: 13684 | 13697 | 13698