Tell the user about the Q&A site.

svn path=/trunk/; revision=35412

On Windows, convert all of our command-line arguments from UTF-16 to
UTF-8 where we don't already do so. In Wireshark use g_utf16_to_utf8
instead of utf_16to8. This should fix bug 5520.

svn path=/trunk/; revision=35411

Prohibit strtoull(): Windows doesn't have it.  (Putting this in just saves waiting for the buildbot to complain.)

svn path=/trunk/; revision=35410

Let's discourage the use of g_warning in dissectors.  Either use one of
the dissector-bug macros, or try your best to dissect a packet that's
"wrong".

svn path=/trunk/; revision=35409

Don't use g_warning() - either you have a dissector bug, and should use
one of the macros to report that, or you have a problem with the packet,
in which case you should note that in the protocol tree, or you have
something you don't understand, in which case you should dissect
whatever of it you do understand and put something appropriate, if
possible, into the protocol tree for the rest.

(And, if the length isn't right, there's not much you can do about it -
you have to trust the length, and manage to fail somewhere else.)

svn path=/trunk/; revision=35408

More misc. CLDAP improvements for Active Directory including updating
field names and adding descriptions, changing the Domain GUID in the "LDAP
ping" response to a FT_GUID instead of FT_BYTES, etc.

svn path=/trunk/; revision=35407

Fix the fuzz failure in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5550 :

In dissect_amqp_0_10_array() if the 'type' is unknown, don't loop (for
potentially a very long time) adding the same element over and over again
(since the type is unknown, we don't know how much to increase the offset so
an exception is never thrown).

svn path=/trunk/; revision=35406

Add URL to Lua website. Whitespace cleanup.

svn path=/trunk/; revision=35405

More minor cleanup/fixes for MS Active Directory's use of CLDAP.

svn path=/trunk/; revision=35404

Minor corrections to NtVer dissection for NETLOGON_NT_VERSION Options Bits
in MSCLDAP packets per Microsoft's MS-ADTS specification, section 7.3.1.1,
revision 26 (11/19/2010).  Also re-format code a bit.

svn path=/trunk/; revision=35403

GEOIP_COUNTRY_EDITION_V6 isn't guaranteed to be defined.

svn path=/trunk/; revision=35402

When showing a DNS query response containing a SRV record, display items
in the tree of the service record broken out.  For example,
"_ldap._tcp.domain.com" shows:

Service: ldap
Protocol: tcp
Name: domain.com

svn path=/trunk/; revision=35401

(After discussion on -dev) revert 35393: the old code is faster (and it is safe).

svn path=/trunk/; revision=35400

strtoull() doesn't exist on Windows, use g_ascii_strtoull() instead.

svn path=/trunk/; revision=35399

Use tvb_memeql() instead of memcmp()+tvb_get_ptr().

svn path=/trunk/; revision=35398

Actually use proto_tree_add_item().

svn path=/trunk/; revision=35397

There's no need to pass the result of tvb_get_ptr() as the 'value' in proto_tree_add_string: just use proto_tree_add_item().

svn path=/trunk/; revision=35396

There's no need to pass the result of tvb_get_ptr() as the 'value' in proto_tree_add_string: just use proto_tree_add_item().

svn path=/trunk/; revision=35395

Rather than creating strtoul{l} functions that can deal with possibly non-NULL
terminated strings (retrieved with tvb_get_ptr()), just use
tvb_get_ephemeral_string() and the standard strtoul{l} functions.

svn path=/trunk/; revision=35394

Use tvb_get_ephemeral_stringz() instead of tvb_get_strsize() plus (sometimes multiple) calls to tvb_get_ptr().

svn path=/trunk/; revision=35393

Rearrange the Intenals and Help menu somewhat.

svn path=/trunk/; revision=35392

Reset the global pdu_type *before* returning.

svn path=/trunk/; revision=35391

Remove an unreached return.

svn path=/trunk/; revision=35390

Remove a duplicate return.

svn path=/trunk/; revision=35389

Remove a break that makes other following statements unreached.

svn path=/trunk/; revision=35388

Remove unreached statement.

svn path=/trunk/; revision=35387

Replace 2 switch()es with duplicated values (but missing most of the case statements--leading the Sun compiler to complain about most of the statements not being reached) with a value_string and a couple of val_to_str() calls.  Also some minor reformatting of the function in question.

