From jc@irbs.com  Sat Dec 16 13:39:28 1995
Received: from irbs.irbs.com (irbs.com [199.182.75.129])
          by freefall.freebsd.org (8.7.3/8.7.3) with SMTP id NAA02751
          for <FreeBSD-gnats-submit@freebsd.org>; Sat, 16 Dec 1995 13:39:23 -0800 (PST)
Received: (from jc@localhost) by irbs.irbs.com (8.6.12/8.6.6) id QAA00570; Sat, 16 Dec 1995 16:38:49 -0500
Message-Id: <199512162138.QAA00570@irbs.irbs.com>
Date: Sat, 16 Dec 1995 16:38:49 -0500
From: John Capo <jc@irbs.com>
Reply-To: jc@irbs.com
To: FreeBSD-gnats-submit@freebsd.org
Subject: ppp predictor-1 memory leak [PATCH included]
X-Send-Pr-Version: 3.2

>Number:         895
>Category:       bin
>Synopsis:       ppp predictor-1 memory leak [PATCH included]
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          closed
>Quarter:
>Keywords:
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sat Dec 16 13:40:02 PST 1995
>Closed-Date:    Sun May 26 22:16:32 PDT 1996
>Last-Modified:  Sun May 26 22:16:48 PDT 1996
>Originator:     John Capo
>Release:        FreeBSD 2.1-STABLE i386
>Organization:
IRBS Engineering
>Environment:

	

>Description:

A link problem, like a silo overflow, that will cause a FCS error
results in a memory leak for each input packet and FCS failures on
all subsequent predictor-1 compressed input packets.


>How-To-Repeat:

Enable and allow predictor-1 compression
Run something that will generate considerable input to ppp
Cause a silo overflow, start Xinside for instance

>Fix:
	

*** pred.c.orig	Mon May 29 23:50:55 1995
--- pred.c	Sat Dec 16 16:27:20 1995
***************
*** 182,187 ****
--- 182,188 ----
      if (len != len1) {	/* Error is detected. Send reset request */
        CcpSendResetReq(&CcpFsm);
        pfree(bp);
+       pfree(wp);
        return;
      }
      cp += olen - 4;
***************
*** 213,218 ****
--- 214,224 ----
        proto = (proto << 8) | *pp++;
      }
      DecodePacket(proto, wp);
+   }
+   else
+   {
+     CcpSendResetReq(&CcpFsm);
+     pfree(wp);
    }
    pfree(bp);
  }
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open->feedback 
State-Changed-By: dfr 
State-Changed-When: Wed Feb 7 03:46:17 PST 1996 
State-Changed-Why:  
Fixed in FreeBSD-stable and FreeBSD-current, please verify. 
State-Changed-From-To: feedback->closed 
State-Changed-By: scrappy 
State-Changed-When: Sun May 26 22:16:32 PDT 1996 
State-Changed-Why:  
confirmed closure by originator 
>Unformatted:
