Rename wtap_dump_file_write_all() to wtap_dump_file_write(), and have
everybody use it; the places using the old wtap_dump_file_write() were
using it in the same way the old wtap_dump_file_write_all() did.

That also lets us get rid of wtap_dump_file_ferror().

Also, have the new wtap_dump_file_write() check for errors from
gzwrite() and fwrite() differently - the former returns 0 on error, the
latter can return a short write on error.

svn path=/trunk/; revision=33113

From Jim Young:
Remove unneeded "gtk_tree_path_free(path);" from case
"GDK_Return:" in tree_view_key_pressed_cb() that was reverted in r32957 to restore X11 functionality lost in refactored code from r32323.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4847

svn path=/trunk/; revision=33112

[Automatic manuf and enterprise-numbers update for 2010-06-06]

svn path=/trunk/; revision=33109

Add 32bit Ubuntu 10.04 to supported platforms

svn path=/trunk/; revision=33108

From Rolf Fiedler: support for writing EyeSDN trace files.

svn path=/trunk/; revision=33107

First check for the new GTK+/OS X integration functions, then the old
ones; it appears that at least one user's -ligemacintegration has both
(see bug 4823), and we should choose the new ones in that case.

Also, always set have_ige_mac if we have the functions.

Fix tpyoes while we're at it.

svn path=/trunk/; revision=33106

From John Fitzgibbon:
When specifying SA keys for AES-CTR, Wireshark expects a key length of 160, 224
or 288 bits, (i.e. 128, 192 or 256 bits, followed by the 32 bit nonce value),
but gcry_cipher_setkey() in packet_ipsec.c fails, as it expects 128, 192 or 256
bits.

Omitting the nonce won't work -- even if Wireshark liked those key lengths,
gcrypt wouldn't be able to decrypt without it.

svn path=/trunk/; revision=33105

Clean up indentation.

svn path=/trunk/; revision=33104

Clarify an error message (and the corresponding comment).

Fix the no-libpcap build.

svn path=/trunk/; revision=33103

Back out r33088 for now.

svn path=/trunk/; revision=33102

From Cal Turney via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4831 :

Add the data read/write length to the NFS tree so it is filterable.

From me: don't bother incrementing the offset just to decrement it again.
Change the hf info a bit.

(Ideally the RPC dissector would add the length to the tree not as a text
item; that is left for future work.)

svn path=/trunk/; revision=33101

Catch some cases that don't currently work.

svn path=/trunk/; revision=33100

From Chris Maynard via bug 1546:

Add units to the rate.

svn path=/trunk/; revision=33099

Thou shalt not have a tap whose listener is always active; that means,
for example, that, if it's a TShark tap, TShark can never be run with
"-w -", as it means that TShark will always have to dissect the packets.

svn path=/trunk/; revision=33098

Add a preference that allows the user to decide if they want the calling and
called GTs (if RI=GT) put in the (pinfo) source and destination (and thus into
the source and destination columns).

This may help (if the PCs change but the GT does not) or hurt (if the GT or RI
change but the PCs do not) TCAP's ability to identify which messages belong to
which TCAP "session."

svn path=/trunk/; revision=33097

Use a gboolean to store a boolean and change the plurality on the preference name

svn path=/trunk/; revision=33096

Add a preference that allows the user to decide if they want the source and
destination address PC or GT (depending on the RI) put in the (pinfo) source and
destination (and thus into the source and destination columns).

This may help (if the IP addresses change but the PC or GT do not) or hurt (if
the PC, GT, or RI change but the IP addresses do not) TCAP's ability to identify
which messages belong to which TCAP "session."

svn path=/trunk/; revision=33094

Add BIN_DIR.  If it is '.' then set WIRESHARK_RUN_FROM_BUILD_DIRECTORY so that we test plugins

svn path=/trunk/; revision=33093

Add a checkapis target.

svn path=/trunk/; revision=33092

Increase LENGTH_OID a bit to give us more room for ACNs (probably more than will ever be used).

svn path=/trunk/; revision=33091

Fix comments.

svn path=/trunk/; revision=33090

tcaphash_context_t.oid is of size LENGTH_OID+1, not LENGTH_OID.  Just use sizeof(tcaphash_context_t.oid) instead of LENGTH_OID so we can use that last byte.  This allows us to retrieve the whole ACN when the ACN is LENGTH_OID bytes long.

