Add changes in 4.2.1.

Merge remote branch 'github/master'

Merge pull request #2 from bodgit/master

Teach PPPoE parser about RFC 4638

Teach PPPoE parser about RFC 4638

Fix a typo (from the Red Hat tcpdump package) and use .LP.

Get rid of an unused variable.

Fix a bunch of "sizeof(sizeof(XXX))".

In some places, there was one too many levels of sizeof() -
sizeof(sizeof(XXX)) is sizeof(size_t), but we wanted the size of type
XXX.

Reviewed-By: Guy Harris <[email protected]>

Add a CARP dissector and a command-line option to dissect proto 112 as CARP.

CARP and VRRP both use IP protocol number 112, so there needs to be a -T
flag to specify that protocol 112 be dissected as CARP rather than VRRP.

Also update the man page.

Redo length checks in ospf6_print_lsa().

Check to make sure we haven't run past the end of the LSA by doing
length checks - and be a bit fussier about length checks.  Do more
end-of-packet checks as well.

Constify some arguments.

Put in missing break.

Reviewed-by: Guy Harris <[email protected]>

Remove unnecessary test.

Also, from me: add a comment explaining why the test isn't being done.

Reviewed-by: Guy Harris <[email protected]>

Clean up option-walking code.

Loop as long as the remaining option list length is not zero, even if
that means we try to process the remaining options if the remaining
length is 1, so that if the option length is bogus, we'll report it.

Check for a valid ESIS_OPTION_ES_CONF_TIME length - it's supposed to be
2.

Fix test output - IP protocol 72 is in /etc/services on at least some UN*Xes.

Reviewed-By: Guy Harris <[email protected]>

Change name of Hilscher link types to say NETANALYER.

That makes the names a bit shorter, and mentions the specific Hilscher
product to which they apply.

Add printers for the Hilscher Ethernet link-layer types.

Merge pull request #1 from jhsmt/master

restore some test cases for forces

Handle systems with getrpcbynumber() but no header file for it.

Glibc 2.14 doesn't install the ONC RPC headers, but it installs the ONC
RPC routines, presumably for binary compatibility.  Don't use
getrpcbynumber() unless we have it *and* the header file to declare it.

Merge branch 'master' of git+ssh://bpf.tcpdump.org/tcpdump/master/git/tcpdump

Conflicts:
Makefile.in

For Babel, recognise both UDP/6696 and UDP/6697.

IANA has reallocated the Babel port; it is now 6696.  This patch makes
tcpdump recognise both the old and the new Babel ports.

remove deleted files from list of distributed files

removed unnecessary file

Get rid of another zero-length array.

Don't compile print-babel.c if we don't have IPv6 support.

Get rid of GCCisms.

Not all C compilers support anonymous unions in structures, zero-length
array members of structures, or __attribute__.

Don't assume we have IPv6 support.

Declare all local variables before any executable statements.

Some C compilers let you get away with that C++-ism; not all do.

Make sure ppi.h is in the release tarball.

Reviewed-By: Guy Harris <[email protected]>

Restore forces3 pcap and associated output

Signed-off-by: Jamal Hadi Salim <[email protected]>

Restore forces2 pcap and associated output

Signed-off-by: Jamal Hadi Salim <[email protected]>

Update the EXTRA_DIST list to reflect what's now in the tests directory.

Changes for 4.2 of tcpdump

Fwd: pcap files

On Tue, Jun 28, 2011 at 3:52 PM, Michael Richardson <[email protected]> wrote:

> I'm not aware of a new file.

The email i responded to had 4 attachments sent by Evangelos.
Those were supposed to replace the files with those exact names.

> Please send github tree, ideally.

You mentioned github to me last time and offered to get me to
learn it in 5 minutes;->  I havent had time and the old school stuff i do
still works.
How about i send you patch #1 to delete the old files (as attached)
and another to re-add with new ones.
Alternatively: I could send one that overrides the existing ones.

cheers,
jamal
> --
> ]       He who is tired of Weird Al is tired of life!           |  firewalls  [
> ]   Michael Richardson, Sandelman Software Works, Ottawa, ON    |net architect[
> ] [email protected] http://www.sandelman.ottawa.on.ca/ |device driver[
>   Kyoto Plus: watch the video <http://www.youtube.com/watch?v=kzx1ycLXQSE>
>                       then sign the petition.
>

commit d93443f24bfb5fd982ff33deb66979bae811db57
Author: Jamal Hadi Salim <[email protected]>
Date:   Tue Jun 28 16:15:49 2011 -0400

    [PATCH] Remove test files using old ForCES ports

Signed-off-by: Jamal Hadi Salim <[email protected]>

ignore more cruft

removed unneeded pcap file

Note that ndo_error doesn't, and *ndo_error shouldn't, return.

