Update config.{guess,sub}, timestamps 2018-07-06,2018-07-03

From repository git://git.savannah.gnu.org/config.git

Add a test from Kamil Frankowicz for the BFD dissector.

 Clean up handling of different packet types.

Only look at the header for control packets; we don't know what the
payload contains for other packet types.

This fixes some cases where we fail to check whether we have a full
header before fetching from the header - we only need to fetch from the
header for control packets, so we now only need to check that we have it
for control packets; make sure we *don't* look at the header for other
packet types.

Update the v0 code to match draft-ietf-bfd-base-01, which was the last
draft that discussed v0.

Update the timestamp in the man page.

[skip ci]

Pointers to packet bytes should be u_char *, not void *

ICMPv6: Fix printing 'Reachable Time' and 'Retrans Timer' as milliseconds

Update 'retrans time' to 'retrans timer'.

See: https://tools.ietf.org/html/rfc4861#section-4.2.

Update the output of some tests accordingly.

Clean up dissection.

Don't use pointers to anything other than octets; there is no guarantee
that the L2TP packet is aligned on a 2-byte or 4-byte boundary, and
there is no need to pretend that we have pointers to aligned values -
we're using the EXTRACT_ macros, which will fetch multi-byte integral
values regardless of the alignment of the pointer.

This also fixes some cases where we were advancing 2 bytes after
processing a 1-byte field - we were incrementing a uint16_t * by 1,
which means advancing it by 2 bytes, and we're now incrementing the
uint8_t * by 1.

Don't cast a 4-byte integer to u_long - EXTRACT_BE_U_4() is guaranteed
to return something printable with %u.

Don't fetch fields dividded into "high" and "low" portions 2 bytes at a
time and reassemble them; the only reason they're divided into "high"
and "low" partitions in the ASCII-art diagrams in RFC 2661 is that those
diagrams tend to show packets in the form of 32-bit words, and those
fields aren't aligned on 32-bit word boundaries, so we can just fetch
those fields with EXTRACT_BE_U_4().

Don't print a sequence of AVPs by recursion; iterate instead.

Add some RFC numbers while we're at it.

Use HTTPS in the www.tcpdump.org URLs.

[skip ci]

Use EXTRACT_ macros, don't reassemble multi-byte quantities ourselves.

ESP: Fix some unreachable code warnings

Found with -Wunreachable-code clang compiler option.

The errors were:
./print-esp.c:263:3: warning: code will never be executed
                                                 [-Wunreachable-code]
                free(input_buffer);
                ^~~~
./print-esp.c:246:3: warning: code will never be executed
                                                 [-Wunreachable-code]
                EVP_CIPHER_CTX_free(ctx);
                ^~~~~~~~~~~~~~~~~~~
./print-esp.c:843:5: warning: code will never be executed
                                                 [-Wunreachable-code]
                                free(input_buffer);
                                ^~~~
./print-esp.c:826:5: warning: code will never be executed
                                                 [-Wunreachable-code]
                                EVP_CIPHER_CTX_free(ctx);
                                ^~~~~~~~~~~~~~~~~~~

DVMRP: Fix an undefined behavior at runtime

Found with -fsanitize=undefined clang compiler option.

The error was:
print-dvmrp.c:238:27: runtime error: left shift of 240 by 24 places cannot
be represented in type 'int'

Sort the tested compiler warning options

syslog: fix a typo in a comment

[skip ci]

Add a missing space for DNS over TCP (GH #688)

[skip ci]

No need to check whether data is present if there isn't any data.

update CREDITS

Move 3 past maintainers to the respective section, fixup formatting for
consistency and specify ordering within sections.

[skip ci]

ICMPv6: Add a length check

If the length is zero, it's invalid, print that.

Add a test with this case.

Get rid of no-longer-used nfsuint64.

nd_uint64_t suffices - it doesn't require 8-byte alignment (XDR doesn't
provide 8-byte alignment for 64-bit quantities).

Use nd_uint types for structures used to overlay packets.

Clean up processing of RPC request header.

Don't just blast through it and do a single check at the end to make
sure we didn't run past the end of the packet; check for the
fixed-length part of the credentials, then check for the variable-length
part of the credentials, and then do the same two steps for the
verifier.

Fix the checks against the on-the-network length while we're at it.

Make roundup2() generally available.

Expand a comment.

Fix test for truncated sname and file.

nd_printztn() returns 0 if the string is truncated; check for it
returning zero, not for returning a non-zero value.

This fix was supplied in GitHub issue #685.

RIP: Add some bounds checks

IEEE 802.11: Add a bounds check

All four BSDs.

Add some EXTRACT_U_1() macro calls

Add two EXTRACT_U_1() macro calls

Fix a typo in the EXTRACT_BE_S_6() macro

S: Signed

Update an email address

Add more nd_print_trunc() calls

Update the output of some tests accordingly.

Include conditionally <config.h> in netdissect-alloc.c

This should suppress the warning reported by Gisle Vanem:

In file included from netdissect-alloc.c:18:
In file included from ./netdissect-alloc.h:22:
./netdissect.h(131,14):  warning: '_strdup' redeclared without 'dllimport'
    attribute: previous 'dllimport' ignored [-Winconsistent-dllimport]
extern char *strdup (const char *str);
             ^
./netdissect-stdinc.h(219,18):  note: expanded from macro 'strdup'
  #define strdup _strdup
                 ^

IP: Stop decoding when truncation occurs in options

We just keep the checksum control.

UDP: Add two bounds checks

ICMPv6: Avoid reinventing ND_TCHECK_*() macros

IP: Print clearer when the version is invalid

Update the output of some tests accordingly.

SUNRPC: Add a bounds check

UDP: Add a bounds check

VJC: Add some bounds checks

MPTCP: Add a bounds check

Simplify some expressions

In the equality case, the difference is also zero.

Add more nd_print_trunc() calls

Update the output of some tests accordingly.

Moreover:
Add a ndo_protocol assignment in pimv2_print().

DCCP: Fix a truncation message

It's a truncated packet, not sure it's invalid.

Make the NFLOG test case unconditional

There is no more HAVE_PCAP_NFLOG_H macro to test.

libpcap has support for NFLOG in current in 1.8.1 and in old releases
1.7.4, 1.6.2 and 1.5.3.

Thus there should be no problem to always do the test.

Add a nd_print_trunc() call

Update the output of a test accordingly.

ES-IS,IS-IS: Add two missing return

Update the output of some tests accordingly.

UDP: Add two missing return

Bluetooth: Add a missing return

We no longer need pcap/nflog.h nor pcap/bluetooth.h, step 2

Remove trailing spaces/tabs

Update closing #endif comment.

We no longer need pcap/nflog.h nor pcap/bluetooth.h.

Cast dport and sport to u_int before shifting them.

The result of the expression is ultimately going to be put into a u_int;
cast them to u_int so that we'll be shifting unsigned values left rather
than int values, to avoid undefined behavior.

This should fix GitHub issue #681.

Encrypted data buffer sizes must also be a multiple of the cipher block size.

This should address the issue in GitHub pull request #680.

Declare the NFLOG pseudo-header ourselves.

It's not specified by a libpcap header that might have a different
layout in different pcap releases, it's specified on the list of
link-layer header types and must remain the same forever (except for
getting additional bits defined), so we don't need to pick it up from
libpcap.

This means we get to use tcpdump's nd_ types; do so.

Declare the Bluetooth pseudo-header ourselves.

It's not specified by a libpcap header that might have a different
layout in different pcap releases, it's specified on the list of
link-layer header types and must remain the same forever (except for
getting additional bits defined), so we don't need to pick it up from
libpcap.

This means we get to use tcpdump's nd_ types; do so.

Add more nd_print_trunc() calls

Update the output of some tests accordingly.

Moreover:
Update a ndo_protocol assignment.

Add a nd_print_trunc() call

Moreover:
Rename a function with _print like in most similar cases

DECnet: Avoid using EXTRACT_ macros on non packet buffer pointers

Remove two now useless variables.

DCCP: Fix printing "Timestamp" and "Timestamp Echo" options

Add some comments.

Moreover:
Put a function definition name at the beginning of the line.

Rx: Use the fn_print_str() function instead of nd_print()

We cannot use nd_print() because the parameter is not a pointer on the
packet buffer.

Add more nd_print_trunc() calls

Update the output of a test accordingly.

Moreover:
Avoid printing useless '\n'.

NTP: Fix extracting the poll value

Use EXTRACT_S_1 for ppoll (signed).

DECnet: Pointers to packet bytes should be u_char *, not char *

Add a cast to the EXTRACT_U_1() macro

This change may avoid sign bit extension problem.

Replace some double parentheses with single parentheses

Pointers to packet bytes should be u_char *, not char *.

timed: Fix bounds check when printing Machine Name

IEEE 802.11: Print the SSID with the fn_print_str() function

We cannot use nd_print() because the parameter is not a pointer on the
packet buffer.

Add the fn_print_str() function

This function print a null-terminated string, filtering out non-printable
characters.
DON'T USE IT with a pointer on the packet buffer because there is no
truncation check. For this use, see the nd_printX() functions.

Add a nd_print_trunc() call

Replace a ND_TTEST_*() with ND_TCHECK_*().
Update the output of some tests accordingly.

Add a nd_print_trunc() call

Moreover:
Replace a ND_TTEST_*() with ND_TCHECK_*().

Replace some ND_TTEST_*() with ND_TCHECK_*()

Add more nd_print_trunc() calls

Update the output of some tests accordingly.

Use more the nd_print_trunc() call

Update the output of a test accordingly.

TCP: Fix a closing string

It was ">", it must be "]"

Update the output of some tests accordingly.

Fix space printing with istr[]

Because istr[] is assign to " (invalid)", avoid adding a space before it.

Moreover:
Avoid also useless '\n'

PGM: Remove unused variable and code

Add more nd_print_trunc() calls

Update the output of some tests accordingly.

TCP: Add a missing 'truncated' message

Add more nd_print_trunc() calls

Update the output of a test accordingly.

Fix truncation checks in CONTRIBUTING

Add a nd_print_trunc() call

Update the output of some tests accordingly.

Add more nd_print_trunc() calls

Update the output of some tests accordingly.

Make the hex-dumping routines for addresses take a uint8_t * argument.

Hopefully, that will convince Coverity that the result of dereferencing
those pointers will have a value between 0x00 and 0xff, and therefore
that shifting that result right by 4 bits will yield a value between 0x0
and 0xf, and therefore that this result can safely be used as an index
into the 16-element hex[] array.

I guess Coverity wants us to realize that there really *are* C
implementations out there with non-8-bit char and unsigned char values,
even though getting tcpdump to work on them will probably be a highly
entertaining exercise (not to mention that the only one I know of that's
actually being *used* are the Unisys Clearpath Dorado series, and
they're one's complement, which is yet *another* place where the port
could be entertaining...).

Add a nd_print_trunc() call

Update the output of a test accordingly.

Moreover:
Add a ndo_protocol reassignment after xxx_print() calls.

Add a nd_print_trunc() call

Update the output of some tests accordingly.

Moreover:
Add a ndo_protocol reassignment after ip_print() call.

Add more nd_print_trunc() calls

Update the output of some tests accordingly.

Moreover:
Add a ndo_protocol field assignment.

Remove some spaces (style)

Update hex_and_ascii_print_with_offset() to static

Add more nd_print_trunc() calls

Update the output of some tests accordingly.

Add more nd_print_trunc() calls

Print truncations with nd_print_trunc() instead of tstr[] strings

Remove the tstr[] strings.
Update the output of some tests accordingly.

Moreover:
Add or update some ndo_protocol fields.

Add the nd_print_trunc() function

Use the ndo->ndo_protocol field to print a " [|protocol]" message.

Change update-test.sh to update multiple tests

Resync SMI list against Wireshark

Remove the safeputs() function

Use the nd_printzp() function instead.

Remove two duplicate lines

Remove the safeputchar() function

Print the characters filtering out non-printable with fn_print_char().

Update the output of some tests accordingly.

Display the "wireless" flag and connection status.