From Peter Wu via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6596 :

Allow reading SSL keylog files with carriage returns.

svn path=/trunk/; revision=52197

Beginnings of win cmake support (not yet working), created by Graham Bloice,
some smaller changes by me.
- README.cmake
  Document how to (one day) run on Windows
- CMakeLists.txt
  Use MSVC compiler flags for MSVC instead of gcc flags
- FindWSWinLibs.cmake
  New: Creates HINTS for finding includes and libraries
  inside the Wireshark support library installation.
- FindXXX.cmake
  Make use of HINTS generated by FindWSWinLibs.cmake

This has not really been tested on Windows as my installation seems to have
automagically downloaded some fixes and is in an inconsistent state since.
Will probably need to reinstall.

svn path=/trunk/; revision=52194

Ensure to release an existing DRX configuration when receiving a RRC Connection Setup message

svn path=/trunk/; revision=52193

Upgrade LPPa dissector to v11.3.0

svn path=/trunk/; revision=52192

Add dissection of new IEs in UL_UNITDATA

svn path=/trunk/; revision=52191

Another step towards tracking UE DRX state

svn path=/trunk/; revision=52190

DTP Dissector Enhancements

1) Corrections to the naming and terminology of DTP, its TLVs, types and values

2) Improvements to the dissection of Trunk Status and Trunk Type TLVs whose values and meaning have not been properly decoded so far

3) Improvements to the dissection of the Domain TLV (now using proto_tree_add_item() to display its value; this also allows for filtering operations)

4) Minor cleanups to the code (mainly renaming the macro names to make them more consistent)

From Peter Paluch, Bug 9156 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9156)

svn path=/trunk/; revision=52189

Tiny indentation fix.

svn path=/trunk/; revision=52188

The pointer into proto_tree_add_oid can be NULL if the length is 0. Exposed
because wmem_alloc(pool, 0) returns NULL (like pretty much every other
malloc-style function) where ep_ or se_ alloc did not.

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

svn path=/trunk/; revision=52187

Fix Function call argument is an uninitialized value warning found by Clang

svn path=/trunk/; revision=52186

Fix some Dead Store (Dead assignement/Dead increment) Warning found by Clang

svn path=/trunk/; revision=52185

Fix some Dead Store (Dead assignement/Dead increment) Warning found by Clang

svn path=/trunk/; revision=52184

Fix some Dead Store (Dead assignement/Dead increment) Warning found by Clang

svn path=/trunk/; revision=52183

From Peter Wu: on systems other than Windows, skip the GUI tests if the
DISPLAY environment variable isn't set.

From me: note that this won't be the right thing to do on OS X if we
switch to a toolkit that doesn't use X11, and that this may or may not
be the right thing to do with toolkits using Wayland or Mir directly.

svn path=/trunk/; revision=52182

Fix some typos, indentation. Don't try to write the NRB if the pointer to our
two lists is itself NULL.

svn path=/trunk/; revision=52181

More emem -> wmem conversion:
- tvb_get_g_stringz()/tvb_get_ephemeral_stringz()/tvb_get_seasonal_stringz() -> tvb_get_stringz()
- tvb_get_g_stringz_enc()/tvb_get_ephemeral_stringz_enc() -> tvb_get_stringz_enc()
- tvb_get_ephemeral_unicode_stringz() -> tvb_get_unicode_stringz()
- tvb_bcd_dig_to_ep_str() -> tvb_bcd_dig_to_wmem_packet_str()
- update docs accordingly

svn path=/trunk/; revision=52180

Fix -Wshadow, indentation

svn path=/trunk/; revision=52179

Remove unused parameter.

svn path=/trunk/; revision=52178

Turn off debuging.

svn path=/trunk/; revision=52177

Get rid of struct addrinfo, use the hastables for name resolution instead.

svn path=/trunk/; revision=52176

Update SCTP PPIDs and use newly assigned value.

svn path=/trunk/; revision=52175

emem -> wmem conversion:
- ep_tvb_get_bits() -> wmem_packet_tvb_get_bits()
- tvb_g_memdup()/ep_tvb_memdup() -> tvb_memdup()
- tvb_fake_unicode()/tvb_get_ephemeral_faked_unicode() -> tvb_get_faked_unicode()
- tvb_get_g_string()/tvb_get_ephemeral_string()/tvb_get_seasonal_string() -> tvb_get_string()
- tvb_get_g_unicode_string()/tvb_get_ephemeral_unicode_string() -> tvb_get_unicode_string()
- tvb_get_ephemeral_string_enc() -> tvb_get_string_enc()
- update docs accordingly

svn path=/trunk/; revision=52172

[Automatic manuf, services and enterprise-numbers update for 2013-09-22]

svn path=/trunk/; revision=52169

Don't declare proto_mac_mgmt_msg_ucd_decoder twice.

svn path=/trunk/; revision=52168

Make dissectors register for their MAC Management Message Type and use dissector_try_uint to do any dissection.  Many "outside" (third-party) message types appear to exist.

Have the makefiles do all the work setting up proto_register_<dissector> and proto_reg_handoff_<dissector>, rather than sprinkled throughout the dissectors with extern declarations.

More is coming to make more functions static (because of current makefile fixes) and cleanup MAC Management dissectors to remove unnecessary/duplicative code, but I wanted to get the build issues squared away.

svn path=/trunk/; revision=52167

Remove U3 references.

svn path=/trunk/; revision=52166

Update to tvb_g_memdup in tvb test suite.

svn path=/trunk/; revision=52165

Add _g_ to the names of functions that allocate glib memory. This is a bit more
explicit, and frees up the "generic" names (like tvb_memdup) for new signatures
that take the appropriate wmem pool.

Majority of the conversion done with sed.

svn path=/trunk/; revision=52164

Lower the default RTMPT max reassembly size. A bigger bandaid for
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6898

svn path=/trunk/; revision=52163

Make it more obvious that the function is using wmem memory

svn path=/trunk/; revision=52162

As pointed out by Kurt in ask.wireshark.org/questions/24886/error-failed-build-dependencies-while-making-rpm :

If we're not building with gtk3 we now need to add --with-gtk2 .

svn path=/trunk/; revision=52161

From Mathias Guettler via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9159 :
ESIS dissector exception due to incorrect length parameter to 'proto_tree_add_uint'

svn path=/trunk/; revision=52160

Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9164 :
Add dissector_try_uint_new() to the list of exported symbols

svn path=/trunk/; revision=52159

Enhance DOCSIS dissector with new fields.  Bug 9149 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9149)

From Jerry Negele

svn path=/trunk/; revision=52158

Bugfix a few items:

1. Correct Interface Flag enumeration
2. Dissect ARP data without making it look like its an ARP packet by disabling column writing.

svn path=/trunk/; revision=52157

Don't use g_strv_length on a single string, just use strlen. Also do the
null-check *before* calling strlen (if it's even necessary?)

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

svn path=/trunk/; revision=52156

Indicate Sequence of 0 length.

svn path=/trunk/; revision=52155

DCH-ID can be 255

svn path=/trunk/; revision=52154

Try to make heuristic slightly faster.

svn path=/trunk/; revision=52153

From Chun-Yeow via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9163 Ability to Decode the Number of Mesh Peerings

Decode the mesh formation information fields related to the number of mesh peerings

Signed-off-by: Chun-Yeow Yeoh <yeohchunyeow@cozybit.com>
svn path=/trunk/; revision=52152

Add Peter Wu to AUTHORS

(Thanks for the work on SSL/TLS/DTLS Ciphers)

svn path=/trunk/; revision=52151

From Peter Wu via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9144 [PATCH] Update TLS Ciphers

Really add support for AEAD ciphers (GCM)

GCM uses counter mode with authentication tags (the latter is currently
not supported). As for the key material, there is no MAC (because the
auth tag is supposed to verify the authenticity).

Finally, correct the GCM cipher suite definitions: IV block size of
4 bytes and GCM instead of CBC mode.

svn path=/trunk/; revision=52150

From Peter Wu via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9144 [PATCH] Update TLS Ciphers

Use IV from record for CBC mode, add padding/IV length check

Add summary of RFCs to make it more obvious why certain parts (IV, MAC,
padding) are used. Merge DTLS and TLS blocks for extracting IV. This
saves an unnecessary memmove() because the input pointer is, well, just
a local variable and can therefore be incremented.

Validate padding and IV lengths before using it. A crash could occur
if the explicit IV is missing (this would make memmove write before its
buffer). The missing padding check had as implication that a misleading
error is returning with a negative length (not exploitable).

Use IV from record for CBC mode, previously it decrypted the first block
incorrectly and then threw this "decrypted" IV away. Now it extracts the
IV and uses this for decrypting the first fragment block. (remember that
CBC xor's the output of the block cipher with the previous ciphertext
(or IV for the first block)).

This is a preparation for GCM which does not have a MAC. The skip_mac
branch is necessary to make the compiler happy in this patch, 'mac'
could otherwise be uninitialised.

svn path=/trunk/; revision=52149

From Peter Wu via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9144 [PATCH] Update TLS Ciphers

Correct cipher suites list, add TLS_ annotations

Add official TLS_ names as comment and correct:
- 6: RC2 is a block cipher using MD5, not stream+SHA.
- 25,26,27: should be SHA instead of MD5
- 98: DES export is a block cipher using 56-bits[1], not stream.
- 99: DES export should be using 56-bits[1].
- 138: removed commented RC4 cipher because it is not a block cipher

Besides these comments and corrections, there are no further changes.

 [1]: http://tools.ietf.org/html/draft-ietf-tls-56-bit-ciphersuites-00

svn path=/trunk/; revision=52148

From Peter Wu via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9144 [PATCH] Update TLS Ciphers

Drop export_cipher and dig_len, cleaner digest access

Removed dig_len as this magic number is dependent on dig. The digests
variable is converted from a string to a structure holding the digest
name and length because of its close dependency.

Introduce another struct+function to get rid of the magic number 0x40
(DIG_MD5).

Removed export_cipher bit as this is dependent on eff_bits < bits.
Verified with:
grep ,KEX_ packet-ssl-utils.c | awk -F, '{bits=$6!=$7;ex=$9;
if ((bits && !ex) || (!bits && ex))print $6, $7, $8, "###", $0}'.

