Fix Gerald's e-mail address (and possibly provoke a copy of the main
repository to the anonymous repository - the current anonymous
repository appears to be out-of-date, which is breaking the buildbot).

svn path=/trunk/; revision=15847

Mark some unused and unremovable arguments as unused.

svn path=/trunk/; revision=15846

Move some headers for stuff defined by a dissector into epan/dissectors
- and rename smb.h to packet-smb.h, as it's packet-smb.c's header file.

svn path=/trunk/; revision=15845

Move a pile of protocol-related headers from the top-level source
directory to the epan directory.  Some of them should perhaps ultimately
be moved to epan/dissectors, if they pertain only to stuff exported by a
particular dissector.

Fix Gerald's e-mail address in files we're moving.

svn path=/trunk/; revision=15844

add an expert info based on QualityCode and a COL_INFO output

svn path=/trunk/; revision=15843

slightly shrink COL_INFO output

svn path=/trunk/; revision=15842

split call id's with a #, if more than one DCE/RPC call is in a data PDU

svn path=/trunk/; revision=15841

Sort each conversation column numerically.  Fix up whitespace.

It looks like we can't put "COPYCMD=/Y" in config.nmake and expect nmake
to do the right thing.  Add a comment, and set COPYCMD explicitly in the
root Makefile.nmake.  The rest of the occurrences of xcopy will have to
be taken care of at some point.

svn path=/trunk/; revision=15840

Aargh - undo r15838 as I missed jmayer's commit of r15835.

svn path=/trunk/; revision=15839

Comment out pidl dissectors for the moment to get the win32 build
going again.  I have most of a patch to get it working but I don't
think it will be finished tonight.

svn path=/trunk/; revision=15838

If we've enabled the display of BER encapsulation tokens, only put them
into the protocol tree once.

Fix the offsets and lengths used to put the variable binding values into
the protocol tree.

svn path=/trunk/; revision=15837

Add a local copy of pidl, add a README.ethereal to include
minimalistic building and usage instructions.

svn path=/trunk/; revision=15836

Move the packet-dcerpc-*.[hc] from pidl/ back to dissectors.
This makes Ethereal build again - there's no real reason that
ethereal fails to build for such a long time on so many platforms.

svn path=/trunk/; revision=15835

wka.tmpl:
        Show MS NLB Adresses individually
manuf:
        Update

svn path=/trunk/; revision=15834

Note problems with some values that appear in, for example, some SNMP
captures.

svn path=/trunk/; revision=15833

Fix a type.

svn path=/trunk/; revision=15832

Add conformance file used for building winreg dissector.

svn path=/trunk/; revision=15831

fixing the fix, it's time to go to bed ...

svn path=/trunk/; revision=15830

bugfix: don't crash (assert), if jumping to the first packet and packet_list is currently empty. Can happen at capture start.

svn path=/trunk/; revision=15829

Don't use u_char - not all platforms define it, and even those that do
might require other files to be included to get it.

svn path=/trunk/; revision=15828

Add a Makefile.

svn path=/trunk/; revision=15827

Add more information regarding call legs.
Make the release cause value filterable.

svn path=/trunk/; revision=15826

leak fix

svn path=/trunk/; revision=15825

Fix fragment length checks.  Fixes bugs 440 and 442-446.

svn path=/trunk/; revision=15824

remove some redundant strcpy()

svn path=/trunk/; revision=15823

remove some redundant strcpy()

svn path=/trunk/; revision=15822

remove a whole bunch of strcpy()     fix minor bug where whe checked the wrong string before we display it

svn path=/trunk/; revision=15821

update for ospf from Senthil Kumar Nagappan

svn path=/trunk/; revision=15820

delete the old dissector

svn path=/trunk/; revision=15819

delete the old dissector and point the makefile to the new dissector

svn path=/trunk/; revision=15818

new  pidl generated atsvc dissector and its conformance file

svn path=/trunk/; revision=15817

Get rid of a no-longer-used variable.

svn path=/trunk/; revision=15816