svn path=/trunk/; revision=35386

Update manuf, which the auto-update script has been missing for the past
few weeks.

svn path=/trunk/; revision=35385

Use some tvb_ accessor functions instead of a standard function plus tvb_get_ptr().

svn path=/trunk/; revision=35384

Handling a too-long length should be handled better.

svn path=/trunk/; revision=35383

Fix (legitimate) warning.

svn path=/trunk/; revision=35382

Use proto_tree_add_item() to add some FT_BYTES to the tree instead of calling proto_tree_add_bytes() with the result of tvb_get_ptr() passed as the 'value'.

svn path=/trunk/; revision=35381

From Steve Huston: add support for AMQP 0-10.

From me: add 0_9 to names for #defines and routines for 0-9, add expert
info for the "you ran past the end of the field table" error.

svn path=/trunk/; revision=35380

Don't pass the result of tvb_get_ptr() into a %s format string: the string may
not be NULL terminated.  Use tvb_get_ephemeral_string() instead.

svn path=/trunk/; revision=35379

Enhance IE dissection.

svn path=/trunk/; revision=35378

Don't pass the result of tvb_get_ptr() into a %s format string: the string may
not be NULL terminated.  Use tvb_get_ephemeral_string() instead.

svn path=/trunk/; revision=35377

Don't pass the result of tvb_get_ptr() into a %s format string: the string may
not be NULL terminated.  Use tvb_get_ephemeral_string() instead.

svn path=/trunk/; revision=35376

Don't pass the result of tvb_get_ptr() into a %s format string: the string may
not be NULL terminated.  Use tvb_get_ephemeral_string() instead.

svn path=/trunk/; revision=35375

Remove a stray debug printf().

svn path=/trunk/; revision=35374

Support FP_R8 over ATM, plus a couple of other tidying-ups.

svn path=/trunk/; revision=35373

Clean up the white space.

svn path=/trunk/; revision=35372

Update the AUTHORS file

svn path=/trunk/; revision=35371

From Francesco Fondelli:
RSVP: support for Vendor Private objects, RFC 3936
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5468

svn path=/trunk/; revision=35370

Put end-of-field-table checks back into dissect_amqp_field_table().

svn path=/trunk/; revision=35369

Use reported length (instead of the captured length) when dissecting error
causes (should have been in rev 35366).

When generating TVB subsets, limit the subset's backing and reported lengths to
the (captured) TVB length and the reported TVB length, respectively.

This allows us to dissect most of the packet in
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5546
before asserting out.

It also yields similar better behavior when the capture is limited by a
snapshot length.

svn path=/trunk/; revision=35368

Reverse r34810 other than the comment because the re-ordering of dns_types[]
caused the numbering to no longer be lined up so the "SRV" record (#33) type
T_SRV was being given the description "EID" instead of "Service Location"

svn path=/trunk/; revision=35367

When generating a tvb subset for chunks, parameters, and error causes, set the:
 - "reported length" to be minimum of the entity's length (taken from the
   packet) or the reported length of the TVB.
 - "backing length" to be the minimum of the reported length (above) or the
   (captured) length of the TVB.

