old DLPI doesn't have dl_sap_length
[ppp.git] / pppd / ccp.c
index 5e90ce6ecf42dc25400d8021058a9fd163429b92..355ba1d9450709e20f80e018b42e30da810734d3 100644 (file)
@@ -1,39 +1,62 @@
 /*
  * ccp.c - PPP Compression Control Protocol.
+ *
+ * Copyright (c) 1994 The Australian National University.
+ * All rights reserved.
+ *
+ * Permission to use, copy, modify, and distribute this software and its
+ * documentation is hereby granted, provided that the above copyright
+ * notice appears in all copies.  This software is provided without any
+ * warranty, express or implied. The Australian National University
+ * makes no representations about the suitability of this software for
+ * any purpose.
+ *
+ * IN NO EVENT SHALL THE AUSTRALIAN NATIONAL UNIVERSITY BE LIABLE TO ANY
+ * PARTY FOR DIRECT, INDIRECT, SPECIAL, INCIDENTAL, OR CONSEQUENTIAL DAMAGES
+ * ARISING OUT OF THE USE OF THIS SOFTWARE AND ITS DOCUMENTATION, EVEN IF
+ * THE AUSTRALIAN NATIONAL UNIVERSITY HAVE BEEN ADVISED OF THE POSSIBILITY
+ * OF SUCH DAMAGE.
+ *
+ * THE AUSTRALIAN NATIONAL UNIVERSITY SPECIFICALLY DISCLAIMS ANY WARRANTIES,
+ * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY
+ * AND FITNESS FOR A PARTICULAR PURPOSE.  THE SOFTWARE PROVIDED HEREUNDER IS
+ * ON AN "AS IS" BASIS, AND THE AUSTRALIAN NATIONAL UNIVERSITY HAS NO
+ * OBLIGATION TO PROVIDE MAINTENANCE, SUPPORT, UPDATES, ENHANCEMENTS,
+ * OR MODIFICATIONS.
  */
 
 #ifndef lint
-static char rcsid[] = "$Id: ccp.c,v 1.1 1994/08/11 01:44:32 paulus Exp $";
+static char rcsid[] = "$Id: ccp.c,v 1.12 1995/10/27 03:42:52 paulus Exp $";
 #endif
 
 #include <syslog.h>
 #include <sys/ioctl.h>
+#include <net/ppp-comp.h>
 
 #include "pppd.h"
-#include "ppp.h"
 #include "fsm.h"
 #include "ccp.h"
 
-fsm ccp_fsm[NPPP];
-ccp_options ccp_wantoptions[NPPP];     /* what to request the peer to use */
-ccp_options ccp_gotoptions[NPPP];      /* what the peer agreed to do */
-ccp_options ccp_allowoptions[NPPP];    /* what we'll agree to do */
-ccp_options ccp_hisoptions[NPPP];      /* what we agreed to do */
+fsm ccp_fsm[NUM_PPP];
+ccp_options ccp_wantoptions[NUM_PPP];  /* what to request the peer to use */
+ccp_options ccp_gotoptions[NUM_PPP];   /* what the peer agreed to do */
+ccp_options ccp_allowoptions[NUM_PPP]; /* what we'll agree to do */
+ccp_options ccp_hisoptions[NUM_PPP];   /* what we agreed to do */
 
 /*
  * Callbacks for fsm code.
  */
