Amend the documentation about the use of the -C -G and -W switches.

When using all three switches (-C -G and -W), the behavior is
practically identical to only using -C and -G, as the -W flag will only
affect the filename suffix.

Fixes #695

Squelch a narrowing warning.

Use a u_char variable for characters in the RST string; no need for it
to be an int.

Have all Internet-checksum computing routines return a uint16_t.

Those checksums are 16-bit; change the return types of those routines
appropriately.

Squelch some narrowing warnings.

Mostly, we just throw in casts of values already known to be in the
range of values supported by the lvalue's type to that type.

Also, for the IPX SAP table, we make the SAP value a uint16_t, as SAP
values are 16-bit.

Make a hash table index unsigned, Just Because.

Umm, wrong.

Add, and use, macros to do locale-independent case mapping.

This means we get the same behavior in Turkish locales (where, if we
aren't in the C locale, we might get lower-case "i" mapped to upper-case
"I with dot" and upper-case "I" mapped to lower-case "i without dot),
and may also suppress some shortening warnings from MSVC.

Revert "Try to squelch a warning about non-constant format strings."

This reverts commit 76ed0bb39212d964946a22380288c34adc09030f.

It doesn't help.

We may just want to suppress the warning, as in "trust us, we know what
we're doing", although it'd be nice if the format string passed to those
functions could be checked to make sure it is one that takes exactly one
integer argument.

Try to squelch a warning about non-constant format strings.

Declare the format-string argument to tok2strbuf(), tok2str(),
bittok2str(), and bittok2str_nosep() as such, to see whether that gets
MSVC not to complain - or if it just gets upset because there's no "..."
argument list for it.

Type and size cleanups.

Make the count of 2-octet values to dump an unsigned value.

Make the variables into which we fetch those octets unsigned as well.

Cast the result of the "is the character printable ASCII?  If so, it's
the character, otherwise it's '.'" to char, to squelch narrowing
warnings.

Squelch a warning.

We're returning the calculated checksum result modulo 65536, given that
Internet checksums are 16-bit; throw in a cast to let MSVC know that
this is intended.

Try to squelch another annoying repeated warning.

Build with the Npcap SDK as well as the WinPcap SDK.

Change tests/hoobr_bfd_print.pcap to test the BFD printer again.

Clamp the on-the-wire length values of the packets at 262144, so that we
hand the packets to the BFD printer, rather than just bailing out before
we ever get there.

Revert "Ethernet: Remove two useless tests"

This reverts commit b7fff7058282c332ba1d8d236c9864ab21b16adf (partially)

The sanity check in print.c/pretty_print_packet() function, packet length
(length) >= capture length (caplen), apply when the ether_print()
function is called by ether_if_print(), netanalyzer_if_print() or
netanalyzer_transparent_if_print().

But the ether_print() function is called in some other cases, thus not
sure length always >= caplen.

Handle radiotap+802.11 for DLT_LINUX_SLL2 as well.

Fix printing of Linux cooked captures with monitor-mode packets.

Apparently, if you have an interface in monitor mode, capturing on the
"any" device can get packets that have a SLL hatype of 803, which is the
ARPHRD_ value for radiotap, and with the payload containing a radiotap
header, followed by an 802.11 header, followed by the 802.11 payload.
Handle that.

ICMPv6: add a comment for the previous commit

[skip ci]

show ICMPv6 echo request ID

DCCP: Fix an undefined behavior at runtime

DCCP: Fix some undefined behaviors at runtime

The errors were like:
print-dccp.c:448:4: runtime error: unsigned integer overflow: 20 - 24
cannot be represented in type 'unsigned int'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior print-dccp.c:448:4

Squelch annoying bogus Clang warnings.

Require sys/capsicum.h to use Capsicum.

My PC-BSD 9.1 VM, at least, has sys/capability.h but not sys/capsicum.h;
we now use sys/capsicum.h, so require it to be present (which it is in
FreeBSD 10 and later).

Fix typo.

Require sys/capsicum.h to use Capsicum.

My PC-BSD 9.1 VM, at least, has sys/capability.h but not sys/capsicum.h;
we now use sys/capsicum.h, so require it to be present (which it is in
FreeBSD 10 and later).

Merge pull request #715 from rbgarga/fix_capsicum_header

Include correct capsicum header

Include correct capsicum header

Current included header `sys/capability.h` is deprecated. All
supported FreeBSD versions have now the proper header `sys/capsicum.h`

Use more nd_print_invalid()

Compile with -Wsign-compare in devel mode if supported (cmake)

Compile with '-Wsign-compare' in devel mode if supported

Add two test files: an empty pcap and an empty pcapng

Travis CI: Add the "./tcpdump -h" command

Add an usage message for the -m option

Moreover:
Fix a space.

Clean up whitespaces

Remove a space (style)

Moreover:
Fix indentation.

Fix nd_snprintf(buf, buflen, "string");

If this is VS prior to 2015 or MinGW, we can't trust snprintf(); we have
to use _snprintf_s(), but that requires us to insert _TRUNCATE as an
argument after the buffer and buffer length and before the format string
and arguments, if any, to the format string.

That means we need to use a vararg macro; however, if we make the format
string a regular argument to the macro, that means that, if there are no
arguments *after* the format string, you end up with an argument list to
_snprintf_s() that ends with "fmt, ", and that's not valid C.

*If* we knew this was GCC or Clang, we could use a GNU C-specific hack,
wherein, if __VA_ARGS__ is preceded by ## and there's a comma before
that, the comma is removed if __VA_ARGS__ is empty, but this might be
Microsoft's C compiler in a version of Visual Studio prior to VS 2015,
which might not support that.

So we have to just have the macro take, as the ... arguments, the format
string and its arguments.

Addresses GitHub issue #713.

Clean up indentation.

Merge pull request #710 from agnosticdev/return-checks

return-checks: Added a return check for malloc and for strsep

return-checks: Added a return check for malloc and for strsep

man page: Update the -ttt and -ttttt options sections

The delta is printed with microsecond or nanosecond resolution depending
on the --time-stamp-precision option (default is microsecond resolution).

Harmonize a message

s/capture size/snapshot length/

It's a follow-up to 8a54b2483b.

Add the snapshot length to the "reading from file ..." message

Use signed formats for signed values

Merge pull request #707 from herwinw/cleanup_win32_GNUMakefile

Fix tab/spaces in GNUmakefile of win32

Change unknown attribute in test

The value 127 is actually assigned to the Location-Information Attribute
(RFC 5580). Change it to a very high value.

Fix tab/spaces in GNUmakefile of win32

Remove unused TCP ports

Merge pull request #704 from nmap/win32-isatty

Ensure isatty is defined on Win32.

Remove unused UDP ports

Ensure isatty is defined on Win32.

SLIP: Remove printing the raw hex dump of the compressed header

ndo_nflag has not this function. We don't currently have an appropriate
flag for this use. Not sure printing the header in hex is currently useful.

Update the output of a test accordingly.

Add a nd_print_trunc() call

ICMPv6: Fix some truncation codes

IP6OPTS: Fix some truncation codes

Moreover:
Update the output of some tests accordingly.
Fix style.

Update a test file.

The bug fix in ccd4a6b3b9165e575b8d425ea8ea96a8cb146df0 means we
correctly set the packet length to be the PDU length, so we can check
the checksum.

Clean up some narrowing warnings.

Cast pointer differences, widen some variables, make some function
returns unsigned as they never return a negative number.

Fix a format while we're at it.

Fix the variable for the packet length.

The length passed in is a u_int, so we should assign it to a u_int, not
a u_short.

SLIP: Add some bounds checks

This should fix GitHub issues #676 and #677.

Update a comment

IP: Simplify an expression

Merge pull request #701 from loganaden/master

Update my email address

Update my email address

Reorganize the dissection code a bit.

Get rid of the global nfserr variable and, instead, have it be local to
interp_reply().  That means one less global variable, which may be good
if any multi-threaded program tries to use libnetdissect in more than
one thread (there are still global variables that would have to be
removed), and gets rid of some cases where we didn't need to set it.

Don't bundle multiple operations inside an if clause.  This squelches
some warnings from MSVC, and makes the code a bit more uniform and a bit
easier to understand (and, in the process of doing that, found a bug
that was fixed in 596aca3d9384e739961df0b5c9ffd9d8161d6c9b).

Fix dissection of NFSv3 WRITE and LINK replies.

Don't report them all as truncated if -v isn't specified.

That wasn't correct.

Clean up the name resolution a bit.

If we're not doing name-to-address resolution, don't bother with the
address-to-name table.

Don't warn about declarations after statements.

We now allow them, and require a compiler that supports them, so there's
no need to warn about them.

Don't warn about declarations after statements.

We now allow them, and require a compiler that supports them, so there's
no need to warn about them.

Add more nd_print_trunc() calls

Fix data types.

If you're accumulating the lengths of strings, as returned by strlen(),
and passing the result to malloc(), it should be accumulated in a
size_t.

Move getopt_long.h to missing.

That's what we did for win_ether_ntohost.h.

Revert "Clean up the code a bit."

This reverts commit a5eca2cbbfa991c0ae26d0b039bf303fa92734ac.

MSVC doesn't complain about it, so leave it alone.

Clean up the code a bit.

This eliminates a warning from MSVC, and makes the flow a little
clearer.

(Yes, it duplicates some code, but compilers have been pretty good at
merging common code sequences, so it might just turn it into the
equivalent of

if (optopt == (int)':')
goto label;
oli = strchr(ostr, optopt);
if (!oli) {
label:
/*
 * Unknown option character.
 */
if (!*place)
++optind;

...
}

although it does mean that the same C code exists in two places.)

BGP: Fix an error string

"[|" is the prefix for a truncated string. This is not the case here.

Remove some useless tests

Because packet length (length) >= capture length (caplen).

(see the sanity checks in print.c, pretty_print_packet() function)

See if a newer MSVC will produce fewer warnings.

Some of the narrowing warnings are false positives; newer versions of
Clang do a better job than older ones of determining the range of values
of expressions, and don't warn if the range is such that it can be
safely narrowed, so maybe the same is true of MSVC.

Do parallel builds, run as a developer.

Tell msbuild to build in parallel.  Use Windows syntax for command-line
flags.  Create the .devel file.

Try to find the 64-bit libraries in a 64-bit build.

Do both 32-bit and 64-bit builds.

Add a nd_print_trunc() call

Add more nd_print_trunc() calls

Add more nd_print_trunc() calls

SLL: Remove two useless tests

Because packet length (length) >= capture length (caplen).

(see the sanity checks in print.c, pretty_print_packet() function)

Fix style

Ethernet: Remove two useless tests

Because packet length (len) >= capture length (caplen).

(see the sanity checks in print.c, pretty_print_packet() function)

Ethernet: Remove two useless tests

Because packet length (length) >= capture length (caplen), when
caplen >= ETHER_HDRLEN, length >= caplen cannot be < ETHER_HDRLEN.

Same for the other test (length < 4).

(see the sanity checks in print.c, pretty_print_packet() function)

Moreover:
Clean up indentation.

Add the nd_print_invalid() function

It prints " (invalid)", used for malformed or corrupted packets.

Moreover:
Update CONTRIBUTING.

Make the invalid snaplen message clearer

More status exit codes for a program using libnetdissect

It can now use status exit codes 0 to 10.

Update a comment with the new macros names

Add the debug macro ND_DEBUG

Don't assume ARM supports unaligned accesses.

Prior to ARMv6, it didn't support it.  ARMv6 supports it, if the "don't
act like ARMv5" bit isn't set in the system control register, but has
another bit in the system control register to cause unaligned accesses
to fault rather than succeed.  ARMv7 and later don't have "act like
ARMv5" mode, but still has the "fault on unaligned access" bit.

At least some OSes might set the "fault on unaligned access" bit; if
your OS *doesn't* set it, feel free to modify the #if to check for your
OS and for ARM.

Clean up white space.

Fix test output files to match Juniper dissector changes.

Clean up indentation.

Remove trailing "_if" from protocol names.

"_if", in function names, is just a convention used for the top-level
routine for a given DLT_ value; it's not part of the protocol name for
the link-layer type.

add test for nfs write verf cookie

test results:
[vmehta@centos72_base tcpdump]# make check | grep nfs
    unaligned-nfs-1                    : passed
    nfs-attr-oobr                      : passed
    hoobr_nfs_printfh                  : passed
    hoobr_nfs_xid_map_enter            : passed
    nfs-seg-fault-1                    : passed
    nfs-write-verf-cookie              : passed <- new test

print verifier cookie for write and commit operation

Fix the fix.

Fix the no-SSL-library version of that test.

Fix tests for BGP changes.