Update .gitignore for other configurations

Add files generated by CMake on Windows to .gitignore

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.

Fix dissection of RFC 4684 routing information.

RFC 4684, sadly, doesn't do a very good job of describing what a "route
target" is.  In practice, it appears to be a Route Target extended
community, as per RFC 4360 section 4 "Route Target Community".  Those do
*not* have the same format as an RFC 4364 Route Descriptor.

So just take the code to dissect an extended community, put it in a
subroutine, change it to directly print information rather than filling
in a static string buffer (C support for strings is spectacularly bad -
large variable-length objects are hard, let's go shopping!), and call
that routine when dissecting RFC 4684 routing information.

Fix spaces

Travis CI: Reduce git clone depth

Squelch a warning on some platforms.

Some platforms define the old V7 routine index() (renamed to strchr() in
System V; the latter name is what's used in ANSI C), and you get
"declaration of 'index' shadows a global declaration" warnings.

Call the variable if_index instead.

RADIUS: constify a couple static arrays

[skip ci]

Fix spaces

CONTRIBUTING: Update about bounds checking

Use more the nd_print_trunc() call

Update the output of some tests accordingly.

EAP: Fixup the check for Code value.

Code value 0 is not assigned, and the code block guarded by the if() is
specific to Request (1) and Response (2) only. This change prevents the
printer from decoding some invalid EAP messages as if they were valid.

[skip ci]

RADIUS: Deduplicate some printing code.

EAP: simplify an if()

[skip ci]

Travis CI: Create .devel for both autotools and CMake

That way, we do development builds for CMake as well, and get the
warnings turned up.

Travis CI: Fold the 'cmake' output

This change allows to see better the possible warnings.

Travis CI: Fix folding the 'configure' output

Travis CI: Simplify the script section

NTP: Fix UTC/GMT timestamps printing according to RFC3339

Add the "Z" suffix for UTC.

According to the RFC3339:
      Z           A suffix which, when applied to a time, denotes a UTC
                  offset of 00:00; often spoken "Zulu" from the ICAO
                  phonetic alphabet representation of the letter "Z".

Update the output of some tests accordingly.

Travis CI: Fix the openssl support in OSX build

A directory name has changed.

Add OSX_SSL_DIR variable to track it.

Report 32-bit timestamps with uppermost bit set as "Error converting time"

This is an update to packet header timestamps printing based on
Guy Harris's proposal.

Theses timestamps are outside the 1970-2038 range; This happens on some
fuzzed captures.

This will allow to have the same printing on Windows and UN*X.

Update needed before 2038.

Update.

-# is used to indicate which packets differ in how they're dissected,
not to ensure the reproducibility of the output.

We also set TZ to GMT0 to do all time-stamp printing in UTC.

Update CONTRIBUTING and TESTLIST according to a recent change

It's a follow-up to 018b2b8c96.

Thanks to Gisle Vanem to spot one of the inconsistencies.

Moreover:
Remove a useless '-#' option in TESTLIST.

Refactor printing the time fractional part ({micro|nano}seconds)

Add the ts_frac_print() function.

Avoid almost duplicate code.

Use UTC/GMT time when building/checking tests files

This will avoid some differences when checking in different time zones.

We now run the tests without the '-t' option. This will allow to get
problems/changes in time printing functions.

Update the output of the tests accordingly.

Moreover:
Add the '-#' option to better identify a packet when there is a difference
in output.

Explicitly check to make sure the header length isn't too short.

Add '-Wunreachable-code-return' warning option in devel mode for cmake

Sort the tested compiler warning options for cmake

Remove the no more used gmt2local() function

Fix local time printing

Using seconds offset from GMT to local time to compute local time give
errors when printing times outside the same daylight saving time period.

Use the localtime() function for default and -tttt cases.
Use the gmtime() function for -ttt and -ttttt cases.

Rename ts_hmsfrac_print() to ts_date_hmsfrac_print().
Remove some variables now useless.
Update some comments.

RADIUS: Decode more MIP6-Feature-Vector bits.

Add more allocations from and a link to the Mobility Capability registry.

Fix spaces

Add a test for the RX/AFS protocol.

The .pcap file is from https://wiki.wireshark.org/SampleCaptures, as
suggested on the openafs-devel mailing list.

Merge pull request #636 from herwinw/radius

Improve printout of MIP6-Feature-Vector

Added RADIUS attributes from RFC 5447

* MIP6-Feature-Vector (124) (with extra attributes from RFC 6572)
* MIP6-Home-Link-Prefix (125)

BGP: Add two comments

Fix some spelling in comments.

[skip ci]

Refresh the CI status badges in README.md.

[skip ci]

Clean up indentation.

Add check for pcap timestamp warnings. (GH #693)

Previously would silently accept any of "host", "adapter", or
"adapter_unsynced" regardless of whether the interface supported
the option.

CLNP: Squelch a cppcheck warning (GH #629)

See also commit f944e1e39.

[print-isoclns.c:906] -> [print-isoclns.c:910]: (warning) Opposite inner
'if' condition leads to a dead code block.

[skip ci]

Don't print timestamps to a buffer and then print the buffer.

Just directly print them.  This means:

1) we don't have to worry about making a big enough buffer;

2) we don't get annoying warnings about string truncation
   due to invalid timestamp values;

3) we use a constant string as a format, so that the compiler
   can more easily do format string vs. format argument checks.

If none of those are defined, there's nothing to dissect.

Wrap everyting in the big #if.

Wrap unused definitions and declarations in #ifdef/#if.

Some stuff is used only by certain dissectors; define/declare it only if
the dissectors that use it are being compiled.

Include pcap-missing.h, to get the declaration of pcap_datalink_val_to_name().

If we're building with an old version of libpcap that doesn't include
pcap_datalink_val_to_name(), we incorporate our own version, but we need
to have it declared before we use it.

Surround some declarations/definitions with #ifdefs.

Some variables are used only if certain pcap APIs are available; don't
define them if the APIs aren't available.

We don't define show_tstamp_types_and_exit() unless we have
pcap_set_tstamp_type(); don't declare it if we don't define it.

Update the list of files in INSTALL.txt.

[skip ci]

Rx: Fix remaining fall-through warnings (GH #618).

Benjamin Kaduk explains in the issue comments:

The first two ("bulk stat" and "callback"; lines 1023 and 1282) should
not fall through, but the other two ("list entry" in various forms,
lines 1741 and 1790) should fall through. The answer lies in the RPC-L
files at [1], [2] and [3] (also line 396), looking at the appropriate IN
or OUT arguments.

1: http://git.openafs.org/?p=openafs.git;a=blob;f=src/fsint/afsint.xg;h=9fe4980c6cffcae457789e363393825e29dcb7b1;hb=HEAD#l594
2: http://git.openafs.org/?p=openafs.git;a=blob;f=src/fsint/afscbint.xg;h=19760bc512c647595662c30e190303b3a20be983;hb=HEAD#l22
3: http://git.openafs.org/?p=openafs.git;a=blob;f=src/vlserver/vldbint.xg;h=ff34d12d45454de328439cab3ed896fd564f0c59;hb=HEAD#l341

Remove a useless warning about the SMB decoder.

In the last couple years it had been proved that any decoder can
potentially have buffer overflows, hence let's not emphasize one of them
more than the others.