-static void ccp_resetci __ARGS((fsm *));
-static int  ccp_cilen __ARGS((fsm *));
-static void ccp_addci __ARGS((fsm *, u_char *, int *));
-static int  ccp_ackci __ARGS((fsm *, u_char *, int));
-static int  ccp_nakci __ARGS((fsm *, u_char *, int));
-static int  ccp_rejci __ARGS((fsm *, u_char *, int));
-static int  ccp_reqci __ARGS((fsm *, u_char *, int *, int));
-static void ccp_up __ARGS((fsm *));
-static void ccp_down __ARGS((fsm *));
-static int  ccp_extcode __ARGS((fsm *, int, int, u_char *, int));
-static void ccp_rack_timeout __ARGS(());
+static void ccp_resetci __P((fsm *));
+static int  ccp_cilen __P((fsm *));
+static void ccp_addci __P((fsm *, u_char *, int *));
+static int  ccp_ackci __P((fsm *, u_char *, int));
+static int  ccp_nakci __P((fsm *, u_char *, int));
+static int  ccp_rejci __P((fsm *, u_char *, int));
+static int  ccp_reqci __P((fsm *, u_char *, int *, int));
+static void ccp_up __P((fsm *));
+static void ccp_down __P((fsm *));
+static int  ccp_extcode __P((fsm *, int, int, u_char *, int));
+static void ccp_rack_timeout __P(());
 
 static fsm_callbacks ccp_callbacks = {
     ccp_resetci,
@@ -54,25 +77,22 @@ static fsm_callbacks ccp_callbacks = {
 };
 
 /*
- * Length of configuration options, which describe possible
- * compression methods.
+ * Do we want / did we get any compression?
  */
-#define CILEN_BSD      3
+#define ANY_COMPRESS(opt)      ((opt).deflate || (opt).bsd_compress \
+                                || (opt).predictor_1 || (opt).predictor_2)
 
 /*
- * Configuration option values for compression methods.
+ * Local state (mainly for handling reset-reqs and reset-acks).
  */
-#define CI_BSD_COMPRESS        0x21
-
-/*
- * Local state (mainly for handling reset-reqs and reset-acks
- */
-static int ccp_localstate[NPPP];
+static int ccp_localstate[NUM_PPP];
 #define RACK_PENDING   1       /* waiting for reset-ack */
 #define RREQ_REPEAT    2       /* send another reset-req if no reset-ack */
 
 #define RACKTIMEOUT    1       /* second */
 
+static int all_rejected[NUM_PPP];      /* we rejected all peer's options */
+
 /*
  * ccp_init - initialize CCP.
  */
@@ -83,7 +103,7 @@ ccp_init(unit)
     fsm *f = &ccp_fsm[unit];
 
     f->unit = unit;
-    f->protocol = CCP;
+    f->protocol = PPP_CCP;
     f->callbacks = &ccp_callbacks;
     fsm_init(f);
 
@@ -92,10 +112,11 @@ ccp_init(unit)
     memset(&ccp_allowoptions[unit], 0, sizeof(ccp_options));
     memset(&ccp_hisoptions[unit],   0, sizeof(ccp_options));
 
+    ccp_wantoptions[0].bsd_compress = 1;
     ccp_wantoptions[0].bsd_bits = 12;  /* default value */
 
     ccp_allowoptions[0].bsd_compress = 1;
-    ccp_allowoptions[0].bsd_bits = MAX_BSD_BITS;
+    ccp_allowoptions[0].bsd_bits = BSD_MAX_BITS;
 }
 
 /*
@@ -109,6 +130,8 @@ ccp_open(unit)
 
     if (f->state != OPENED)
        ccp_flags_set(unit, 1, 0);
+    if (!ANY_COMPRESS(ccp_wantoptions[unit]))
+       f->flags |= OPT_SILENT;
     fsm_open(f);
 }
 
@@ -152,7 +175,24 @@ ccp_input(unit, p, len)
     u_char *p;
     int len;
 {
-    fsm_input(&ccp_fsm[unit], p, len);
+    fsm *f = &ccp_fsm[unit];
+    int oldstate;
+
+    /*
+     * Check for a terminate-request so we can print a message.
+     */
+    oldstate = f->state;
+    fsm_input(f, p, len);
+    if (oldstate == OPENED && p[0] == TERMREQ && f->state != OPENED)
+       syslog(LOG_NOTICE, "Compression disabled by peer.");
+
+    /*
+     * If we get a terminate-ack and we're not asking for compression,
+     * close CCP.
+     */
+    if (oldstate == REQSENT && p[0] == TERMACK
+       && !ANY_COMPRESS(ccp_gotoptions[unit]))
+       ccp_close(unit);
 }
 
 /*
@@ -166,15 +206,15 @@ ccp_extcode(f, code, id, p, len)
     int len;
 {
     switch (code) {
-    case RESETREQ:
+    case CCP_RESETREQ:
        if (f->state != OPENED)
            break;
        /* send a reset-ack, which the transmitter will see and
           reset its compression state. */
-       fsm_sdata(f, RESETACK, id, NULL, 0);
+       fsm_sdata(f, CCP_RESETACK, id, NULL, 0);
        break;
 
-    case RESETACK:
+    case CCP_RESETACK:
        if (ccp_localstate[f->unit] & RACK_PENDING && id == f->reqid) {
            ccp_localstate[f->unit] &= ~(RACK_PENDING | RREQ_REPEAT);
            UNTIMEOUT(ccp_rack_timeout, (caddr_t) f);
@@ -206,7 +246,44 @@ static void
 ccp_resetci(f)
     fsm *f;
 {
-    ccp_gotoptions[f->unit] = ccp_wantoptions[f->unit];
+    int ok;
+    ccp_options *go = &ccp_gotoptions[f->unit];
+    u_char opt_buf[16];
+
+    *go = ccp_wantoptions[f->unit];
+    all_rejected[f->unit] = 0;
+
+    /*
+     * Check whether the kernel knows about the various
+     * compression methods we might request.
+     */
+    if (go->bsd_compress) {
+       opt_buf[0] = CI_BSD_COMPRESS;
+       opt_buf[1] = CILEN_BSD_COMPRESS;
+       opt_buf[2] = BSD_MAKE_OPT(BSD_CURRENT_VERSION, BSD_MIN_BITS);
+       if (ccp_test(f->unit, opt_buf, CILEN_BSD_COMPRESS, 0) <= 0)
+           go->bsd_compress = 0;
+    }
+    if (go->deflate) {
+       opt_buf[0] = CI_DEFLATE;
+       opt_buf[1] = CILEN_DEFLATE;
+       opt_buf[2] = DEFLATE_MAKE_OPT(DEFLATE_MIN_SIZE);
+       opt_buf[3] = DEFLATE_CHK_SEQUENCE;
+       if (ccp_test(f->unit, opt_buf, CILEN_DEFLATE, 0) <= 0)
+           go->deflate = 0;
+    }
+    if (go->predictor_1) {
+       opt_buf[0] = CI_PREDICTOR_1;
+       opt_buf[1] = CILEN_PREDICTOR_1;
+       if (ccp_test(f->unit, opt_buf, CILEN_PREDICTOR_1, 0) <= 0)
+           go->predictor_1 = 0;
+    }
+    if (go->predictor_2) {
+       opt_buf[0] = CI_PREDICTOR_2;
+       opt_buf[1] = CILEN_PREDICTOR_2;
+       if (ccp_test(f->unit, opt_buf, CILEN_PREDICTOR_2, 0) <= 0)
+           go->predictor_2 = 0;
+    }
 }
 
 /*
@@ -218,7 +295,10 @@ ccp_cilen(f)
 {
     ccp_options *go = &ccp_gotoptions[f->unit];
 
-    return (go->bsd_compress? CILEN_BSD: 0);
+    return (go->bsd_compress? CILEN_BSD_COMPRESS: 0)
+       + (go->deflate? CILEN_DEFLATE: 0)
+       + (go->predictor_1? CILEN_PREDICTOR_1: 0)
+       + (go->predictor_2? CILEN_PREDICTOR_2: 0);
 }
 
 /*
@@ -230,15 +310,76 @@ ccp_addci(f, p, lenp)
     u_char *p;
     int *lenp;
 {
+    int res;
     ccp_options *go = &ccp_gotoptions[f->unit];
     u_char *p0 = p;
 
+    /*
+     * Add the compression types that we can receive, in decreasing
+     * preference order.  Get the kernel to allocate the first one
+     * in case it gets Acked.
+     */
+    if (go->deflate) {
+       p[0] = CI_DEFLATE;
+       p[1] = CILEN_DEFLATE;
+       p[2] = DEFLATE_MAKE_OPT(go->deflate_size);
+       p[3] = DEFLATE_CHK_SEQUENCE;
+       for (;;) {
+           res = ccp_test(f->unit, p, CILEN_DEFLATE, 0);
+           if (res > 0) {
+               p += CILEN_DEFLATE;
+               break;
+           }
+           if (res < 0 || go->deflate_size <= DEFLATE_MIN_SIZE) {
+               go->deflate = 0;
+               break;
+           }
+           --go->deflate_size;
+           p[2] = DEFLATE_MAKE_OPT(go->deflate_size);
+       }
+    }
     if (go->bsd_compress) {
        p[0] = CI_BSD_COMPRESS;
-       p[1] = CILEN_BSD;
-       p[2] = go->bsd_bits;
-       p += 3;
+       p[1] = CILEN_BSD_COMPRESS;
+       p[2] = BSD_MAKE_OPT(BSD_CURRENT_VERSION, go->bsd_bits);
+       if (p != p0) {
+           p += CILEN_BSD_COMPRESS;    /* not the first option */
+       } else {
+           for (;;) {
+               res = ccp_test(f->unit, p, CILEN_BSD_COMPRESS, 0);
+               if (res > 0) {
+                   p += CILEN_BSD_COMPRESS;
+                   break;
+               }
+               if (res < 0 || go->bsd_bits <= BSD_MIN_BITS) {
+                   go->bsd_compress = 0;
+                   break;
+               }
+               --go->bsd_bits;
+               p[2] = BSD_MAKE_OPT(BSD_CURRENT_VERSION, go->bsd_bits);
+           }
+       }
     }
+    /* XXX Should Predictor 2 be preferable to Predictor 1? */
+    if (go->predictor_1) {
+       p[0] = CI_PREDICTOR_1;
+       p[1] = CILEN_PREDICTOR_1;
+       if (p == p0 && ccp_test(f->unit, p, CILEN_PREDICTOR_1, 0) <= 0) {
+           go->predictor_1 = 0;
+       } else {
+           p += CILEN_PREDICTOR_1;
+       }
+    }
+    if (go->predictor_2) {
+       p[0] = CI_PREDICTOR_2;
+       p[1] = CILEN_PREDICTOR_2;
+       if (p == p0 && ccp_test(f->unit, p, CILEN_PREDICTOR_2, 0) <= 0) {
+           go->predictor_2 = 0;
+       } else {
+           p += CILEN_PREDICTOR_2;
+       }
+    }
+
     *lenp = p - p0;
 }
 
