Attachment 4872 Details for Bug 6828 – git am format patch for 3.4.3.

[patch] git am format patch for 3.4.3.

0001-Fix-bug-6828-infinite-timeout-occurs-when-byte-loc.patch (text/plain), 1.94 KB, created by Jeremy Allison on 2009-10-20 20:12:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Allison

Created: 2009-10-20 20:12:03 UTC

Size: 1.94 KB

patch

obsolete

>From 75decce10c87790f5e26daa5bc19428d4a65a4fe Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>Date: Tue, 20 Oct 2009 18:10:30 -0700
>Subject: [PATCH] Fix bug 6828 - infinite timeout occurs when byte lock held outside of samba
> Jeremy.
>
>---
> source3/smbd/blocking.c |   26 ++++++++++++++++++--------
> 1 files changed, 18 insertions(+), 8 deletions(-)
>
>diff --git a/source3/smbd/blocking.c b/source3/smbd/blocking.c
>index 4284993..f4d88d8 100644
>--- a/source3/smbd/blocking.c
>+++ b/source3/smbd/blocking.c
>@@ -48,6 +48,22 @@ static void brl_timeout_fn(struct event_context *event_ctx,
> }
> 
> /****************************************************************************
>+ We need a version of timeval_min that treats zero timval as infinite.
>+****************************************************************************/
>+
>+static struct timeval timeval_brl_min(const struct timeval *tv1,
>+					const struct timeval *tv2)
>+{
>+	if (timeval_is_zero(tv1)) {
>+		return *tv2;
>+	}
>+	if (timeval_is_zero(tv2)) {
>+		return *tv1;
>+	}
>+	return timeval_min(tv1, tv2);
>+}
>+
>+/****************************************************************************
>  After a change to blocking_lock_queue, recalculate the timed_event for the
>  next processing.
> ****************************************************************************/
>@@ -70,19 +86,13 @@ static bool recalc_brl_timeout(void)
> 			 */
>                         if (blr->blocking_pid == 0xFFFFFFFF) {
> 				struct timeval psx_to = timeval_current_ofs(10, 0);
>-				next_timeout = timeval_min(&next_timeout, &psx_to);
>+				next_timeout = timeval_brl_min(&next_timeout, &psx_to);
>                         }
> 
> 			continue;
> 		}
> 
>-		if (timeval_is_zero(&next_timeout)) {
>-			next_timeout = blr->expire_time;
>-		}
>-		else {
>-			next_timeout = timeval_min(&next_timeout,
>-						   &blr->expire_time);
>-		}
>+		next_timeout = timeval_brl_min(&next_timeout, &blr->expire_time);
> 	}
> 
> 	if (timeval_is_zero(&next_timeout)) {
>-- 
>1.5.4.3
>

Actions: View

Attachments on bug 6828: 4866 | 4872 | 4873 | 4889 | 4890 | 4891 | 4899 | 4900