Removed space before SIG_RSA for cipher 51 for consistency with others.

svn path=/trunk/; revision=52147

Add *.lai file to wireshark

svn path=/trunk/; revision=52146

Zero the key in ntlmssp, because we don't otherwise fill all of it. I suspect
the tvb_memcpy on the next line should be taking more data, but I don't know
enough about the protocol to be sure. This is the least disruptive way to fix
the last valgrind error from
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8941

svn path=/trunk/; revision=52145

Enum was used after all. Restore it and namespace it with GLUSTER_ to avoid the
collision.

svn path=/trunk/; revision=52144

From Mark Vitale via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9141

Decode AFS RXAFSCB_GetCapabilities RPC requests with their replies.

svn path=/trunk/; revision=52143

Remove unused enum that started conflicting with /usr/include/dirent.h on my
system.

svn path=/trunk/; revision=52142

Revert "Prefer newer Lua versions"

The original version was OK, obviously.

svn path=/trunk/; revision=52141

Prefer newer Lua versions

svn path=/trunk/; revision=52140

Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9138 (Malformed NORM packet) having to do with Wireshark assuming that all remaining bytes in the packet are part of the "cc_node_list", rather than stopping dissection according to the hdr_len field in the NORM Common Message Header.

Ref: http://tools.ietf.org/html/rfc3940#section-4.2.3.4

#BACKPORT(1.10, 1.8)

svn path=/trunk/; revision=52139

Rename PCAP files to .pcap; set mime-type appropriately.  Should these files be in the test/captures directory?

svn path=/trunk/; revision=52138

Missed one file from dfilter-test.py
Updated license in ancient tool.

svn path=/trunk/; revision=52137

Update dfilter-test.py to use a much more modern test harness,
the "unittest" module that comes with Python. Specifically, this
takes advantage of a couple of features in the "unittest" in
Python 2.7. The tests are all the same as before, but much
better managed.

This is in preparation for some work on the display filter code.

svn path=/trunk/; revision=52136

Fix a couple of EXPORT_DEPENDS

svn path=/trunk/; revision=52135

Add the dependeny handling for windows builds as well

svn path=/trunk/; revision=52134

Missed 1 character replacement as part of r52132.

svn path=/trunk/; revision=52133

Remove all non-ASCII characters from all .asn files.  I didn't see the harm in replacing them all as opposed to only those not copied verbatim from other sources.

svn path=/trunk/; revision=52132

From Jason Wzhy via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9157 :
BSSGP: Fix dissection of Trace Type IE in SGSN-INVOKE-TRACE message

svn path=/trunk/; revision=52131

emem -> wmem

svn path=/trunk/; revision=52130

Fix remaining dependencides for -exp.cnf files.

svn path=/trunk/; revision=52129

Fix off by 1 error when dissecting Data Radio Bearer identity number

svn path=/trunk/; revision=52128

More dependeny updates for asn1 -exp.cnf files.
Only three remaining: p7, p22, x509ce - thanks to Chris Maynard for the list.

svn path=/trunk/; revision=52127

Remove some unneeded #includes...

svn path=/trunk/; revision=52126

Regenerate all files where the .cnf files had whitespace changes.

svn path=/trunk/; revision=52125

Reference 'BACnetVendorIdentifiers' value_string using
an extended value string.

svn path=/trunk/; revision=52124

One of the wmem patches only updated the generated packet-snmp.c
but not the source. Fix that.

svn path=/trunk/; revision=52123

