aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGerald Combs <gerald@wireshark.org>2016-09-07 09:40:26 -0700
committerGerald Combs <gerald@wireshark.org>2016-09-07 16:53:44 +0000
commit5368c50be46d4a44986d12bdfc0a35b42c0f34fc (patch)
tree6ac3e53db94898e02db4c3cac86c6adbc2ea0c94
parent432500ed68982c989d1a84e7e2b1686b4b04e757 (diff)
downloadwireshark-5368c50be46d4a44986d12bdfc0a35b42c0f34fc.tar.gz
wireshark-5368c50be46d4a44986d12bdfc0a35b42c0f34fc.tar.bz2
wireshark-5368c50be46d4a44986d12bdfc0a35b42c0f34fc.zip
Change-Id: If308a7181cbec1e256d5aacf2092c3066f7a80dd Reviewed-on: https://code.wireshark.org/review/17559 Reviewed-by: Gerald Combs <gerald@wireshark.org>
-rw-r--r--ChangeLog993
-rw-r--r--docbook/asciidoc.conf2
-rw-r--r--docbook/release-notes.asciidoc7
-rw-r--r--version.conf12
4 files changed, 321 insertions, 693 deletions
diff --git a/ChangeLog b/ChangeLog
index 4a190bf03d..8d3766e26c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,818 +1,443 @@
-commit 987fb23
-Author: Peter Wu <peter@lekensteyn.nl>
-Date: Wed Aug 31 16:51:16 2016 +0200
+commit 432500e
+Author: Makoto Shimamura <makoto.shimamura@toshiba.co.jp>
+Date: Tue Sep 6 22:36:33 2016 +0900
- autotools: really disable warnings by default
+ OCFS2: dissector incorrectly takes pdu length as little endian
- The v2.2.0rc1 release tarball accidentally enables warnings as errors,
- this led to build failures due to warnings that occur on 32-bit builds.
+ This patch fixes incorrect endian conversion in pdu length.
+ Actually pdu length is big endian.
- Change-Id: Ib12f5762e9b30a681b193014ed78dc73bc794b0f
- Reviewed-on: https://code.wireshark.org/review/17420
- Reviewed-by: Gerald Combs <gerald@wireshark.org>
+ Ping-Bug: 12122
+ Change-Id: I9f8827293e684a5b4c957138f5879efdd140c500
+ Reviewed-on: https://code.wireshark.org/review/17533
+ Reviewed-by: Anders Broman <a.broman58@gmail.com>
+ (cherry picked from commit c95c536f47fbf0daac4f6f6258ed8902766a9a3c)
+ Reviewed-on: https://code.wireshark.org/review/17548
+ Reviewed-by: Makoto Shimamura <makoto.shimamura@toshiba.co.jp>
-commit 50c51c8
-Author: Peter Wu <peter@lekensteyn.nl>
-Date: Wed Aug 31 00:04:38 2016 +0200
+commit 597f5a4
+Author: Gerald Combs <gerald@wireshark.org>
+Date: Tue Sep 6 15:59:24 2016 -0700
- ipv6: fix build with old 32-bit glib headers
-
- glib versions before 2.31.2 expand GUINT_TO_POINTER(x) as (gpointer)x
- instead of (gpointer)(guint)x, resulting in these errors:
+ Autotools: Distribute packaging/wix.
- packet-ipv6.c:2129: error: cast to pointer from integer of different size
- packet-ipv6.c:2185: error: cast to pointer from integer of different size
+ …otherwise you can't compile from the tarball on Windows.
- Change-Id: I1b25060ed329ce05e4f1ab75e32a56feda8e3435
- Reviewed-on: https://code.wireshark.org/review/17401
- Reviewed-by: Michael Mann <mmann78@netscape.net>
- Petri-Dish: Michael Mann <mmann78@netscape.net>
+ Change-Id: I99c73b09d9468f0da753c05451fa54348680963f
+ Reviewed-on: https://code.wireshark.org/review/17544
+ Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: João Valverde <j@v6e.pt>
+ Reviewed-by: Gerald Combs <gerald@wireshark.org>
-commit 8b7444e
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Tue Aug 30 17:59:59 2016 -0700
+commit e553366
+Author: Mirko Parthey <mirko.parthey@web.de>
+Date: Mon Sep 5 16:39:45 2016 +0200
- Use "ecatf" as the protocol name, to parallel everything else.
+ ISAKMP: Fix handling of cert requests without CA
- The fields have names beginning with "ecatf.", the dissector is called
- "ecatf", and it's only the frame layer of EtherCAT anyway, so just call
- it "ecatf".
+ Check IKEv1 Certificate Request Payloads for an empty
+ Certificate Authority field, which is allowed by RFC 2408.
+ Suppress dissection of this field if it is indeed empty.
- Change-Id: I2f127363fd115c307f0525f612fe184a30d46c55
- Reviewed-on: https://code.wireshark.org/review/17406
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit d78197955050f0ddf4a91379018f41b58923c56b)
- Reviewed-on: https://code.wireshark.org/review/17407
+ Change-Id: Ifb997e460a4c12003215fde86c374cfc769c5d72
+ Reviewed-on: https://code.wireshark.org/review/17501
+ Reviewed-by: Michael Mann <mmann78@netscape.net>
+ Petri-Dish: Michael Mann <mmann78@netscape.net>
+ Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
+ Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
+ (cherry picked from commit 70f3737c3e4d9402cb2bb67cdd892e0e7e0ee991)
+ Reviewed-on: https://code.wireshark.org/review/17504
+
+commit ae6ae02
+Author: Anthony Coddington <anthony.coddington@endace.com>
+Date: Tue Aug 30 11:04:23 2016 +1200
+
+ ERF: Fix dissector abort on short meta tags and typos
+
+ Fix dissector abort on short tags.
+ Fix value typo in hash mode enum.
+ Differentiate unexpectedly short value, zero length (deliberate invalid)
+ and off-end-of-record tags through expertinfo.
+ Continue to use proto_tree_add_*() length mismatch warnings for unxepectedly
+ long tags for now.
+ Change WWN tags to FT_BYTES for now as they are 16 not 8 byte WWN. Not
+ currently implemented outside Wireshark anyway.
+
+ Ping-Bug: 12303
+ Change-Id: I79fe4332f0c1f2aed726c69acdbc958eb9e08816
+ Reviewed-on: https://code.wireshark.org/review/17382
+ Reviewed-by: Anthony Coddington <anthony.coddington@endace.com>
+ Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
+ Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
+ Reviewed-by: Anders Broman <a.broman58@gmail.com>
+ (cherry picked from commit e2b0aa21ea0f5ed136e802546c9861f2e2609fd0)
+ Reviewed-on: https://code.wireshark.org/review/17494
-commit 6b6cf99
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Tue Aug 30 15:51:54 2016 -0700
+commit 0aebbc3
+Author: Pascal Quantin <pascal.quantin@gmail.com>
+Date: Sun Sep 4 22:21:42 2016 +0200
- Don't do any Decode As stuff for dissector tables not used with Decode As.
-
- Have all dissector tables have a "supports Decode As" flag, which
- defaults to FALSE, and which is set to TRUE if a register_decode_as()
- refers to it.
-
- When adding a dissector to a dissector table with a given key, only add
- it for Decode As if the dissector table supports it.
+ S1AP: fix dissection of TransportLayerAddress IE when it contains both IPv4 and IPv6
- For non-FT_STRING dissector tables, always check for multiple entries
- for the same protocol with different dissectors, and report an error if
- we found them.
-
- This means there's no need for the creator of a dissector table to
- specify whether duplicates of that sort should be allowed - we always do
- the check when registering something for "Decode As" (in a non-FT_STRING
- dissector table), and just don't bother registering anything for "Decode
- As" if the dissector table doesn't support "Decode As", so there's no
- check done for those dissector tables.
-
- Change-Id: I4e98bfe0061f1014cd7ffdf4c51d9b6e8a2a5689
- Reviewed-on: https://code.wireshark.org/review/17405
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
+ Change-Id: I8c257d79b4e842da6853ea3c53ae479f33a8047a
+ Reviewed-on: https://code.wireshark.org/review/17491
+ Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
+ Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
+ Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
+ (cherry picked from commit 87519e4c06260f78ecd55defb86bb20c3f4d7697)
+ Conflicts:
+ epan/dissectors/asn1/s1ap/s1ap.cnf
+ epan/dissectors/packet-s1ap.c
+ Reviewed-on: https://code.wireshark.org/review/17492
-commit 0158d62
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Tue Aug 30 13:38:03 2016 -0700
+commit 556a68a
+Author: Gerald Combs <gerald@wireshark.org>
+Date: Sun Sep 4 08:22:38 2016 -0700
- Support Decode As for {SSL,TLS}-over-TCP.
+ [Automatic update for 2016-09-04]
- We register dissectors for "Decode As" for {SSL,TLS}-over-TCP, so we
- should actually set up the "Decode As" stuff for it.
-
- Change-Id: I2a738667efdec1007069df74885a4fe8fc3fcbab
- Reviewed-on: https://code.wireshark.org/review/17400
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit afa2605e4359509a8e188f3d7ae118246e9e0ac7)
- Reviewed-on: https://code.wireshark.org/review/17404
-
-commit 1ba7176
-Author: Juanjo Martin <juanjo@rti.com>
-Date: Tue Aug 30 01:57:32 2016 +0200
-
- RTPS: Allowing duplicates in the rtps.type_name table
+ Update manuf, services enterprise-numbers, translations, and other items.
- Change-Id: Ie7a35b0bb3275a0895fad646bf9a81406eadf37c
- Reviewed-on: https://code.wireshark.org/review/17393
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit 1dbd2216edfe34e3dbb1f8967d325e7fa00ecb51)
- Reviewed-on: https://code.wireshark.org/review/17403
+ Change-Id: I2ff2a8a9f66f3c1e1ec26aa46b2f585c1794e19c
+ Reviewed-on: https://code.wireshark.org/review/17485
+ Reviewed-by: Gerald Combs <gerald@wireshark.org>
-commit 7968175
+commit bedf118
Author: Peter Wu <peter@lekensteyn.nl>
-Date: Tue Aug 30 11:12:44 2016 +0200
+Date: Sun Sep 4 13:40:20 2016 +0200
- time_util: fix -Wshadow issue
+ ssl,http: rename http-over-ssl to http-over-tls
+
+ This name is displayed in the SSL prototcol tree (Application Data
+ Protocol: http-over-tls), rename to avoid possible user confusion.
- time_util.c was already fixed, but the header was missing the change,
- breaking the build on a very old compiler.
+ Modify the SSL dissector such that both "http" and "http-over-tls"
+ invoke the same dissector function.
- Change-Id: I95685c9a3e25dcb7567f2551b92f20c8792a6e47
- Reviewed-on: https://code.wireshark.org/review/17384
- Petri-Dish: Dario Lombardo <lomato@gmail.com>
+ Change-Id: I2d52890a8ec8fa88b6390b133a11df607a5ec3dc
+ Reviewed-on: https://code.wireshark.org/review/17481
+ Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Dario Lombardo <lomato@gmail.com>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
- (cherry picked from commit a892c90367a9f1b1d3d962cfe958174b8fefcaee)
- Reviewed-on: https://code.wireshark.org/review/17390
+ (cherry picked from commit 7c7497a77e30f8098bcaffd0a9c96be24855c0e8)
+ Reviewed-on: https://code.wireshark.org/review/17482
-commit a9e19bc
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Sat Aug 6 20:07:04 2016 -0700
+commit f44b4f8
+Author: Pascal Quantin <pascal.quantin@gmail.com>
+Date: Sun Sep 4 11:52:24 2016 +0200
- Avoid colliding with the (old) UNIX stime() system call.
+ irda: restore a line removed by mistake in ge4c059f
- Rename "stime" to "sys_time", and, for consistency, do the same with
- "utime". This failed when I tried a build on an Ubuntu 12.04 LTS
- machine.
-
- Change-Id: Ib955b723506346aa742db0de4d26dc157bcd4fc8
- Reviewed-on: https://code.wireshark.org/review/16935
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit 66711eb2612ddc7f523b7cc5990e1674a755cb00)
- Reviewed-on: https://code.wireshark.org/review/17389
- Reviewed-by: Peter Wu <peter@lekensteyn.nl>
+ Bug: 12821
+ Change-Id: If6423c7f8443eccd6ddb4b699bbc327a965d0d74
+ Reviewed-on: https://code.wireshark.org/review/17477
+ Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
+ (cherry picked from commit 29a243d61cb1ae9d6eda404dad2038b85cfdb487)
+ Reviewed-on: https://code.wireshark.org/review/17479
-commit 5f59439
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Sat Jul 30 13:50:04 2016 -0700
+commit 4f6214a
+Author: Gerald Combs <gerald@wireshark.org>
+Date: Thu Sep 1 13:51:13 2016 -0700
- Use correct relative path.
-
- The build is done in epan/wmem, so libwsutil.la is in ../../wsutil.
+ Qt: Conversation time column updates.
- Change-Id: I053f6b925829e9c5e9ded014fbc716abc527b46a
- Reviewed-on: https://code.wireshark.org/review/16790
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit c2ac9c5c0316a27f54830fae0b70cf7825c6d99c)
- Reviewed-on: https://code.wireshark.org/review/17388
- Reviewed-by: Peter Wu <peter@lekensteyn.nl>
-
-commit b25ce3d
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Sat Jul 30 13:37:03 2016 -0700
-
- ifdef out a variable used only on Windows.
+ Add a checkbox which lets you toggle between absolute and relative start
+ times. Use the local time for now. Fixes bug 11618.
- Also, fix identation - tab stops are set every 8 spaces, not every 4
- spaces, in UN*X, which is why the mode lines specify that.
+ Adjust our time precision based on the capture file's time precision.
+ Fixes bug 12803.
- Change-Id: If7cfc2cdae26feb2d5c13e31a32ed950354fb15b
- Reviewed-on: https://code.wireshark.org/review/16789
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit f6322a4977181103431637703f11649e5023d6b0)
- Reviewed-on: https://code.wireshark.org/review/17387
- Reviewed-by: Peter Wu <peter@lekensteyn.nl>
-
-commit 1eb6828
-Author: Gerald Combs <gerald@wireshark.org>
-Date: Thu Apr 7 15:31:21 2016 -0700
-
- Add string function times to wmem_test.
-
- The system, GLib, and wmem string functions can perform differently,
- particularly on Windows. Start adding performance tests to wmem_test so
- that we can see the differences.
-
- With this change applied "wmem_test --verbose" prints out the following
- on a Windows 7 x64 VM here. wmem_test is linked against GLib 2.4.20.
-
- (MINPERF:g_printf_string_upper_bound (via g_snprintf) 1 string: u 327.602 ms s 0
- .000 ms)
- (MINPERF:g_printf_string_upper_bound (via g_snprintf) 5 strings: u 1419.609 ms s
- 0.000 ms)
- (MINPERF:g_printf_string_upper_bound (via g_snprintf) mixed args: u 1606.810 ms
- s 0.000 ms)
- (MINPERF:_snprintf_s upper bound 1 string: u 124.801 ms s 0.000 ms)
- (MINPERF:_snprintf_s upper bound 5 strings: u 140.401 ms s 0.000 ms)
- (MINPERF:_snprintf_s upper bound mixed args: u 124.801 ms s 0.000 ms)
- (MINPERF:g_strdup_printf 2 strings: u 702.005 ms s 0.156 ms)
- (MINPERF:g_strconcat 2 strings: u 78.000 ms s 0.000 ms)
- (MINPERF:g_strdup_printf 5 strings: u 1419.609 ms s 0.156 ms)
- (MINPERF:g_strconcat 5 strings: u 93.601 ms s 0.156 ms)
- (MINPERF:wmem_strdup_printf 2 strings: u 343.202 ms s 0.312 ms)
- (MINPERF:wmem_strconcat 2 strings: u 93.601 ms s 0.468 ms)
- (MINPERF:wmem_strdup_printf 5 strings: u 327.602 ms s 8.268 ms)
- (MINPERF:wmem_strconcat 5 strings: u 62.400 ms s 3.432 ms)
-
- Change-Id: Id9b23918829db1719d141e7f830b9eba6245a25b
- Reviewed-on: https://code.wireshark.org/review/14857
+ Bug: 11618
+ Bug: 12803
+ Change-Id: I0049d6db6e4d0b6967bf35e6d056a61bfb4de10f
+ Reviewed-on: https://code.wireshark.org/review/17471
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
- (cherry picked from commit 183d7f3b78de30581390244a72c6ab328e6338f0)
- Reviewed-on: https://code.wireshark.org/review/17386
- Reviewed-by: Peter Wu <peter@lekensteyn.nl>
-commit f3a5b33
-Author: Roland Knall <roland.knall@br-automation.com>
-Date: Mon Aug 1 16:57:51 2016 +0200
+commit fa7a6e3
+Author: Peter Wu <peter@lekensteyn.nl>
+Date: Mon Aug 29 18:38:45 2016 +0200
- extcap: Remove g_spawn_check_exit_status
+ gtk: fix compilation with GTK+ before 2.18
- This function is not supported in the min GLIB version, therefore
- the code is to be removed
+ Removed in v1.99.2rc0-399-g43f09e6, but some includes are still needed.
- Change-Id: Ie39170bfc0662e5a477cbc45d7eadebcf2c70d4e
- Reviewed-on: https://code.wireshark.org/review/16827
- Petri-Dish: Roland Knall <rknall@gmail.com>
- Reviewed-by: Dario Lombardo <lomato@gmail.com>
- Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Roland Knall <rknall@gmail.com>
- (cherry picked from commit d62ad9dd47d368f7545c3cc91f44ecc07476d2e1)
- Reviewed-on: https://code.wireshark.org/review/17383
- Reviewed-by: Anthony Coddington <anthony.coddington@endace.com>
-
-commit 0962de8
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Mon Aug 29 19:49:58 2016 -0700
-
- Don't use a no-longer-extant variable.
+ Basically I looked for the functions defined in the
+ ui/gtk/old-gtk-compat.h header file that were in files below ui/gtk/.
+ Then I matched it against the removed part in the above commit and added
+ the header back at its old place. In two other cases, the header was
+ only needed for Windows.
- Change-Id: I41c1a37248335d983da58b0b657a28ec521be290
- Reviewed-on: https://code.wireshark.org/review/17378
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit 8e9b6f4abe06a32bb5c8e8842403c4cf8dccd765)
- Reviewed-on: https://code.wireshark.org/review/17379
-
-commit 5d9f67f
-Author: Pascal Quantin <pascal.quantin@gmail.com>
-Date: Mon Aug 29 11:31:11 2016 +0200
-
- tap-iostat.c: ensure that interval is set to its maximum value when using 0
+ If the above commit did not reference the file, I checked which function
+ was in use and added the header on top, removing redundant conditionals.
- Otherwise the statistics will be wrong if the capture duration is greater
- than G_MAXINT32 and it the user specifies an interval of 0
+ Reported and tested by a user of GTK+ 2.12.12 on top of v2.0.5.
- Bug: 12778
- Change-Id: I83a0f627ec0bb7c535446c17afa486835091ab8b
- Reviewed-on: https://code.wireshark.org/review/17367
- Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
- Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
+ Change-Id: I649eec1e5531070f88c99d893c4920306f56d849
+ Reviewed-on: https://code.wireshark.org/review/17371
+ Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Michael Mann <mmann78@netscape.net>
- (cherry picked from commit 24fd16d86553279ca3fe8f651d85cace14edfc6c)
- Reviewed-on: https://code.wireshark.org/review/17369
-
-commit 6deac90
-Author: Gerald Combs <gerald@wireshark.org>
-Date: Sun Aug 28 08:22:26 2016 -0700
-
- [Automatic update for 2016-08-28]
-
- Update manuf, services enterprise-numbers, translations, and other items.
-
- Change-Id: I582a41fbd6f895572b3c5822cf04fb3c5fb03c2b
- Reviewed-on: https://code.wireshark.org/review/17359
- Reviewed-by: Gerald Combs <gerald@wireshark.org>
+ Reviewed-by: Peter Wu <peter@lekensteyn.nl>
+ (cherry picked from commit 77ae30ffc598a56e71e0f57e151d0b6801110b03)
+ Reviewed-on: https://code.wireshark.org/review/17438
-commit 0180c29
-Author: Mirko Parthey <mirko.parthey@web.de>
-Date: Sun Aug 28 14:49:52 2016 +0200
+commit 7058a80
+Author: Michael Mann <mmann78@netscape.net>
+Date: Sun Jul 24 14:16:19 2016 -0400
- ISAKMP: fix # of SPIs field name in Delete payload
+ Pacify GTK checkAPIs.pl warnings.
- Fix the "Number of SPIs" field name in the Delete payload.
- References: RFC 2408, RFC 7296
+ Add "Wireshark" macros to old-gtk-compat.h for GTK APIs that have been deprecated.
+ The macros are setup by version number to limit their proliferation (not that I
+ suspect much development will really be done there since GTK is deprecated).
+ Just want to make buildbots happy for the time being.
- Change-Id: I205fb830275fc011e6605fdae53c6b9141e1628b
- Reviewed-on: https://code.wireshark.org/review/17353
- Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
+ Change-Id: I095f850065166a0bc2e2456fb2e886ab64fdd97d
+ Reviewed-on: https://code.wireshark.org/review/16635
+ Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
- (cherry picked from commit 294fcd11bfc2a76306e0c6ea8e8f53445df940cf)
- Reviewed-on: https://code.wireshark.org/review/17356
-
-commit c66deb9
-Author: Stig Bjørlykke <stig@bjorlykke.org>
-Date: Sat Aug 27 11:53:29 2016 +0200
-
- Qt: Support "Resolve Names" with multiple custom column fields
-
- Make "Resolve Names" enabled if at least one of the custom column
- fields can be resolved.
-
- Change-Id: I702471be5d90c91f71209923e6abcc6921dca850
- Reviewed-on: https://code.wireshark.org/review/17350
- Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
- (cherry picked from commit 5fe3e6af39fc55650ff9563906d016aa7becbaf1)
- Reviewed-on: https://code.wireshark.org/review/17351
-
-commit 4229951
-Author: Alexis La Goutte <alexis.lagoutte@gmail.com>
-Date: Fri Aug 26 10:41:45 2016 +0200
-
- QUIC: Fix some typo (copy/paste error) on GOAWAY/CONNECTION_CLOSE Frame
-
- Issue reported by Lucas Perdue
-
- Change-Id: I9c4ede6ba2fb0303aab05f1d59835e5a8b386a3e
- Reviewed-on: https://code.wireshark.org/review/17340
- Reviewed-by: Lucas Pardue <lucas.pardue@bbc.co.uk>
+ Reviewed-by: Balint Reczey <balint@balintreczey.hu>
Reviewed-by: Michael Mann <mmann78@netscape.net>
- (cherry picked from commit df28c8266ff9ebc404fd53408050c507a442ac04)
- Reviewed-on: https://code.wireshark.org/review/17349
- Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
-
-commit c71ac83
-Author: Peter Wu <peter@lekensteyn.nl>
-Date: Thu Aug 25 18:03:35 2016 +0200
-
- qt: fix crash on Search Packet while redissecting
-
- Before redissection, PacketList::freeze() is called which clears the
- model. This results in a NULL-deref when pressing Ctrl-F (Packet
- Search) or Ctrl-G (Go To Packet). Reproducer: in a large capture file,
- enter some display filter (e.g. "udp") and immediately press Ctrl-F.
-
- Thanks to Github user SNAPESNATCH for the initial report via IRC that
- included a helpful stack trace on Windows.
-
- Change-Id: If7334d6df4e9591fb1f2a52e3e2f837285b2959f
- Reviewed-on: https://code.wireshark.org/review/17326
- Petri-Dish: Peter Wu <peter@lekensteyn.nl>
- Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
+ (cherry picked from commit ee7f9c33f63532bc69899a0750177756be53c0c1)
+ Reviewed-on: https://code.wireshark.org/review/17437
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
- (cherry picked from commit 09be46361bab677842cc598405cbc7a44b974f1f)
- Reviewed-on: https://code.wireshark.org/review/17342
-commit 608befb
+commit 6d29c12
Author: Stig Bjørlykke <stig@bjorlykke.org>
-Date: Thu Aug 25 15:39:40 2016 +0200
+Date: Fri Sep 2 14:10:31 2016 +0200
- xmpp/xml: Store item length in xml_frame.
+ Qt: Preserve capture filter when preferences changed
- The created XML proto_item can be faked (if not visible and not referenced),
- so ensure we store the correct item length to be used in XMPP.
+ Avoid that the last entry from recent.capture_filter is added to the
+ capture filter combo when editing preferences or changing profile.
- This will avoid an invalid "Malformed Packet" for some XMPP packets.
+ This bug was introduced in gb7897dde.
- Change-Id: I79d805b725dbeb93f26a38b72bdcc84187aee16f
- Reviewed-on: https://code.wireshark.org/review/17324
+ Change-Id: I38a32386765c9e7ffaa93d006ff0ef7b78ac8252
+ Reviewed-on: https://code.wireshark.org/review/17453
Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
- (cherry picked from commit ce9a4b212ec7b75985cef3b9ab362808e6fd25c5)
- Reviewed-on: https://code.wireshark.org/review/17338
+ Reviewed-by: Peter Wu <peter@lekensteyn.nl>
+ Reviewed-by: Anders Broman <a.broman58@gmail.com>
+ (cherry picked from commit 770aaf1dde2c57687beb0ebe9a3af9003c3c2c14)
+ Reviewed-on: https://code.wireshark.org/review/17468
Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
-commit c84ce03
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Thu Aug 25 16:58:25 2016 -0700
-
- Handle ETH_P_CANFD as well as ETH_P_CAN.
-
- Both of them need to have the CAN ID/flags field of the header
- byte-swapped as necessary to make sure it's in the *reading* host's byte
- order, not the *writing* host's byte order, if the two are different.
-
- Change-Id: Iac1589fdd9fe4d9ee6fbac8d821b48694d68919b
- Reviewed-on: https://code.wireshark.org/review/17333
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit 17f977ec5695259ec02c7e6e4cd24e3e2ed263c1)
- Reviewed-on: https://code.wireshark.org/review/17334
-
-commit ddbae1c
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Thu Aug 25 14:39:42 2016 -0700
-
- One more change for the removal of LINKTYPE_SOCKETCAN_HOSTENDIAN.
-
- Change-Id: If1615e23efa30119fff5eb2935335c2cde34b89e
- Reviewed-on: https://code.wireshark.org/review/17330
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit 27372ca7379b3ff5cccf1488d96a72d06dfba0d1)
- Reviewed-on: https://code.wireshark.org/review/17331
-
-commit 6d074ae
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Thu Aug 25 14:14:11 2016 -0700
+commit a6bd069
+Author: Peter Wu <peter@lekensteyn.nl>
+Date: Wed Aug 31 17:55:15 2016 +0200
- Go back to having only one SocketCAN LINKTYPE_ value.
+ diam_dict.l: remove use of strdup
- Libpcap just backed out the "host-endian" SocketCAN LINKTYPE_ value; we
- don't need it any more.
+ This should fix crashes on Windows, _strdup should not be mixed with
+ g_free. This was only uncovered in v2.3.0rc0-474-ga04b6fc, before that
+ ddict_free was never called.
- Change-Id: I33a7dc21207a0009e20b4abaefe1119eb649c39a
- Reviewed-on: https://code.wireshark.org/review/17327
+ Change-Id: I34111385c82715de70fb42fe44b99b89e132a374
+ Reviewed-on: https://code.wireshark.org/review/17423
Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit c8157aa6b4afdbc2f84779ffbeea4f8d3bdc8666)
- Reviewed-on: https://code.wireshark.org/review/17328
+ (cherry picked from commit 5c2d82a6c24a06da0132fb7bd493ce4e58594f0d)
+ Reviewed-on: https://code.wireshark.org/review/17466
-commit 2e3c898
-Author: Balint Reczey <balint@balintreczey.hu>
-Date: Tue Aug 23 13:53:23 2016 +0200
+commit 44e097d
+Author: Peter Wu <peter@lekensteyn.nl>
+Date: Mon Aug 29 01:34:22 2016 +0200
- Documentation with unset license is under project's license (GPL-2+)
+ xml: fix some memleaks
- doc/idl2deb.pod is licensed explicitly under GPL-2+ it is just
- not recognized by the check.
+ No more memleaks reported for the attachment in bug 12790 :-)
- Change-Id: Iabc7ee6bd6fe9080d6cdd28a29bb4c5f97b0ece9
- Reviewed-on: https://code.wireshark.org/review/17268
- Reviewed-by: Balint Reczey <balint@balintreczey.hu>
- Petri-Dish: Balint Reczey <balint@balintreczey.hu>
- Reviewed-by: Dario Lombardo <lomato@gmail.com>
- Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- (cherry picked from commit 40ac3c86825d4a15e585035fbae6e9a86424ee4e)
- Reviewed-on: https://code.wireshark.org/review/17283
+ Change-Id: I8472e442143b332edfacdf9ef3b8b893f1ec4386
+ Ping-Bug: 12790
+ Reviewed-on: https://code.wireshark.org/review/17365
Reviewed-by: Michael Mann <mmann78@netscape.net>
+ Petri-Dish: Michael Mann <mmann78@netscape.net>
+ Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
+ Reviewed-by: Peter Wu <peter@lekensteyn.nl>
+ (cherry picked from commit 961f743d69b323aa217a6b39184485d6a0bfb2d5)
+ Reviewed-on: https://code.wireshark.org/review/17465
+ Reviewed-by: Guy Harris <guy@alum.mit.edu>
-commit 1c16105
-Author: Gerald Combs <gerald@wireshark.org>
-Date: Tue Aug 23 16:55:16 2016 -0700
+commit 0b45da6
+Author: Peter Wu <peter@lekensteyn.nl>
+Date: Sun Aug 28 22:19:29 2016 +0200
- Fuzz test: Make Valgrind error detection more verbose.
+ diameter: fix 400kb leaked memory on exit
- When we set VG_ERR_CNT=1 print the reason.
+ Before:
- Change-Id: Icb8f2a1e7074044521873b116fc891cc4be4b204
- Reviewed-on: https://code.wireshark.org/review/17286
- Reviewed-by: Gerald Combs <gerald@wireshark.org>
-
-commit e8d5252
-Author: Michał Skalski <mskalski13@gmail.com>
-Date: Tue Aug 23 01:12:59 2016 +0200
-
- isakmp Fix IKEv2 modification UAT crash
+ SUMMARY: AddressSanitizer: 399684 byte(s) leaked in 17208 allocation(s).
- Problem is that not all fields of UAT are updated on change (only those
- defined in UAT definition with ikev2_uat_flds, specifically pointers to
- encr_spec and auth_spec in ike2_decr_data_t were set to NULL.
+ After addressing to-do by calling ddict_free:
- Fixed by re-setting pointers after update table callback was called.
+ SUMMARY: AddressSanitizer: 3024 byte(s) leaked in 256 allocation(s).
- Also fixed memory leaks after UAT modification.
+ After fixing all remaining leaks cases in the flex file for diameter:
- Bug was partially resolved with change
- Ibdab979b5959eb561635cbcb446e17138baca87b
- https://code.wireshark.org/review/17078
+ SUMMARY: AddressSanitizer: 735 byte(s) leaked in 58 allocation(s).
- which eliminated crash, but decryption still didn't work after UAT
- modification (DISSECTOR BUG was displayed).
+ Not bad huh :-)
- Bug: 12748
- Change-Id: I8209edd8e214d62e34b641fdd2e046b9ff4c95eb
- Reviewed-on: https://code.wireshark.org/review/17249
- Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
- Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl>
+ Ping-Bug: 12790
+ Change-Id: I0c730ad77ae15c69390bc6cf0a3a985395a64771
+ Reviewed-on: https://code.wireshark.org/review/17364
+ Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
- (cherry picked from commit 505bcefd20501ae0811f6074f2113b52788d78c5)
- Conflicts:
- epan/dissectors/packet-isakmp.c
- Reviewed-on: https://code.wireshark.org/review/17299
- Reviewed-by: Michał Skalski <mskalski13@gmail.com>
- Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
+ Reviewed-by: Michael Mann <mmann78@netscape.net>
+ Reviewed-by: Jeff Morriss <jeff.morriss.ws@gmail.com>
+ Reviewed-by: Peter Wu <peter@lekensteyn.nl>
+ (cherry picked from commit a04b6fcb3db901734ed948134c973996786be8b7)
+ Reviewed-on: https://code.wireshark.org/review/17464
+ Reviewed-by: Guy Harris <guy@alum.mit.edu>
-commit fb6b0e6
-Author: Pascal Quantin <pascal.quantin@gmail.com>
-Date: Mon Aug 15 18:55:45 2016 +0200
+commit 357e063
+Author: Peter Wu <peter@lekensteyn.nl>
+Date: Sun Aug 28 19:15:01 2016 +0200
- ISAKMP: Fix Result of operation is garbage or undefined found by Clang Analyzer
+ tvbparse: fix memleak
- Change-Id: I47233968dfc87494b3c959896622db5cd31244a7
- Reviewed-on: https://code.wireshark.org/review/17065
+ Use same wmem_epan_scope() as "w" (tvbparse_wanted_t).
+
+ Change-Id: I73fdb1fb3b55a91b7bb0fc36e435024c6f0b3d73
+ Ping-Bug: 12790
+ Reviewed-on: https://code.wireshark.org/review/17361
+ Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
- Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Mirko Parthey <mirko.parthey@web.de>
- Reviewed-by: Anders Broman <a.broman58@gmail.com>
- (cherry picked from commit 3c8f657e01f1548fbf9cc5740374552aa8c6fdc6)
- Reviewed-on: https://code.wireshark.org/review/17267
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
+ (cherry picked from commit 0bae8424fe651ea5de1f7e160666781e60ce3c86)
+ Reviewed-on: https://code.wireshark.org/review/17463
+ Reviewed-by: Guy Harris <guy@alum.mit.edu>
-commit 5c07723
-Author: Mirko Parthey <mirko.parthey@web.de>
-Date: Tue Jul 19 00:43:19 2016 +0200
+commit 297a4ea
+Author: Peter Wu <peter@lekensteyn.nl>
+Date: Sun Aug 28 19:20:59 2016 +0200
- Fix IKEv1 decryption and initialization vectors
-
- Generate initialization vectors during Wireshark's first pass,
- in order to improve reliability and efficiency. (bug #12699)
- Remove a dissection shortcut in first pass to allow early analysis
- of the transform payloads, which is needed to find out the crypto
- algorithms.
+ pcapng: do not leak blocks
- Introduce a hash table to store an IV per message ID. (bug #12610)
- Fix handling of the initial phase 1 IV. (bug #12620)
- Cache IVs as per-packet data instead of a list.
+ pcapng_open and pcapng_read have 'wblock' allocated on the stack, so if
+ they return, they do not have to set wblock.block to NULL.
- Use the wmem allocator to avoid memory leaks.
- Ensure libgcrypt resource cleanup on error paths.
+ pcapng_read_block always sets wblock->block to NULL and may initialize
+ it for SHB, IDB, NRB and ISB. Be sure to release the memory for IDB and
+ ISB. It is better to have more wtap_block_free calls on a NULL value
+ than missing them as this would be a memleak (on the other hand, do not
+ release memory that is stored elsewhere such as SHB and NRB).
- Rely only on IKE attributes to find the decryption algorithm,
- do not consider the length of the user-configured secret at this point.
-
- Split code into small functions.
-
- Ping-Bug: 12610
- Ping-Bug: 12620
- Ping-Bug: 12699
- Change-Id: I7137943da42d54816808312cd1716a7e05478cbb
- Reviewed-on: https://code.wireshark.org/review/16846
- Reviewed-by: Peter Wu <peter@lekensteyn.nl>
+ Ping-Bug: 12790
+ Change-Id: I081f841addb36f16e3671095a919d357f4bc16c5
+ Reviewed-on: https://code.wireshark.org/review/17362
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Anders Broman <a.broman58@gmail.com>
- (cherry picked from commit 561a2f056bd76bfcb4f89a05542ffce32eed5049)
- Reviewed-on: https://code.wireshark.org/review/17198
- Reviewed-by: Mirko Parthey <mirko.parthey@web.de>
-
-commit fef8471
-Author: Mirko Parthey <mirko.parthey@web.de>
-Date: Tue Aug 23 16:29:57 2016 +0200
-
- ISAKMP: fix DOI field read size
-
- Read 4 bytes from the packet instead of 1 because that is the
- correct size of the DOI field.
- Reference: RFC 2408
-
- Change-Id: I5745363811bb46af307a925d688ec36cfb29984b
- Reviewed-on: https://code.wireshark.org/review/17271
- Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
- Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
- Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Anders Broman <a.broman58@gmail.com>
- (cherry picked from commit 0fcca6f0c8392b16ebfefdc69086d02e1dba2c7e)
- Reviewed-on: https://code.wireshark.org/review/17295
-
-commit 7dc3d01
-Author: Pascal Quantin <pascal.quantin@gmail.com>
-Date: Tue Aug 23 22:10:08 2016 +0200
-
- call_heur_dissector_direct: do not trigger an assert if heuristic dissector rejects packet
-
- This can happen for example is the heuristics changed between the Wireshark
- version used to export PDUs, and the one used to open the file.
- Instead, call data dissector.
-
- Change-Id: I29f7754f883fd710c3557a610583ef988ca13e43
- Reviewed-on: https://code.wireshark.org/review/17280
- Reviewed-by: Anders Broman <a.broman58@gmail.com>
- (cherry picked from commit 2095b325998f1da06a4e80efeb186ec99d6e39ed)
- Reviewed-on: https://code.wireshark.org/review/17294
- Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
+ Reviewed-by: Peter Wu <peter@lekensteyn.nl>
+ (cherry picked from commit 14312835c63a3e2ec9d311ed1ffee5285141f4f9)
+ Reviewed-on: https://code.wireshark.org/review/17462
+ Reviewed-by: Guy Harris <guy@alum.mit.edu>
-commit a7e78b4
-Author: Alexis La Goutte <alexis.lagoutte@gmail.com>
-Date: Tue Aug 23 19:41:46 2016 +0200
+commit f40ffe1
+Author: Stig Bjørlykke <stig@bjorlykke.org>
+Date: Fri Sep 2 15:45:53 2016 +0200
- SCSI: Mode Sense 10: Wrong block descriptor length
+ Qt: Clear open files list when clear recent menu
- Issue reported by Sharon Samuel Enoch
+ Remove all recent files from the main welcome page when clearing
+ recent files from the menu.
- Bug:12780
- Change-Id: I94ad5355cdfa4d8cd3915c9e261931ff56dc765b
- Reviewed-on: https://code.wireshark.org/review/17272
- Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
- Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
+ Change-Id: Ic410a729e63d82ee58b6bbb31f7e4a658b17d794
+ Reviewed-on: https://code.wireshark.org/review/17456
+ Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Anders Broman <a.broman58@gmail.com>
- (cherry picked from commit 20908f55f6c93fb79fbd39e60affa2d4428b4951)
- Reviewed-on: https://code.wireshark.org/review/17292
+ Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
+ (cherry picked from commit fb7a60e611b4cea46978d4f2452fe9ad7a1916ed)
+ Reviewed-on: https://code.wireshark.org/review/17460
-commit 2a4150a
-Author: Chris Bontje <cbontje@gmail.com>
-Date: Tue Aug 23 08:42:59 2016 -0600
+commit 5b399d3
+Author: Stig Bjørlykke <stig@bjorlykke.org>
+Date: Fri Sep 2 15:21:46 2016 +0200
- [lg8979] - Fix dissection of 'Analog Group Change Report' Responses (FC2)
-
- They follow the same dissection as normal 'Analog Change Report' Responses (FC0)
+ Qt: Add newly saved files to the recent files
- Change-Id: I854084f43fd0cc52ba02b6f1e760a63033ab48dd
- Reviewed-on: https://code.wireshark.org/review/17270
- Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
- Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
- Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
- (cherry picked from commit e11d4a891373c0f47062dd526a1f5ff6baa09ba6)
- Reviewed-on: https://code.wireshark.org/review/17281
-
-commit f9598b8
-Author: Pascal Quantin <pascal.quantin@gmail.com>
-Date: Wed Aug 24 08:10:27 2016 +0200
-
- IPMI: do not use col_set_str with non const strings
+ Update the recent files list when saving unsaved capture files.
- Bug: 12782
- Change-Id: Ia082ccf5355d7f8dd6073861c59c804fecc96266
- Reviewed-on: https://code.wireshark.org/review/17289
- Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
- (cherry picked from commit 5213496250aceff086404c568e3718ebc0060934)
- Reviewed-on: https://code.wireshark.org/review/17290
+ Change-Id: I469fa1f2ce3216c66de328c0b3558c9f9db115e0
+ Reviewed-on: https://code.wireshark.org/review/17459
+ Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
-commit 96e712f
+commit de86f27
Author: Gerald Combs <gerald@wireshark.org>
-Date: Tue Aug 23 10:44:32 2016 -0700
+Date: Thu Sep 1 15:12:18 2016 -0700
- Update the release notes.
+ CMake: Move version declarations to the top of CMakeLists.txt.
- Add missing new protocols, move a "what's new item", add a bug.
+ This lets me be more lazy during releases.
- Change-Id: I84e4cf36591a503d96032c6be9ba14bdc0dc016c
- Reviewed-on: https://code.wireshark.org/review/17273
- Petri-Dish: Gerald Combs <gerald@wireshark.org>
- Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
+ Change-Id: Ie321454db362e08c7420943a1b07a53599818b7e
+ Reviewed-on: https://code.wireshark.org/review/17451
Reviewed-by: Gerald Combs <gerald@wireshark.org>
-
-commit 02f6c94
-Author: Pascal Quantin <pascal.quantin@gmail.com>
-Date: Tue Aug 23 19:41:48 2016 +0200
-
- MAC LTE: fix dissection of CE Mode A RAR message
-
- The Msg3 PUSCH narrowband index parameter depends on the UL bandwidth
-
- Change-Id: Ib57c85ffbd4c108e9c8f3d14fa53a48f0df1b0e6
- Reviewed-on: https://code.wireshark.org/review/17274
- Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
+ Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
- (cherry picked from commit 12cae9dc80a09ba0ed75b421d328143ac98a298b)
- Reviewed-on: https://code.wireshark.org/review/17276
-
-commit 934bcae
-Author: João Valverde <joao.valverde@tecnico.ulisboa.pt>
-Date: Mon Aug 8 16:48:14 2016 +0100
-
- checklicenses: Fix a FIXME
-
- We do not whitelist GPLv2 only code because of the
- incompatibility with GPLv3.
-
- Move the path exception to its proper place. These files are
- dual-licensed in the sense of "choose one or the other", not
- "both licenses are in use".
-
- If the exception becomes unnecessary later it can be removed,
- but let's get rid of the FIXME until then.
-
- Change-Id: Ia040284b72c2ff6588b0544896cc37ae19a61613
- Reviewed-on: https://code.wireshark.org/review/16957
- Reviewed-by: João Valverde <j@v6e.pt>
- (cherry picked from commit e3009b0b1c78b25337c915fb66232314f769688c)
- Reviewed-on: https://code.wireshark.org/review/17264
- Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
- Reviewed-by: Peter Wu <peter@lekensteyn.nl>
- Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
-
-commit ac1b8b3
-Author: Michael Mann <mmann78@netscape.net>
-Date: Tue Jul 26 09:27:07 2016 -0400
-
- Move airpdcap's print_debug_line() to airpdcap_debug.h
-
- It saves a little on file pollution and the g_warning call isn't
- bothered in the header file.
-
- Change-Id: Ia9bdd96d9d93bbba6811769c4e6e1ed9124c2e5a
- Reviewed-on: https://code.wireshark.org/review/16698
Reviewed-by: Michael Mann <mmann78@netscape.net>
- Petri-Dish: Michael Mann <mmann78@netscape.net>
- Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Anders Broman <a.broman58@gmail.com>
- (cherry picked from commit 9a73978ba5bbb78026721500aaf332f53e19d0cd)
- Reviewed-on: https://code.wireshark.org/review/17266
- Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
- Reviewed-by: Peter Wu <peter@lekensteyn.nl>
- Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
-commit 562457a
+commit b348c38
Author: Peter Wu <peter@lekensteyn.nl>
-Date: Mon Aug 22 22:29:21 2016 +0200
+Date: Wed Aug 31 11:11:56 2016 +0200
- isakmp: fix type confusion
+ gtk: fix 32-bit build
- Caught by ASAN with WIRESHARK_DEBUG_WMEM_OVERRIDE=simple set.
+ 32-bit glib before 2.31.2 expand GUINT_TO_POINTER(x) as (gpointer)x.
+ add explicit cast since curr_layer_num is 8 bits. Fixes
+ v1.99.10rc0-179-g1d7bcb2.
- Manually inspection of all type casts to decrypt_data_t and
- ikev2_decrypt_data_t showed no other users that do not check
- isakmp_version first.
+ Storing a 64-bit integer into a 32-bit pointer makes the compiler
+ complain. Add explicit cast. Fixes v1.99.1rc0-76-ged0b19b.
- Change-Id: If889afff85a20e31222d33cbea8db3a91a77f389
- Reviewed-on: https://code.wireshark.org/review/17246
+ Change-Id: I75fdf17882a0f5ddce7d3b3e74b1bf80ff6cd4ae
+ Reviewed-on: https://code.wireshark.org/review/17417
Petri-Dish: Peter Wu <peter@lekensteyn.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
- Reviewed-by: Michał Skalski <mskalski13@gmail.com>
- Reviewed-by: Anders Broman <a.broman58@gmail.com>
- (cherry picked from commit 52910f4e3fa7d771a671cd30bd5320d70aee3eec)
- Reviewed-on: https://code.wireshark.org/review/17260
- Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
- Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
-
-commit f60c4b2
-Author: Balint Reczey <balint@balintreczey.hu>
-Date: Tue Aug 23 03:01:32 2016 +0200
+ (cherry picked from commit e3c3d7a4afeeb8bc0519717e6c85c038fb99170a)
+ Reviewed-on: https://code.wireshark.org/review/17435
- debian: Fix libwscodecs' symbols file
-
- Change-Id: I168a630643fc7a6cbc27d7879006d31acca30e37
- Reviewed-on: https://code.wireshark.org/review/17263
- Petri-Dish: Balint Reczey <balint@balintreczey.hu>
- Reviewed-by: Balint Reczey <balint@balintreczey.hu>
- Tested-by: Balint Reczey <balint@balintreczey.hu>
-
-commit 5a1c9a0
-Author: Balint Reczey <balint@balintreczey.hu>
-Date: Tue Aug 23 02:49:34 2016 +0200
-
- debian: Refresh patches
-
- Change-Id: I7b756f3f88e664687602be6ced77f0d9a29a1238
- Reviewed-on: https://code.wireshark.org/review/17262
- Reviewed-by: Balint Reczey <balint@balintreczey.hu>
-
-commit dba08c5
-Author: Balint Reczey <balint@balintreczey.hu>
-Date: Tue Aug 23 02:25:17 2016 +0200
-
- debian: Make wireshark-common depend on proper libwiretap package
-
- Change-Id: I251f4807732fc091629899b6c792bc53a43d3e64
- Reviewed-on: https://code.wireshark.org/review/17261
- Reviewed-by: Balint Reczey <balint@balintreczey.hu>
-
-commit 2697dbb
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Mon Aug 22 20:36:24 2016 -0700
-
- See if *this* convinces the compiler that valuelen and value will be set.
-
- Change-Id: I2a7715ac3255502f244a0d0a7e588b3a44c34f11
- Reviewed-on: https://code.wireshark.org/review/17258
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit bee0cd776219fc3cd6ac6a932a1d25b944c299a0)
- Reviewed-on: https://code.wireshark.org/review/17259
-
-commit e23a7d0
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Mon Aug 22 19:28:24 2016 -0700
+commit 60649ac
+Author: Peter Wu <peter@lekensteyn.nl>
+Date: Tue Aug 30 19:24:15 2016 +0200
- Fix handling of parameters with no value.
-
- Return a value length of 0 if there's no parameter value.
-
- (And don't return anything if the pointers through which we return them
- are null.)
+ erf: fix build against older glib
- If no value is present, return NULL from ws_find_media_type_parameter().
+ Commit v2.1.0rc0-2561-g3db7b1e started using g_int64_equal which does
+ not exist before glib 2.22. Note that master requires 2.22 and does not
+ need this fix.
- Change-Id: I32b57623d7651bcf065af5b81f2390a600988b21
- Reviewed-on: https://code.wireshark.org/review/17255
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit ff00166a96d887e03899577caae3a355c26b0837)
- Reviewed-on: https://code.wireshark.org/review/17256
-
-commit 1c167f8
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Mon Aug 22 18:25:33 2016 -0700
-
- Add cast to squelch "building with a C++ compiler" warnings.
-
- Change-Id: Ia95c40096018479aec99fafd93d7b95d31ba4723
- Reviewed-on: https://code.wireshark.org/review/17253
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit 4458c48cf4b94da8626361499a044d7c8c096538)
- Reviewed-on: https://code.wireshark.org/review/17254
+ Change-Id: Ie89432a6cd688b89dd346ce5cdecf96a76e92692
+ Reviewed-on: https://code.wireshark.org/review/17395
+ Reviewed-by: João Valverde <j@v6e.pt>
+ Reviewed-by: Peter Wu <peter@lekensteyn.nl>
-commit d6f83a3
-Author: Guy Harris <guy@alum.mit.edu>
-Date: Mon Aug 22 17:53:10 2016 -0700
+commit 2755421
+Author: Jaap Keuter <jaap.keuter@xs4all.nl>
+Date: Wed Aug 31 21:06:34 2016 +0200
- Handle quoted-pairs in quoted-strings correctly.
-
- Backslash, in a quoted-string, escapes quotes (and any other characters,
- although the only ones that *need* escaping are a double-quote and a
- backslash).
+ OSSP: Retrieve OUI at proper offset in packet
- This means that the value of a parameter isn't just the raw characters
- from the parameters string; for a quoted string, it needs to be
- un-escaped, and for a *non*-quoted string, it has to stop at the first
- non-token character (you can put comments in). So
- ws_find_media_type_parameter() must return an allocated string with the
- actual value.
-
- Get rid of index_of_char(); it doesn't do anything that strchr() does.
-
- Change-Id: I36328ea71c28fe6ac4918a8e73c281a25f6be844
- Reviewed-on: https://code.wireshark.org/review/17251
- Reviewed-by: Guy Harris <guy@alum.mit.edu>
- (cherry picked from commit 0b3a96222b39ea4df1fdc899a950c50ec56a1077)
- Reviewed-on: https://code.wireshark.org/review/17252
+ Bug: 12801
+ Change-Id: Ic70f0d93bbffc99ceacf6b2963b04d7477e4914b
+ Reviewed-on: https://code.wireshark.org/review/17425
+ Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
+ Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl>
+ Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
+ Reviewed-by: Michael Mann <mmann78@netscape.net>
+ (cherry picked from commit cb2f8fc8914b89d7455a4a7f5e99b9de64f32556)
+ Reviewed-on: https://code.wireshark.org/review/17429
-commit d24bf49
+commit a075602
Author: Gerald Combs <gerald@wireshark.org>
-Date: Mon Aug 22 15:54:41 2016 -0700
+Date: Wed Aug 31 12:45:41 2016 -0700
- 2.2.0rc1 → 2.2.0rc2.
+ 2.2.0rc2 → 2.2.0rc3.
- Change-Id: I8272c2a8eb05fd5215999657728f56cf09950920
- Reviewed-on: https://code.wireshark.org/review/17248
+ Change-Id: I1c4f6a2857534aff6cfb01d7102edbd8140e759d
+ Reviewed-on: https://code.wireshark.org/review/17427
Reviewed-by: Gerald Combs <gerald@wireshark.org>
diff --git a/docbook/asciidoc.conf b/docbook/asciidoc.conf
index af2a8ebabc..5c1d36b640 100644
--- a/docbook/asciidoc.conf
+++ b/docbook/asciidoc.conf
@@ -2,7 +2,7 @@
[attributes]
-wireshark-version=2.2.0rc2
+wireshark-version=2.2.0
[replacements]
diff --git a/docbook/release-notes.asciidoc b/docbook/release-notes.asciidoc
index ab8c25dcf2..fad96c7422 100644
--- a/docbook/release-notes.asciidoc
+++ b/docbook/release-notes.asciidoc
@@ -1,8 +1,6 @@
= Wireshark {wireshark-version} Release Notes
// AsciiDoc quick reference: http://powerman.name/doc/asciidoc
-This is the third release candidate for Wireshark 2.2.0.
-
== What is Wireshark?
Wireshark is the world's most popular network protocol analyzer. It is
@@ -26,6 +24,11 @@ used for troubleshooting, analysis, development and education.
=== New and Updated Features
The following features are new (or have been significantly updated)
+since version 2.2.0rc2:
+
+* No major changes since 2.2.0rc2.
+
+The following features are new (or have been significantly updated)
since version 2.2.0rc1:
"Decode As" supports SSL (TLS) over TCP.
diff --git a/version.conf b/version.conf
index 9c98f42453..22b4e20962 100644
--- a/version.conf
+++ b/version.conf
@@ -1,7 +1,7 @@
# Interim releases: Enable packaging, add a "rc" to the version.
-enable: 1
-pkg_format: rc2-%#
-pkg_enable: 1
+#enable: 1
+#pkg_format: rc2-%#
+#pkg_enable: 1
# Release candidates
#enable: 1
@@ -9,6 +9,6 @@ pkg_enable: 1
#pkg_enable: 1
# Final release: Disable package version stamps.
-#enable: 1
-#pkg_format:
-#pkg_enable: 0
+enable: 1
+pkg_format:
+pkg_enable: 0