From Peter Valchev: fix editcap to assign the result of "getopt()" to an
"int" and to check "getopt()"s return value with -1 rather than EOF.

Fix other "getopt()" loops to check against -1 as well (EOF is -1 on
most if not all platforms, but the Single UNIX Specification says
"getopt()" returns -1, so we should check against -1, not EOF).

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4793 f5534014-38df-0310-8fa8-9805f1628bb7

In the Q.931-over-TPKT-over-TCP dissector, if the TCP segment we're
handed looks as if it contains only a TPKT header (4 bytes long, and
those 4 bytes look like a TPKT header according to "is_tpkt()"), call
the "dissect TPKT over a TCP stream" routine.  If we're doing
reassembly, that routine will force a reassembly because the TPKT
payload isn't in that segment, and the various heuristic XXX-over-TPKT
dissectors will be called again, this time with enough data for them to
say whether the TPKT payload is for them or not; if we're not doing
reassembly, we'll dissect the TPKT header and then call the "dissect a
Q.931 PDU" routine, which will throw an exception because there isn't
any payload from which to fetch data (and that's what we want to
happen).

In the "dissect TPKT over a TCP stream" routine, if reassembly is
enabled, do the check to see if we need to do reassembly to get the
payload before dissecting the TPKT header, so that we don't dissect the
TPKT header and then decide "oops, we need some more data to get the
TPKT payload".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4792 f5534014-38df-0310-8fa8-9805f1628bb7

