From Didier Gautheron:
Part 2
Extracted from optimizations patch http://wiki.wireshark.org/Development/Optimization
Optimize expert info.
Slightly changed by me.

svn path=/trunk/; revision=29478

From Didier Gautheron:
Part 1
Extracted from optimizations patch http://wiki.wireshark.org/Development/Optimization
Optimize expert info.
Slightly changed by me.

svn path=/trunk/; revision=29477

As pointed out by Alexis La Goutte via PM, r29474 didn't
support the different MAC formats (eui48 and eui64) properly.
Now, eui48 is printed as mac, the rest is still handled as
blob.

svn path=/trunk/; revision=29476

We shouldn't conditionalise on _filtering_ tap listerners to determine whether tapping should be performed or not. We also need to handle _normal_ tap listerners. This change backs out the regression introduced in r29435.

Thanks to Didier: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3916

svn path=/trunk/; revision=29475

Add support for Radio Mac/Wireless Specific Information optional
header fields. This patch lacks handling of padding since a) I don't
have a trace containing padding and b) I don't understand the
wording in the rfc (it's to ambigous for my liking).

svn path=/trunk/; revision=29474

From   Jochen Bartl:
EIGRP dissector enhancements (IPv6, Stub routing, Authentication TLVs)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3907

svn path=/trunk/; revision=29473

From   Jan Lübbe:
packet-gsm_a_rr.c: flip gsm_a_rr_mv_vals, 0 means vaild.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3915

svn path=/trunk/; revision=29472

From Didier Gautheron:
Extracted from optimizations patch http://wiki.wireshark.org/Development/Optimization
Make col_expr const gchar.

svn path=/trunk/; revision=29471

Fix a memmory leak.

svn path=/trunk/; revision=29470

Add new_packet_list_recent_write_all().

svn path=/trunk/; revision=29469

Properly handle closing/destruction of RTP player window.

svn path=/trunk/; revision=29468

Make copy summary work for new_packet_list.
XXX The enum in .h file does not seem to work???

svn path=/trunk/; revision=29467

Minor clean up.

svn path=/trunk/; revision=29466

Search for includes and libs in /opt/local/ (MacPorts).

svn path=/trunk/; revision=29465

Add an expert info noting that an ANSI message with the national indicator set to 0 (international) is unusual.

svn path=/trunk/; revision=29464

Make build bot happy?

svn path=/trunk/; revision=29463

Typo: parameter_tvb to be used.

svn path=/trunk/; revision=29462

From Mathias de Paulis Nilsson:
Updates to  V8.5.0.

svn path=/trunk/; revision=29461

From Swapnil Barai:
Minor Bug in Bearer Context in Packet-gtpv2.c fixed.

svn path=/trunk/; revision=29460

Display Microsoft's Remote Desktop Protocol (RDP) extension to the X.224
connection request that shows the session load balancing information.  There
appears to be more dissecting that can be done for this field, but I don't
have any sample captures to test this.

svn path=/trunk/; revision=29459

As pointed out by Guy, we should probably explain why the gulong -> size_t cast is 'safe'

svn path=/trunk/; revision=29458

Change window_new() to dlg_window_new() and destroy the window with the top level window.

svn path=/trunk/; revision=29457

setbit() is (AFAIK) defined in <sys/param.h> (OpenBSD) so rename to dect_setbit() to avoid clash. getbit() is also renamed for consistency.

Fixes the issue reported by Stan: http://www.wireshark.org/lists/wireshark-users/200908/msg00128.html

svn path=/trunk/; revision=29456

Remove #if 0

svn path=/trunk/; revision=29455

Prepare to store column data from pinfo.

svn path=/trunk/; revision=29454

Move the NULL pointer 'tree' check under TRY_TO_FAKE_THIS_ITEMs umbrella

svn path=/trunk/; revision=29453

Don't do PROTO_REGISTRAR_GET_NTH twice.

svn path=/trunk/; revision=29452

Add proper type cast.

Thanks to Martin for reporting this.

svn path=/trunk/; revision=29451

Add missing call to packets_bar_update() after thaw'ing

svn path=/trunk/; revision=29450

[Automatic manuf and enterprise-numbers update for 2009-08-16]

svn path=/trunk/; revision=29447

Switch a bunch of dissectors over to using tvb_new_subset_remaining()

svn path=/trunk/; revision=29446

Add a new function, tvb_new_subset_remaining(), which is a sligtly optimized version of tvb_new_subset(). The latter can be mapped to the former by tvb_new_subset(tvb, offset, -1 /* backing_length */, -1 /* reported_length */). We can disable some bounds checking because 'backing_length' and 'reported_length' are hardcoded to -1.

The current implementation of tvb_new_subset_remaining() only has the THROW_ON(reported_length < 1) check removed when compared to tvb_new_subset(). So there's room for improvement in this function. We should be able to disable some more (redundant) bounds checking.

svn path=/trunk/; revision=29445

Remove unused parameters

svn path=/trunk/; revision=29444

Remove the non-thread safe usage of 'last_tvb' variable in tvb_new_subset()

svn path=/trunk/; revision=29443

Make some of the check_XXX functions accept the length/reported length of the tvbuff as arguments instead the tvbuff. We'll need this feature in order to remove the non-thread usage of the 'last_tvb' variable in tvb_new_subset()

svn path=/trunk/; revision=29442

Remove a non thread-safe usage (useful when/if we ever support threading) of a static tvbuff in tvb_new_real_data(). The current version uses a static 'last_tvb' to keep track of the last allocated tvbuff. This is needed because some of the function we call can throw an exception. This patch improves this strategy by throwing an exception (if needed) before we try to allocate the tvbuff. This way we avoid a memleak _and_ we don't have to track the 'last_tvb' tvbuff.

svn path=/trunk/; revision=29441

Use the slice allocator if GLIB >= 2,10,0

svn path=/trunk/; revision=29440

Switch over to using GPtrArray instead of manually g_renew'ing. This switch has some benefits:

1) We don't have to manually resize
2) GPtrArray uses a sligtly more efficient allocation scheme. It increases the size of the array exponentially which reduces the amount of memcpy's we have to do when we resize the array

svn path=/trunk/; revision=29439

Mark function paramter as unused with _U_ to please gcc.

svn path=/trunk/; revision=29438

Try to please build bot

svn path=/trunk/; revision=29437

Prepare for storing column data in packet list store.

svn path=/trunk/; revision=29436

Inline a few checks in the add_packet_to_packet_list() function since it's called in tight loop. This will help us to avoid calling functions that simply returns. This is especially the case when we're dealing with the new packet list due to its dissect-on-demand nature where colour/column construction is delayed until the row becomes visible.

svn path=/trunk/; revision=29435

Use the slice allocator if GLIB >= 2,10,0 as suggested by Anders Broman

svn path=/trunk/; revision=29434

Add a missing DISSECTOR_ASSERT in tvb_init()

svn path=/trunk/; revision=29433

Make mem chunk static

svn path=/trunk/; revision=29432

Use HFILL instead of hard coding the values

svn path=/trunk/; revision=29431

Switch over to using GLib's g_malloc/g_free in the name of consistency.

svn path=/trunk/; revision=29430

This should have been part of r29428. This patch forces proto_field_is_referenced() to respect the fake_protocols flag set by Protocol Hierarchy stats tap.

svn path=/trunk/; revision=29429

The frame dissector contains a kludge to workaround the fact that the Protocol Hierarchy stats tap always needs the protocol node to appear even though no color, display filter etc. reference it. This is no longer needed due to r29380.

svn path=/trunk/; revision=29428

This patch introduces packet_add_new_data_source() which effectively deprecates add_new_data_source(). This is based on the following observation:

1) The tvb + name (aka. data_source) is only used when the protocol tree is visible

The current implementation of add_new_data_source() doesn't take this into account and simply allocates a data_source regardless. This is what packet_add_new_data_source() tries to rectify.

A couple of dissectors have already been switched over to the new packet_add_new_data_source(). Many are still missing. Help appreciated!

svn path=/trunk/; revision=29427

Micro optimize by storing PACKET_LIST(tree_model) into a local variable and using that instead of computing PACKET_LIST(tree_model) each time.

svn path=/trunk/; revision=29426

Avoid superfluous g_strdup()

svn path=/trunk/; revision=29425

Enable 'Mark Packet' in the Edit menu

svn path=/trunk/; revision=29424

Add support for mark/unmark all frames in our new packet list

svn path=/trunk/; revision=29423

Find next/previous mark is also supported so don't exclude it

svn path=/trunk/; revision=29422

Don't exclude functionality we already support

svn path=/trunk/; revision=29421

Allow s1ap not to be dissected (to avoid seeing errors, possibly due to proprietary variants).

svn path=/trunk/; revision=29420

Make 'Apply as filter'/'Prepare filter' work for new packet list.

svn path=/trunk/; revision=29419

Remove the hacky row_from_iter() function and use new_packet_list_get_record() instead. Alas, new_packet_list_get_record() is also somewhat hacky in its nature, but hey at least we're using one less hacky thing.

svn path=/trunk/; revision=29418

Make mark/unmark frame work with the new filter model

svn path=/trunk/; revision=29417

Remove unused functions/prototypes

svn path=/trunk/; revision=29416

New packet list: right clicking on a row now selects the given row.

svn path=/trunk/; revision=29415

Make first/last frame work again with the new filter model.

svn path=/trunk/; revision=29414

Make next/previous frame work again with the new filter model.

