From thompson@tgsoft.com  Sat Jan 11 06:54:55 1997
Received: from squirrel.tgsoft.com (squirrel.tgsoft.com [207.167.64.183])
          by freefall.freebsd.org (8.8.4/8.8.4) with ESMTP id GAA19738
          for <FreeBSD-gnats-submit@freebsd.org>; Sat, 11 Jan 1997 06:54:53 -0800 (PST)
Received: (from thompson@localhost) by squirrel.tgsoft.com (8.8.3/8.6.12) id GAA15414; Sat, 11 Jan 1997 06:56:40 -0800 (PST)
Message-Id: <199701111456.GAA15414@squirrel.tgsoft.com>
Date: Sat, 11 Jan 1997 06:56:40 -0800 (PST)
From: mark thompson <thompson@tgsoft.com>
Reply-To: thompson@tgsoft.com
To: FreeBSD-gnats-submit@freebsd.org
Subject: nit in ij-ppp
X-Send-Pr-Version: 3.2

>Number:         2449
>Category:       bin
>Synopsis:       ij-ppp in auto mode goes into failure loop
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    brian
>State:          closed
>Quarter:
>Keywords:
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sat Jan 11 07:00:01 PST 1997
>Closed-Date:    Wed May 7 16:40:45 PDT 1997
>Last-Modified:  Wed May  7 16:41:32 PDT 1997
>Originator:     mark thompson
>Release:        FreeBSD 2.1-960627-SNAP i386
>Organization:
tgsoft
>Environment:

Using user-mode ppp in auto mode.

>Description:

If ppp ever gets to the line:
      LogPrintf(LOG_PHASE_BIT, "** 1 Too many ECHO packets are lost. **\n");
It will get there every time until killed, because the counter "lqrsendcnt"
does not get reset.

>How-To-Repeat:

Best way to reproduce this is to have the receiving end refuse to
anwer pings for a while, then bring it back.

>Fix:
	
diff -r -u ../FreeBSD.2.2/src/usr.sbin/ppp/lqr.c Root/usr/src/usr.sbin/ppp/lqr.c
--- ../FreeBSD.2.2/src/usr.sbin/ppp/lqr.c	Sat May 11 13:48:33 1996
+++ Root/usr/src/usr.sbin/ppp/lqr.c	Fri Jan 10 05:06:26 1997
@@ -189,6 +189,7 @@
   struct lcpstate *lcp = &LcpInfo;
   int period;
 
+  lqrsendcnt = 0;	/* start waiting all over for ECHOs */
   lqmmethod = LQM_ECHO;
   if (Enabled(ConfLqr))
     lqmmethod |= LQM_LQR;


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-bugs->brian 
Responsible-Changed-By: brian 
Responsible-Changed-When: Tue Apr 15 09:58:49 PDT 1997 
Responsible-Changed-Why:  
I'll look at merging the diffs. 
State-Changed-From-To: open->closed 
State-Changed-By: brian 
State-Changed-When: Wed May 7 16:40:45 PDT 1997 
State-Changed-Why:  
The patch has been submitted to -current. 
It will go into 2.2 once it's been proven in -current. 
>Unformatted:
