Add program_name field in the netdissect_options structure

A program that use the library should set it. Done for tcpdump.
ndo_error() and ndo_warning() print now 'ndo->program_name'.

Moreover:
Fix indent

Remove obsolete comments

Fix a comment

Rename cstr[] to istr[] like invalid string

Moreover:
Hamonise the output for error messages
Add istr[] in print-babel.c

DHCPv6: Update Status Codes with RFCs/IANA names

Use the word 'invalid' for 'malformed' or 'corrupted' packets

An invalid packet could be:
1) built malformed originally by the sender or a fuzz tester,
2) became corrupted in transit.

Printers must include 'netdissect.h', not 'interface.h'

Functions in libnetdissect must use ndo_error() function

amend some comments in util-print.c

When I needed to print a string and didn't remember which of the three
functions fn_print(), fn_printn() and fn_printzp() was the right one
for the data, every time it would end up in reading through all of them
and forgetting the difference shortly after the commit.

Just having it explained in the comments should work better.

Printers must use ndo_error() function (via ndo->ndo_error)

Keep the two line numbers in esp_print_decode_onesecret() separate.

Have the line number for the file we're opening for "file" be separate
from the line number we're passed.  That avoids warnings, and makes it
clearer *which* line number we're using.

Avoid collison with error(), clean up code a bit.

Rename the variable to "error_status", as that's what it represents, and
as that doesn't collide with the error() function.  Don't set it and
then not use the resulting value.

Avoid collisions with isascii().

Those variables are counts, so just give them names that reflects that.

Avoid colliding with a tcpdump routine's name.

Just call the variable "data", not "print_data"; we're obviously
printing it.

Don't collide with strlen().

strlen() is a standard C function, so don't use its name for a variable.

Get rid of warnings on systems that provide index().

Rename the variable "index" to "idx", so that if the environment in
which we're compiling tcpdump happens to declare the index() function
(the old V7 name for the function called strchr() in S3/S5 and ANSI C),
we don't get compiler warnings.

The official #define for 32-bit and 64-bit Windows is _WIN32.

It's _WIN32, with a leading underscore, not WIN32.  See, for example:

https://sourceforge.net/p/predef/wiki/OperatingSystems/

and

https://msdn.microsoft.com/en-us/library/b0084kay.aspx

*Some* environments may also define WIN32, but we shouldn't depend on
that.

Fix to reflect dBm radiotap values now being displayed as "dBm" rather than "dB".

Merge pull request #479 from rlucia/patch-1

dBm values get printed as dB

dBm values get printed as dB

This is a very old bug, and I think it's time to get fixed :-)

Add a test that makes unaligned accesses.

From GitHub issue #478, in which tcpdump crashed on SPARC due to making
an unaligned access.

Don't assume the ONC RPC header is nicely aligned.

Use UNALIGNED_MEMCPY() to extract the XID from it; otherwise, this might
crash on machines that require strict alignment (e.g., SPARC machines).

Fixes GitHub issue #478.

Compile with '-Wpedantic' in devel mode as an attempt to get gcc-ism

Fix warnings as "comma at end of enumerator list"

The warnings were:
comma at end of enumerator list [-Wpedantic]

Fix a warning as "ISO C90 forbids mixed declarations and code"

The warning was:
./tcpdump.c: In function 'droproot':
./tcpdump.c:496:3: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
   int ret = capng_change_id(pw->pw_uid, pw->pw_gid, CAPNG_NO_FLAG);
   ^

Compile with '-Wdeclaration-after-statement' in devel mode

Fix warnings as "ISO C90 does not support the '%T' gnu_strftime format"

The complete warnings were:
ISO C90 does not support the '%T' gnu_strftime format [-Wformat=]

Fix warnings as "ISO C90 does not support the '%lf' gnu_printf format"

The complete warnings were:
ISO C90 does not support the '%lf' gnu_printf format [-Wformat=]

Babel: address a couple compiler warnings

677:31: warning: variable ‘router_id’ set but not used
676:72: warning: variable ‘hopc’ set but not used

update CREDITS

Babel: add decoder for source-specific extension.

Fix typo in comment.

Give more details about packet time stamps.

Don't speak of "Ethernet" and "wire", as you might not be sniffing an
Ethernet or, indeed, any form of wired network.

Note that not only could there be a delay between the point at which the
interface is finished receiving the packet and when an interrupt is
delivered (whether due to bus delays, polling rather than immediate
interrupts being used, or delays in the CPU responding to the interrupt,
or more than one of those) but also a delay between the point at which
the kernel responds to the interrupt and the point at which it actually
applies a time stamp to the packet.