Move OUIs from LLDP dissector to oui.h.

Get rid of remaining sprintf'ing into on-the-stack buffers.

Note where some items came from.

svn path=/trunk/; revision=15815

Use ep_strdup_printf() to generate the "not found in value_string table"
string for val_to_str().

svn path=/trunk/; revision=15814

Tag ep_strdup_printf() and se_strdup_printf() as printf-like functions,
so we get compile-time checking of arguments when possible.

svn path=/trunk/; revision=15813

A first go at importing some pidl generated code into ethereal.  This
commit replaces the hand written dcerpc initshutdown and winreg
interfaces with autogenerated ones.

The pidl generated code is still a bit rought around the edges but will
hopefully improve with time.

svn path=/trunk/; revision=15812

Add PRIX64, as we now use it.

svn path=/trunk/; revision=15811

Replace the spnego dissector with an asn2eth generated one.

svn path=/trunk/; revision=15810

Rename TLV_INFO_MASK to TLV_INFO_LEN_MASK, to match TLV_TYPE_MASK.

Use "tvb_format_stringzpad()" rather than "tvb_format_text()" on
strings, as some of them are null-padded.  (XXX - are they null-padded
or null-terminated?)

Clean up the TLV length checks, and do many more such checks.

svn path=/trunk/; revision=15809

We already define ETHERTYPE_LLDP in "etypes.h" - use that, don't define
our own.

Get rid of MAC_to_str() - ether_to_str() suffices.

Don't copy to on-the-stack fixed-size string buffers, just save the
results of various formatting calls as pointers.

Use tvb_format_text() to get displayable versions of text strings.

Use tvb_get_ipv4() and tvb_get_ipv6().

Don't put an item into the protocol tree as text and then put in a
hidden item that, if unhidden, would display the same or almost exactly
the same; just put the item into the tree unhidden.

svn path=/trunk/; revision=15808

Properly handle 64-bit integer constants (they don't happen by magic,
and the stuff you have to do is compiler-dependent, so use
G_GINT64_CONSTANT()).

Properly handle formatting 64-bit quantities (they don't format
correctly by magic, and the stuff you have to do is compiler-dependent,
so use PRI[duoxX]64).

Don't copy the result of tvb_bytes_to_str() to a string buffer and then
only use it as an argument to a "%s" - just use the result directly.

Don't copy constant strings to a buffer - just use the strings directly.

Fetch 64-bit integral quantities with tvb_get_ntoh64(); they're
presumably in a standard byte order, which is probably big-endian for
Internet-family protocols, and using tvb_memcpy() is *guaranteed* to be
wrong on some platforms.

Fix up a format string.

Fix up indentation a bit.

svn path=/trunk/; revision=15807

Squelch some warnings, fix some typoes.

svn path=/trunk/; revision=15806

fix wtap_dump_flush function name and don't check for it's return value. We don't do this check in Ethereal either and we will recognize problems at next write anyway ...

svn path=/trunk/; revision=15805

added compression support for capture file output. The Save/As dialog now has a checkbox "Compress with gzip"

currently limited to Ethereal and all the variants of libpcap filetypes only.

We might want to add output compression support to the other tools as well (tethereal, mergecap, ...).

We might also want to add support for the other filetypes, but this is only possible if the filetype functions doesn't use special output operations like fseek.

One bug is still left: if the input and output filetypes while saving are the same, Ethereal currently optimizes this by simply copy the binary file instead of using wiretap (so it will be faster but it will ignore the compress setting).

Don't know a good workaround for this, as I don't know a way to find out if the input file is currently compressed or not. One idea might be to use a heuristic on the filesize (compared to the packet size summmary). Another workaround I see is to remove this optimization, which is of course not the way I like to do it ...

svn path=/trunk/; revision=15804

add E_OUTOFMEMORY

svn path=/trunk/; revision=15803

From Peter Johansson
I added a description for the call to DISSECTOR_ASSERT in proto.c to be
printed to stderr.

svn path=/trunk/; revision=15802

Fix a warning.

svn path=/trunk/; revision=15801

