diff options
| author | Francois-Xavier Le Bail <devel.fx.lebail@orange.fr> | 2022-02-20 11:44:03 +0100 |
|---|---|---|
| committer | Francois-Xavier Le Bail <devel.fx.lebail@orange.fr> | 2022-02-20 11:44:25 +0100 |
| commit | c4e44860201f66026ca4d6b8187ac73e33188c99 (patch) | |
| tree | 5305989795c1b31e21d3a67129e491c3d78e651b | |
| parent | 2800b8968362329f5fc586142edc1c64ef8b48b4 (diff) | |
| download | tcpdump-c4e44860201f66026ca4d6b8187ac73e33188c99.tar.gz | |
RADIUS: Use GET_U_1() to replace a direct dereference
Remove some now unnecessary ND_TCHECK_*().
| -rw-r--r-- | print-radius.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/print-radius.c b/print-radius.c index 501ac299..f74b7a6a 100644 --- a/print-radius.c +++ b/print-radius.c @@ -104,7 +104,7 @@ #define PRINT_HEX(bytes_len, ptr_data) \ while(bytes_len) \ { \ - ND_PRINT("%02X", *ptr_data ); \ + ND_PRINT("%02X", GET_U_1(ptr_data)); \ ptr_data++; \ bytes_len--; \ } @@ -1339,11 +1339,9 @@ print_attr_strange(netdissect_options *ndo, return; } ND_PRINT("User_challenge ("); - ND_TCHECK_8(data); len_data = 8; PRINT_HEX(len_data, data); ND_PRINT(") User_resp("); - ND_TCHECK_8(data); len_data = 8; PRINT_HEX(len_data, data); ND_PRINT(")"); @@ -1363,15 +1361,12 @@ print_attr_strange(netdissect_options *ndo, ND_PRINT(", Min password length: %u", GET_U_1(data)); data++; ND_PRINT(", created at: "); - ND_TCHECK_4(data); len_data = 4; PRINT_HEX(len_data, data); ND_PRINT(", expires in: "); - ND_TCHECK_4(data); len_data = 4; PRINT_HEX(len_data, data); ND_PRINT(", Current Time: "); - ND_TCHECK_4(data); len_data = 4; PRINT_HEX(len_data, data); break; @@ -1382,7 +1377,6 @@ print_attr_strange(netdissect_options *ndo, ND_PRINT("ERROR: length %u != 8", length); return; } - ND_TCHECK_8(data); len_data = 8; PRINT_HEX(len_data, data); break; |