svn path=/trunk/; revision=33089

From Matt Poduska via bug 1957 (with minor changes):

New dissector (plugin) to support decode of the EPCglobal Low-Level Reader
protocol (see llrp.org for more information). This dissector has passed fuzz
testing.

svn path=/trunk/; revision=33088

According to RFC 3220 section 7 the max UDVM message size is 65536,
not 65535. Hopefully fixes bug 4837.

svn path=/trunk/; revision=33087

From  Samu Varjonen: Whitespace cleanup

Tabs converted to 8 spaces and trailing whitespaces removed, etc.

svn path=/trunk/; revision=33086

If BIN_DIR is ".", set WIRESHARK_RUN_FROM_BUILD_DIRECTORY so that we
test plugins.

svn path=/trunk/; revision=33085

Description for netscape-cert-type bit 4 (e.g. see docs.sun.com/source/816-5531-10/poli_ext.htm#1078885)

svn path=/trunk/; revision=33084

RCS 3280, which obsoletes RFC 2459, allows an IA5String CHOICE in the DisplayText of a UserNotice.

svn path=/trunk/; revision=33083

From Holger Freyther via bug 4841:
Add a new dissector for the NexusWare C7 MTP over UDP/TCP protocol. One of
NexusWare's example applications provide a way to forward MTP Level 3 messages
via UDP/TCP. This is a dissector for this protocol (which is lacking an IANA
assigned port).

svn path=/trunk/; revision=33082

Set the right DPI for PDF production.

svn path=/trunk/; revision=33081

From  Sven Eckelmann: Remove recursion for aggregated packets in batman dissector
"Different developers wanted to have the recursive dissector for aggregated
 packets changed to a iterative approach."

See Bug #4836: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4836

svn path=/trunk/; revision=33080

From Cal Turney: The NFS FH hash (nfs.fh.hash) incorrectly matches multiple filehandles.
"The method used in packet-nfs.c to calculate a 32-bit hash representing the
 32-byte filehandle is faulty in that the hash often matches multiple
 filehandles."

"This patch uses CRC-32 to calculate the hash.
 We (EMC GNS) have tested this patch for the past two years and we have not
 found a single case where the hash matched more than one filehandle."

See Bug #4839: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4839

svn path=/trunk/; revision=33079

Bring more texts and examples up to date.

svn path=/trunk/; revision=33078

Fix a gcc -Wshadow warning

svn path=/trunk/; revision=33077

Set the (pinfo) address if a PC is present (for now: regardless of whether routing is on GT or not).

svn path=/trunk/; revision=33076

From  Sven Eckelmann: Change recursion for aggregated packets to iteration.
See Bug #2631: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2631

svn path=/trunk/; revision=33075

Improved readability when displaying seconds with hours and minutes.

svn path=/trunk/; revision=33074

- Add tap-sv.c and packet-sv.c to the list of files to build
- Whitespace fixes
- Add Fedora 64 bit to the list of platforms where cmake should
  work

svn path=/trunk/; revision=33073

Use g_hash_table_new_full().

svn path=/trunk/; revision=33072

From Pascal Quantin:
Avoid triggering a malformed packet error when decoding a simple UE terminated Detach Request when the EMM cause IE is not present:

svn path=/trunk/; revision=33071

Whitespace/indentation/code style cleanup.

svn path=/trunk/; revision=33070

From Cal Turney via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4832 :

Add field 'nfs.ops.count' in the detail pane of NFSv4 calls and replies that
displays the number of operations in NFSv4 COMPOUND requests/replies.

From me: change the blurb wording a bit.

svn path=/trunk/; revision=33069

From Cal Turney via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4830 :

Display the fsid (filesystem ID) in decimal as well as hex in the "attributes"
section of the header in NFSv3/v4 replies.

svn path=/trunk/; revision=33068

From Peter Harris via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4820 :

Mesa moved its API description file again.

Also, while I'm here, I noticed that they stopped using signed tags, so add
--tags to "git describe" for a better description of the mesa version used to
build the X11 dissector.

svn path=/trunk/; revision=33067

From:  Sven Eckelmann:Add support for v10 packet format
 "v10 of the packet format was added recently and could be added
  to the wireshark dissector"
See: Bug #4833: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4833

From me:
- Fix a compile error: ...version=tvb_get_ntohs...
- Fix a missing = NULL.

svn path=/trunk/; revision=33066

Sorry, I forgot that 8.6. UDVM Cycles says

Note that the number of UDVM cycles MUST NOT be increased if a
request for additional compressed data fails.

so we *shouldn't* increase the cycle count further when
SIGCOMP_INSTR_INPUT_BYTES fails.  That does *NOT* of course, mean that
we shouldn't increase the cycle count by 1 for a failed INPUT-BYTES
instruction - that would leave UDVM vulnerable to infinite loops (as per
bug 4826), and I *really* doubt that was their intent; presumably, it
means it should not be increased *by the number of cycles for the
additional data*:

Additionally, if the UDVM successfully requests n bits of
compressed data using one of the INPUT instructions then the
number of available UDVM cycles is increased by n *
cycles_per_bit once the instruction has been executed.

if the attempt to get that additional data fails.

svn path=/trunk/; revision=33065

Add back a few 'if (check_col()...' removed by me in a previous commit.
Possibly it's better to use an 'if (checkcol()' if the enclosed call to
'col_...' has "complex" arguments.

svn path=/trunk/; revision=33064

From Reinhard Speyerer via bug 4779:

From reading the rawshark(1) manpage my assumption was that rawshark
could be used like
$ /usr/bml/bin/rawshark -s -r test.pcap -d encap:EN10MB ...

However rawshark either expects the -r argument to be -
(read from stdin) or a pipe which results in the following error
message:

rawshark: ".../test.pcap" is neither an interface nor a pipe

The proposed rawshark.pod patch updates the -r description to
the implemented rawshark functionality.

The patch also applies to the current SVN version.

svn path=/trunk/; revision=33063

From Sven Eckelmann: Synchronize bat dissector with batadv dissector
 "Different people made changes to enhance the batman-adv dissector. It seems
 that the batman dissector wasn't touched and misses those changes. Following
 patchset should improve the dissector the same way Gerald Combs, Guy Harris and
 Bill Meier improved batman-adv."
See Bug #4384: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4834

From me: Add back a few 'if (check_col()) ...';
 I probably went just a bit too far in changes I made in packet-batadv.c

svn path=/trunk/; revision=33062

Increment the cycle count no matter what. Increase the cycle count
further when SIGCOMP_INSTR_INPUT_BYTES fails. Fixes the infinite loop
found in bug 4826.

svn path=/trunk/; revision=33061

From  Sven Eckelmann:  Fix endless recursive loop
See Bug #2631: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2631

svn path=/trunk/; revision=33060

From  David Bond:
Added the IEEE defined ether types for TRILL and Layer 2 ISIS. Also had ISIS
register itself with the ethertype dissector.

References:
http://www.postel.org/pipermail/rbridge/2010-May/003998.html
http://tools.ietf.org/html/draft-ietf-isis-layer2-05
http://tools.ietf.org/html/draft-ietf-trill-rbridge-protocol-16

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4825

svn path=/trunk/; revision=33059

Set Subversion properties right.

svn path=/trunk/; revision=33058

Misc changes:
- Clear info col before beginning dissection;
- Remove some uneeded forward declarations;
- Remove some ... = NULL initializers;
- Display protocol as BATADV_??? if unknown type;
- '(gwflags & 0x7C) >> 3'  ==> '(gwflags & 0x78) >> 3'

svn path=/trunk/; revision=33057

From Chris Maynard via bug 4014:

This trivial patch merely displays the ICMP ID and sequence # in the Info
column.

svn path=/trunk/; revision=33056

check_col() no longer required;
Use col_add_fstr() instead of col_clear()/col_append_fstr() combo.

svn path=/trunk/; revision=33055

Added packet-batadv.c

svn path=/trunk/; revision=33054

Fix incorrect fetch of version field.

svn path=/trunk/; revision=33053

From Sven Eckelmann via but 2631:

The wireless meshing protocol B.A.T.M.A.N. Advanced changed their packet format
in such a way that now versions can be identified and so correct dissection of
the packets can be supported by wireshark.
Since it is a ever moving target it is very possible that the packet format is
changing slightly. The dissector was written in such a way that new version can
be supported relative easy.
I hope that it sufficient for the inclusion in wireshark.

I tried to fuzzing it some hours and no error was reported.

From me:

Initialize our dissector handles.

Merge packet-batadv.h into packet-batadv.c. It isn't included anywhere else.

Fuzz 500 passes using attached capture files.

svn path=/trunk/; revision=33052

Initialize no_vs_avps_ext with a pointer to a valid 'match' function so we don't crash in val_to_str_ext() if an AVP from an unknown vendor is encountered.

svn path=/trunk/; revision=33051

Definition of certificatePolicies certificate extension and removal of warnings

svn path=/trunk/; revision=33050

CA Version certificate extension

svn path=/trunk/; revision=33049

Bug 3597 - implicit octet string that is constructed causes PRES/FTAM dissect failure

Introduced some state to remember last dissected Tag/Length so that they can be recalled if an IMPLICIT tag is encountered and stripped. This allows its to be determined if the value has a constructed value - and so can be reassembled.

In this case, it is a IMPLICIT constructed OCTET STRING at the presentation layer.

Many thanks to Fred Gruman for identifying - and apologies for the delay in commiting.

svn path=/trunk/; revision=33048

Minor change to remember the object identifier.

svn path=/trunk/; revision=33047

Updated get_ber_length to handle multiple nested indefinite length encodings.

svn path=/trunk/; revision=33046

Clean up a bit, fix indent.

svn path=/trunk/; revision=33045

Update value_strings with IE names.

svn path=/trunk/; revision=33043

From Jim Young via bug 4331:

This patch adds a new '-S' option to editcap that will rewrite timestamps of
packets to insure that the new capture file is in strict chronological order.

This option's primary use case is to fixup the occasional timestamps that have
a negative delta time relative to previous packet.

This feature is related to (but does not depend on) capinfos enhancement
submitted in bug #4315 which helps identify tracefiles with "out-of-order"
packets.

svn path=/trunk/; revision=33042

From Jim Young via bug 4049:

This patch adds a new '-o' option to capinfos (enabled by default) to report if
the packets within a particular capture file are in strict chronological time
order or not.

svn path=/trunk/; revision=33041

Rebuild the IEC 61850 dissector.

svn path=/trunk/; revision=33040

From Michael Bernhard via bug 3398 with minor fixups:

I've created a ASN.1 dissector for the IEC 61850 Sampled Values protocol. It
dissects ethernet frames of the IEC 61850-9-2LE specification form the UCA
International User Group.

There is also a new TAP for tshark (-R sv) which extracts the important
information of the frame and allows to create plots (with external tools) of
the sampled values.

I've developed under Linux (Ubuntu 8.10) but everything should be in place for
successful compilation under Windows.

It would be great if this dissector could be included in wireshark. I'm looking
forward for your comments.

svn path=/trunk/; revision=33039

Remove the now unused value_strings.

svn path=/trunk/; revision=33038

- Traffic flow template miss dissected, use de_sm_tflow_temp()
  insted.
- Dissect vendor id of private message.

svn path=/trunk/; revision=33037

From Kovarththanan Rajaratnam via bug 3500:

Sébastien's initial commit [1] didn't contain support for embedding Python on
Windows.

[1] http://anonsvn.wireshark.org/viewvc?view=rev&revision=28529

From me:

Comment out PYTHON_EMBED for now.

Start a list of known Python+CRT versions.

Add get_wspython_dir to libwireshark.def.

svn path=/trunk/; revision=33036

Per Jaap's suggestion on -dev list: remove "with new_packet_list" from
version output since it has been the default for a while now and show
"with old_packet_list" if compiled as such.

svn path=/trunk/; revision=33035

There's no need to assign numerical values to the enum values.

svn path=/trunk/; revision=33034

From Herbert Lischka:
Lori made a simple change to the bacapp.c file for segmentation parsing.
Basically all fragments have some common data in the APDU which is useful to
see parsed in the tree.

I added support for vendor numbers bigger than 255.

Corrected Info column for "message fragment" and "message reassembled"

svn path=/trunk/; revision=33033

The Permitted-Actions-Attribute is exported from ftam-exp.cnf as DISPLAY = BASE_HEX. This causes a runtime error and so we override the definition here until we can find the correct fix.

svn path=/trunk/; revision=33032

Fixed a comment.

svn path=/trunk/; revision=33031

From Pascal Quantin:
Decode the EIA0 (null integrity algorithm) bit in EMM messages. This was introduced in 3GPP 24.301 CR-0422 (Release 9.2) and is also used by the LTSI (LTE/SAE Trial Initiative) for interoperability tests between constructors (with Release 8 equipments). I also updated the description of EEA0 as stated in 24.302 release 9.2.

svn path=/trunk/; revision=33030

capture() can, in some cases, return FALSE when the capture succeeds;
back out the change to check its return value until we fix that.

Also back out the test suite changes to look for an error exit for
invalid capture filters and interfaces.

svn path=/trunk/; revision=33029

Wireshark requires routines from libm, such as ceil(), so link with it;
we shouldn't rely on getting it pulled in by other shared libraries, as

1) there's no guarantee that the other shared libraries we use
   are linked with it or will continue to be linked with it;

2) there's no guarantee that we're even linking dynamically;