From Brian Bogora:
Our company (Mitel) has developed an LLDP-MED plug-in for Ethereal.

svn path=/trunk/; revision=15800

From Dinesh G Dutt

Support some new messages and fix a couple of bugs.

packet-fc.h - Added a #define for a Cisco MDS-specific frame called OHMS (online health mgmt srv) packet-fc.c - Support for OHMS frame, fixed an incorrect "malformed frame" error caused on ACK1 frames (they
                   don't contain anything but (encap hdr + FC hdr + encap trailer), fixed incorrect detection of
                   last-data-frame

packet-fcct.h - Support for new service type, "Fabric Controller", used in conjunction with FC-SW3 ESS message,
                     exported the service name value string definitions and
packet-fcct.c - Support for recognizing "Fabric Controller" service type and "vendor-specific" service

packet-fcswils.h - Support for ESS & MRRA messages, defined as part of FC-SW3 packet-fcswils.c - Support for ESS & MRRA messages, defined as part of FC-SW3

packet-scsi.c - Support for Verify and Write&Verify SBC commands.

svn path=/trunk/; revision=15799

Do the variable binding with dissect_ber_x() routines rather than asn1 ones.

svn path=/trunk/; revision=15798

Dont't carsh if dissect_ber_null() isn't passed a hf_id.

svn path=/trunk/; revision=15797

Allow dissection of dcerpc of short frames   instead of aborting as soon as it is detected the pdu is "short"

svn path=/trunk/; revision=15796

for octet strings that span beyond the end of a "short" tvb,   create a new subset tvb of what we have and use that  instead of bailing out completely.

svn path=/trunk/; revision=15795

when we have raw keberos inside the spnego blob,   pass it as is over to
the kerberos dissector  and do not strip the ber tag and lengths off

svn path=/trunk/; revision=15794

Have cf_retap_packets() take an argument that indicates whether to
generate columns; use cf_retap_packets instead of cf_redissect_packets()
when running taps (the general flow graph stat uses the Info column).

svn path=/trunk/; revision=15793

fix #388: add /desktopicon and /quicklaunchicon command line options to the NSIS installer

update the User's Guide accordingly

svn path=/trunk/; revision=15792

Move "Expert Info" from "Statistics" to "Analyze" menu item.

*significantly* improve performance (100000 infos from ~5min to 25sec!)

Add a simple severity based filter mechanism.

replace // style comments by /**/

svn path=/trunk/; revision=15791

Turn around the sequence of PATH, so the former PATH will be in front of the newly added things.

This way, the cygwin link.exe command will be behind the link.exe from MSVC.

Also added a definition of the docbook dir, floating around in my personal file for a while. This will be needed if the User's Guide is included in the NSIS installer later.

svn path=/trunk/; revision=15790

In smb_trans_defragment(), throw an exception if we encounter a too-large
fragment size.  The limit is conservatively set at 65536 bytes.  It may
have to be increased.  Fixes bug 421.

Add an entry to the release notes.

svn path=/trunk/; revision=15789

implement a way to add menu items to the "Analyze" menu

svn path=/trunk/; revision=15788

don't crash with a NULL pointer exception, if "do not decode as" is used

svn path=/trunk/; revision=15787

add to the protocol item a short description what a "Short Frame" is

svn path=/trunk/; revision=15786

increment index of IID's in COL_INFO

svn path=/trunk/; revision=15785

use the same name for menu item and dialog box title, use the right capitalization for both

svn path=/trunk/; revision=15784

Get application ID from the packet.

svn path=/trunk/; revision=15783

From Peter Johansson
PATH contents one had before starting the  build, that PATH is added to the new PATH variable 5 additional times
instead of 1.
I have made changes (supplied) to config.nmake that takes care of this
problem.

svn path=/trunk/; revision=15782

don't simply remove the expert_info output at BoundsError, but add the comment from Guy why this (usually) isn't a bug (because of a short snapshot length).

svn path=/trunk/; revision=15781

From Ankur Aggarwal:
First phase of testing revealed the following errors

