]> git.ozlabs.org Git - ppp.git/blobdiff - modules/vjcompress.c
fix counting bug
[ppp.git] / modules / vjcompress.c
index 4b164b30b180cf2968ff65e42e63a4bf22ce7452..66d3402886c22cdd81dc27b7996a81e1e7a150c6 100644 (file)
  * Modified June 1993 by Paul Mackerras, paulus@cs.anu.edu.au,
  * so that the entire packet being decompressed doesn't have
  * to be in contiguous memory (just the compressed header).
+ */
+
+/*
+ * This version is used under SunOS 4.x, Digital UNIX, AIX 4.x,
+ * and SVR4 systems including Solaris 2.
  *
- * $Id: vjcompress.c,v 1.2 1994/09/19 04:20:13 paulus Exp $
+ * $Id: vjcompress.c,v 1.9 1996/06/26 00:53:17 paulus Exp $
  */
 
 #include <sys/types.h>
 #include <sys/param.h>
+
+#ifdef SVR4
+#ifndef __GNUC__
+#include <sys/byteorder.h>     /* for ntohl, etc. */
+#else
+/* make sure we don't get the gnu "fixed" one! */
+#include "/usr/include/sys/byteorder.h"
+#endif
+#endif
+
+#ifdef __osf__
+#include <net/net_globals.h>
+#endif
 #include <netinet/in.h>
+
+#ifdef AIX4
+#define _NETINET_IN_SYSTM_H_
+typedef u_long  n_long;
+#else
 #include <netinet/in_systm.h>
+#endif
+
 #include <netinet/ip.h>
 #include <netinet/tcp.h>
 
 #define ovbcopy bcopy
 #endif
 
+#ifdef __osf__
+#define getip_hl(base) (((base).ip_vhl)&0xf)
+#define getth_off(base)        ((((base).th_xoff)&0xf0)>>4)
+
+#else
+#define getip_hl(base) ((base).ip_hl)
+#define getth_off(base)        ((base).th_off)
+#endif
+
 void
 vj_compress_init(comp, max_state)
     struct vjcompress *comp;
@@ -101,19 +135,23 @@ vj_compress_init(comp, max_state)
 
 #define DECODEL(f) { \
        if (*cp == 0) {\
-               (f) = htonl(ntohl(f) + ((cp[1] << 8) | cp[2])); \
+               u_int32_t tmp = ntohl(f) + ((cp[1] << 8) | cp[2]); \
+               (f) = htonl(tmp); \
                cp += 3; \
        } else { \
-               (f) = htonl(ntohl(f) + (u_long)*cp++); \
+               u_int32_t tmp = ntohl(f) + (u_int32_t)*cp++; \
+               (f) = htonl(tmp); \
        } \
 }
 
 #define DECODES(f) { \
        if (*cp == 0) {\
-               (f) = htons(ntohs(f) + ((cp[1] << 8) | cp[2])); \
+               u_short tmp = ntohs(f) + ((cp[1] << 8) | cp[2]); \
+               (f) = htons(tmp); \
                cp += 3; \
        } else { \
-               (f) = htons(ntohs(f) + (u_long)*cp++); \
+               u_short tmp = ntohs(f) + (u_int32_t)*cp++; \
+               (f) = htons(tmp); \
        } \
 }
 
@@ -122,7 +160,7 @@ vj_compress_init(comp, max_state)
                (f) = htons((cp[1] << 8) | cp[2]); \
                cp += 3; \
        } else { \
-               (f) = htons((u_long)*cp++); \
+               (f) = htons((u_int32_t)*cp++); \
        } \
 }
 