3) it fails on Fedora 13 as part of a change to catch programs
   that implicitly (or explicitly) assume, in their build
   procedures, that they'll get libraries linked in as a result
   of linking with other libraries.

svn path=/trunk/; revision=33028

From Ian Schorr:
NFS Dissector needs FH support for EMC Celerra.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4813

svn path=/trunk/; revision=33027

From Chris Maynard:
Fix for bug 4810 so PPP/USB dissector registers for IF_CLASS_CDC_DATA
- in addition to IF_CLASS_UNKNOWN and IF_CLASS_VENDOR_SPECIFIC, for those rare
occasions that the right set of circumstances arise where PPP traffic will be
(correctly) encapsulated in a USB frame of that interface class.

svn path=/trunk/; revision=33026

[Automatic manuf and enterprise-numbers update for 2010-05-30]

svn path=/trunk/; revision=33023

From Matteo Valdina
There are two typos in the RTCP TMMBR packets:
1. In the dissector of RTCP TMMBR is written "messured" instead of measured.
2. In the text title field is written TMMBN instead on TMMBR and vice-versa.

svn path=/trunk/; revision=33022

From Chris Maynard:
Fix for bug 4807 to handle compressed (NULL) address and control bytes.

The problem seems to be that the address and control bytes are missing,
presumably because they are compressed per section 3.2 of
http://tools.ietf.org/html/rfc1662.  The attached patch tries to account for
that.