svn path=/trunk/; revision=15780

From Stefan Metzmacher
Some updates to the winsrepl-dssector to add COL_INFO strings...

svn path=/trunk/; revision=15779

From Alejandro Vaquero.

svn path=/trunk/; revision=15778

From Piros Lucian:
A new dissector - cimd dissector. CIMD stands for Computer Interface to Message Distribution and it's used to transfer short messages between applications and Nokia Short Message Service Center.

svn path=/trunk/; revision=15777

Move the definition of GNUC_FORMAT_CHECK() to its own header, use it in
column-utils.h, and add it to expert.h, so we check the arguments to
"expert_add_info_format()", at least if the format argument is a
constant string.

Fix some more calls to "expert_add_info_format()" to pass it a format
string.

Don't record BoundsError exceptions as expert events - they merely
reflect a capture done with a snapshot length too short to capture all
of the packet (any case where it's caused by something else is a bug).

svn path=/trunk/; revision=15776

Don't pass an arbitrary string from a packet to a routine expecting a
format string - the arbitrary string might contain "%" characters, so
the routine might then fetch arbitrary junk and try to use it as, for
example, a string pointer.

svn path=/trunk/; revision=15775

Convert some more "tvb_memcpy()"s fetching IPv4 and IPv6 addresses to
calls to "tvb_get_ipv4()" and "tvb_get_ipv6()".

Fix a call in the BGP dissector to properly fetch an IEEE floating-point
number.

Update some I-D info.

svn path=/trunk/; revision=15774

When jumping (e.g. using menu item Go/Go to Packet...) to a new packet which is currently not displayed, this newly selected packet will become the first in the packet list.

This is inconvenient, as most of the time (at least) I'm interested not only in the packets behind the newly selected one, but also to have some history *before* it.

So this change will scroll the packet list to have the selected packet after the first third of the packet list.

This change won't take effect if the new packet is already visible (only the selection is changed) or it's near the beginning or end of the packet list (so the whole beginning/end of the list is shown).

svn path=/trunk/; revision=15772

add number of defragmented bytes to the expert info output

svn path=/trunk/; revision=15771

Get rid of "dtd_grammar.out" on a "make distclean".

Get rid of all the Flex-generated (and Lemon-generated) files on a
"nmake -f Makefile.nmake distclean".

svn path=/trunk/; revision=15770

Add a header file to declare routines defined in lexer and used in
parser, or vice versa.

svn path=/trunk/; revision=15769

Again, "g_strdown()" doesn't return a value in GLib 1.2[.x].

svn path=/trunk/; revision=15768

"g_strdown()" doesn't return a value in GLib 1.2[.x]; don't depend on it
doing so.

svn path=/trunk/; revision=15767

The previous checkin for packet-isup.c was accidentally checked in along
with some unrelated changes; it got rid of some calls to get IPv4/IPv6
addresses and add them to the protocol tree, replacing them with
proto_tree_add_item() calls, and fixed up the length of one protocol
tree item.

Note that we have (at least) three count 'em three different routines
for dissecting/displaying NSAPs (there might be more - I might have
missed some), and suggest that we might want to reduce that to one.

Update the URL for ICP values for IPv{4,6} addresses inside NSAPs.

Fix the offset in the call to add the IPv6 address part of an NSAP
containing such an address, and the length in the call adding the DSP
for an NSAP containing an IPv4 address.

Fix up indentation a bit.

svn path=/trunk/; revision=15766

In now supports also:
  - http://www.ietf.org/internet-drafts/draft-ietf-tsvwg-sctpimpguide-15.txt
  - http://www.ietf.org/internet-drafts/draft-ietf-tsvwg-addip-sctp-13.txt
  - http://www.ietf.org/internet-drafts/draft-stewart-sctp-pktdrprep-02.txt
  - http://www.ietf.org/internet-drafts/draft-ietf-tsvwg-sctp-auth-01.txt
  - http://www.ietf.org/internet-drafts/draft-ladha-sctp-nonce-02.txt

svn path=/trunk/; revision=15765

Don't collide with the "regular" IPv6 defintions in at least some
platforms in ipv6-utils.h.

svn path=/trunk/; revision=15764

fix fault defragmentation the same way as in the request/response path

Unfortunately, I don't have a capture file to test this...

svn path=/trunk/; revision=15763

fix reassembling problem I've introduced yesterday, by using fragment_add_seq_next() function instead of fragment_add()

in addition, I had to implement fragment_get_reassembled() in addition to fragment_get(), which works with reassembled_table

svn path=/trunk/; revision=15762

Fix a typo.

svn path=/trunk/; revision=15761

Frame numbers are unsigned, and they start at 1; 0 is what's used for
"unknown" for frame numbers.  Note that in epan/frame_data.h, and make
the frame number in experts unsigned, and use 0 for "unknown", and
display it as an unsigned number - and, if it's 0, don't display it at
all.

Fix the signature of "expert_dlg_draw()" to match what a tap's draw
routine's signature is expected to be.

svn path=/trunk/; revision=15760

Add some braces to squelch a compiler warning.

Fix up indentation.

svn path=/trunk/; revision=15759

Replace tvb_memcpy() calls that extract IPv4 addresses into a guint32,
and that extract IPv6 addresses into a "struct e_in6_addr", with
tvb_get_ipv4() and tvb_get_ipv6() calls - except for some that we
remove, by using proto_tree_add_item(), rather than replacing.

Have epan/tvbuff.h include epan/ipv6-utils.h, to define "struct
e_in6_addr" (not necessary to declare the tvbuff routines, but including
it there means "struct e_in6_addr" is guaranteed to be defined before
those declarations, so we don't get compiler complaints if we define it
*after* those declarations).