Tcpdump's ndo_error() doesn't return.  Any other ndo_error routine
supplied to netdissect shouldn't, either, as printers expect it not to.

Use "-H", not "-h", for the 802.11s option, and improve error reporting.

I was rudely surprised to find that "tcpdump -h" wasn't printing a usage
message, and I'm the person who *added* the "-h" option.  Make it "-H",
and add an explicit "-h" option to print a usage message, so nobody else
makes the same mistake.  Also, don't clear opterr, so that if you give
an illegal command-line option, you get an explanatory error message.

from Weesan Lee <[email protected]>: display pim bidir support

Handle ICMP6 checksums more like TCP checksums.

Instead of printing the result of icmp6_cksum() if it's non-zero, print
the checksum field value and the value it should have had.  That means
that what we print is the same regardless of whether we're running on a
big-endian or little-endian machine.

Fix some warnings.

Add a routine to do the "checksum with pseudo-header" stuff for IPv4.

Clean up some other stuff while we're at it.

Use nextproto6_cksum() for XXX-over-IPv6 checksums.

Get rid of duplicated checksums with IPv6 pseudo-headers.

Handle UDP checksums more like TCP checksums.

Instead of printing the result of udp_cksum() if it's non-zero, print
the checksum field value and the value it should have had.  That means
that what we print is the same regardless of whether we're running on a
big-endian or little-endian machine.

Also, just as we did with TCP:

Check -v and -K, and the fragmented flag, up front; then check the IP
version etc..  Don't check for IPv6 if we already know it's IPv4.  Fetch
the checksum field only once.

Update some test files for the new output format.

Get rid of $Id$ - you can't get that with Git.

Clean up the checksum checking.

Check -v and -K, and the fragmented flag, up front; then check the IP
version etc..  Don't check for IPv6 if we already know it's IPv4.  Fetch
the checksum field only once.

add support for the RPKI/Router Protocol as per draft-ietf-sidr-rpki-rtr-12

Go with Wireshark's Internet checksum routine.

The Wireshark routine is based on the BSD in-kernel portable checksum
routine (thus BSD-licensed); it takes a vector of pointers and lengths
and checksums the concatenation of the buffers in question (just as the
BSD in-kernel routine checksums a chain of mbufs).

This simplifies the "with a pseudo-header" checksums; hopefully it'll
fix up the problems being seen on some big-endian platforms, which might
be due to hand-calculating some or all of the checksum and doing so
incorrectly.  It also gets rid of some code that might be dereferencing
unaligned pointers.

Update sflow test output to reflect recent changes to print-sflow.c.

Fix some problems with the tests.

Get rid of some no-longer-necessary uudecodes (Git can store binary
files such as pcap files, so we no longer need to uuencode them, and the
uuencoded files are no longer around), and handle the "-X" and "-XX"
flag tests (where we had to rename the "should be" output files to avoid
collisions on case-insensitive file systems such as the default local
file system on the desktop UN*X with the biggest market share).

The ifSpeed field of a generic interface counter in sFlow is 64 bits.

The "overlay" definition in print-sflow.c is correct, but the actual
extract for printing is using EXTRACT_32BITS rather than EXTRACT_64BITS,
which leads to an incorrect report for speed.

Reviewed-By: Guy Harris <[email protected]>

Don't use "struct netdissect_options" in interface.h.

Use it in netdissect.h, where it's defined; otherwise, we get a bunch of
warnings when compiling modules that include interface.h but don't yet
include netdissect.h.

From: Darren Reed <[email protected]>
To: [email protected]
Date: Sat, 09 Apr 2011 12:51:14 +1000
Subject: [tcpdump-workers] Printing PPI packets

Printing PPI packets with tcpdump does not turn out
to be that hard.

My simple tests have produced the output as below.

It would be worthwhile having some changes made into
the tcpdump code base that were similar to the attached
that print them out.

removed need for uudecode

added babal test cases

Merge branch 'master' of git+ssh://bpf.tcpdump.org/tcpdump/master/git/tcpdump

updated forces test cases

Fix endianness of some parameters.

Reviewed-By: Guy Harris <[email protected]>

Add error checking and support for DCB Exchange protocol (DCBX) version 1.01.

From Kaladhar Musunuru <[email protected]>:

Added support for DCB Exchange protocol (DCBX) version 1.01.
http://www.ieee802.org/1/files/public/docs2008/az-wadekar-dcbx-capability-exchange-discovery-protocol-1108-v1.01.pdf

DCBX protocol exchanges control state machine and generic feature state
machine parameters as Organizationally specific TLVs.  The OUI used for
the DCBX TLV 1.01 is 0x001B21.  Following TLVs are decoded:

- Control state
- Priority Groups (PG)
- Priority-based Flow Control (PFC)
- Application Protocol (APP)

From me:

Add a bunch of additional error checking, and sort the main switch
statement by TLV code (and thus by the order in the specification).
Also update/add indications of what standards document what items.

Babel dissector, from Juliusz Chroboczek and Grégoire Henry.

Reviewed-By: Guy Harris <[email protected]>

Try to handle radiotap Rate values > 127 better.

Use the same heuristic Wireshark uses, i.e. assume that FreeBSD only
reports MCS indices between 0 and 15, and that any other values with the
0x80 bit set are rate values.  (Yes, those do exist.)

Also note, as I did in Wireshark, that it might be possible to extract
from the XChannel and Flags field the additional information to convert
an MCS index from the Rate field into a rate.  (Whether that's possible
depends on what platforms using the FreeBSD convention do with the
channel width and guard interval information.)

Merge branch 'master' of git+ssh://bpf.tcpdump.org/tcpdump/master/git/tcpdump

Add printer for the Babel routing protocol (RFC 6126).

This version includes a couple of fixes from Wireshark.

added sflow test cases

Support RX flags, MCS and the vendor namespace, and fix Rate.

Based on patches from an anonymous donor, support the radiotap RX flags and
MCS fields, and the vendor namespace, and, if Channel and XChannel are
both present, use XChannel, not Channel.

Do not try to look up a rate for an MCS value from the Rate field; you
cannot map an MCS value to a rate without also knowing the channel width
and guard interval length.

Handle the enterprise field of the sflow flow and counter record format.

Thanks to some traces sent my way by Gavin McCullagh, and a comparison
against the output of inMon's sflowtool, I can confidently say "Yes
Virginia, there is an enterprise other than zero."  Which means lest we
start trying to decode something as what it is not, we best actually
look at the enterprise field and make sure it is one we recognize.

To enable printing of non-expanded samples I've shuffled a bunch of code
around and created a bunch of smaller routines to more easily support
printing of both expanded and non-expanded counter and flow samples.
I've done simple testing of non-expanded counter and flow, and expanded
counter, but I don't have expanded flow at present with which to test.
So, that part of the change is only compile/eyeball tested.

Reviewed-and-tweaked-by: Guy Harris <[email protected]>

Fix the handling of unknown tagged parameters in management frames.

Fetch the element length early in the process, rather than fetching it
each time we need it - even after we've advanced the pointer we are
using to fetch it (doing the latter means we fetch the wrong value when
we subtract it from the remaining length).

Process the flags even if parsing the full radiotap header fails.

If we were able to get the flags, we should process them, so we know
whether there's Atheros padding after the 802.11 header or if there's an
FCS at the end of the packet, and properly process them; if we weren't
able to get the flags, they're 0, so we're not doing anything
differently, and that's the best we can do.

Don't return value from a filed cpack_uint8() call.

If the cpack_uint8() call to fetch the flags for
IEEE80211_RADIOTAP_FLAGS fails, don't return u.u8, as we didn't fetch
anything into it.

Check the sample, flow, and counter lengths.

When dissecting a sample, or a flow or counter within the sample, make
sure we don't run past the length in the sample/flow/counter header.

This also means we count the sample length down as we process the
contents of the sample, so we only add what, if anything, remains of the
sample to the pointer into the packet, and subtract that from the
remaining data length in the packet, which means we properly handle
packets with multiple samples.

Include <tcpdump-stdinc.h> when checking for headers.

Now that we're setting V_INCLS correctly, we don't need to include
"tcpdump-stdinc.h" - including <tcpdump-stdinc.h> works.

This also picks up configure changes from the previous configure.in
change.

