s/u_short/u_int16_t/ for KAME-origin source codes

[tcpdump] / print-atalk.c

diff --git a/print-atalk.c b/print-atalk.c
index 5134ebc19933234e3f7f5290ebc6dc0906e94fb9..274e0fb571d235e9f40cbe5d1e34b47703059517 100644 (file)
--- a/print-atalk.c
+++ b/print-atalk.c
@@ -23,7 +23,7 @@
 
 #ifndef lint
 static const char rcsid[] =
 
 #ifndef lint
 static const char rcsid[] =

-    "@(#) $Header: /tcpdump/master/tcpdump/print-atalk.c,v 1.51 1999-11-21 09:36:48 fenner Exp $ (LBL)";
+    "@(#) $Header: /tcpdump/master/tcpdump/print-atalk.c,v 1.54 2000-04-02 23:50:48 itojun Exp $ (LBL)";

 #endif
 
 #ifdef HAVE_CONFIG_H
 #endif
 
 #ifdef HAVE_CONFIG_H


@@ -71,13 +71,13 @@ static struct tok type2str[] = {
 };
 
 struct aarp {
 };
 
 struct aarp {

-       u_short htype, ptype;
-       u_char  halen, palen;
-       u_short op;
-       u_char  hsaddr[6];
-       u_char  psaddr[4];
-       u_char  hdaddr[6];
-       u_char  pdaddr[4];
+       u_int16_t       htype, ptype;
+       u_int8_t        halen, palen;
+       u_int16_t       op;
+       u_int8_t        hsaddr[6];
+       u_int8_t        psaddr[4];
+       u_int8_t        hdaddr[6];
+       u_int8_t        pdaddr[4];

 };
 
 static char tstr[] = "[|atalk]";
 };
 
 static char tstr[] = "[|atalk]";


@@ -166,9 +166,9 @@ aarp_print(register const u_char *bp, u_int length)
 
        printf("aarp ");
        ap = (const struct aarp *)bp;
 
        printf("aarp ");
        ap = (const struct aarp *)bp;

-       if (ap->htype == 1 && ap->ptype == ETHERTYPE_ATALK &&
+       if (ntohs(ap->htype) == 1 && ntohs(ap->ptype) == ETHERTYPE_ATALK &&

            ap->halen == 6 && ap->palen == 4 )
            ap->halen == 6 && ap->palen == 4 )

-               switch (ap->op) {
+               switch (ntohs(ap->op)) {

 
                case 1:                         /* request */
                        (void)printf("who-has %s tell %s",
 
                case 1:                         /* request */
                        (void)printf("who-has %s tell %s",


@@ -185,8 +185,9 @@ aarp_print(register const u_char *bp, u_int length)
                            AT(pdaddr), AT(psaddr));
                        return;
                }
                            AT(pdaddr), AT(psaddr));
                        return;
                }

-       (void)printf("len %d op %d htype %d ptype %#x halen %d palen %d",
-           length, ap->op, ap->htype, ap->ptype, ap->halen, ap->palen );
+       (void)printf("len %u op %u htype %u ptype %#x halen %u palen %u",
+           length, ntohs(ap->op), ntohs(ap->htype), ntohs(ap->ptype),
+           ap->halen, ap->palen);

 }
 
 static void
 }
 
 static void


@@ -507,11 +508,11 @@ ataddr_string(u_short atnet, u_char athost)
                while (fgets(line, sizeof(line), fp)) {
                        if (line[0] == '\n' || line[0] == 0 || line[0] == '#')
                                continue;
                while (fgets(line, sizeof(line), fp)) {
                        if (line[0] == '\n' || line[0] == 0 || line[0] == '#')
                                continue;

-                       if (sscanf(line, "%d.%d.%d %s", &i1, &i2, &i3,
+                       if (sscanf(line, "%d.%d.%d %256s", &i1, &i2, &i3,

                                     nambuf) == 4)
                                /* got a hostname. */
                                i3 |= ((i1 << 8) | i2) << 8;
                                     nambuf) == 4)
                                /* got a hostname. */
                                i3 |= ((i1 << 8) | i2) << 8;

-                       else if (sscanf(line, "%d.%d %s", &i1, &i2,
+                       else if (sscanf(line, "%d.%d %256s", &i1, &i2,

                                        nambuf) == 3)
                                /* got a net name */
                                i3 = (((i1 << 8) | i2) << 8) | 255;
                                        nambuf) == 3)
                                /* got a net name */
                                i3 = (((i1 << 8) | i2) << 8) | 255;


@@ -538,7 +539,8 @@ ataddr_string(u_short atnet, u_char athost)
                if (tp2->addr == i) {
                        tp->addr = (atnet << 8) | athost;
                        tp->nxt = newhnamemem();
                if (tp2->addr == i) {
                        tp->addr = (atnet << 8) | athost;
                        tp->nxt = newhnamemem();

-                       (void)sprintf(nambuf, "%s.%d", tp2->name, athost);
+                       (void)snprintf(nambuf, sizeof(nambuf), "%s.%d",
+                           tp2->name, athost);

                        tp->name = savestr(nambuf);
                        return (tp->name);
                }
                        tp->name = savestr(nambuf);
                        return (tp->name);
                }


@@ -546,10 +548,11 @@ ataddr_string(u_short atnet, u_char athost)
        tp->addr = (atnet << 8) | athost;
        tp->nxt = newhnamemem();
        if (athost != 255)
        tp->addr = (atnet << 8) | athost;
        tp->nxt = newhnamemem();
        if (athost != 255)

-               (void)sprintf(nambuf, "%d.%d.%d",
+               (void)snprintf(nambuf, sizeof(nambuf), "%d.%d.%d",

                    atnet >> 8, atnet & 0xff, athost);
        else
                    atnet >> 8, atnet & 0xff, athost);
        else

-               (void)sprintf(nambuf, "%d.%d", atnet >> 8, atnet & 0xff);
+               (void)snprintf(nambuf, sizeof(nambuf), "%d.%d", atnet >> 8,
+                   atnet & 0xff);

        tp->name = savestr(nambuf);
 
        return (tp->name);
        tp->name = savestr(nambuf);
 
        return (tp->name);


@@ -569,7 +572,7 @@ ddpskt_string(register int skt)
        static char buf[8];
 
        if (nflag) {
        static char buf[8];
 
        if (nflag) {

-               (void)sprintf(buf, "%d", skt);
+               (void)snprintf(buf, sizeof(buf), "%d", skt);

                return (buf);
        }
        return (tok2str(skt2str, "%d", skt));
                return (buf);
        }
        return (tok2str(skt2str, "%d", skt));