The patch also fixes another bug.  If the hdlc framing character, 0x7e, is not
present, but the address and control fields are, then the offset might need to
be 3 bytes instead of only 2 bytes if the control field is escaped.  Sometimes
it is escaped and sometimes it isn't, so we need to handle both cases.

svn path=/trunk/; revision=33021

Increase the base size to 100 000 we are using more than that.

svn path=/trunk/; revision=33020

Include glib.h (for g_snprintf).

svn path=/trunk/; revision=33019

Add braces to avoid an ambiguous else

svn path=/trunk/; revision=33018

Put the optional objects in EXTRA..SOURCES instead of EXTRA_DIST.

Put the optional objects in a _DEPENDENCIES rule so they actually get built
when needed.

Use libtool to make these objects.  Remove AC variables that are no longer
needed.

svn path=/trunk/; revision=33017

Make inet_aton an optional object on *NIX again

svn path=/trunk/; revision=33016

Remove a couple more references to the optional objects (that are now in wsutil)

svn path=/trunk/; revision=33015

From Karl Heinz Wolf via bug 3571 (with minor changes):

Patch for the SIP dissector: the raw SIP message can be shown with our without
the "\r\n". User may want to remove the \r\n for clarity (by selecting a
checkbox).

svn path=/trunk/; revision=33014

Add a missing variable from wsgetopt

svn path=/trunk/; revision=33013

Move some code (including the optional objects) into libwsutil

svn path=/trunk/; revision=33012

From Jakub Zawadzki via bug 4273:

After appending new record set packet_list->columnized to FALSE.

From me:

Use g_ptr_array_sort_with_data instead of g_qsort_with_data in case the
GPtrArray internals ever change.

svn path=/trunk/; revision=33011

Fix the Windows build

svn path=/trunk/; revision=33010

Remaining fixes for
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4788
- Compile the python code directly into epan - don't link it in as
  a static lib.
- Call make-init-lua.pl with the top level directory instead of the
  current directory. Change make-init-lua.pl accordingly.

svn path=/trunk/; revision=33009

Add proto_help.c to the list of files.

svn path=/trunk/; revision=33008