Give a URL for a paper about that protocol.

Compile with -Wshadow

update reference for UDP port 3799

Merge pull request #470 from qnet-herwin/radius_rfc3580

I haven't included any unit tests, because the trivial change this is. It has been tested locally and it has been seen working there. It doesn't break any existing unit tests either.

Fun fact: this value is used to assign a VLAN to a port or WLAN user, I wouldn't be surprised if it's the most common used value for Tunnel-Type.

Add Value 13 "VLAN" to Tunnel-Type RADIUS attribute

Expand a comment to indicate what the "16 bits" are.

NetBIOS Name Service can run over TCP.

Clean up the port definitions.

From Bill Parker: sort the port definitions, add some additional
ports, and use #ifndef/#endif around all definitions.

From me: use consistent indentation, add the RFCs for NetBIOS-over-TCP
to the #defines for the NBT ports, and don't keep the IPv6 ports
separate.

ISOCLNS: Fix some warnings

The warnings were:
./print-isoclns.c: In function 'osi_print_cksum':
./print-isoclns.c:3107:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
             || length > ndo->ndo_snaplen
                       ^
./print-isoclns.c:3108:32: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
             || checksum_offset > ndo->ndo_snaplen
                                ^

Add Bill Parker to the credits.

Add checks for strdup() failure.

Add checks for strdup() failure.

Add checks for strdup() failure.

We're not doing setjmp/longjmp, so no need for static variables.

We stopped doing setjmp/longjmp a while ago (it could cause issues on
multiple platforms), so we don't need to make pointer variables static
in the lookup routines.

refine use of nameser.h

Move the port number #define's to the TCP and UDP files such that they
don't require nameser.h anymore. Update the TCP printer to disregard the
multicast DNS port as it is UDP-only.

Fix warnings as "declaration of 'x' shadows a global declaration"

The warnings were:
./tcpdump.c: In function 'show_tstamp_types_and_exit':
./tcpdump.c:220:56: warning: declaration of 'pd' shadows a global declaration [-Wshadow]
 show_tstamp_types_and_exit(const char *device, pcap_t *pd)
                                                        ^
./tcpdump.c:200:16: warning: shadowed declaration is here [-Wshadow]
 static pcap_t *pd;
                ^
./tcpdump.c: In function 'show_dlts_and_exit':
./tcpdump.c:253:48: warning: declaration of 'pd' shadows a global declaration [-Wshadow]
 show_dlts_and_exit(const char *device, pcap_t *pd)
                                                ^
./tcpdump.c:200:16: warning: shadowed declaration is here [-Wshadow]
 static pcap_t *pd;
                ^

Fix warnings as "declaration of 'x' shadows a previous local"

RX: Fix 168 warnings as "declaration of 'x' shadows a previous local"

An example of these warnings is:
./print-rx.c: In function 'fs_print':
./print-rx.c:774:35: warning: declaration of 'i' shadows a previous local [-Wshadow]
 #define UINTOUT() { unsigned long i; \
                                   ^
./print-rx.c:916:4: note: in expansion of macro 'UINTOUT'
    UINTOUT();
    ^
./print-rx.c:883:16: warning: shadowed declaration is here [-Wshadow]
  unsigned long i;
                ^

BGP: Fix a warning

Thanks to Gisle Vanem to catch this.

The warning was:
./print-bgp.c: In function 'bgp_attr_print':
./print-bgp.c:2214:35: warning: declaration of 'atype' shadows a parameter [-Wshadow]
                     u_int aflags, atype, alenlen, alen;
                                   ^
./print-bgp.c:1322:22: warning: shadowed declaration is here [-Wshadow]
                u_int atype, const u_char *pptr, u_int len)
                      ^

TCP: Use quoted include for 'signature.h' instead of angle-bracketed one

Travis: Restore 'make install' in the build script

Moreover:
Delete trailing spaces
Delete redundant empty lines

make build container happy

Merge pull request #469 from gvanem/patch-2

Fix print-icmp.c warning

Merge pull request #468 from gvanem/patch-1

Fix print-dccp.c warning

update the DNSSEC test case after the last change

Merge pull request #467 from wessels/master

add a DNSSEC test case

This is related to GH #445 and GH #467.

Fix print-icmp.c warning

Fix compilation warning:
```
print-icmp.c(559): warning C4456: declaration of 'vec' hides previous local declaration
print-icmp.c(344): note: see declaration of 'vec'
```

Update print-dccp.c