@@ -253,14 +394,48 @@ ccp_ackci(f, p, len)
     int len;
 {
     ccp_options *go = &ccp_gotoptions[f->unit];
+    u_char *p0 = p;
 
+    if (go->deflate) {
+       if (len < CILEN_DEFLATE
+           || p[0] != CI_DEFLATE || p[1] != CILEN_DEFLATE
+           || p[2] != DEFLATE_MAKE_OPT(go->deflate_size)
+           || p[3] != DEFLATE_CHK_SEQUENCE)
+           return 0;
+       p += CILEN_DEFLATE;
+       len -= CILEN_DEFLATE;
+       /* XXX Cope with first/fast ack */
+       if (len == 0)
+           return 1;
+    }
     if (go->bsd_compress) {
-       if (len != 3 || p[0] != CI_BSD_COMPRESS
-           || p[1] != CILEN_BSD || p[2] != go->bsd_bits)
+       if (len < CILEN_BSD_COMPRESS
+           || p[0] != CI_BSD_COMPRESS || p[1] != CILEN_BSD_COMPRESS
+           || p[2] != BSD_MAKE_OPT(BSD_CURRENT_VERSION, go->bsd_bits))
+           return 0;
+       p += CILEN_BSD_COMPRESS;
+       len -= CILEN_BSD_COMPRESS;
+       /* XXX Cope with first/fast ack */
+       if (p == p0 && len == 0)
+           return 1;
+    }
+    if (go->predictor_1) {
+       if (len < CILEN_PREDICTOR_1
+           || p[0] != CI_PREDICTOR_1 || p[1] != CILEN_PREDICTOR_1)
+           return 0;
+       /* XXX Cope with first/fast ack */
+       if (p == p0 && len == 0)
+           return 1;
+    }
+    if (go->predictor_2) {
+       if (len < CILEN_PREDICTOR_2
+           || p[0] != CI_PREDICTOR_2 || p[1] != CILEN_PREDICTOR_2)
            return 0;
-       p += 3;
-       len -= 3;
+       /* XXX Cope with first/fast ack */
+       if (p == p0 && len == 0)
+           return 1;
     }
+
     if (len != 0)
        return 0;
     return 1;
@@ -283,20 +458,42 @@ ccp_nakci(f, p, len)
     memset(&no, 0, sizeof(no));
     try = *go;
 
-    if (go->bsd_compress && len >= CILEN_BSD && p[0] == CI_BSD_COMPRESS
-       && p[1] == CILEN_BSD) {
+    if (go->deflate && len >= CILEN_DEFLATE
+       && p[0] == CI_DEFLATE && p[1] == CILEN_DEFLATE) {
+       no.deflate = 1;
+       /*
+        * Peer wants us to use a different code size or something.
+        * Stop asking for Deflate if we don't understand his suggestion.
+        */
+       if (DEFLATE_METHOD(p[2]) != DEFLATE_METHOD_VAL
+           || DEFLATE_SIZE(p[2]) < DEFLATE_MIN_SIZE
+           || p[3] != DEFLATE_CHK_SEQUENCE)
+           try.deflate = 0;
+       else if (DEFLATE_SIZE(p[2]) < go->deflate_size)
+           go->deflate_size = DEFLATE_SIZE(p[2]);
+       p += CILEN_DEFLATE;
+       len -= CILEN_DEFLATE;
+    }
+
+    if (go->bsd_compress && len >= CILEN_BSD_COMPRESS
+       && p[0] == CI_BSD_COMPRESS && p[1] == CILEN_BSD_COMPRESS) {
        no.bsd_compress = 1;
        /*
-        * Peer wants us to use a different number of bits.
+        * Peer wants us to use a different number of bits
+        * or a different version.
         */
-       if (p[2] < go->bsd_bits)
-           try.bsd_bits = p[2];
-       p += CILEN_BSD;
-       len -= CILEN_BSD;
+       if (BSD_VERSION(p[2]) != BSD_CURRENT_VERSION)
+           try.bsd_compress = 0;
+       else if (BSD_NBITS(p[2]) < go->bsd_bits)
+           try.bsd_bits = BSD_NBITS(p[2]);
+       p += CILEN_BSD_COMPRESS;
+       len -= CILEN_BSD_COMPRESS;
     }
 
     /*
-     * Have a look at any remaining options...???
+     * Predictor-1 and 2 have no options, so they can't be Naked.
+     *
+     * XXX What should we do with any remaining options?
      */
 
     if (len != 0)
@@ -321,13 +518,41 @@ ccp_rejci(f, p, len)
 
     try = *go;
 
-    if (go->bsd_compress && len >= CILEN_BSD && p[0] == CI_BSD_COMPRESS
-       && p[1] == CILEN_BSD) {
-       if (p[2] != go->bsd_bits)
+    /*
+     * Cope with empty configure-rejects by ceasing to send
+     * configure-requests.
+     */
+    if (len == 0 && all_rejected[f->unit])
+       return -1;
+
+    if (go->deflate && len >= CILEN_DEFLATE
+       && p[0] == CI_DEFLATE && p[1] == CILEN_DEFLATE) {
+       if (p[2] != DEFLATE_MAKE_OPT(go->deflate_size)
+           || p[3] != DEFLATE_CHK_SEQUENCE)
+           return 0;           /* Rej is bad */
+       try.deflate = 0;
+       p += CILEN_DEFLATE;
+       len -= CILEN_DEFLATE;
+    }
+    if (go->bsd_compress && len >= CILEN_BSD_COMPRESS
+       && p[0] == CI_BSD_COMPRESS && p[1] == CILEN_BSD_COMPRESS) {
+       if (p[2] != BSD_MAKE_OPT(BSD_CURRENT_VERSION, go->bsd_bits))
            return 0;
        try.bsd_compress = 0;
-       p += CILEN_BSD;
-       len -= CILEN_BSD;
+       p += CILEN_BSD_COMPRESS;
+       len -= CILEN_BSD_COMPRESS;
+    }
+    if (go->predictor_1 && len >= CILEN_PREDICTOR_1
+       && p[0] == CI_PREDICTOR_1 && p[1] == CILEN_PREDICTOR_1) {
+       try.predictor_1 = 0;
+       p += CILEN_PREDICTOR_1;
+       len -= CILEN_PREDICTOR_1;
+    }
+    if (go->predictor_2 && len >= CILEN_PREDICTOR_2
+       && p[0] == CI_PREDICTOR_2 && p[1] == CILEN_PREDICTOR_2) {
+       try.predictor_2 = 0;
+       p += CILEN_PREDICTOR_2;
+       len -= CILEN_PREDICTOR_2;
     }
 
     if (len != 0)
@@ -351,14 +576,14 @@ ccp_reqci(f, p, lenp, dont_nak)
     int *lenp;
     int dont_nak;
 {
-    int ret, newret;
+    int ret, newret, res;
     u_char *p0, *retp;
-    int len, clen, type;
+    int len, clen, type, nb;
     ccp_options *ho = &ccp_hisoptions[f->unit];
     ccp_options *ao = &ccp_allowoptions[f->unit];
 
     ret = CONFACK;
-    p0 = p;
+    retp = p0 = p;
     len = *lenp;
 
     memset(ho, 0, sizeof(ccp_options));
@@ -375,22 +600,119 @@ ccp_reqci(f, p, lenp, dont_nak)
            clen = p[1];
 
            switch (type) {
+           case CI_DEFLATE:
+               if (!ao->deflate || clen != CILEN_DEFLATE) {
+                   newret = CONFREJ;
+                   break;
+               }
+
+               ho->deflate = 1;
+               ho->deflate_size = nb = DEFLATE_SIZE(p[2]);
+               if (DEFLATE_METHOD(p[2]) != DEFLATE_METHOD_VAL
+                   || p[3] != DEFLATE_CHK_SEQUENCE
+                   || nb > ao->deflate_size) {
+                   newret = CONFNAK;
+                   nb = ao->deflate_size;
+               } else {
+                   /*
+                    * Check whether we can do Deflate with the window
+                    * size they want.  If the window is too big, reduce
+                    * it until the kernel can cope and nak with that.
+                    */
+                   for (;;) {
+                       if (nb < DEFLATE_MIN_SIZE) {
+                           newret = CONFREJ;
+                           p[2] = DEFLATE_MAKE_OPT(ho->deflate_size);
+                           break;
+                       }
+                       p[2] = DEFLATE_MAKE_OPT(nb);
+                       res = ccp_test(f->unit, p, CILEN_DEFLATE, 1);
+                       if (res != 0) {
+                           if (res < 0)
+                               newret = CONFREJ;
+                           break;
+                       }
+                       newret = CONFNAK;
+                       --nb;
+                   }
+               }
+               if (newret == CONFNAK && !dont_nak) {
+                   if (nb >= DEFLATE_MIN_SIZE) {
+                       p[2] = DEFLATE_MAKE_OPT(nb);
+                       p[3] = DEFLATE_CHK_SEQUENCE;
+                   } else {
+                       newret = CONFREJ;
+                   }
+               }
+               break;
+
            case CI_BSD_COMPRESS:
-               if (!ao->bsd_compress || clen != CILEN_BSD) {
+               if (!ao->bsd_compress || clen != CILEN_BSD_COMPRESS) {
                    newret = CONFREJ;
                    break;
                }
 
                ho->bsd_compress = 1;
-               ho->bsd_bits = p[2];
-               if (ho->bsd_bits < MIN_BSD_BITS
-                   || ho->bsd_bits > ao->bsd_bits) {
+               ho->bsd_bits = nb = BSD_NBITS(p[2]);
+               if (BSD_VERSION(p[2]) != BSD_CURRENT_VERSION
+                   || nb > ao->bsd_bits) {
                    newret = CONFNAK;
-                   if (!dont_nak)
-                       p[2] = (ho->bsd_bits < MIN_BSD_BITS? MIN_BSD_BITS:
-                               ao->bsd_bits);
+                   nb = ao->bsd_bits;
+               } else {
+                   /*
+                    * Check whether we can do BSD_Compress with the code
+                    * size they want.  If the code size is too big, reduce
+                    * it until the kernel can cope and nak with that.
+                    */
+                   for (;;) {
+                       if (nb < BSD_MIN_BITS) {
+                           newret = CONFREJ;
+                           p[2] = BSD_MAKE_OPT(BSD_CURRENT_VERSION,
+                                               ho->bsd_bits);
+                           break;
+                       }
+                       p[2] = BSD_MAKE_OPT(BSD_CURRENT_VERSION, nb);
+                       res = ccp_test(f->unit, p, CILEN_BSD_COMPRESS, 1);
+                       if (res != 0) {
+                           if (res < 0)
+                               newret = CONFREJ;
+                           break;
+                       }
+                       newret = CONFNAK;
+                       --nb;
+                   }
+               }
+               if (newret == CONFNAK && !dont_nak) {
+                   if (nb >= BSD_MIN_BITS) {
+                       p[2] = BSD_MAKE_OPT(BSD_CURRENT_VERSION, nb);
+                   } else {
+                       newret = CONFREJ;
+                   }
+               }
+               break;
+
+           case CI_PREDICTOR_1:
+               if (!ao->predictor_1 || clen != CILEN_PREDICTOR_1) {
+                   newret = CONFREJ;
+                   break;
                }
 
+               ho->predictor_1 = 1;
+               if (ccp_test(f->unit, p, CILEN_PREDICTOR_1, 1) <= 0) {
+                   newret = CONFREJ;
+               }
+               break;
+
+           case CI_PREDICTOR_2:
+               if (!ao->predictor_2 || clen != CILEN_PREDICTOR_2) {
+                   newret = CONFREJ;
+                   break;
+               }
+
+               ho->predictor_2 = 1;
+               if (ccp_test(f->unit, p, CILEN_PREDICTOR_2, 1) <= 0) {
+                   newret = CONFREJ;
+               }
                break;
 
            default:
@@ -398,12 +720,13 @@ ccp_reqci(f, p, lenp, dont_nak)
            }
        }
 
+       if (newret == CONFNAK && dont_nak)
+           newret = CONFREJ;
        if (!(newret == CONFACK || newret == CONFNAK && ret == CONFREJ)) {
            /* we're returning this option */
-           if (newret != ret) {
+           if (newret == CONFREJ && ret == CONFNAK)
                retp = p0;
-               ret = newret;
-           }
+           ret = newret;
            if (p != retp)
                BCOPY(p, retp, clen);
            retp += clen;
@@ -413,8 +736,12 @@ ccp_reqci(f, p, lenp, dont_nak)
        len -= clen;
     }
 
-    if (ret != CONFACK)
-       *lenp = retp - p0;
+    if (ret != CONFACK) {
+       if (ret == CONFREJ && *lenp == retp - p0)
+           all_rejected[f->unit] = 1;
+       else
+           *lenp = retp - p0;
+    }
     return ret;
 }
 
@@ -425,7 +752,14 @@ static void
 ccp_up(f)
     fsm *f;
 {
+    ccp_options *go = &ccp_gotoptions[f->unit];
+    ccp_options *ho = &ccp_hisoptions[f->unit];
+
     ccp_flags_set(f->unit, 1, 1);
+    if (ANY_COMPRESS(*go) || ANY_COMPRESS(*ho))
+       syslog(LOG_NOTICE, "%s enabled",
+              ANY_COMPRESS(*go)? ANY_COMPRESS(*ho)? "Compression":
+              "Receive compression": "Transmit compression");
 }
 
 /*
@@ -455,7 +789,7 @@ int
 ccp_printpkt(p, plen, printer, arg)
     u_char *p;
     int plen;
-    void (*printer) __ARGS((void *, char *, ...));
+    void (*printer) __P((void *, char *, ...));
     void *arg;
 {
     u_char *p0, *optend;
@@ -495,10 +829,33 @@ ccp_printpkt(p, plen, printer, arg)
            len -= optlen;
            optend = p + optlen;
            switch (code) {
+           case CI_DEFLATE:
+               if (optlen >= CILEN_DEFLATE) {
+                   printer(arg, "deflate %d", DEFLATE_SIZE(p[2]));
+                   if (DEFLATE_METHOD(p[2]) != DEFLATE_METHOD_VAL)
+                       printer(arg, " method %d", DEFLATE_METHOD(p[2]));
+                   if (p[3] != DEFLATE_CHK_SEQUENCE)
+                       printer(arg, " check %d", p[3]);
+                   p += CILEN_DEFLATE;
+               }
+               break;
            case CI_BSD_COMPRESS:
-               if (optlen >= CILEN_BSD) {
-                   printer(arg, "bsd %d", p[2]);
-                   p += CILEN_BSD;
+               if (optlen >= CILEN_BSD_COMPRESS) {
+                   printer(arg, "bsd v%d %d", BSD_VERSION(p[2]),
+                           BSD_NBITS(p[2]));
+                   p += CILEN_BSD_COMPRESS;
+               }
+               break;
+           case CI_PREDICTOR_1:
+               if (optlen >= CILEN_PREDICTOR_1) {
+                   printer(arg, "predictor 1");
+                   p += CILEN_PREDICTOR_1;
+               }
+               break;
+           case CI_PREDICTOR_2:
+               if (optlen >= CILEN_PREDICTOR_2) {
+                   printer(arg, "predictor 2");
+                   p += CILEN_PREDICTOR_2;
                }
                break;
            }
@@ -517,8 +874,16 @@ ccp_printpkt(p, plen, printer, arg)
 }
 
 /*
- * We have received a packet that the decompressor failed to decompress.
- * Issue a reset-req (if we haven't issued one recently).
+ * We have received a packet that the decompressor failed to
+ * decompress.  Here we would expect to issue a reset-request, but
+ * Motorola has a patent on resetting the compressor as a result of
+ * detecting an error in the decompressed data after decompression.
+ * (See US patent 5,130,993; international patent publication number
+ * WO 91/10289; Australian patent 73296/91.)
+ *
+ * So we ask the kernel whether the error was detected after
+ * decompression; if it was, we take CCP down, thus disabling
+ * compression :-(, otherwise we issue the reset-request.
  */
 void
 ccp_datainput(unit, pkt, len)
@@ -530,12 +895,25 @@ ccp_datainput(unit, pkt, len)
 
     f = &ccp_fsm[unit];
     if (f->state == OPENED) {
-       if (!(ccp_localstate[unit] & RACK_PENDING)) {
-           fsm_sdata(f, RESETREQ, f->reqid = ++f->id, NULL, 0);
-           TIMEOUT(ccp_rack_timeout, (caddr_t) f, RACKTIMEOUT);
-           ccp_localstate[unit] |= RACK_PENDING;
-       } else
-           ccp_localstate[unit] |= RREQ_REPEAT;
+       if (ccp_fatal_error(unit)) {
+           /*
+            * Disable compression by taking CCP down.
+            */
+           syslog(LOG_ERR, "Lost compression sync: disabling compression");
+           ccp_close(unit);
+       } else {
+           /*
+            * Send a reset-request to reset the peer's compressor.
+            * We don't do that if we are still waiting for an
+            * acknowledgement to a previous reset-request.
+            */
+           if (!(ccp_localstate[f->unit] & RACK_PENDING)) {
+               fsm_sdata(f, CCP_RESETREQ, f->reqid = ++f->id, NULL, 0);
+               TIMEOUT(ccp_rack_timeout, (caddr_t) f, RACKTIMEOUT);
+               ccp_localstate[f->unit] |= RACK_PENDING;
+           } else
+               ccp_localstate[f->unit] |= RREQ_REPEAT;
+       }
     }
 }
 
@@ -549,7 +927,7 @@ ccp_rack_timeout(arg)
     fsm *f = (fsm *) arg;
 
     if (f->state == OPENED && ccp_localstate[f->unit] & RREQ_REPEAT) {
-       fsm_sdata(f, RESETREQ, f->reqid, NULL, 0);
+       fsm_sdata(f, CCP_RESETREQ, f->reqid, NULL, 0);
        TIMEOUT(ccp_rack_timeout, (caddr_t) f, RACKTIMEOUT);
        ccp_localstate[f->unit] &= ~RREQ_REPEAT;
     } else