svn path=/trunk/; revision=15758

rename fragment_add_dcerpc -> fragment_add_dcerpc_dg to avoid confusion a bit,
as connection oriented (cn) and connectionless (dg) DCE/RPC uses different ways to handle defragmentation and this function is only used for dg

svn path=/trunk/; revision=15757

Get rid of dependency on format-oid.h

svn path=/trunk/; revision=15756

some things fixed, leftover from code cleanup (thanks to the buggy MSVC dependencies)

svn path=/trunk/; revision=15755

I'm adding the "Expert Info" prototype now, as it seems to be in a state where others might have a look and probably already find it useful :-). Anyway, we can easily disable it at one or two places in the code if it get's in our way of a new release.

Please see: http://wiki.ethereal.com/Development/ExpertInfo for a complete overview of the intended feature and it's current state of implementation.

While I'm working on this, I've also added some more status result codes to the DCE/RPC and DCOM dissectors.

svn path=/trunk/; revision=15754

Add "tvb_get_ipv4()" and "tvb_get_ipv6()" addresses, to fetch IPv4 and
IPv6 addresses.  Use "tvb_get_ipv4()" in the WINS Replication dissector,
so that it gets the right answer on little-endian *AND* big-endian
machines.

svn path=/trunk/; revision=15753

In "dissect_sip()", return 0 if "dissect_sip_common()" returns a
negative value, so we reject the packet as "not SIP".

Add a "dissect_sip_tcp_heur()" that's heuristic - it'll return FALSE if
the first packet doesn't look like SIP, but once it's dissected one
packet it'll just treat subsequent SIP packets as continuations if they
don't look right.

Clean up indentation a bit.

svn path=/trunk/; revision=15752

propperly handle the errors when loading a file fails.

svn path=/trunk/; revision=15751

don't use fgets() as MSVC does not implement it. use fgetc instead.

svn path=/trunk/; revision=15750

fix MSVC slash/backslash and other such problems, now getting:

dtd_preparse.obj : error LNK2001: Nichtaufgeloestes externes Symbol _fgetln

MSVC doesn't support fgetln!

svn path=/trunk/; revision=15749

Make sure lemon is built before trying to use it

svn path=/trunk/; revision=15748

fix the dtd_grammar.c rule

svn path=/trunk/; revision=15747