The intent was to wait for ping_time *seconds*, but the argument for
sbus_conn_send() is in *milliseconds*. The result of this was that we
were timing out much too quickly if the process on the other end was at
all busy.
From d13b66f22c9b9ff99c90ef6b6da7edbaa2e32d53 Mon Sep 17 00:00:00 2001
From: Stephen Gallagher <sgall...@redhat.com>
Date: Mon, 17 Oct 2011 08:25:11 -0400
Subject: [PATCH] MONITOR: fix timeout conversion

---
 src/monitor/monitor.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/monitor/monitor.c b/src/monitor/monitor.c
index 9fb277e3b7a9de56950036c6803de2a535a65987..1b7f87a94df7a4e74c87155dd917383988446e7a 100644
--- a/src/monitor/monitor.c
+++ b/src/monitor/monitor.c
@@ -2185,7 +2185,7 @@ static int service_send_ping(struct mt_svc *svc)
     }
 
     ret = sbus_conn_send(svc->conn, msg,
-                         svc->ping_time,
+                         svc->ping_time * 1000, /* milliseconds */
                          ping_check, svc, NULL);
     dbus_message_unref(msg);
     return ret;
-- 
1.7.6.4

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to