diff common-session.c @ 1143:924ff1b959a2

only update keepalive timeout post-auth (when keepalives are sent)
author Matt Johnston <matt@ucc.asn.au>
date Fri, 07 Aug 2015 21:02:49 +0800
parents f6d3a16ecc71
children a550e1e2e2a9
line wrap: on
line diff
--- a/common-session.c	Tue Aug 04 08:20:50 2015 +0800
+++ b/common-session.c	Fri Aug 07 21:02:49 2015 +0800
@@ -533,11 +533,11 @@
 }
 
 static void update_timeout(long limit, long now, long last_event, long * timeout) {
-	TRACE(("update_timeout limit %ld, now %ld, last %ld, timeout %ld",
+	TRACE2(("update_timeout limit %ld, now %ld, last %ld, timeout %ld",
 		limit, now, last_event, *timeout))
 	if (last_event > 0 && limit > 0) {
 		*timeout = MIN(*timeout, last_event+limit-now);
-		TRACE(("update to %ld", *timeout))
+		TRACE2(("new timeout %ld", *timeout))
 	}
 }
 
@@ -554,15 +554,15 @@
 		update_timeout(AUTH_TIMEOUT, now, ses.connect_time, &timeout);
 	}
 
-	update_timeout(opts.keepalive_secs, now, 
-		MAX(ses.last_packet_time_keepalive_recv, ses.last_packet_time_keepalive_sent),
-		&timeout);
+	if (ses.authstate.authdone) {
+		update_timeout(opts.keepalive_secs, now, 
+			MAX(ses.last_packet_time_keepalive_recv, ses.last_packet_time_keepalive_sent),
+			&timeout);
+	}
 
 	update_timeout(opts.idle_timeout_secs, now, ses.last_packet_time_idle,
 		&timeout);
 
-	TRACE(("timeout %ld", timeout))
-
 	/* clamp negative timeouts to zero - event has already triggered */
 	return MAX(timeout, 0);
 }