@@ -135,7 +173,7 @@ vj_compress_tcp(ip, mlen, comp, compress_cid, vjhdrp)
     u_char **vjhdrp;
 {
     register struct cstate *cs = comp->last_cs->cs_next;
-    register u_int hlen = ip->ip_hl;
+    register u_int hlen = getip_hl(*ip);
     register struct tcphdr *oth;
     register struct tcphdr *th;
     register u_int deltaS, deltaA;
@@ -165,7 +203,7 @@ vj_compress_tcp(ip, mlen, comp, compress_cid, vjhdrp)
     INCR(vjs_packets);
     if (ip->ip_src.s_addr != cs->cs_ip.ip_src.s_addr ||
        ip->ip_dst.s_addr != cs->cs_ip.ip_dst.s_addr ||
-       *(int *)th != ((int *)&cs->cs_ip)[cs->cs_ip.ip_hl]) {
+       *(int *)th != ((int *)&cs->cs_ip)[getip_hl(cs->cs_ip)]) {
        /*
         * Wasn't the first -- search for it.
         *
@@ -186,7 +224,7 @@ vj_compress_tcp(ip, mlen, comp, compress_cid, vjhdrp)
            INCR(vjs_searches);
            if (ip->ip_src.s_addr == cs->cs_ip.ip_src.s_addr
                && ip->ip_dst.s_addr == cs->cs_ip.ip_dst.s_addr
-               && *(int *)th == ((int *)&cs->cs_ip)[cs->cs_ip.ip_hl])
+               && *(int *)th == ((int *)&cs->cs_ip)[getip_hl(cs->cs_ip)])
                goto found;
        } while (cs != lastcs);
 
@@ -200,7 +238,7 @@ vj_compress_tcp(ip, mlen, comp, compress_cid, vjhdrp)
         */
        INCR(vjs_misses);
        comp->last_cs = lcs;
-       hlen += th->th_off;
+       hlen += getth_off(*th);
        hlen <<= 2;
        if (hlen > mlen)
            return (TYPE_IP);
@@ -232,7 +270,7 @@ vj_compress_tcp(ip, mlen, comp, compress_cid, vjhdrp)
      */
     oth = (struct tcphdr *)&((int *)&cs->cs_ip)[hlen];
     deltaS = hlen;
-    hlen += th->th_off;
+    hlen += getth_off(*th);
     hlen <<= 2;
     if (hlen > mlen)
        return (TYPE_IP);
@@ -240,9 +278,9 @@ vj_compress_tcp(ip, mlen, comp, compress_cid, vjhdrp)
     if (((u_short *)ip)[0] != ((u_short *)&cs->cs_ip)[0] ||
        ((u_short *)ip)[3] != ((u_short *)&cs->cs_ip)[3] ||
        ((u_short *)ip)[4] != ((u_short *)&cs->cs_ip)[4] ||
-       th->th_off != oth->th_off ||
+       getth_off(*th) != getth_off(*oth) ||
        (deltaS > 5 && BCMP(ip + 1, &cs->cs_ip + 1, (deltaS - 5) << 2)) ||
-       (th->th_off > 5 && BCMP(th + 1, oth + 1, (th->th_off - 5) << 2)))
+       (getth_off(*th) > 5 && BCMP(th + 1, oth + 1, (getth_off(*th) - 5) << 2)))
        goto uncompressed;
 
     /*
@@ -392,8 +430,9 @@ vj_uncompress_err(comp)
  * "Uncompress" a packet of type TYPE_UNCOMPRESSED_TCP.
  */
 int
-vj_uncompress_uncomp(buf, comp)
+vj_uncompress_uncomp(buf, buflen, comp)
     u_char *buf;
+    int buflen;
     struct vjcompress *comp;
 {
     register u_int hlen;
@@ -401,7 +440,12 @@ vj_uncompress_uncomp(buf, comp)
     register struct ip *ip;
 
     ip = (struct ip *) buf;
-    if (ip->ip_p >= MAX_STATES) {
+    hlen = getip_hl(*ip) << 2;
+    if (ip->ip_p >= MAX_STATES
+       || hlen + sizeof(struct tcphdr) > buflen
+       || (hlen += getth_off(*((struct tcphdr *)&((char *)ip)[hlen])) << 2)
+           > buflen
+       || hlen > MAX_HDR) {
        comp->flags |= VJF_TOSS;
        INCR(vjs_errorin);
        return (0);
@@ -409,9 +453,6 @@ vj_uncompress_uncomp(buf, comp)
     cs = &comp->rstate[comp->last_recv = ip->ip_p];
     comp->flags &=~ VJF_TOSS;
     ip->ip_p = IPPROTO_TCP;
-    hlen = ip->ip_hl;
-    hlen += ((struct tcphdr *)&((int *)ip)[hlen])->th_off;
-    hlen <<= 2;
     BCOPY(ip, &cs->cs_ip, hlen);
     cs->cs_hlen = hlen;
     INCR(vjs_uncompressedin);
@@ -438,9 +479,9 @@ vj_uncompress_tcp(buf, buflen, total_len, comp, hdrp, hlenp)
     register u_int hlen, changes;
     register struct tcphdr *th;
     register struct cstate *cs;
-    register struct ip *ip;
     register u_short *bp;
     register u_int vjlen;
+    register u_int32_t tmp;
 
     INCR(vjs_compressedin);
     cp = buf;
@@ -463,7 +504,7 @@ vj_uncompress_tcp(buf, buflen, total_len, comp, hdrp, hlenp)
        }
     }
     cs = &comp->rstate[comp->last_recv];
-    hlen = cs->cs_ip.ip_hl << 2;
+    hlen = getip_hl(cs->cs_ip) << 2;
     th = (struct tcphdr *)&((u_char *)&cs->cs_ip)[hlen];
     th->th_sum = htons((*cp << 8) | cp[1]);
     cp += 2;
@@ -475,15 +516,19 @@ vj_uncompress_tcp(buf, buflen, total_len, comp, hdrp, hlenp)
     switch (changes & SPECIALS_MASK) {
     case SPECIAL_I:
        {
-           register u_int i = ntohs(cs->cs_ip.ip_len) - cs->cs_hlen;
-           th->th_ack = htonl(ntohl(th->th_ack) + i);
-           th->th_seq = htonl(ntohl(th->th_seq) + i);
+           register u_int32_t i = ntohs(cs->cs_ip.ip_len) - cs->cs_hlen;
+           /* some compilers can't nest inline assembler.. */
+           tmp = ntohl(th->th_ack) + i;
+           th->th_ack = htonl(tmp);
+           tmp = ntohl(th->th_seq) + i;
+           th->th_seq = htonl(tmp);
        }
        break;
 
     case SPECIAL_D:
-       th->th_seq = htonl(ntohl(th->th_seq) + ntohs(cs->cs_ip.ip_len)
-                          - cs->cs_hlen);
+       /* some compilers can't nest inline assembler.. */
+       tmp = ntohl(th->th_seq) + ntohs(cs->cs_ip.ip_len) - cs->cs_hlen;
+       th->th_seq = htonl(tmp);
        break;
 
     default:
@@ -502,8 +547,10 @@ vj_uncompress_tcp(buf, buflen, total_len, comp, hdrp, hlenp)
     }
     if (changes & NEW_I) {
        DECODES(cs->cs_ip.ip_id);
-    } else
-       cs->cs_ip.ip_id = htons(ntohs(cs->cs_ip.ip_id) + 1);
+    } else {
+       cs->cs_ip.ip_id = ntohs(cs->cs_ip.ip_id) + 1;
+       cs->cs_ip.ip_id = htons(cs->cs_ip.ip_id);
+    }
 
     /*
      * At this point, cp points to the first byte of data in the