Properly define {u_}intN_t types when testing pcap/*.h headers.

pcap/bluetooth.h and pcap/usb.h require those types, so we check for
those types before checking those headers, and we include
"tcpdump-stdinc.h" in the tests, so the appropriate headers to define
those types get pulled in.

Expand srcdir in V_INCLS at configure time, not make time.

That way, if we temporarily set CPPFLAGS to include V_INCLS, it'll work
correctly.

Fix one place where we didn't save and restore CPPFLAGS when modifying it.

Have empty.uu explicitly be a uuencoding of /dev/null.

That way, uudecode will write the decoded output there, rather than to a
file named "empty" that gets left around.  That also means we don't need
to redirect the output of uudecode to /dev/null, so don't do that.

Check for uudecode at make time by uudecoding a uuencoded empty file.

Doing it at make time means you don't have to re-run the configure
script if you add uudecode to your system, and doing it by uudecoding a
uuencoded empty file means we don't depend on uudecode supporting
"--help".

Don't assume uudecode supports a --help option; not all uudecodes do.

Instead, check for uudecode in the configure script, and set the
CHECK_UUDECODE configuration variable to "true" if we find it and to a
command that prints an error message and exits with 1 if we don't find
it, and substitute that variable into the Makefile as the first command
for "make check".

RESEND: [PATCH 1/1] ForCES fix SCTP TML port

The old ports for SCTP TML ports were replaced at RFC editor
publication time. Caught by folks preparing for an Interop
tommorow.
While we could add a backward compat support it seems meaningless
if those ports will never ever be used in production.
I will send new test/pcaps at a later point and remove the existing
ones once this patch goes in.

cheers,
jamal

commit 40ecad0e866e7567716e6e57158c5274c0786a21
Author: Jamal Hadi Salim <[email protected]>
Date:   Thu Feb 3 22:00:23 2011 -0500

    Conform to new IANA assignment of ForCES SCTP TML

Signed-off-by: Jamal Hadi Salim <[email protected]>

Updated VERSION

Merge branch 'master' of git+ssh://bpf.tcpdump.org/tcpdump/master/git/tcpdump

Fix plurals in packet count messages.

Based on a patch from [email protected], but with a macro
PLURAL_SUFFIX() defined to return either "s" or "" (rather than possibly
printing a NUL character with %c), and with that macro used in a couple
of cases where the equivalent had been done by hand, and with one case
the patch missed fixed as well.

From Minto Jeyananth <[email protected]>: add support for IS-IS Generic Cryptographic Authentication as per RFC 5310

From Minto Jeyananth <[email protected]>: fix the IS-IS MT flag tokentable

Fix a comment, and note a potential issue.

Don't clear stuff we'll be overwriting in its entirety.

We only need to clear out the tha structure if we support IPv6, so that
the addresses are IPv6 addresses, but we only have IPv4 addresses, so
that the rest of the address bytes need to be set somehow.

Also, explain what we're doing, so people don't ask why we're not
zeroing it if we don't support IPv6.

libpcap doesn't have that comment; remove it here.

Squelch warnings about datarootdir.

And another quoting of AC_LANG_SOURCE.

Put the call to AC_LANG_SOURCE in AC_C___ATTRIBUTE__ in quotes.

That's what they do in the autoconf manual; perhaps it'll squelch some
warnings I'm seeing on NetBSD 5.1 with autoconf 2.68.

Ignore os-proto.h - it might be a symlink to os/os-XXX.h for some XXX.

Get rid of unnecessary configure script tests and missing/XXX.h headers.

This also fixes configure-script problems on OSes that have IPv6 support
but don't have <netinet6/in6.h> - no standard I could find requires that
header for IPv6 support (the Single UNIX Specification speaks of
<netinet/in.h> for both IPv4 and IPv6, and RFCs 2292 and 3542 don't
mention <netinet6/in6.h>, just <netinet/in.h> and <netinet6/ip6.h>).

Fix e-mail address for tcpdump-workers.

Thanks to Cameron Elliott for pointing out that the address hadn't been
updated.

Add test pcap+output for nested and recursive pathdata

Signed-off-by: Jamal Hadi Salim <[email protected]>
Signed-off-by: Michael Richardson <[email protected]>

Use proper type (16bit) when extracting path id count

Signed-off-by: Jamal Hadi Salim <[email protected]>
Signed-off-by: Michael Richardson <[email protected]>

Fix mixing recursive and flat Path Data

 +--PD
 .  +--PD
 .  |
 .  |
 .  +--PD
 .
 .
 +--PD
 |
 |
 +--PD

or
 +--PD
 .  +--PD
 .     +
 .     |
 .     +--PD
 .         +
 .         |
 .         +-- PD
 .         +
 .         |
 .         +-- PD
 +--PD
 |
 |
 +--PD

etc

Signed-off-by: Jamal Hadi Salim <[email protected]>
Signed-off-by: Michael Richardson <[email protected]>

Fix indent space when we have padded data

See if you can spot the difference between

    Pathdata: Flags 0x0 ID count 1
      ID#01: 3
     FULLDATA TLV (Length 10 DataLen 6 pad 2 Bytes)
       [
       0x0000:  0002 0406 080a 0000
       ]

and:
    Pathdata: Flags 0x0 ID count 1
      ID#01: 3
      FULLDATA TLV (Length 10 DataLen 6 pad 2 Bytes)
       [
       0x0000:  0002 0406 080a 0000
       ]

Signed-off-by: Jamal Hadi Salim <[email protected]>
Signed-off-by: Michael Richardson <[email protected]>

Point people to the autoconf documentation on shell substitution.

Hopefully this will keep people from sending "helpful" patches that
break things.

Handle POSIX quoting rules.

See section 11.6 "Shell Substitutions" in the autoconf documentation,
especially the

There is just no portable way to use double-quoted strings
inside double-quoted back-quoted expressions (pfew!).

part.

ip6_print() now takes a netdissect_options pointer as its first argument.

mistype of RPL type codes: off by one. fixed