svn path=/trunk/; revision=29413

Typos.
Fixes:
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3878

svn path=/trunk/; revision=29412

Don't attach our original packet list model in new_packet_list_thaw(). It's pointless because we override it in filter_function().

svn path=/trunk/; revision=29411

From Artem Tamazov via. 3899:

Remove superfluous /MP

svn path=/trunk/; revision=29410

Make display filtering work for NEW_PACKET_LIST. This didn't work previously because we would simply append the new filtered packets onto the existing ones.

svn path=/trunk/; revision=29409

Always add packet to the new packet list regardless of whether the packet passed the display filter test or not. We do this because we do the filtering inside the new packet list.

svn path=/trunk/; revision=29408

Start to implement display filtering.

svn path=/trunk/; revision=29407

Get the pointer to the packet list record and use the data directly.

svn path=/trunk/; revision=29406

Add needed header due to stack allocated 'edt' structure

svn path=/trunk/; revision=29405

Introduce epan_dissect_init()/epan_dissect_cleanup(). These are used to initialise/cleanup stack allocated 'edt' structures. This should speed up dissection since we avoid some malloc traffic.

svn path=/trunk/; revision=29404

Make sure that we find the "data" dissector during handoff else we might crash if the received payload was less than 2 bytes.

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

svn path=/trunk/; revision=29403

fix build failure without GnuTLS

svn path=/trunk/; revision=29402

use G_GSIZE_MODIFIER instead of %zd which is not supported by MSVC

svn path=/trunk/; revision=29401

Fix SSL decryption failure if client and server have the same TCP port

svn path=/trunk/; revision=29400

From Gerasimos Dimitriadis:
DTAP decoding of Mobile Station Classmark 3 (R7 and R8)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3891

svn path=/trunk/; revision=29399

From Gerasimos Dimitriadis:
DTAP decoding of Mobile Station Classmark 3 (R6).
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3886

svn path=/trunk/; revision=29398

White space changes

svn path=/trunk/; revision=29397

DISSECTOR_ASSERT on FT_PCRE in construct_match_selected_string().

svn path=/trunk/; revision=29396

Temporarily disable dissector bug checking. Fixing the ones we've found
so far will take a while and in the meantime looking for dissector
assertions is keeping us from finding more serious bugs.

svn path=/trunk/; revision=29395

From William Preston via. 3549:

A typo in 2 of the field names causes the apn not to be displayed.

svn path=/trunk/; revision=29394

Fix COL_INFO and COL_PROTOCOL for Teredo Direct IPv6 Connectivity Test

svn path=/trunk/; revision=29393

fix memoryleak causing crash with long SSL traces

svn path=/trunk/; revision=29392

Restore the formatting of the saved graph.

svn path=/trunk/; revision=29391

Complete the fix in r29389.

svn path=/trunk/; revision=29390

Instead of calling DISSECTOR_ASSERT_NOT_REACHED, just say the subchannel
is unknown. Fixes bug 3864.

svn path=/trunk/; revision=29389

Add some length checks.

svn path=/trunk/; revision=29388

From Ivan Sy (with minor modifications):
Show Teredo Connectivity tests.
Fix offset of ICMPv6 sequence number.

svn path=/trunk/; revision=29387

From swapnil:
Small Bug in TFT (Flow label) and ULI.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3876

svn path=/trunk/; revision=29386

Use the right size for FT_UINT16.

svn path=/trunk/; revision=29385

Add some length checks. Fixes a few recent fuzz bugs.

svn path=/trunk/; revision=29384

Add a bunch of length checks. Fixes bug 3826.

svn path=/trunk/; revision=29383

(Trivial change) move ensure_contiguous() to reflect call hierarchy

svn path=/trunk/; revision=29382

Turn a comment in fast_ensure_contiguous() into a DISSECTOR_ASSERT() to make the implied requirement explicit.

svn path=/trunk/; revision=29381

TRY_TO_FAKE_THIS_ITEM now fakes FT_PROTOCOL per default. If there are any users (e.g. proto_hier_stats.c, others?) that relies on FT_PROTOCOL nodes being created they'll now need to call the newly introduced epan_dissect_fake_protocols() to disable this optimization.

Also make use of TRY_TO_FAKE_THIS_ITEM in proto_tree_add_text_node(), proto_tree_add_none_format() and proto_tree_add_protocol_format().

svn path=/trunk/; revision=29380

Be a little more explicit in our description of tvb_get_ptr.

svn path=/trunk/; revision=29379

Don't coerce a (void *) to a (guint32 *).
The (void *) pointer in this case may point to
"unaligned" data.

svn path=/trunk/; revision=29378

From Gerasimos Dimitriadis:
DTAP decoding of Mobile Station Classmark 3 (R4 and R5).
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3866

svn path=/trunk/; revision=29377