diff options
| author |   dlg <dlg@openbsd.org> | 2019-04-02 11:10:54 +0000 |
|---|---|---|
| committer | dlg <dlg@openbsd.org> | 2019-04-02 11:10:54 +0000 |
| commit | db14ce8c61c23bd2eaafb0eba002f86db13ddf3e (patch) | |
| tree | 02c7cdec16818935f2f061cdf5e1f7df002468df | |
| parent | annoying white space (diff) | |
| download | wireguard-openbsd-db14ce8c61c23bd2eaafb0eba002f86db13ddf3e.tar.xz wireguard-openbsd-db14ce8c61c23bd2eaafb0eba002f86db13ddf3e.zip | |
print MPLS_MCAST as mpls, rather than unknown
The caveat with this is that MPLS and MPLS_MCAST look exactly the
same. I could prefix the MCAST line with "multicast" or something,
but like everything else in MPLS the meaning of that protocol type
is supposed to have changed. It's no longer meant to indicate
multicast mpls, but mpls with a label selected by an upstream. So
what's the right thing to do?
noticed by Mitchell Krome, who used this breakage to identify another
problem, which won't happen anymore.
| -rw-r--r-- | usr.sbin/tcpdump/print-gre.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/usr.sbin/tcpdump/print-gre.c b/usr.sbin/tcpdump/print-gre.c index 8af67644ab5..c4e3be0d49f 100644 --- a/usr.sbin/tcpdump/print-gre.c +++ b/usr.sbin/tcpdump/print-gre.c @@ -1,4 +1,4 @@ -/* $OpenBSD: print-gre.c,v 1.22 2019/02/05 10:57:48 dlg Exp $ */ +/* $OpenBSD: print-gre.c,v 1.23 2019/04/02 11:10:54 dlg Exp $ */ /* * Copyright (c) 2002 Jason L. Wright (jason@thought.net) @@ -260,6 +260,7 @@ gre_print_0(const u_char *p, u_int length) ip6_print(p, length); break; case ETHERTYPE_MPLS: + case ETHERTYPE_MPLS_MCAST: mpls_print(p, length); break; case ETHERTYPE_TRANSETHER: |