"is_tpkt()" is always used to check a TCP segment to see if it might
have a TPKT header at the beginning, so there's not need for it to have
an offset as an argument; its callers don't have to know how big the
TPKT header is (or we can put a #define in "packet-tpkt.h" for it).  Get
rid of the second argument.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4791 f5534014-38df-0310-8fa8-9805f1628bb7

From John Mackenzie: put missing initializations of table entries in
"plugins.c".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4790 f5534014-38df-0310-8fa8-9805f1628bb7

Dissect the payload of EAPOL Key messages.

Put in a comment noting that the payload of EAPOL Encapsulated ASF Alert
messages should perhaps be dissected as SNMP Trap messages.

Put the type of the message into the Info column.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4789 f5534014-38df-0310-8fa8-9805f1628bb7

Add some necessary "volatile" declarations.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4788 f5534014-38df-0310-8fa8-9805f1628bb7

Get rid of an unnecessary "volatile".

Give the type field a value_string array, and use that field when
putting the type into the protocol tree.

Display the data under the EAPOL tree, as it's part of the EAPOL PDU.

Just use "next_tvb" as the tvbuff for the data; don't create a new one
that, the different fourth argument to "tvb_new_subset()"
nonwithstanding, refers to the same data as "next_tvb".

Call the EAP dissector through a handle.

Nobody directly calls the EAP dissector any more, they all call it
through handles; make it static.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4787 f5534014-38df-0310-8fa8-9805f1628bb7

From Adam Sulmicki: dissect EAP messages inside RADIUS.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4786 f5534014-38df-0310-8fa8-9805f1628bb7

Fixed up value_string for SMBE_unknownlevel error.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4785 f5534014-38df-0310-8fa8-9805f1628bb7

Set "snaplen", not "has_snaplen", to the snapshot length specified in
the capture preferences dialog box.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4783 f5534014-38df-0310-8fa8-9805f1628bb7

Catch the ReportedBoundsError exception in the DNS and TPKT dissectors
when dissecting messages over TCP, so that an error in one message
doesn't stop us from dissecting the next message in the segment, if any.

Put an XXX comment before the code that constructs the tvbuff for each
message inside a TCP segment, noting that we really want tvbuffs to have
three lengths and to have a new type of exception thrown if you go past
the second length but not past the reported length.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4782 f5534014-38df-0310-8fa8-9805f1628bb7

COPS-PR extension support, from Kari Tiirikainen.

Add support for desegmentation of COPS messages and for multiple COPS
messages per TCP segment.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4781 f5534014-38df-0310-8fa8-9805f1628bb7

From Adam Sulmicki: additional AVPs for RADIUS. and making
RD_TP_CONNECT_INFO a RADIUS_STRING rather than a RADIUS_STRING_TAGGED.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4780 f5534014-38df-0310-8fa8-9805f1628bb7

From Adam Sulmicki <adam[AT]cfar.umd.edu>:
Add more type values for EAP.
Fix off-by-one bug when displaying Code of EAP message.

Get rid of an unnecessary "volatile".

Give the code and type fields value_string arrays, and use them when
putting the code and type into the protocol tree.

Base the decision of whether to put the type field into the tree on the
request code, not on the length of the packet.

Display the Type-Data field, under that name, under the EAP tree, as
it's part of the EAP PDU.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4779 f5534014-38df-0310-8fa8-9805f1628bb7

Handle TPKT packets split across segment boundaries, and multiple TPKT
packets per segment.

Instead of having a routine for dissectors such as the Q.931 dissector
to call to dissect the TPKT header, have a routine that does all the
reassembly and multiple-packets-per-segment work, and have the Q.931
dissector call it.  Export "is_tpkt()", and the new routine, to plugins.

Add preferences for TPKT and Q.931 reassembly.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4778 f5534014-38df-0310-8fa8-9805f1628bb7

Properly set the length of the top-level item.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4777 f5534014-38df-0310-8fa8-9805f1628bb7

Redo the DNS-over-TCP code to handle the DNS-over-TCP header being split
across segment boundaries and to, for each DNS-over-TCP PDU, create a
tvbuff containing the header and the body of the PDU, handing that to
the DNS PDU dissector.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4776 f5534014-38df-0310-8fa8-9805f1628bb7

From Dermot Bradley:

support for Openwave-specific WSP headers;

support for Openwave-specific field names;

support for additional content types from Openwave;

support for additional language values.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4775 f5534014-38df-0310-8fa8-9805f1628bb7

Properly handle the "no data" data frames.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4774 f5534014-38df-0310-8fa8-9805f1628bb7

AiroPeek captures have an extra 4 bytes of gunk at the end of every
packet; don't hand that gunk to the 802.11 dissector.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4773 f5534014-38df-0310-8fa8-9805f1628bb7

Added a preference to allow for setting the COPS tcp port.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4772 f5534014-38df-0310-8fa8-9805f1628bb7

Initial quota support, from Ronnie Sahlberg.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4771 f5534014-38df-0310-8fa8-9805f1628bb7

Use the right format (one with a numerical format item rather than a
string format item) as the third argument to "val_to_str()".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4770 f5534014-38df-0310-8fa8-9805f1628bb7

The number of bytes of tagged parameters should be computed by seeing
how many bytes remain in the packet starting at the initial offset of
the tagged parameters, not by seeing how many bytes remain in the packet
starting 4 bytes later.  (If you're trying to avoid counting a CRC that
appears at the end of the packet data, then you need to be sure there
*is* a CRC first; this may require using a different DLT_ type, in
libpcap, for those captures.)

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4769 f5534014-38df-0310-8fa8-9805f1628bb7

Export dissect_nt_sec_desc() so it can be used by other dissectors.

A NT security descriptor can be either revision 1 or 2 and seem to be the
same format on the wire.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4768 f5534014-38df-0310-8fa8-9805f1628bb7

Add first attempt at creating a tool to take a [t]ethereal core file
and using gdb, extract the packet that was being dissected when the
core file was created. It works in simple cases; it will probably
fail in many other cases. Right now it only creates libpcap files, and uses
text2pcap to do so.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4767 f5534014-38df-0310-8fa8-9805f1628bb7

Be more paranoid, and check to make sure the length of the string or OID
isn't so large as to cause the end offset to wrap around.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4766 f5534014-38df-0310-8fa8-9805f1628bb7

Throw in some sanity checking to make sure a bogus length in an ASN.1
string or OID data type doesn't cause us to hand a bogus length to a
memory allocator.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4765 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg: add the actual SMB command to the structure that
remembers SMBs for request/response matching, and make sure the request
and the response have the same type (or that the response has a
different type but is a valid response to the request).

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4763 f5534014-38df-0310-8fa8-9805f1628bb7

Sun's "yp.x" appears to lie when it claims that the argument to a FIRST
call is a "ypreq_key"; the argument appears to be a "ypreq_nokey"
instead.

The response to an ALL call isn't a single item, it's a sequence of
items - all but the last item has the "more" field true, the last has it
false.  Show all the items.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4762 f5534014-38df-0310-8fa8-9805f1628bb7

Export "conversation_set_dissector()" in the plugin API table.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4760 f5534014-38df-0310-8fa8-9805f1628bb7

Fixed small bug in translation of IXCD in OSPF GMPLS LSA.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4759 f5534014-38df-0310-8fa8-9805f1628bb7

From Andrew Hood: translate enterprise OIDs in SNMP traps to strings if
possible.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4758 f5534014-38df-0310-8fa8-9805f1628bb7

Update a comment.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4757 f5534014-38df-0310-8fa8-9805f1628bb7

Fix the declaration of "rec_dissector_t" to include a return value type.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4756 f5534014-38df-0310-8fa8-9805f1628bb7

Fix up the declaration of "addr_tvb_new_real_data" to match
"tvb_new_real_data()".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4755 f5534014-38df-0310-8fa8-9805f1628bb7

For TCP segments that are reassembled into larger packets, show the data
as raw TCP segment data under the TCP protocol tree item, rather than as
a top-level data item - and do so even for the last of the segments
reassembled into that packet.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4754 f5534014-38df-0310-8fa8-9805f1628bb7

Add support for reassembling RPC-over-TCP fragments, and do that in both
RPC and NDMP.

Show the RPC-over-TCP fragment header as a tree with bitfields below it.

Add a routine to show a reported bounds error as an "Unreassembled
Packet" or a "Malformed Packet" depending on whether "pinfo->fragmented"
is set, and have NBNS and RPC use that.

Add "ett_ndmp_file_stats" to the list of ett_ values to be initialized
(it wasn't in that list, and wasn't getting initialized).

When freeing up various hash tables and memory chunks in the RPC
dissector, zero out the pointers to them, just to make sure we don't try
to free them again.

Always destroy the TCP segment key and address memory chunks in
"tcp_desegment_init()", regardless of whether TCP desegmentation is
enabled - we don't *allocate* them if TCP desegmentation isn't enabled,
but we should free them even if it's not enabled.  Also, when we free
them, set the pointers to them to null, so we don't double-free them.

Supply to subdissectors called from the TCP dissector the sequence
number of the first byte handed to the sub dissector.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4753 f5534014-38df-0310-8fa8-9805f1628bb7

Have "alloc_field_info()" take a pointer to the field length as an
argument, so if the length was supplied as -1, it can set it to the
length of data remaining in the tvbuff, so that its callers can use that
length when getting the value for the field, rather than leaving the
length in the "field_info" structure as -1.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4752 f5534014-38df-0310-8fa8-9805f1628bb7

Don't give tvbuffs names; instead, give data sources names, where a
"data source" has a name and a top-level tvbuff, and frames can have a
list of data sources associated with them.

Use the tvbuff pointer to determine which data source is the data source
for a given field; this means we don't have to worry about multiple data
sources with the same name - the only thing the name does is label the
notebook tab for the display of the data source, and label the hex dump
of the data source in print/Tethereal output.

Clean up a bunch of things discovered in the process of doing the above.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4749 f5534014-38df-0310-8fa8-9805f1628bb7

With the tvbuffication of all dissectors, the "packet_info" structure no
longer contains length fields, so there's no need to pass a "packet_info
*" argument to "set_actual_length()".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4748 f5534014-38df-0310-8fa8-9805f1628bb7

Add support for SNA-over-Ethernet (Ethernet type 80d5).

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4747 f5534014-38df-0310-8fa8-9805f1628bb7

From Joerg Mayer: remove trailing blanks from hex dump in
print/Tethereal.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4742 f5534014-38df-0310-8fa8-9805f1628bb7

AiroPeek puts variable-length, not padded fixed-length, 802.11 headers
into captures.

The data rate header is in .5 megabyte/second units, not .5
kilobyte/second units.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4741 f5534014-38df-0310-8fa8-9805f1628bb7

Put in a missing "break;" statement.  (Testing the untested AiroPeek
support revealed that I'd left it out; putting it back made it work.)

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4740 f5534014-38df-0310-8fa8-9805f1628bb7

Undo the change of "UcpHandleByte(hf_ucp_parm_NT);" to
"UcpHandleString(hf_ucp_parm_NT);" - the field really is one character
long, as per the (correct) change from FT_STRING to FT_UINT8.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4739 f5534014-38df-0310-8fa8-9805f1628bb7

Comment-out FT_UCS2_LE, and remove the already commented-out FT_TEXT_ONLY.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4738 f5534014-38df-0310-8fa8-9805f1628bb7

Add ncp2222 as a type of packet than randpkt can produce.
If type passed in with -t option is unknown, die instead of using
Ethernet as a default.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4737 f5534014-38df-0310-8fa8-9805f1628bb7

Register hf_ucp_parm_NT as an FT_UINT8 instead of FT_STRING,
and use the vals_parm_NT value string in that registration.
Thanks to Marcin Gryszkalis <mgryszkalis@cerint.pl> for the bug report.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4736 f5534014-38df-0310-8fa8-9805f1628bb7

FIRST_PROTO_PREFS_PAGE needs to be incremented by 2 in order for
the "Protocol Preferences" pop-up menu option to work properly.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4735 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg: add decoding of the (known) individual bits of
the Account-Control bits.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4734 f5534014-38df-0310-8fa8-9805f1628bb7

From Tim Potter:

  - For selected read and write SMBs, display the byte count and offset
    in the info column.  This makes browsing file read/writes easier to
    understand.

  - In dissect_nt_sids() sometimes the version number is 3 but the rest
    of the sid format remains the same.  This is purely by observation -
    I have no documentation to confirm this.

  - Use a GString instead of a fixed buffer in dissect_nt_sids().

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4733 f5534014-38df-0310-8fa8-9805f1628bb7

From Laurent Rabret: print the correct length in the Extended
Communities attribute in a BGP Update message.

Also, get rid of an extra space before a colon in the display for that
attribute, which isn't in other attributes.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4732 f5534014-38df-0310-8fa8-9805f1628bb7

"tvb_ieee_to_long()" returns a "long", not a "long long", so you can't
print it with "%lld".

In any case, not all platforms on which Ethereal can be built support
64-bit integral data types, and, even on those that do, not all of them
support "%ll[doux]" as the format for printing those types, so do the
arithmetic in floating point by multiplying the bytes/s values by 8.0,
and print with "%.0f" instead.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4731 f5534014-38df-0310-8fa8-9805f1628bb7

GMPLS extensions for OSPF Opaque LSAs

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4730 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg:

fix various alignment problems;

change/prettify a few of the structures;

other updates.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4729 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg: display the opnum also for reply packets.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4728 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg: display FILETIME values of 0x8000000000000000 as
"Infinity (absolute time)", and values of 0x7fffffffffffffff as
"Infinity (relative time)".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4727 f5534014-38df-0310-8fa8-9805f1628bb7

Have the private data for the SCSI dissection routines be a pointer to a
structure containing a 32-bit conversation ID (which uniquely identifies
conversations between a SCSI initiator and target) and a 32-bit task ID
(which uniquely identifies a task within that conversation).

Have the NDMP dissector create conversations when it sees an "execute
CDB" request, and use the conversation index as the conversation ID and
the sequence number for requests and reply sequence for replies as the
task ID.

Have it use "dissect_scsi_payload()" to dissect the payload of "execute
CDB" requests and replies.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4726 f5534014-38df-0310-8fa8-9805f1628bb7

Dissect "execute CDB" requests and replies.

Get rid of extra trailing white space in some protocol tree items.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4725 f5534014-38df-0310-8fa8-9805f1628bb7

Move all the stuff in "packet-scsi.h" that's solely for the use of
"packet-scsi.c" into "packet-scsi.c"; the iSCSI dissector doesn't need
its own versions of a pile of static data structures used only by
"packet-scsi.c", nor does it need a pile of typedefs and #defines used
only by "packet-scsi.c".

The iSCSI dissector *does* use "scsi_status_val", so make that
non-static, and export it from "packet-scsi.h".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4724 f5534014-38df-0310-8fa8-9805f1628bb7

Have a Q.931-over-TPKT TCP heuristic dissector.

Have the Q.931 PDU dissector, if it's Q.931-over-TPKT, check for
user-user IEs with a protocol discriminator of "X.208 and X.209 coded
user information" and, if it sees one, call an H.225.0 Call Setup
dissector if it could find the handle for it.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4723 f5534014-38df-0310-8fa8-9805f1628bb7

Display the fragment length, and fragment ordinal number, in decimal.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4722 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg:

SAMR updates;

a bugfix in dissect_ndr_pointer() (should not check referent id
for aliases for unique pointers);

enhancement to dissect_ndr_pointer() to make it possible to
hand a generic int value to the dissector for the pointer object
in a similar way as hf_index values are passed through the
pointer layer.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4721 f5534014-38df-0310-8fa8-9805f1628bb7

dissect_frame() can throw a ReportedBoundsError; catch it in
dissect_packet().

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4720 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg:

fix to LookupRids to match what the IDL file says;

fix to "dissect_ndr_uint64()" to specify the right length to
"proto_tree_add_item()";

give the protocol tree items for array header counts and offsets
the correct offsets in the packet.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4719 f5534014-38df-0310-8fa8-9805f1628bb7

From Todd Sabin:

dissect dcerpc UDP replies correctly - use the opnum from the
request, ont the reply (the opnum from the request is frequently
wrong in Microsoft's DCE RPC implementation);

don't crash if the packet isn't found in the hash tables;

dissect SamrLookupDomain requests properly.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4718 f5534014-38df-0310-8fa8-9805f1628bb7

Put ARP hardware addresses that are MAC addresses into the protocol tree
as FT_ETHER fields, not FT_BYTES fields.

Only label Appletalk protocol addresses as "{Source,Target} ID"; label
the others as "{Source,Target} protocol address", and have different
fields for Appletalk and other protocol addresses.

Don't put addresses into the protocol tree if they're zero-length.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4717 f5534014-38df-0310-8fa8-9805f1628bb7

Put ARP hardware addresses that are MAC addresses into the protocol tree
as FT_ETHER fields, not FT_BYTES fields.

Don't treat Experimental Ethernet addresses as being Ethernet addresses
(they were 8 bytes long, not 48 bytes long).

Put ARP protocol addresses that are IPv4 addresses into the protocol
tree as FT_IPv4 fields, not FT_BYTES fields.

Fix the checks for MAC hardware addresses and IPv4 protocol addresses,
done to see if we should say a given Ethernet address has the same name
as a given IP address, to check both the type and length.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4716 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg: display the mailslot header items only if we have
a tvbuff with the mailslot header.

Fix indentation.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4715 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg: display the SAMR "Account Control" field in hex.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4714 f5534014-38df-0310-8fa8-9805f1628bb7

Make the fields that correspond to Boolean bitfields be FT_BOOLEANs.

Display the VLDB flags in hex, not decimal.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4713 f5534014-38df-0310-8fa8-9805f1628bb7

Support for the new restart TLV, from Hannes Gredler.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4712 f5534014-38df-0310-8fa8-9805f1628bb7

rename some macros, start getting rid of redundant macros, add more vldb dissection

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4711 f5534014-38df-0310-8fa8-9805f1628bb7

Fixes from Ronnie Sahlberg.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4710 f5534014-38df-0310-8fa8-9805f1628bb7

Have Wiretap set the snapshot length to 0 if it can't be derived from
reading the capture file.  Have callers of "wtap_snapshot_length()"
treat a value of 0 as "unknown", and default to WTAP_MAX_PACKET_SIZE (so
that, when writing a capture file in a format that *does* store the
snapshot length, we can at least put *something* in the file).

If we don't know the snapshot length of the current capture file, don't
display a value in the summary window.

Don't use "cfile.snap" as the snapshot length option when capturing -
doing so causes Ethereal to default, when capturing, to the snapshot
length of the last capture file that you read in, rather than to the
snapshot length of the last capture you did (or the initial default of
"no snapshot length").

Redo the "Capture Options" dialog box to group options into sections
with frames around them, and add units to the snapshot length, maximum
file size, and capture duration options, as per a suggestion by Ulf
Lamping.  Also add units to the capture count option.

Make the snapshot length, capture count, maximum file size, and capture
duration options into a combination of a check box and a spin button.
If the check box is not checked, the limit in question is inactive
(snapshot length of 65535, no max packet count, no max file size, no max
capture duration); if it's checked, the spinbox specifies the limit.
Default all of the check boxes to "not checked" and all of the spin
boxes to small values.

Use "gtk_toggle_button_get_active()" rather than directly fetching the
state of a check box.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4709 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg: add dissectors for the rest of SAMR.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4708 f5534014-38df-0310-8fa8-9805f1628bb7

Support in Wiretap for DLT_HHDLC, from Tomas Kukosa.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4707 f5534014-38df-0310-8fa8-9805f1628bb7

Additional cipher suite names for SSL, from Nathan Neulinger and Scott
Renfro.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4706 f5534014-38df-0310-8fa8-9805f1628bb7

If you compile with IBM's compiler on AIX 5.1, the compiler apparently
somehow doesn't like the structure member name "fh_len", for some reason
(probably some #define in some header file); change that (and other
"fh_len" variables) to "fh_length" instead.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4705 f5534014-38df-0310-8fa8-9805f1628bb7

Use "value_string" tables to map EAP code and type values to strings -
and fix up the table for EAP types.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4704 f5534014-38df-0310-8fa8-9805f1628bb7

Old-style (non-tvbuffified) dissectors haven't been supported since
0.9.0 was released; get rid of the typedef for "old_dissector_t".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4703 f5534014-38df-0310-8fa8-9805f1628bb7

zlib 1.0.8 - the version that comes with X11 - does have "gzseek()",
even if it doesn't have "gzgets()", so one might think we could use it
by using our own replacement for "gzgets()".

One would be wrong to think so, however, as the "gzseek()" it has
doesn't actually work when reading uncompressed files.

zlib 1.0.9 has "gzgets()", and fixes that bug, so we rever to checking
for "gzgets()" rather than "gzseek()", so that we don't accept pre-1.0.9
versions of zlib, and we get rid of our "gzgets()" replacement.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4702 f5534014-38df-0310-8fa8-9805f1628bb7

From Ronnie Sahlberg:

the count fields in SAMR replies aren't array max_count values,
so don't display them as such;

Put conformant and conformant varying array length/offset/etc.
values into the protocol tree.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4701 f5534014-38df-0310-8fa8-9805f1628bb7

"log" is a name that belongs to the ANSI C89 standard; it refers to a
function that computes the natural logarithm of a double.  Using it as
the name of a pointer to a routine to do logging can cause namespace
collisions; in fact, it *does* cause them on AIX.  Rename the function
argument to "logfunc".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4700 f5534014-38df-0310-8fa8-9805f1628bb7

The typedef in "epan/value_string.h" declares "value_string" to be a
typedef for "struct _value_string"; as such, the incomplete structure
declaration in "epan/proto.h" should declare "struct _value_string", not
"struct value_string", and casts and declarations in that header should
also use "struct _value_string", not "struct value_string".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4699 f5534014-38df-0310-8fa8-9805f1628bb7

add idle to rx ack value strings

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4698 f5534014-38df-0310-8fa8-9805f1628bb7

Have configure check for pod2man, thus creating a POD2MAN makefile
variable, allowing it to be overridden during make.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4697 f5534014-38df-0310-8fa8-9805f1628bb7

Clean up a bunch of things:

use value_string tables and make numerical fields numeric,
with associated value_string tables, rather than strings;

make a signed integer field signed, which means we don't have to
format the item for it in the protocol tree ourselves;

give it a long protocol name, and make the short protocol name
all-caps, as with other short protocol names.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4696 f5534014-38df-0310-8fa8-9805f1628bb7

"Raw IP" could be IPv4 or IPv6; look at the first byte of the packet
(i.e., the byte of the IP header containing the IP version number) to
see which it is, and handle the packet either as IPv4, IPv6, or "other"
based on the IP version number.

The "if" chain for handling various packet types using DLT_RAW doesn't
need returns after every clause; it should just return after falling off
the end (and shouldn't call "g_assert_not_reached()" once we do that).

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4695 f5534014-38df-0310-8fa8-9805f1628bb7

From Ricardo Barroetave�a: support dissectors that are handed
reassembled TCP data being able to indicate that they need still more
reassembly, so that, for example, a dissector can indicate that it needs
reassembly in order to dissect a header that says how long the PDU is
and, when that reassembly is done and it dissects the header, it can
then indicate that it needs more reassembly to get the entire PDU.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4694 f5534014-38df-0310-8fa8-9805f1628bb7

Get rid of an extra space.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4693 f5534014-38df-0310-8fa8-9805f1628bb7

Catch ReportedBoundsError exceptions when dissecting NDMP messages, and
report the error but don't re-throw the exception; that way, we can
continue to dissect additional NDMP messages in the frame or reassembled
chunk of data, even if one of them happens to be too short for what's in
it.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4692 f5534014-38df-0310-8fa8-9805f1628bb7

In C89, you can't declare variables at arbitrary points in code; move
the declaration of "tmp" in the "OUT_STRING()" to the beginning of the
block.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4691 f5534014-38df-0310-8fa8-9805f1628bb7

dissect flags, more vldb fixes

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4690 f5534014-38df-0310-8fa8-9805f1628bb7

Fix string parsing from rx packets. Previously was calculating the next
offset incorrectly, which resulted in the rest of the packet being shifted
by a few bytes, and therefore all garbage.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4689 f5534014-38df-0310-8fa8-9805f1628bb7

add packet flag for slow-start/jumbogram

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4688 f5534014-38df-0310-8fa8-9805f1628bb7

add a couple rx opcodes

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4687 f5534014-38df-0310-8fa8-9805f1628bb7

Sigh.  Strings in SDP may have tabs in them, so we need to use
"format_text()" to display them, so that the tabs show up as "\t".  (We
eventually need to use that when displaying string fields - and arrange
that it properly handle the character set of the font being used for
display, in Ethereal, and the character set of the user's locale, in
Tethereal).

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4685 f5534014-38df-0310-8fa8-9805f1628bb7

Update version to 0.9.1.  Add NEWS entries for 0.9.1 changes.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4684 f5534014-38df-0310-8fa8-9805f1628bb7