Attachment 14015 Details for Bug 13318 – WIP patch for 4.7, should also apply to master

[patch] WIP patch for 4.7, should also apply to master

bug13318-v47.patch (text/plain), 1.65 KB, created by Ralph Böhme on 2018-03-02 15:38:31 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ralph Böhme

Created: 2018-03-02 15:38:31 UTC

Size: 1.65 KB

patch

obsolete

>From af0f67205bf4ec776de6a21481a31220dca9d964 Mon Sep 17 00:00:00 2001
>From: Ralph Boehme <slow@samba.org>
>Date: Fri, 2 Mar 2018 15:50:29 +0100
>Subject: [PATCH] s3: smbd/durable: remove dev and inode check from
> vfs_default_durable_reconnect_check_stat()
>
>On a cluster filesystem the device numbers may differ on the cluster
>nodes. We already verify the file_id in vfs_default_durable_reconnect(),
>so we can safely remove the dev/inode checks.
>
>Bug: https://bugzilla.samba.org/show_bug.cgi?id=13318
>---
> source3/smbd/durable.c | 24 ------------------------
> 1 file changed, 24 deletions(-)
>
>diff --git a/source3/smbd/durable.c b/source3/smbd/durable.c
>index 602a96e1fe5..2a5ddc00268 100644
>--- a/source3/smbd/durable.c
>+++ b/source3/smbd/durable.c
>@@ -305,30 +305,6 @@ static bool vfs_default_durable_reconnect_check_stat(
> {
> 	int ret;
> 
>-	if (cookie_st->st_ex_dev != fsp_st->st_ex_dev) {
>-		DEBUG(1, ("vfs_default_durable_reconnect (%s): "
>-			  "stat_ex.%s differs: "
>-			  "cookie:%llu != stat:%llu, "
>-			  "denying durable reconnect\n",
>-			  name,
>-			  "st_ex_dev",
>-			  (unsigned long long)cookie_st->st_ex_dev,
>-			  (unsigned long long)fsp_st->st_ex_dev));
>-		return false;
>-	}
>-
>-	if (cookie_st->st_ex_ino != fsp_st->st_ex_ino) {
>-		DEBUG(1, ("vfs_default_durable_reconnect (%s): "
>-			  "stat_ex.%s differs: "
>-			  "cookie:%llu != stat:%llu, "
>-			  "denying durable reconnect\n",
>-			  name,
>-			  "st_ex_ino",
>-			  (unsigned long long)cookie_st->st_ex_ino,
>-			  (unsigned long long)fsp_st->st_ex_ino));
>-		return false;
>-	}
>-
> 	if (cookie_st->st_ex_mode != fsp_st->st_ex_mode) {
> 		DEBUG(1, ("vfs_default_durable_reconnect (%s): "
> 			  "stat_ex.%s differs: "
>-- 
>2.13.6
>

Actions: View

Attachments on bug 13318: 14015 | 14313