Fun and games with const.

svn path=/trunk/; revision=54817

Explain casting away const.

svn path=/trunk/; revision=54816

Update to version V12.3.0 (2013-12).

svn path=/trunk/; revision=54815

Don't cast away constness.

svn path=/trunk/; revision=54814

Don't cast away constness.

svn path=/trunk/; revision=54813

Reject pcap files that claim on-the-wire packet sizes > 64MB. This fixes many
heuristic cases broken in r49999 when we permitted packets > 64KB, since that
relaxed so severely the definition of a valid packet header.

64MB is an arbitrary and perhaps suboptimal number, but it seems to do the right
thing in all the examples I have handy.

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

svn path=/trunk/; revision=54812

Eliminate fvalue_set() in favor of routines that take arguments
appropriate for particular FT_ types.  This lets us do some more type
checking and lets us use const pointers when appropriate.

Constify a bunch of stuff, and don't cast away constness.

svn path=/trunk/; revision=54811

Make it more clear that we're faking a git description for now.

svn path=/trunk/; revision=54810

Copy over recent changes from trunk-1.8 along with some local
modifications.

  ------------------------------------------------------------------------
  r54807 | gerald | 2014-01-14 16:35:34 -0800 (Tue, 14 Jan 2014) | 2 lines

  Start using the output of 'git describe'.
  ------------------------------------------------------------------------
  r54805 | gerald | 2014-01-14 15:38:16 -0800 (Tue, 14 Jan 2014) | 2 lines

  Start adding support for plain git.
  ------------------------------------------------------------------------

svn path=/trunk/; revision=54809

Fix typo (Direcly => Directly)

svn path=/trunk/; revision=54804

Add filetap/abi* to .gitignore

svn path=/trunk/; revision=54803

Don't cast away constness.

svn path=/trunk/; revision=54802

We have to cast away constness here, because C doesn't have
parameterized types.

svn path=/trunk/; revision=54801

Don't cast away constness.

svn path=/trunk/; revision=54800

Regenerate ANSI TCAP dissector from the changes in r54499.

svn path=/trunk/; revision=54799

Don't cast away constness.

svn path=/trunk/; revision=54798

Don't cast away constness.

svn path=/trunk/; revision=54797

Don't cast away constness.

svn path=/trunk/; revision=54796

Consistently use "guint8" for "8-bit byte".

Constify.

For routines that manipulate sequences of 8-bit bytes, have them take
guint8 pointers rather than void pointers.

Don't cast away constness.

svn path=/trunk/; revision=54795

Don't cast away constness.

svn path=/trunk/; revision=54794

Don't cast away constness.

svn path=/trunk/; revision=54793

Don't make claims of constness that can't be guaranteed in terms the C
compiler can verify.

svn path=/trunk/; revision=54792

Fix -Wsometimes-uninitialized found by Clang

packet-pdcp-lte.c:1397:9: error: variable 'decrypted_data' is used uninitialized whenever 'if' condition is false
packet-pdcp-lte.c:1469:50: note: uninitialized use occurs here
    decrypted_tvb = tvb_new_child_real_data(tvb, decrypted_data, payload_length, payload_length);
packet-pdcp-lte.c:1353:27: note: initialize the variable 'decrypted_data' to silence this warning

packet-pdcp-lte.c:1397:9: error: variable 'payload_length' is used uninitialized whenever 'if' condition is false
packet-pdcp-lte.c:1469:66: note: uninitialized use occurs here
    decrypted_tvb = tvb_new_child_real_data(tvb, decrypted_data, payload_length, payload_length);
packet-pdcp-lte.c:1354:24: note: initialize the variable 'payload_length' to silence this warning

svn path=/trunk/; revision=54791

Fix unused const variable found by Clang 3.4

svn path=/trunk/; revision=54790

Upgrade LTE RRC dissector to v11.6.0

svn path=/trunk/; revision=54789

SNOW3G is not available on default Wireshark

svn path=/trunk/; revision=54788

Fix dissection of 3GPP2 SMS encoded in UCS2

svn path=/trunk/; revision=54787

Make sure we allocate bytes for snow3g at least rounded up to next 4 byte boundary

svn path=/trunk/; revision=54786

Update a comment

svn path=/trunk/; revision=54785

Add snow3g integrity (not quite working for me), but rework how integrity and ciphering functions and conditional compilation is done

svn path=/trunk/; revision=54784

UCS2 encoded SMS is directly read from data -> remove PROTO_ITEM_SET_GENERATED()

svn path=/trunk/; revision=54783

Provide hook for calling EEA1 implementation (Snow3G).  Implementation not supplied due to licensing conditions.  TODO: EIA1.

svn path=/trunk/; revision=54782

Whoever in GLibland was responsible for defining the signature of
g_snprintf() owes the world a huge apology.  They *could* have just used
size_t, or they *could* at least have, if they insisted on having GLib
types to mirror regular types, made gsize an alias for size_t and,
whatever they did with gsize, they *could* have used it as the argument
for the size of the buffer for g_snprintf(), but no, they had to use
gulong.

svn path=/trunk/; revision=54781

Don't cast away constness.

svn path=/trunk/; revision=54780

Don't cast away constness.

svn path=/trunk/; revision=54779

But *do* cast const void * to the appropriate type.

svn path=/trunk/; revision=54778

Don't cast away constness.

svn path=/trunk/; revision=54777

Don't assign away constness.

svn path=/trunk/; revision=54776

Don't cast away constness.

svn path=/trunk/; revision=54775

Don't cast away constness.

svn path=/trunk/; revision=54774

Don't cast away constness.

svn path=/trunk/; revision=54773

gsize, not gusize.  Whatever.

svn path=/trunk/; revision=54772

Glib, gotta love it.  No, gusize isn't the same as size_t, it's long,
which is Too Damn Short on Windows.  (Microsoft, gotta love 'em.  I
guess even more Windows programmers than UN*X programmers thought long
was 32 bits, now and forever, world without end, amen.)

svn path=/trunk/; revision=54771

Don't cast away constness.

svn path=/trunk/; revision=54770

Don't cast away constness.

svn path=/trunk/; revision=54769

Don't cast away constness.

svn path=/trunk/; revision=54768

Have ip6_to_str() and tvb_ip6_to_str() return const pointers, as the
ip_to_str equivalents do.

Don't cast away constness, and don't make it necessary to do so.

svn path=/trunk/; revision=54767

Don't cast away constness.

svn path=/trunk/; revision=54766

Don't cast away constness.

svn path=/trunk/; revision=54765

Don't cast away constness.  In one place, this requires that we use one
pointer while we're filling in a string and then assigning it to a const
pointer when we're done.

Don't use guint16 as a size value; it does no good (it doesn't magically
protect you againt too-long strings), it runs the potential risk of
overflow, it doesn't save time for arithmetic (32-bit or 64-bit
arithmetic may well be faster, and will at least be as fast), and it
doesn't save space (if it fits in a register, you save nothing, and if
it's on the stack, there will probably be padding.

svn path=/trunk/; revision=54764

Cast away constness.

svn path=/trunk/; revision=54763

Constification.

svn path=/trunk/; revision=54762

Don't cast away constness.

svn path=/trunk/; revision=54761

Don't cast away constness.

svn path=/trunk/; revision=54760

And more constification.

svn path=/trunk/; revision=54759

More constification.

svn path=/trunk/; revision=54758

Don't make promises of constness that we can't honor.

svn path=/trunk/; revision=54757

Don't cast away constness when you don't have to.

svn path=/trunk/; revision=54756

Squelch some casting-away-constness warnings.

svn path=/trunk/; revision=54755

Clean up references to an address and squelch some warnings.

svn path=/trunk/; revision=54754

Don't cast away constness.

svn path=/trunk/; revision=54753

More prototype cleanup.

svn path=/trunk/; revision=54752

Make a routine not used outside the SNMP dissector static.

svn path=/trunk/; revision=54751

In a source file that defines external functions, the header file that
declares the functions must be included, in order to make sure the
declarations match the function signature.  Make it so.

Said header declares pipe_input_cb_t, so we don't have to do it
ourselves.

svn path=/trunk/; revision=54750

In a source file that defines external functions, the header file that
declares the functions must be included, in order to make sure the
declarations match the function signature.  Make it so.

svn path=/trunk/; revision=54749

Detangle the #ifdefs a bit.

svn path=/trunk/; revision=54748

In a source file that defines external functions, the header file that
declares the functions must be included, in order to make sure the
declarations match the function signature.  Make it so.

Don't cast away constness when passing something to a routine that takes
a const pointer argument.

svn path=/trunk/; revision=54747

Make a routine not used outside this file static.

svn path=/trunk/; revision=54746

In a source file that defines external functions, the header file that
declares the functions must be included, in order to make sure the
declarations match the function signature.  Make it so.

svn path=/trunk/; revision=54745

Include ui/progress_dlg.h to get the declaration of
delayed_create_progress_dlg().

svn path=/trunk/; revision=54744

#if 0 out an unused routine.

svn path=/trunk/; revision=54743

In a source file that defines external functions, the header file that
declares the functions must be included, in order to make sure the
declarations match the function signature.  Make it so.

svn path=/trunk/; revision=54742

Make a routine used only in this file static.

svn path=/trunk/; revision=54741

In a source file that defines external functions, the header file that
declares the functions must be included, in order to make sure the
declarations match the function signature.  Make it so - which means
creating the header file in the first place.

Also, declare proto_register_wimax_compact_dlmap_ie() early in
wimax_compact_dlmap_ie_decoder.c, similar to what we do in
wimax_copact_ulmap_ie_decoder.c.

svn path=/trunk/; revision=54739

tvparse_until() is used, so don't make it static.

svn path=/trunk/; revision=54738

Upgrade LPP dissector to v11.5.0

svn path=/trunk/; revision=54737

Update SVN properties

svn path=/trunk/; revision=54736

