Fix some comments.

svn path=/trunk/; revision=7097

Rename "dissect_ndr_char_string()" and "dissect_ndr_wchar_string()" to
"dissect_ndr_char_cvstring()" and "dissect_ndr_wchar_cvstring()", to
indicate that they're for conformant varying strings.

Rename "dissect_ndr_character_array()" to "dissect_ndr_cvstring()", to
indicate that it's for conformant varying strings.

svn path=/trunk/; revision=7096

Rename "dissect_ndr_char_array" and "disect_ndr_wchar_array" to
"dissect_ndr_char_string" and "dissect_ndr_wchar_string", to make it
clearer what it does.

svn path=/trunk/; revision=7095

From Thierry Pelle: updates (use the correct RFC 2472 term - "interface
identifier", not "interface token" - and print the octets of the
interface identifier with zero padding).

svn path=/trunk/; revision=7094

Thou Shalt Not Ever Make An Offset An 8-bit Quantity If Thou Art
Comparing It Against An 8-bit Or Longer Length To Make Sure It Doesn't
Go Past The Length, because if the length is 255, it can't ever go past
it as it'll overflow if it does.

svn path=/trunk/; revision=7093

Catch ReportedBoundsError when dissecting even non-encrypted stub data,
so that even if the stub data is bad, we still dissect and show the
verifier.

svn path=/trunk/; revision=7092

Fix a typo in the multiple-include protection in "packet-dcerpc-nt.h".

Rename "dissect_ndr_element_array()" to "dissect_ndr_character_array()",
move it out of "packet-dcerpc-nt.c" to "packet-dcerpc.c", and have it
use the standard DCE RPC array max count/offset/count fields rather than
their own private versions of those fields.  Give it an option to create
a subtree, and an argument to specify the field to use for the actual
data buffer, and export it.

Move the routines for handling arrays of "char" and "wchar" as strings
out of "packet-dcerpc-nt.c" to "packet-dcerpc.c".

Add a routine to handle an array of "char" as an opaque blob of bytes.

Use "dissect_ndr_character_array()" to dissect character strings in MAPI
(the strings in question are ASCII, not Unicode), and use the routine to
handle an array of "char" as an opaque blob of bytes to dissect
encrypted data (again, it's bytes, not 16-bit quantities).  Show them as
encrypted data, not unknown data.

Use "dissect_ndr_character_array()" to dissect a form name in
"dissect_form_name()" in the SPOOLSS dissector.

svn path=/trunk/; revision=7091

Decorate several layers of the tree with the account name in
"USER_INFO_21" and in "LOOKUP_NAMES".

svn path=/trunk/; revision=7090

Decorate the top-level items for the account name in
POLICY_ACCOUNT_DOMAIN_INFO and the domain in TRUSTED_DOMAIN with the
string.

svn path=/trunk/; revision=7089

Decorate the top-level tree items for "LSA_TRANSLATED_NAME" and an
"LSA_TRUST_INFORMATION" with the account name.

svn path=/trunk/; revision=7088

Fix callers to dissect_nt_sec_desc() to use new function interface.

Dissection of security descriptors in SPOOLSS RPC calls now display
the correct meaning of the specific access mask bits.

svn path=/trunk/; revision=7087

Move dissect_nt_access_mask() from packet-dcerpc-nt.c to packet-smb.c

Give dissect_nt_sec_desc() and dissect_nt_access_mask() a specific rights
function parameter for dissecting specific access rights.

Fix callers in packet-smb.c to use the new interface.

svn path=/trunk/; revision=7086

Add the RFC 3203 FORCERENEW message type, as suggested by Suresh K.

svn path=/trunk/; revision=7085

From Olivier Biot: WBXML/WMLC support.

svn path=/trunk/; revision=7084

The "Subdirectory" bit in search attributes means "subdirectories only",
and in file attributes means "this is a subdirectory"; don't mix the
two.

The 1-byte and 2-byte search attributes appear to have the same bit
definitions (except, obviously, for those that are in the topmost byte).

svn path=/trunk/; revision=7083

- protect against multiple inclusion
- remove incorrect and unused definition
  of tapping_is_active

svn path=/trunk/; revision=7082

Various XXXDirectoryID values always appear to be big-endian (including
the ones not specified as such when used), so declare them as such
rather than specifying them as such when used.

The SearchSequenceWord also appears to be big-endian.

Note that we're not cracking the bits of a DirectoryAttributes field.

The "Subdirectory" bit in search attributes is really "Subdirectories
Only", as in "just show me subdirectories".

Note some confusion about whether the bit numbers on the Novell Web site
for search attributes are bit numbers or bit flags.

Note that we appear to have gotten back attributes for a file rather
than a directory in at least one search that had "Subdirectories Only"
set - unless the problem is that the numbers in the Novell spec are bit
numbers rather than bit flags.

Update some items that claimed to have a 2-byte search attributes field
to have a 1-byte search attributes field instead, to match the spec on
the Novell site (of course, the spec could be wrong...).

svn path=/trunk/; revision=7081

An object of type 0x5555 is apparently a "Site Lock".

svn path=/trunk/; revision=7080

Sometimes the trace stuff doesn't appear in connect messages.

svn path=/trunk/; revision=7079

Cleaned up print job filterable fields.

Set item len for devicemode dissector.

Fixed dissection of relative strings so that the actual value of
the string is assigned to the hf item instead of the empty string.

Dissect JOB_INFO_2 structure.

svn path=/trunk/; revision=7078

FileSize appears to be big-endian in DOSFileEntryStruct and FileInstance
structures; make it so (which makes it big-endian on other structures as
well).

The same appears to be true of the UpdateID.

There doesn't appear to be a reserved word between the
SearchSequenceWord and DirectoryID fields of a DirectoryInstance or a
FileInstance.

The dates and times in a DirectoryInstance or a FileInstance appear to
be big-endian; note that we need to add the ability to make those
instances big-endian but leave others little-endian.

Note that there's a sequence of NameSpaceNames and a sequence of
DataStreamNames in the reply to a Get Name Space Operation request.

A Login Object request has a 2-byte object type followed by a
ClientName, just a UserName.  The same applies to a Get Object
Connection List request.

Note that the DirectoryInstance or FileInstance in a File Search
Continue reply appears to be garbage if the reply doesn't succeed with
"OK".

svn path=/trunk/; revision=7077

Get rid of unused ett_ variables (for fields that don't have subfields).

Put in URLs for the RMI and serialization protocols.

Move value_string arrays, and declarations of static routines, out of
the header file into the .c file.

For integral values with associated value_string tables, just put them
into the protocol trees as integral values and associate the
value_string table with the field.

Use "proto_tree_add_item()" whenever possible.

Use the #define for "JRMI" rather than hardcoding it.

svn path=/trunk/; revision=7076

Deleted RPC request/response proto items from the spoolss dissector as
it's done better in the dcerpc dissector.

svn path=/trunk/; revision=7075

Make the dcerpc.request_in and dcerpc.response_in fields FT_FRAMENUM's.

svn path=/trunk/; revision=7074

Fixed bug in dissect_printerdata_data()

Display something useful in COL_INFO when dissecting REG_BINARY
printerdata.

Display the value needed field in value subtree.

svn path=/trunk/; revision=7073

 Add a list for heuristic dissectors for COTP/ISO 8473
 (non inactive subset). So there are now two different
 lists : "cotp" and the legacy "cotp_is". Cotp_is is
 currently only used by H1 dissector. "cotp" will be
 used at least by some (possibly unpublished) plugins.

svn path=/trunk/; revision=7072

From Teemu Rinta-aho: draft 20 MIPv6 support (now in a file of its own).

svn path=/trunk/; revision=7071

Remove out of date comment.

Display server name in COL_INFO for OpenPolicy2 request.

svn path=/trunk/; revision=7070

Convert to new DCERPC string handling functions.  There are still some
cosmetic bugs to work out though.

svn path=/trunk/; revision=7069

Replace the confusing collection of Windows DCERPC string handling
functions with something a little less confusing.

We now have two sets of functions to dissect strings:

 - dissect_ndr_wchar_array() which dissects NDR arrays of wide characters
   (uint32, uint32, uint32, buffer)

 - dissect_ndr_counted_string() which dissects a "counted string"
   (uint16, uint16, pointer to array of wchar)

There are three contexts for dissecting counted strings:

 1. "in-line" using dissect_ndr_counted_string()
 2. as a callback to dissect_ndr_pointer()
 3. as a callback to dissect_ndr_pointer_cb()

Context 2 is used when you have a pointer to a counted string.
Context 3 is when you wish to perform some special processing with the
returned string.

svn path=/trunk/; revision=7068

Decorate one level up with the string in
"samr_dissect_pointer_UNICODE_STRING()".

Put in a comment asking why that routine exists.

svn path=/trunk/; revision=7067

From Pasi Kovanen: display flow label IE in GTP v0 in hex.

svn path=/trunk/; revision=7066

Update a comment.

svn path=/trunk/; revision=7065

Fix up the construct of AS strings (the opening curly bracket wasn't
being put in for sets).

svn path=/trunk/; revision=7064

Decorate a PRIV_NAME_ARRAY with the privilege names in the array.

svn path=/trunk/; revision=7063

Decorate a DELTA_DOMAIN structure's top-level item with the domain name.

svn path=/trunk/; revision=7062

Put in a missing assignment of the value of
"dissect_ndr_nt_UNICODE_STRING_cb()" to "offset".

svn path=/trunk/; revision=7061

Fix up a couple of references to "tvb" that should refer to "tr_tvb".

Note that the whole hack for dealing with Linux bogosities should
perhaps have an option to turn it off, as sometimes it detects
bogosities that aren't there.

svn path=/trunk/; revision=7060

Fix up some field lengths.

Put in a note about one capture I've seen.

svn path=/trunk/; revision=7059

If "dissect_ndr_nt_STRING()" is being used to dissect a byte string
rather than a character string, don't decorate anything above that item
in the tree with the "string".

svn path=/trunk/; revision=7058

Top-level share entries are now decorated with the share name; put a
colon after "Share:".

svn path=/trunk/; revision=7057

bit-plane is a bitmask; display it in hex.

svn path=/trunk/; revision=7056

The handle returned by the OpenDomain RPC now has the SID in the handle
name.

Made OpenGroup and CreateGroup handle names contain appropriate RID values.

Added some more COL_INFO bits & pieces (info levels etc).

svn path=/trunk/; revision=7055

Modify dissect_nt_sid() function to return a string version of the SID.

Fix all callers to use the new function form.

svn path=/trunk/; revision=7054

Get rid of BASE_BIN - it's just the same as BASE_DEC, but people seemed
to be using it for stuff that should be hex, and for stuff that should
be Boolean.  Use BASE_DEC if it should be decimal, BASE_HEX if it should
be hex, and make it Boolean if it should be Boolean.

svn path=/trunk/; revision=7053

Add WTAP_ENCAP_FRELAY_WITH_PHDR for use with Frame Relay capture files
that have direction information.

Support writing WTAP_ENCAP_FRELAY_WITH_PHDR and WTAP_ENCAP_PPP_WITH_PHDR
captures out in libpcap format - we throw away the direction
information, but so it goes.

When reading/writing Windows Sniffer format, read and write the
direction flag.

svn path=/trunk/; revision=7052

Fixed memory leak in dissect_unistr2().

Fixed double free bug in GetPrinterData occuring when applying a filter.

Cleaned up same bit of code in OpenPrinterEx.

svn path=/trunk/; revision=7051

Don't append string to upper-level proto_items if item == NULL.

svn path=/trunk/; revision=7050

The MemberType field in various bindery operations is 2 bytes, not 1
byte or 4 bytes.

svn path=/trunk/; revision=7049

Add support for writing Frame Relay files in NetXRay format 2.x.

svn path=/trunk/; revision=7048

Remove unprotected use of GCC __attribute__ in an unused #define

svn path=/trunk/; revision=7047

Put in URLs for copies of the now-expired I-D for MSN Messenger, and a
note that it really should be dissected like other FTP-flavored
protocols.  (Or is that "SMTP-flavored"?  Which was the first Internet
protocol to have that "command/response-with-3-digit-error-code" form?)

svn path=/trunk/; revision=7046

From Chris Waters: MSN Messenger support.

svn path=/trunk/; revision=7045

AC_ARG_ENABLE takes 4 argument: The 3rd specifies what to do in case a
configure option is given on the command line. The value of the arguement
is passwd in the enableval variable. The 4th argument tells what to do in
case no command line argument was given.
This causes --disable-gtk2 (which is the default) to behave differently
from the case when no option is given.
I do not really understand where the difference in the behaviour of the
generated codes comes from, but I definitely see a difference.

Fixed all occurrences where the 3rd arguement was empty.

svn path=/trunk/; revision=7044

Generalize "cb_str_postprocess()" to allow the string to be appended to
items N levels up from the item being processed, and use that to
decorate the tree as it was decorated before.

svn path=/trunk/; revision=7043

FT_STRING and FT_STRINGZ have no base, so use BASE_NONE.

The payload isn't a text string, it's an array of bytes; make it
FT_BYTES, not FT_STRING.

svn path=/trunk/; revision=7042

NDR strings begin with a uint32, so they start on a 4-byte boundary.

svn path=/trunk/; revision=7041

Display random useful tidbits in COL_INFO.

svn path=/trunk/; revision=7040

Factor out srvsvc_dissect_pointer_UNICODE_STRING() and replace it with
calls to our dissect_ndr_str_pointer_item() helper function.

svn path=/trunk/; revision=7039

Get rid of the "sap_server_ident" structure - just fetch stuff from that
structure in the packet as required.

The server name in that structure is not necessarily null-terminated;
use "%.48s", not "%s", to print it.

svn path=/trunk/; revision=7038

Clean up the handling of null buffers in getprinter, getform,
getjob and getprinterdriver2.

svn path=/trunk/; revision=7037

Put the computer_name in COL_INFO when dissecting a
BROWSE_REQUEST_ANNOUNCE packet.

svn path=/trunk/; revision=7036

Add some missing specific configure options.

svn path=/trunk/; revision=7035

Fix a couple of checks for families with renamed preferences.

svn path=/trunk/; revision=7034

Add the FAQ to the help menu

svn path=/trunk/; revision=7033

From Huagang Xie: more robust parsing for illegal SSH packet.

svn path=/trunk/; revision=7032

More completely explain the "name" argument to the
"prefs_register_XXX_preference" routines, to note that it should *NOT*
include the protocol name - the protocol name is automatically prepended
to it, with a "." separator, as the preference is registered in a module
whose name is that of the protocol.

svn path=/trunk/; revision=7031

Panic if a preference starts with the name of the module to which it
belongs, as that's redundant.

Fix a bunch of cases where that was done, and map the old name to the
new name.

Instead of marking "mtp3.mtp3_standard" as obsolete, map it to
"mtp3.standard".

svn path=/trunk/; revision=7030

Removed some out of date comments.

svn path=/trunk/; revision=7029

* Fix compilation errors with some installation of NET-SNMP
* Fix some compilation warnings (conservative changes here)

svn path=/trunk/; revision=7028

Cleanup of filterable fields for printer data and devicemodes.

Added a stack of value_strings for devicemode fields that didn't
have them already.

svn path=/trunk/; revision=7027

Added constants for devicemode value_strings.

svn path=/trunk/; revision=7026

Get rid of the extra "pcli." in the "pcli.pcli.udp_port" preference.
Map "pcli.pcli.udp_port" to "pcli.udp_port".

svn path=/trunk/; revision=7025

Add the MEGACO and PCLI plugins.

svn path=/trunk/; revision=7024

Booleans are 1 byte, not 4 bytes.

svn path=/trunk/; revision=7023

Minor corrections and modifications.

svn path=/trunk/; revision=7022

Thou shalt not cast pointers of unknown alignment to pointers to
multi-byte integral quantities and blithely dereference them, lest thou
enrage those of us who work on machines that trap on unaligned
references.

svn path=/trunk/; revision=7021

Do the usual "isprint()" workaround for versions of GTK+ that assume
UTF-8 strings.

svn path=/trunk/; revision=7020

From Huagang Xie: SSH version 1 support.

svn path=/trunk/; revision=7019

Convert dissector to callback based version of dissect_ndr_pointer().

This fixes a bunch of neat stuff that was broken after the conversion
to the dissect_ndr_* functions like printer handle tracking by name
and many COL_INFO things.

svn path=/trunk/; revision=7018

Convert dissector to callback based version of dissect_ndr_pointer().
Strings that used to call with levels != -1 should call the
callback helper which will append the string to the pointer item.

svn path=/trunk/; revision=7017

Defined a useful callback function for dissect_ndr_pointer() which can
be used to help out in most DCERPC strings.  The cb_str_postprocess()
function reads the callback_args and either appends the value of the
string to the COL_INFO field, appends it to the NDR pointer item, or
saves the string to dcv->private_data.  Calling cb_str_postprocess()
with CB_STR_ITEM is the same as calling dissect_ndr_pointer() with
levels = 0.

Renamed some of the pointer dissection functions with a suffix of _cb
and created helpers of the original which call the _cb function with
NULL args.  This should help minimise the amount of code changes in
the bulk of the DCERPC dissectors.

svn path=/trunk/; revision=7016

Replace the 'levels' argument to dissect_ndr_pointer() with a callback
function and a void * callback args.  The callback is executed after
the dissection of the ndr pointer buffer which may be called,
depending on the number of pointers in the structure, after the return
of the dissect_ndr_pointer() call.

The callback function is of type:

void (dcerpc_callback_fnct_t)(packet_info *pinfo, proto_tree *tree,
proto_item *item, tvbuff_t *tvb, int start_offset, int end_offset,
void *callback_args);

where the proto tree and item are the tree and item created by
dissect_ndr_pointer() and the tvb plus offsets are the buffer pointed
to by the pointer.

svn path=/trunk/; revision=7015

From Akira Endoh: BGP support for draft-ietf-idr-as4bytes-06.txt and
draft-ietf-idr-dynamic-cap-03.txt.

Fix his AUTHORS entry to put the (presumed) given name first and family
name last.

svn path=/trunk/; revision=7014

Fix a similar problem in the RFC 3332 version of the protocol.

svn path=/trunk/; revision=7013

Compilation warning fixes

svn path=/trunk/; revision=7012

Code cleaning & compilation warning fixes

svn path=/trunk/; revision=7011

"hf_afs_bos_status" is an FT_INT32, so it can't be put into the tree
with "proto_tree_add_uint()", which "OUT_UINT()" uses; use "OUT_INT()"
instead.

svn path=/trunk/; revision=7010

From Jeff Morriss: dissect parameters regardless of whether a protocol
tree is being generated or not, so that subdissectors get called.

svn path=/trunk/; revision=7009

Make the list of column format descriptions "static", and size it based
on the number of column formats - and fix that list by adding a missing
comma between "Circuit ID" and "Src PortIdx".

Add to "col_format_desc()" a check that the argument is valid.

svn path=/trunk/; revision=7008

Check "$prefix/include/pcap" before checking "$prefix/include" for
"pcap.h", in case $prefix is set to "/usr/include" or
"/usr/local/include" on a system with the Red Hat "feature" of burying
pcap.h in a "pcap" subdirectory.

svn path=/trunk/; revision=7007

From Yaniv Kaul: add names and types for ZLIB compression and AES
ciphers, and categorize compression methods in the reserved and private
range as such.

svn path=/trunk/; revision=7006

From Huagang Xu: treat all SSH sesions where the version cannot be
determined as having an unknown version of SSH rather than as using
SSHv1.

svn path=/trunk/; revision=7005

From Hannes Gredler: support MPLS over CHDLC.

svn path=/trunk/; revision=7004

Update my email address in various places since my old one no longer works

svn path=/trunk/; revision=7003

Call the GSSAPI dissector when disecting GSSAPI transaction keys.  The TKEY
request works quite nicely but I'm not sure what to call for the response.
Currently it just displays "Unknown header: (cls=2, cons=1, tag=1) which
looks like it's some more ASN.1 data.

svn path=/trunk/; revision=7002

SSH dissector, from Huagang Xie.

svn path=/trunk/; revision=7001

From Dave Richards: BACNET-over-ARCNET is BACNET-over-802.2-over-ARCNET,
not BACNET-directly-over-ARCNET.

svn path=/trunk/; revision=7000

Add a missing include packet-dcerpc-nt.h

svn path=/trunk/; revision=6999

Remove an unused function prototype

svn path=/trunk/; revision=6998