From Kelly Carmichael <[email protected]>: remove a bogus increment of

the packet data pointer from the code to print a 32-bit RADIUS numerical
attribute value (the increment made sense for TUNNEL_TYPE and
TUNNEL_MEDIUM attributes, which have an 8-bit tag and 24-bit value, and
the increment skips over the tag after printing it; however, for other
attributes, with a 32-bit value, it's bogus).

diff --git a/CREDITS b/CREDITS
index ef546d4ea560013253a653b705e8ab477167aa06..f7df588d801ab889241fef8b2ecd4d4e261b72b1 100644 (file)
--- a/CREDITS
+++ b/CREDITS
@@ -33,6 +33,7 @@ Additional people who have contributed patches:
        Jeffrey Hutzelman               <[email protected]>
        Jim Hutchins                    <[email protected]>
        Juergen Schoenwaelder           <[email protected]>
+       Kelly Carmichael                <[email protected]>
        Ken Hornstein                   <[email protected]>
        Kevin Steves                    <[email protected]>
        Kris Kennaway                   <[email protected]>

diff --git a/print-radius.c b/print-radius.c
index 09952087947a9149b3177423247f21c55c350654..86e9cd896c75604f3a4a08a682abff8b86d1b0cc 100644 (file)
--- a/print-radius.c
+++ b/print-radius.c
@@ -23,7 +23,7 @@
 
 #ifndef lint
 static const char rcsid[] =
-    "$Id: print-radius.c,v 1.8 2001-06-25 21:13:33 itojun Exp $";
+    "$Id: print-radius.c,v 1.9 2001-07-15 19:27:45 guy Exp $";
 #endif
 
 #ifdef HAVE_CONFIG_H
@@ -498,7 +498,6 @@ print_attr_num(register u_char *data, u_int length, u_short attr_code )
       }
       else
       {
-        data++;
          data_value = EXTRACT_32BITS(data);
       }
       if ( data_value <= (attr_type[attr_code].siz_subtypes - 1 +