dissect_CBA_Connection_Data() isn't used outside packet-dcom-cba-acco.c,
so make it static - and get rid of the #if 0'ed out declaration in
packet-dcom-cba-acco.h.

svn path=/trunk/; revision=54735

No reason for decode_bgp_rd()'s return value to be non-const, so make it
const.

svn path=/trunk/; revision=54734

Move dissect_pn_uint16_ret_item() to packet-pn.c, to be with its sibling
dissect_pn_uint16().  Declare it in packet-pn.h.

svn path=/trunk/; revision=54733

Fix Dead Store (Dead assignement/Dead increment) warning found by Clang

svn path=/trunk/; revision=54732

Fix Dead Store (Dead assignement/Dead increment) warning found by Clang

svn path=/trunk/; revision=54731

dispatchService() is imported from other code; declare it in
opcua_transport_layer.h, not opcua_transport_layer.c.

svn path=/trunk/; revision=54730

From Kevin Loewen via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9631

Add dissector for Pathport Protocol

From me :
* Fix indent (use 4 spaces)
* Remove unused variable and fix Clang warning

svn path=/trunk/; revision=54729

From Miroslav Miklus via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9504 Enhance BGP dissector : bgp-ls dissector (draft-ietf-idr-ls-distribution-04)

BGPTYPE_LINK_STATE_ATTR is temporarily set to 99, would need change when IANA allocate a Path Attribute value for BGP-LS

From me :
* Fix indent
* fix arg encoding (via encoding-args tools)

svn path=/trunk/; revision=54728

#if 0 out unused routines claimed to be "untested" and #if 0'ed out in
epan/tvparse.h, and static routines used by that code.

svn path=/trunk/; revision=54727

Get rid of unused routine (the initialization is done in
geoip_db_post_update_cb() - as the comment says, "and also once on
startup").

svn path=/trunk/; revision=54726

In a source file that defines external functions, the header file that
declares the functions must be included, in order to make sure the
declarations match the function signature.  Make it so - which means
creating the header file in the first place.

That means luaopen_bit() doesn't need to, and shouldn't be, declared in
wslua.h.

Have make-reg.pl generate a #include of lua_bitop.h.  Also fix the "this
is autogenerated" warning.

svn path=/trunk/; revision=54725

f_k() is used outside packet-gsm_a_rr.c, so it can't be static.

In a source file that defines external functions, the header file that
declares the functions must be included, in order to make sure the
declarations match the function signature.  Make it so - which means
creating the header file in the first place.

That means f_k() doesn't need to, and shouldn't be, declared in
packet-gsm_rlcmac.h.  Include packet-gsm_a_rr.h in packet-gsm_rlcmac.c.

svn path=/trunk/; revision=54724

lsarpc_dissect_sec_desc_buf() and lsarpc_dissect_struct_dom_sid2() are
used in the Netlogon dissector, so don't make them static.

Now that we've added HEADER START/HEADER END to Pidl, use it to declare
those routines in packet-dcerpc-lsa.h.  Don't declare them in
packet-dcerpc-netlogon.c, as they're now declared in the header.

svn path=/trunk/; revision=54723

Fix crash when exporting to CSV file

svn path=/trunk/; revision=54722

Document {CODE,HEADER} {START,END}.

Clean up indentation.

svn path=/trunk/; revision=54721

Add HEADER START and HEADER END, similar to CODE START and CODE END, to
allow stuff to be put into the .h file; that way, if you put helper
functions into the .c file, for use by dissectors other than the one
being generated, you can put declarations for them into the .h file.

Do some checks for mis-nesting of CODE and HEADER blocks.

Clean up the "this is generated code" header.

svn path=/trunk/; revision=54720

Make a routine not used outside this file static.

svn path=/trunk/; revision=54719

Make a routine not used outside this file static.

svn path=/trunk/; revision=54718

In a source file that defines external functions, the header file that
declares the functions must be included, in order to make sure the
declarations match the function signature.  Make it so - which means
creating the header file in the first place.

svn path=/trunk/; revision=54717

In a source file that defines external functions, the header file that
declares the functions must be included, in order to make sure the
declarations match the function signature.  Make it so.

They *didn't* match; fix that.

Do *not* use 16-bit tvbuff offsets; there is no advantage to that (and,
in fact, 32-bit arithmetic may be easier on some platforms), and it has
the disadvantage that you're more likely to have the offset overflow.

dissect_cbs_serial_number() returns the new offset, not the length of
the item it dissected; don't add its return value to the offset, set the
offset to its return value.

Make dissect_cbs_message_identifier() return the new offset as well, for
consistency, and don't add *its* return value to the offset.

svn path=/trunk/; revision=54714

In a file that defines external routines, the header file that declares
them should be included, to make sure the declaration and definition
match.  Make it so.

svn path=/trunk/; revision=54713

Make routines not used outside packet-dcerpc-lsa.c static.

svn path=/trunk/; revision=54712

Squelch some warnings.

svn path=/trunk/; revision=54711