Switch the inline functions that use tvbuffs and emem back to macros.
This should hopefully fix linking with Solaris Studio, which doesn't
remove unused inline functions by default. Fix a copy+paste error.

svn path=/trunk/; revision=52122

Fix proto_registrar_dump_protocols() bug introduced by the
implementation of BASE_VAL64_STRING value_strings.

Fixes 'tshark -G values' crash.

svn path=/trunk/; revision=52121

When building packet-dap.c on an empty build tree, packet-dap.c would
be incorrect on the first pass and only be correct when remade.
The reason was that disp-exp.cnf was created incorrectly on first pass.
disp-exp.cnf required dop-exp.cnf to build correctly, so add a mechanism
to add dependencies for export files.

Note: Only the cmake change has been tested, the autofoo stuff requires
  testing.

svn path=/trunk/; revision=52120

Update 'BACnetVendorIdentifiers' value strings from
BACnet vendor ID list dated September 16, 2013.
http://www.bacnet.org/VendorID/BACnet%20Vendor%20IDs.htm

svn path=/trunk/; revision=52119

Fix dependency problems (continuation of r52110 and r52112).

svn path=/trunk/; revision=52118

Fix a difference to Makefile.common

svn path=/trunk/; revision=52117

Add svn id tag

svn path=/trunk/; revision=52116

Don't call gethostbyaddr() we have already read the local hosts file, settle for that.

svn path=/trunk/; revision=52115

From Mark Vitale via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9142

Fix decoding of AFS Volume Name.

svn path=/trunk/; revision=52114

From Mark Vitale via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9140

Add reassembly support for AFS.

From me: minor tweaks to conform to other reassemblable protocols; indentation
fixes; modelines

svn path=/trunk/; revision=52113

Copy over rev 52110 to cmake

svn path=/trunk/; revision=52112

Forgot to check in regenerated dissector for dap.

svn path=/trunk/; revision=52111

Fix dependency problem reported by Joerg Mayer on wireshark-dev.
Ref: http://www.wireshark.org/lists/wireshark-dev/201309/msg00151.html

svn path=/trunk/; revision=52110

Whitespace changes

svn path=/trunk/; revision=52109

address_equal → addresses_equal. Switch from address macros to functions
in the Qt code.

svn path=/trunk/; revision=52108

Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9155 :
Ensure that codec_str string is always initialized

svn path=/trunk/; revision=52107

Convert each of the address.h macros to inline functions.

Document each function. Note that we now call tvb_get_ptr() before
modifying the address in tvb_set_address() and tvb_set_address_hf(). The
caller doesn't have to worry about doing that any more. Add
add_address_to_hash64().

svn path=/trunk/; revision=52106

Don't print a warning about missing svn id when the argument is not
a file. Warn about the "wrong" filetype and continue

svn path=/trunk/; revision=52105

Read the local services file.

svn path=/trunk/; revision=52104

Don't call getservbyport() on unresolved ports - it's expensive.
We should read the services file in initialize_services instead.

svn path=/trunk/; revision=52103

Add stream information to the hint label. Update our hint more often.
Add a "to do" list.

svn path=/trunk/; revision=52102

Trivial: Fix typo.

svn path=/trunk/; revision=52101

Remove a bunch more set-but-no-longer-used variables.

svn path=/trunk/; revision=52100

Remove set-but-no-longer-used variable.

svn path=/trunk/; revision=52099

Convert proto_tree_add_uint_format to proto_tree_add_uint_format_value if hf_ field name is the first part of the formatted string.  This was all manual inspection and most cases were either:
1. Case sensitivity differences between hf_ field name and formatted string.
2. Unnecessary whitespace between hf_ field name and colon in formatted string

There are cases where the hf_ field name doesn't quite match the proto_tree_add_uint_format, but it's close enough that one of them should be "right", I'm just not sure which is, I just know the string in proto_tree_add_uint_format is the one displayed.

svn path=/trunk/; revision=52098

Fix some Dead Store (Dead assignement/Dead increment) Warning found by Clang

svn path=/trunk/; revision=52097

From Peter Wu via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9144

Update TLS Ciphers

The attached trivial patch corrects the digest (SHA) length for TLS_DHE_DSS_EXPORT1024_WITH_DES_CBC_SHA

svn path=/trunk/; revision=52096

Make sure we don't tap our packets twice. Give the graph has focus after
we redraw it.

svn path=/trunk/; revision=52095

Fix graph type selection.

svn path=/trunk/; revision=52094

Another attempt at axis scaling. Tweak the appearance a little.

svn path=/trunk/; revision=52093

When caclulating throughput, only subtract what we've added. Try to fix
axis scaling.

svn path=/trunk/; revision=52092