X-Git-Url: https://git.ozlabs.org/?p=ppp.git;a=blobdiff_plain;f=pppd%2Fccp.c;h=90f83fad1429e4d453307223e49db1bdfa825d51;hp=421542d47569cf946ccf676b0449f83738f37117;hb=b45067455a4e74d002f8d321ac261df95ad8ad7e;hpb=97f76825ddab9381b3fa7ad72b8f15469a6bc095 diff --git a/pppd/ccp.c b/pppd/ccp.c index 421542d..90f83fa 100644 --- a/pppd/ccp.c +++ b/pppd/ccp.c @@ -23,42 +23,39 @@ * ON AN "AS IS" BASIS, AND THE AUSTRALIAN NATIONAL UNIVERSITY HAS NO * OBLIGATION TO PROVIDE MAINTENANCE, SUPPORT, UPDATES, ENHANCEMENTS, * OR MODIFICATIONS. - * - * $Id: ccp.c,v 1.3 1994/09/01 00:16:20 paulus Exp $ */ #ifndef lint -static char rcsid[] = "$Id: ccp.c,v 1.3 1994/09/01 00:16:20 paulus Exp $"; +static char rcsid[] = "$Id: ccp.c,v 1.6 1994/10/24 04:31:11 paulus Exp $"; #endif #include #include #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, @@ -89,10 +86,23 @@ static fsm_callbacks ccp_callbacks = { */ #define CI_BSD_COMPRESS 0x21 +/* + * Information relating to BSD Compress configuration options. + */ +#define BSD_NBITS(x) ((x) & 0x1F) +#define BSD_VERSION(x) ((x) >> 5) +#define BSD_CURRENT_VERSION 1 +#define BSD_MAKE_OPT(v, n) (((v) << 5) | (n)) + +/* + * Do we want / did we get any compression? + */ +#define ANY_COMPRESS(opt) ((opt).bsd_compress) + /* * 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 */ @@ -108,7 +118,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); @@ -134,7 +144,7 @@ ccp_open(unit) if (f->state != OPENED) ccp_flags_set(unit, 1, 0); - if (!ccp_wantoptions[unit].bsd_compress) + if (!ANY_COMPRESS(ccp_wantoptions[unit])) f->flags |= OPT_SILENT; fsm_open(f); } @@ -179,7 +189,16 @@ 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."); } /* @@ -240,8 +259,8 @@ ccp_resetci(f) if (go->bsd_compress) { opt_buf[0] = CI_BSD_COMPRESS; opt_buf[1] = CILEN_BSD; - opt_buf[2] = go->bsd_bits; - if (!ccp_test(f->unit, opt_buf, 3, 0)) + opt_buf[2] = BSD_MAKE_OPT(BSD_CURRENT_VERSION, go->bsd_bits); + if (!ccp_test(f->unit, opt_buf, CILEN_BSD, 0)) go->bsd_compress = 0; } } @@ -273,8 +292,11 @@ ccp_addci(f, p, lenp) if (go->bsd_compress) { p[0] = CI_BSD_COMPRESS; p[1] = CILEN_BSD; - p[2] = go->bsd_bits; - p += 3; + p[2] = BSD_MAKE_OPT(BSD_CURRENT_VERSION, go->bsd_bits); + if (ccp_test(f->unit, p, CILEN_BSD, 0)) + p += CILEN_BSD; + else + go->bsd_compress = 0; } *lenp = p - p0; } @@ -292,11 +314,11 @@ ccp_ackci(f, p, len) ccp_options *go = &ccp_gotoptions[f->unit]; 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 || p[0] != CI_BSD_COMPRESS || p[1] != CILEN_BSD + || p[2] != BSD_MAKE_OPT(BSD_CURRENT_VERSION, go->bsd_bits)) return 0; - p += 3; - len -= 3; + p += CILEN_BSD; + len -= CILEN_BSD; } if (len != 0) return 0; @@ -320,14 +342,17 @@ 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->bsd_compress && !no.bsd_compress && len >= CILEN_BSD + && p[0] == CI_BSD_COMPRESS && p[1] == CILEN_BSD) { 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]; + 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; len -= CILEN_BSD; } @@ -358,9 +383,9 @@ 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) + if (go->bsd_compress && len >= CILEN_BSD + && p[0] == CI_BSD_COMPRESS && p[1] == CILEN_BSD) { + if (p[2] != BSD_MAKE_OPT(BSD_CURRENT_VERSION, go->bsd_bits)) return 0; try.bsd_compress = 0; p += CILEN_BSD; @@ -390,7 +415,7 @@ ccp_reqci(f, p, lenp, dont_nak) { int ret, newret; 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]; @@ -419,19 +444,22 @@ ccp_reqci(f, p, lenp, dont_nak) } 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; + nb = ao->bsd_bits; + } else if (nb < MIN_BSD_BITS) { + newret = CONFREJ; } else if (!ccp_test(f->unit, p, CILEN_BSD, 1)) { - if (ho->bsd_bits > MIN_BSD_BITS) + if (nb > MIN_BSD_BITS) { + --nb; newret = CONFNAK; - else + } else newret = CONFREJ; } if (newret == CONFNAK && !dont_nak) { - p[2] = (ho->bsd_bits < ao->bsd_bits? MIN_BSD_BITS: - ao->bsd_bits); + p[2] = BSD_MAKE_OPT(BSD_CURRENT_VERSION, nb); } break; @@ -461,10 +489,6 @@ ccp_reqci(f, p, lenp, dont_nak) /* * CCP has come up - inform the kernel driver. */ -static char *up_strings[] = { - "no ", "receive ", "transmit ", "" -}; - static void ccp_up(f) fsm *f; @@ -473,8 +497,10 @@ ccp_up(f) ccp_options *ho = &ccp_hisoptions[f->unit]; ccp_flags_set(f->unit, 1, 1); - syslog(LOG_INFO, "%scompression enabled", - up_strings[(go->bsd_compress? 1: 0) + (ho->bsd_compress? 2: 0)]); + if (go->bsd_compress || ho->bsd_compress) + syslog(LOG_NOTICE, "%s enabled", + go->bsd_compress? ho->bsd_compress? "Compression": + "Receive compression": "Transmit compression"); } /* @@ -488,7 +514,6 @@ ccp_down(f) UNTIMEOUT(ccp_rack_timeout, (caddr_t) f); ccp_localstate[f->unit] = 0; ccp_flags_set(f->unit, 1, 0); - syslog(LOG_NOTICE, "Compression disabled."); } /* @@ -505,7 +530,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; @@ -547,7 +572,8 @@ ccp_printpkt(p, plen, printer, arg) switch (code) { case CI_BSD_COMPRESS: if (optlen >= CILEN_BSD) { - printer(arg, "bsd %d", p[2]); + printer(arg, "bsd v%d %d", BSD_VERSION(p[2]), + BSD_NBITS(p[2])); p += CILEN_BSD; } break; @@ -568,10 +594,15 @@ ccp_printpkt(p, plen, printer, arg) /* * 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 that, so instead we log a message - * and take CCP down :-(. (See US patent 5,130,993; international - * patent publication number WO 91/10289; Australian patent 73296/91.) + * 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) @@ -583,26 +614,25 @@ ccp_datainput(unit, pkt, len) f = &ccp_fsm[unit]; if (f->state == OPENED) { - syslog(LOG_ERR, "Lost compression sync: disabling compression"); - ccp_close(unit); - } -} - -/* - * ccp_req_reset - Request that the peer reset the compression - * dictionary. - */ -void -ccp_req_reset(f) - fsm *f; -{ - if (f->state == OPENED) { - if (!(ccp_localstate[f->unit] & RACK_PENDING)) { - fsm_sdata(f, 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; + 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, 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; + } } }