This prevents tvb_new_subset() from generating an exception if the entity's
length (in the packet) is bogus (bigger than what's in the TVB) which allows
dissection to continue to a point where we can show the user what the problem
is.

When dissecting chunks, add an expert info if the item's length is bigger than
what was on the wire (the reported length).  (The same could also be done for
parameters and error causes.)

This makes captures like that in https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5546
easier to understand.

It also starts getting this dissector using the reported length instead of the
(captured) TVB length (there's more to be done with other tvb subsets).

svn path=/trunk/; revision=35366

Drop Python dissectors directory flag from programs that don't have dissectors.
Win32: remove superfluous dumpcap build dependancy on wiretap.

svn path=/trunk/; revision=35365

Check hf entries *after* removing comments (in case there are commented-out entries).

svn path=/trunk/; revision=35364

Detect (and error on) NULL or empty abbreviations.  Wireshark aborts on the former and warns on the latter.

svn path=/trunk/; revision=35363

Give a couple of fields abbreviations.

svn path=/trunk/; revision=35362

Maeke use of dissect_e212_mcc_mnc_ep_str().

svn path=/trunk/; revision=35361

Introduce dissect_e212_mcc_mnc_ep_str()

Which can be useful to make nice lables like:
+ Serving Network : MCC 311 United States of America, MNC 28

svn path=/trunk/; revision=35360

From Alexis La Goutte via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5544 :

Include the packet-ipv6.h changes too.

svn path=/trunk/; revision=35359

Fix build.

svn path=/trunk/; revision=35358

Don't use DISSECTOR_ASSERT() to catch malformed packets.  Fixes bug
4048.

svn path=/trunk/; revision=35357

Instead of going over the Report segment twice, once to get the fields
in the segment and once to put them into the protocol tree, just do it
once.  That also means we don't need to allocate an array for all the
reception claims, so we don't have to worry about the claim count (other
than making sure it's non-negative), and that we won't abort the
dissection until we run past the end of the packet.

(The rest of the dissector should be changed to work that way as well.)

svn path=/trunk/; revision=35356

From Alexis La Goutte:
A Patch to enchance the ICMPv6 Router Renumbering for IPv6 (RFC 2894) Dissector
- Make RR field filterable
- Add RR Result Message
- ...

svn path=/trunk/; revision=35355

Add more info to the tree.

svn path=/trunk/; revision=35354

Free dumpcap from linking to wiretap.

svn path=/trunk/; revision=35353

Prettify dissection of User Location Info (ULI). (Complete)

svn path=/trunk/; revision=35352

Prettify dissection of User Location Info (ULI).

svn path=/trunk/; revision=35351

From Hadar Shoham:
Add support to EuroCableLabs at bootp opt 60 sub 23.
according to PKT-SP-PROV1.5-I04-090624, paragraph 10.23 Device MIB Support:
type 2 is indicating EuroCableLabs.

svn path=/trunk/; revision=35350

State in the developer documentation that the tvb_fake_unicode() and
tvb_get_ephemeral_fake_unicode() functions have been superceded by
tvb_get_unicode_string() and tvb_get_ephemeral_unicode_string() respectivey.

svn path=/trunk/; revision=35349

Remove the other use of tvb_fake_unicode() from the DCE/RPC dissector and
replace it with tvb_get_unicode_string().

svn path=/trunk/; revision=35348

Fix (or at least quiet Valgrind about) https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5542 :

Always initialize *all* the outputs of get_sdp_type().

svn path=/trunk/; revision=35347

Switch one of the places in the DCERPC dissector from using
tvb_fake_unicode() to tvb_get_unicode_string().

svn path=/trunk/; revision=35346

Document new tvb_get_unicode_string() and tvb_get_ephemeral_string()
functions.

svn path=/trunk/; revision=35345

Introduce two new functions for Unicode (UTF-16) string handling:

tvb_get_unicode_string()
tvb_get_ephemeral_unicode_string()

These function like their counterparts, tvb_get_string and
tvb_get_epemeral_string, for standard strings.

Also update comment on what the first such function,
tvb_get_ephemeral_unicode_stringz does regarding updating lengthp.

svn path=/trunk/; revision=35344

Partially fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5542 :

Fix one of the "Conditional jump or move depends on uninitialised value(s)"
errors from Valgrind: always initialize *val in get_sdp_type().

svn path=/trunk/; revision=35343

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

Don't go modifying the bitstring if its length is 0 (meaning that there is only
the NULL terminator).

svn path=/trunk/; revision=35342

Mark unused parameters as such.

svn path=/trunk/; revision=35341

Mark unused parameters as such.

svn path=/trunk/; revision=35340

Comment out a faulty line.

svn path=/trunk/; revision=35339

Enhance IE dissection and a number of bug fixes.

svn path=/trunk/; revision=35338

We must always return an error code on an error; otherwise, our caller
will see random crap as the error code.

However, if we're skipping a "TCPIPTRACE-W-BUFFERSFUL" error, if the
"error" we get is an end-of-file indication, that's *not* an error.

It is, however, ultimately a "we dropped some packets" indication; add a
comment noting that we should eventually treat it as such.

svn path=/trunk/; revision=35337

Put the query list items under the top-level query list item (which
should've been done in the previous checkin).

Display the "request type" field in a Read Partition request correctly
(only the upper 2 bits matter, and they're an enumerated value).

svn path=/trunk/; revision=35336

If we have no value to dissect, because the AVP has the wrong length for
the value's type, return NULL as the label, rather than returning an
uninitialized value as the label.

svn path=/trunk/; revision=35335

Add relevant docbook tags in the source.

svn path=/trunk/; revision=35334

Include dumpcap in the description.

svn path=/trunk/; revision=35333

Do length checking for Integer32, Integer64, Unsigned64, Float32, and
Float64, as well as for Unsigned32.

svn path=/trunk/; revision=35332

Note that this is ultimately not necessarily the right fix.

svn path=/trunk/; revision=35331

For those cases where we interpret the contents of an OCTET STRING,
which could be of arbitrary length - even if it's not supposed to be! -
as a value of some other type, by adding them as a registered field,
first check to make sure the length of the field is appropriate for the
type and, if not, show a dissection error, rather than showing a
dissector-bug assertion when we call proto_tree_add_item().

This fixes a bunch of dissector-bug assertions that show up with
malformed BER-encoded packets.

Also, fix a typo, and expand a comment.

svn path=/trunk/; revision=35330

Fix indentation.

svn path=/trunk/; revision=35329

Fix some cut-and-pasteos and other typoes.

svn path=/trunk/; revision=35328

[Automatic manuf, services and enterprise-numbers update for 2011-01-02]

svn path=/trunk/; revision=35325

Add missing packet-btsdp.h

svn path=/trunk/; revision=35324

From Allan M. Madsen via. bug 5445:

Bluetooth profiles and protocols above RFCOMM and L2CAP can not be dissected correctly because the required information (server channel and dynamic PSM value mappings to services/profiles) about the type of data carried in the payload is not available. RFCOMM is currently hardcoded to handoff all payload data to the obex dissector though it may carry e.g. handsfree, dial-up networking or serial port profile related data.

The patch consists of modifcations to the following dissectors:

btsdp: Extraction of RFCOMM server channel and L2CAP dynamic PSM with service mapping is provided to RFCOMM and L2CAP through a tap interface. In addition, the packet list info is beautyfied and extended with more details for better
overview.

btl2cap: Adds a new dissector table with services and dynamic PSM mapping which is filled by a tap listner catching the info from btsdp. More info added to packet list.

btrfcomm: Adds a new dissector table with services and server channel mapping which is filled by a tap listner catching the info from btsdp. Dissectors for handsfree, dial-up netorking and serial port profiles (all based on RFCOMM) are also added.

btobex: Registers several obex based profiles (e.g. obex push, file transfer, basic printing etc.) in both RFCOMM and L2CAP. Some cleanup.

svn path=/trunk/; revision=35323

New Year.

svn path=/trunk/; revision=35322

Add packet-json.c

svn path=/trunk/; revision=35321

Update some URLs, add another spec.

At least for one of the DOCSIS suboptions, do length checking.

svn path=/trunk/; revision=35320

A Query List has more than one has multiple QCODEs; you cannot handle
that by doing the entire list as a single FT_UINT8 field, you must
handle each QCODE, in a loop, as that field.

svn path=/trunk/; revision=35319

From FRAsse via bug 5539:

There's a buffer overflow in ENTTEC DMX Data RLE, leading to crashes and
potential code execution.

From me: ep_allocate our buffers.

svn path=/trunk/; revision=35318

From Jakub Zawadzki via bug 4753: New dissector: JSON, w/minor changes by me:
1) Add links to RFC 4627 and the json.org web site.
2) Comment out hf_json_member_key to keep tools/checkhf.pl happy.
3) Avoid duplicate (application/json) from being displayed in Info column.

svn path=/trunk/; revision=35317

Whops, packet-sgsap.c is not ASN1.

svn path=/trunk/; revision=35316

Added missing packet-sgsap.c

svn path=/trunk/; revision=35315

Add current list of ROHC profile names, taken from:
http://www.iana.org/assignments/rohc-pro-ids/rohc-pro-ids.txt

As raised by bug 5536, we now at least show which profile is present.
Supporting all of the specific message types is another matter...

svn path=/trunk/; revision=35314

It's not a good idea to use tcph->th_seglen if you haven't set it.

Fixes bug 4211.

svn path=/trunk/; revision=35313

Update the export dialog screen shots.

svn path=/trunk/; revision=35312

Corrected comment for col_custom_occurrence.

svn path=/trunk/; revision=35311