Fix warning:
```
print-dccp.c(500): warning C4456: declaration of 'cp' hides previous local declaration
print-dccp.c(277): note: see declaration of 'cp'
```

interface.h,netdissect.h: Cleaning unused items

BGP: Replace printf by ND_PRINT

Fix display of timestamps with -ttt and -ttttt options

- Fix display of some nanoseconds timestamps
For example, bad print 00:-16:-2.000851566 is now 00:00:00.037851566

- Fix display of timestamp of a packet when it is lower than previous one
For example, bad print 00:00:-1.000999790 is now -00:00:00.000000210

BGP: add decoding of ADD-PATH capability

This implements the capability part of draft-ietf-idr-add-paths-10 and
seems to work for a packet capture I am looking into. The problem with
the "extended NLRI encodings" defined in the same document is that they
are going to use a different structure for the two previously assigned
path attributes, which makes decoding of an UPDATE difficult without
having both relevant OPENs from the same session.

refine tok2str() buffer use

Switch RRCP and BGP functions from tok2strbuf() to tok2str() to avoid
unnecessary local buffer management. The latter function does it in a
way to allow up to 4 calls to itself from printf(). After that
tok2strbuf() remains used by tok2str() only and can be refined later.

Remove a duplicate macro and place the remaining definition into the
right file.

struct netdissect_options: Remove unused members 'ndo_infodelay' and 'ndo_info'

Remark: ^T is a tcpdump function, not a packet-dissection mechanism.

struct netdissect_options: Remove unused member 'ndo_aflag'

Moreover:
Remove redundant empty line.

Fix display of nanoseconds timestamps with -tt option (GH issue 466)

Add 'ts_unix_format' function.

Fix a non-reentrant code in a function

Delete the 'static char buf' array in ts_format().
buf is now a parameter of ts_format().
Moreover:
Decrease a too long string.
Fix a space in a comment.

For DNS messages, represent the "DNSSEC OK" bit as "DO" instead of "OK"

"DO" is a better description of the DNSSEC OK bit in the EDNS0 OPT record.  See RFC 3225 section 3.  There may be other bits allocated in the future that call themselves "Something OK".

Map *ONLY* ASCII letters; don't map anything with the 8th bit set.

Do case-insensitive comparisons assuming ASCII strings.

Do the case-insensitive comparisons in a locale-independent fashion that
only maps ASCII letters, in the standard English-language fashion; that
way, we don't get bitten by, for example, Turkish having separate "i
with dot" and "i without dot" letters, with lower-case "i with dot" being
mapped to upper-case "I with dot" rather than being mapped to "I".

Get rid of macros already defined in netdissect.h

Move more prototypes in netdissect.h

Move some prototypes in netdissect.h

Get rid of macros already defined in netdissect.h

Checksum stuff must go in netdissect.h (remainder)

Checksum stuff must go in netdissect.h

Delete trailing spaces/tabs

Fix a Coverity warning

Test already done by 'ND_TCHECK2(tptr[3], length - 3);'

Coverity: build script: Update the download URL of 7.6 release

BGP: Add support for the AIGP attribute (RFC7311)

Based on a patch from Hannes Gredler

Merge pull request #455 from brooksdavis/gndo-cleanup

Remove global netdissect_object and remove non-dissection related fields

Merge pull request #457 from brooksdavis/no-sandbox-message

Remove "capability mode sandbox enabled" message.

Merge branch 'master' of github.com:the-tcpdump-group/tcpdump

add testcases for IS-IS POI (rfc6232)

Update test output for the previous change

add support for IS-IS Purge Originator Identifier as per RFC6232.

Don't assume you have <net/if_pflog.h> if you have <net/pfvar.h>.

Debian/kFreeBSD ships <net/pfvar.h> but not <net/if_pflog.h>.

SFLOW: Fix bounds checking

Remove "capability mode sandbox enabled" message.

We have removed all instances of these messages in FreeBSD as they serve
little purpose and break some comsumers.

RPL: Fix 'Consistency Check' control code

RPL: Fix suboption print

OLSR: Fix indent and comments

Avoid too long lines.
Not all C compilers accept C++/C99 comments by default.

olsr: add a test to cover a HNA sgw case

Signed-off-by: Ferry Huberts <[email protected]>

olsr: fix printing of smart-gateway HNAs in IPv4

Signed-off-by: Ferry Huberts <[email protected]>

olsr: fix 'Advertised networks' count

Signed-off-by: Ferry Huberts <[email protected]>

Source and destination addresses were backwards.