X-Git-Url: https://git.tcpdump.org/tcpdump/blobdiff_plain/65a0c47dc8103f3794e07dc35bd72b52a67a76c7..f67e46a634dd3bdcefa9fc09dbef6baae1b1d1ed:/print-macsec.c diff --git a/print-macsec.c b/print-macsec.c index 431ae42c..0c9ea05f 100644 --- a/print-macsec.c +++ b/print-macsec.c @@ -21,17 +21,12 @@ /* \summary: MACsec printer */ -#ifdef HAVE_CONFIG_H #include -#endif -#include - -#include +#include "netdissect-stdinc.h" #include "netdissect.h" #include "addrtoname.h" -#include "ethertype.h" #include "extract.h" #define MACSEC_DEFAULT_ICV_LEN 16 @@ -104,7 +99,7 @@ static void macsec_print_header(netdissect_options *ndo, if (GET_U_1(sectag->tci_an) & MACSEC_TCI_SC) ND_PRINT(", sci " SCI_FMT, GET_BE_U_8(sectag->secure_channel_id)); - + ND_PRINT(", "); } @@ -123,7 +118,7 @@ int macsec_print(netdissect_options *ndo, const u_char **bp, u_int short_length; save_protocol = ndo->ndo_protocol; - ndo->ndo_protocol = "MACsec"; + ndo->ndo_protocol = "macsec"; /* we need the full MACsec header in the capture */ if (caplen < MACSEC_SECTAG_LEN_NOSCI) { @@ -152,7 +147,7 @@ int macsec_print(netdissect_options *ndo, const u_char **bp, } else sectag_len = MACSEC_SECTAG_LEN_NOSCI; - if ((GET_U_1(sectag->short_length) & ~MACSEC_SL_MASK) != 0 || + if ((GET_U_1(sectag->short_length) & ~MACSEC_SL_MASK) != 0 || GET_U_1(sectag->tci_an) & MACSEC_TCI_VERSION) { nd_print_invalid(ndo); ndo->ndo_protocol = save_protocol; @@ -219,6 +214,13 @@ int macsec_print(netdissect_options *ndo, const u_char **bp, } *lengthp -= MACSEC_DEFAULT_ICV_LEN; *caplenp -= MACSEC_DEFAULT_ICV_LEN; + /* + * Update the snapend thus the ICV field is not in the payload for + * the caller. + * The ICV (Integrity Check Value) is at the end of the frame, after + * the secure data. + */ + ndo->ndo_snapend -= MACSEC_DEFAULT_ICV_LEN; /* * If the SL field is non-zero, then it's the length of the @@ -245,7 +247,7 @@ int macsec_print(netdissect_options *ndo, const u_char **bp, if (*caplenp > short_length) *caplenp = short_length; } - + ndo->ndo_protocol = save_protocol; return -1; }