This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "wireshark with Osmocom specific dissectors". The branch, pespin/osmux-stats has been updated discards 0987ace7f58717e0da75b25adffa902b87fc10f5 (commit) discards 5c9660df9684dc762d5dd4f8d1b8350882288487 (commit) discards 22f6e11f617b4036a5a5ed955f1c7a7c63865db8 (commit) discards 103190de10caf79b18e38e0ed22d2dd2daf69121 (commit) via ea2231458a55fd730398aa3570236490358f68a5 (commit) via ad60fe0238a877cb29af36fd59cd872364b19bfb (commit) via 4814685f204e3d9019ee6bec73aca354d2b53bf2 (commit) via ba92095873914d2ec942a9b9363e2392f68d88e6 (commit) via f15b779d0b904526eb174893bc68c86eb08811d1 (commit) via 129bdb5a164a6386c35ff387e9d8f0d3d6a12dbf (commit) via 79eab8ca070f978415126f85b0777ab4ab02f0a2 (commit) via 871a8a2cccf79fae30c30a99fa21a4fa4dfbae16 (commit) via 804d0cdfa362a0243ee2f24b39af87539983c954 (commit) via 1ecf3b6645924f242b040fe518071ea62fc16e0a (commit) via 1be20bc4c241d82c3a8fd5617f22afb784f98b31 (commit) via f751918476bdde65f2289b86245a3c30dace6730 (commit) via d13b8ea13cf708a7770e7f51912ce16d0b43e459 (commit) via 537eddbc407d891430d9f4a2d462c6c3c0753307 (commit) via 559952b9030e6221dfccaf5ff2d1c85a24f2825f (commit) via 12057373d750909cd2770ebab76111f6eb1fb5f6 (commit) via fc1f62a88c37f56a532b683e1b0841b6ac1cc0f0 (commit) via a78ff1093c9a776a601eb873b63ac1f52806b40b (commit) via 5d45e77eefdf799db5aa4f3d9e75e4e4f53e5d1c (commit) via 5e944ee2a86145bf42db1386e4b497facfd3e0b6 (commit) via 8bab958e195a071bc5e5c016c631d6485e23fc2e (commit) via 9d6083efd16d7fffdeb52d6d9a91cc1bebe24598 (commit) via 3d9cadd392ff1fc3907fbe024b6f03c892abc240 (commit) via b8040a44ce2560f2d93007e066af4db8d4ef14f5 (commit) via c25b4f2f3fbec3e305638a90c93a90cdce931178 (commit) via 09209056ce0fd4ef0a4f5ac1ccc2fa1b12bba49c (commit) via 2d44f7f3035fd6b44b413d2f2a741686d8c190f3 (commit) via 0972db2f9d5a82d0545f3b22f03d4c8388fda354 (commit) via 5cd7ea6af225e8ed968c7db8cebad0fc8c0293af (commit) via 8c576051fd3edf3783eeb981a0c83a7f9e18a845 (commit) via 33f3866fb436ae104a5dc63cb2a3262574383421 (commit) via ea3b356eb2ed8afc9ad370db71067e9451bb4671 (commit) via 666366e069ab52308139048a70591284e97e2095 (commit) via a7b0674076cdf034cc44fe97572e3a8f6e0e837f (commit) This update added new revisions after undoing existing revisions. That is to say, the old revision is not a strict subset of the new revision. This situation occurs when you --force push a change and generate a repository containing something like this: * -- * -- B -- O -- O -- O (0987ace7f58717e0da75b25adffa902b87fc10f5) \ N -- N -- N (ea2231458a55fd730398aa3570236490358f68a5) When this happens we assume that you've already had alert emails for all of the O revisions, and so we here report only the revisions in the N branch from the common base, B. Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://cgit.osmocom.org/wireshark/commit/?id=ea2231458a55fd730398aa35702364… commit ea2231458a55fd730398aa3570236490358f68a5 Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Fri May 12 11:40:04 2017 +0200 gsm_osmux: Use proper sizes for structo smuxh fields Change-Id: I93de7ffdbd3c43494bc6a5dd1f44f6f45d6b54f8 http://cgit.osmocom.org/wireshark/commit/?id=ad60fe0238a877cb29af36fd59cd87… commit ad60fe0238a877cb29af36fd59cd872364b19bfb Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Wed May 10 20:24:56 2017 +0200 gsm_osmux: Add support to follow Osmux streams and show its statistics Change-Id: I5262b3b1ac5a6f5bc6ac932eedbb889847131d9c http://cgit.osmocom.org/wireshark/commit/?id=4814685f204e3d9019ee6bec73aca3… commit 4814685f204e3d9019ee6bec73aca354d2b53bf2 Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Tue May 9 17:50:48 2017 +0200 gsm_osmux: Support multiple OSmux frames per packet Also split and show all AMR payloads Change-Id: I5bde4e87dc51fd4a996a14117813ea89d7005d38 http://cgit.osmocom.org/wireshark/commit/?id=ba92095873914d2ec942a9b9363e23… commit ba92095873914d2ec942a9b9363e2392f68d88e6 Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Tue May 9 14:19:10 2017 +0200 gsm_osmux: Add RTP Marker field GSM Osmux recently got support to bring RTP Marker (M) bit. It should be mostly backwards compatible as the first bit was reserved and not used before for the FieldType. Support was added in libosmo-netif commit e98afe5808176efb60298a2f764e8e11efaf580b This bit is documented in the protocol documentation recently available: http://ftp.osmocom.org/docs/latest/osmux-reference.pdf Change-Id: Ia0508971519b3df9499d963404bb8a0e3c4b9c33 http://cgit.osmocom.org/wireshark/commit/?id=f15b779d0b904526eb174893bc68c8… commit f15b779d0b904526eb174893bc68c86eb08811d1 Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Tue May 9 14:04:18 2017 +0200 GTK+Qt: Show non-plugin stats_tree entries on the menu bar Change-Id: I565b0d2d43dd98781f77a302a20bd841e3e4650e http://cgit.osmocom.org/wireshark/commit/?id=129bdb5a164a6386c35ff387e9d8f0… commit 129bdb5a164a6386c35ff387e9d8f0d3d6a12dbf Author: Peter Wu <peter(a)lekensteyn.nl> Date: Fri May 5 11:46:07 2017 +0200 dns: improve loop detection in label decompression Previously the number of allowed pointers within a message is equal to the data in a tvb (16575 in one example). This is still expensive, so implement an alternative detection mechanism that looks for a direct self-loop and limits the total pointers to about 256. Bug: 13633 Change-Id: I803873e24ab170c7ef0b881d3bdc9dfd4014de97 Link: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=1206 Reviewed-on: https://code.wireshark.org/review/21507 Petri-Dish: Peter Wu <peter(a)lekensteyn.nl> Tested-by: Petri Dish Buildbot <buildbot-no-reply(a)wireshark.org> Reviewed-by: Martin Kaiser <wireshark(a)kaiser.cx> Reviewed-by: Peter Wu <peter(a)lekensteyn.nl> http://cgit.osmocom.org/wireshark/commit/?id=79eab8ca070f978415126f85b0777a… commit 79eab8ca070f978415126f85b0777ab4ab02f0a2 Author: Guy Harris <guy(a)alum.mit.edu> Date: Thu May 11 03:56:44 2017 -0700 Force HAVE_REMOTE to be defined when including pcap.h with remote support. WinPcap made the mistake of having stuff in its public header fines depend on a configuration #define, HAVE_REMOTE; this means that we need to forcibly define it when building with remote capture support. The tip of the libpcap master branch does not have that botch; hopefully future versions of libpcap-for-Windows will be based on that libpcap and thus lack that botch as well. Defining HAVE_REMOTE in config.h is not the right fix, as it makes it look like a *Wireshark* configuration option that code in Wireshark should test, rather than a *WinPcap* configuration option that the pcap.h that ships with the WinPcap SDK should have been changed, as part of the build process, to correctly define or not, so that users of WinPcap don't have to define it themselves. Change-Id: I62d1eca6d3c900d0dcc9fbc011db77f595a86313 Reviewed-on: https://code.wireshark.org/review/21593 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=871a8a2cccf79fae30c30a99fa21a4… commit 871a8a2cccf79fae30c30a99fa21a4fa4dfbae16 Author: Guy Harris <guy(a)alum.mit.edu> Date: Thu May 11 03:50:02 2017 -0700 Another shadowing fix for abort(). Change-Id: I991bd24ab8b64bdfeae275941d79b716da3309f9 Reviewed-on: https://code.wireshark.org/review/21592 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=804d0cdfa362a0243ee2f24b39af87… commit 804d0cdfa362a0243ee2f24b39af87539983c954 Author: Guy Harris <guy(a)alum.mit.edu> Date: Thu May 11 03:07:28 2017 -0700 Squelch warning from some compilers. Change-Id: Iacd5832e3b63fa24907423d2b7df40e81fafb0de Reviewed-on: https://code.wireshark.org/review/21591 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=1ecf3b6645924f242b040fe518071e… commit 1ecf3b6645924f242b040fe518071ea62fc16e0a Author: Guy Harris <guy(a)alum.mit.edu> Date: Thu May 11 02:47:58 2017 -0700 Note whether we found remote pcap support in libpcap. Change-Id: Ida20f7164d3132a72fdd6547905cd0af4e451917 Reviewed-on: https://code.wireshark.org/review/21590 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=1be20bc4c241d82c3a8fd5617f22af… commit 1be20bc4c241d82c3a8fd5617f22afb784f98b31 Author: Christoph Schlosser <christoph(a)schlosser.xyz> Date: Wed Apr 12 11:15:39 2017 +0200 EPL: Dissect read/write multiple parameter SDOs The dissector only dissected Write Multiple Parameter by Index requests correctly. Now it is possible to dissect the response to the request and Read Multiple Parameter by Index request and responses. Bug: 13677 Change-Id: I13aae241690cd3ac9dfbe8129ef56d0fc9016301 Reviewed-on: https://code.wireshark.org/review/21034 Reviewed-by: Christoph Schlosser <christoph(a)schlosser.xyz> Petri-Dish: Roland Knall <rknall(a)gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply(a)wireshark.org> Reviewed-by: Anders Broman <a.broman58(a)gmail.com> http://cgit.osmocom.org/wireshark/commit/?id=f751918476bdde65f2289b86245a3c… commit f751918476bdde65f2289b86245a3c30dace6730 Author: Pascal Quantin <pascal.quantin(a)gmail.com> Date: Wed May 10 18:10:42 2017 +0200 GSM RLC/MAC: fix global-buffer-overflow error reported by ASAN After a M_NEXT_EXIST(), you should always have another variable (like M_UINT, M_TYPE, ...) and not immediately end the structure with CSN_DESCR_END. Note that this is a workaround as the current description does not allow a proper parsing anyway (as the structure is reentrant). But at least it solves the buffer overflow. Bug: 13692 Change-Id: I8e607aadcced2d8cc636dc10f9690c5c679ad581 Reviewed-on: https://code.wireshark.org/review/21585 Petri-Dish: Pascal Quantin <pascal.quantin(a)gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply(a)wireshark.org> Reviewed-by: Anders Broman <a.broman58(a)gmail.com> http://cgit.osmocom.org/wireshark/commit/?id=d13b8ea13cf708a7770e7f51912ce1… commit d13b8ea13cf708a7770e7f51912ce16d0b43e459 Author: Guy Harris <guy(a)alum.mit.edu> Date: Wed May 10 14:15:55 2017 -0700 Get rid of HAVE_REMOTE. We never test for it - we test for HAVE_PCAP_REMOTE - so there's no need to set it in config.h. While we're at it, note that "PCAP" in "HAVE_PCAP_REMOTE" can refer to libpcap as well as WinPcap, given that, at least in the tip of the master branch, you can configure remote support in libpcap, although it's not enabled by default (it needs to be vetted for security, as it increases the attack surface of a machine running the server *and* of an application using libpcap with remote support). Change-Id: I3c96cf16bbda19ec7c085f74cffc6f125198d45b Reviewed-on: https://code.wireshark.org/review/21589 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=537eddbc407d891430d9f4a2d462c6… commit 537eddbc407d891430d9f4a2d462c6c3c0753307 Author: Guy Harris <guy(a)alum.mit.edu> Date: Wed May 10 13:46:58 2017 -0700 Define YYMALLOCARGTYPE appropriately. Define it, so the generated parser knows what argument type the function passed to MateParserAlloc() takes. Use it when declaring MateParserAlloc(). Change-Id: Ice18fd6b5fdbdb31f527e5d6eb06e78594d4565b Reviewed-on: https://code.wireshark.org/review/21588 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=559952b9030e6221dfccaf5ff2d1c8… commit 559952b9030e6221dfccaf5ff2d1c85a24f2825f Author: Guy Harris <guy(a)alum.mit.edu> Date: Wed May 10 13:39:17 2017 -0700 Clean up MateParser routine declarations. Remove the declarations from mate_parser.l, and have mate_parser.h include mate.h at the beginning of the file, instead. Move the #if'ed version of the declaration of MateParserAlloc() to mate.h. Change-Id: I03ffdd5f093b179ffc0cb0e43eac093f7e4af65c Reviewed-on: https://code.wireshark.org/review/21587 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=12057373d750909cd2770ebab76111… commit 12057373d750909cd2770ebab76111f6eb1fb5f6 Author: Guy Harris <guy(a)alum.mit.edu> Date: Tue May 9 18:42:05 2017 -0700 This is specified in section 8 of the DIX v2.0 Ethernet spec. Note that, and give a URL for the spec. Change-Id: I60953ddf2e7934ac273ba808ae4aa662f562917e Reviewed-on: https://code.wireshark.org/review/21583 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=fc1f62a88c37f56a532b683e1b0841… commit fc1f62a88c37f56a532b683e1b0841b6ac1cc0f0 Author: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Date: Tue May 9 19:28:18 2017 +0000 RADIUS: sync with FreeRADIUS commit d4e9cc75313f05678d0f2ac41962f46c99751e1f Author: Alan T. DeKok <aland(a)freeradius.org> Date: Sun May 7 12:56:57 2017 -0400 added one more attribute Change-Id: I5821076c2d4f02d82c393a768edd0c77da582a2d Reviewed-on: https://code.wireshark.org/review/21576 Reviewed-by: Michael Mann <mmann78(a)netscape.net> http://cgit.osmocom.org/wireshark/commit/?id=a78ff1093c9a776a601eb873b63ac1… commit a78ff1093c9a776a601eb873b63ac1f52806b40b Author: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Date: Tue May 9 18:56:33 2017 +0200 nge: add modelines info Change-Id: Ib75f637c36be575a85e75fee6282d1a16455bfd6 Reviewed-on: https://code.wireshark.org/review/21572 Petri-Dish: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Reviewed-by: Charlie Lenahan <clenahan(a)sonicbison.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply(a)wireshark.org> Reviewed-by: Michael Mann <mmann78(a)netscape.net> http://cgit.osmocom.org/wireshark/commit/?id=5d45e77eefdf799db5aa4f3d9e75e4… commit 5d45e77eefdf799db5aa4f3d9e75e4e4f53e5d1c Author: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Date: Tue May 9 18:56:01 2017 +0200 nge: fix no previous prototype for ‘proto_register_nge/handoff_nge’ [-Wmissing-prototypes] Change-Id: I2e7cecad96390f709c6a135b6059c01712971dfe Reviewed-on: https://code.wireshark.org/review/21571 Reviewed-by: Charlie Lenahan <clenahan(a)sonicbison.com> Reviewed-by: Michael Mann <mmann78(a)netscape.net> http://cgit.osmocom.org/wireshark/commit/?id=5e944ee2a86145bf42db1386e4b497… commit 5e944ee2a86145bf42db1386e4b497facfd3e0b6 Author: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Date: Tue May 9 20:05:10 2017 +0200 mate (grammar): fix no previous prototype for function 'MateParser/Alloc/Free' [-Wmissing-prototypes] Change-Id: I87ff11b7c04cb3b6963d4c8c16df2c3d60a0aec8 Reviewed-on: https://code.wireshark.org/review/21574 Petri-Dish: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply(a)wireshark.org> Reviewed-by: Michael Mann <mmann78(a)netscape.net> http://cgit.osmocom.org/wireshark/commit/?id=8bab958e195a071bc5e5c016c631d6… commit 8bab958e195a071bc5e5c016c631d6485e23fc2e Author: Guy Harris <guy(a)alum.mit.edu> Date: Tue May 9 15:07:26 2017 -0700 You can't have a field be both integer and string. For an inter-frame gap that couldn't be determined, just add it with proto_tree_add_uint_format_value() and "Cannot be determined" as the way the value is displayed. (But why add it *at all* if it couldn't be determined?) Change-Id: I491f9a6fe430141a15568e9a7dec3253a1c78cf5 Reviewed-on: https://code.wireshark.org/review/21582 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=9d6083efd16d7fffdeb52d6d9a91cc… commit 9d6083efd16d7fffdeb52d6d9a91cc1bebe24598 Author: Guy Harris <guy(a)alum.mit.edu> Date: Tue May 9 14:46:24 2017 -0700 Give fields better names than ".bitN". ".bitN" names 1) don't indicate what the bit actually *means* and 2) run the risk of collisions if the bit in question has a different meaning in different bitsets. Combine the "receive case" and "transmit case" bitsets in some cases where the only difference between the two sets is that some bits are present only in one set, but if a bit is present in both sets, it has the same meaning. Change-Id: Ie1cb9d076e431b30b64ef05acab39efe38193d33 Reviewed-on: https://code.wireshark.org/review/21581 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=3d9cadd392ff1fc3907fbe024b6f03… commit 3d9cadd392ff1fc3907fbe024b6f03c892abc240 Author: Guy Harris <guy(a)alum.mit.edu> Date: Tue May 9 14:39:47 2017 -0700 De-capitalize "error" - it's not capitalized in other field names. Change-Id: I6f5aaaba47ab8d36124aa10c63ceec7cf1f70867 Reviewed-on: https://code.wireshark.org/review/21580 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=b8040a44ce2560f2d93007e066af4d… commit b8040a44ce2560f2d93007e066af4db8d4ef14f5 Author: Guy Harris <guy(a)alum.mit.edu> Date: Tue May 9 14:28:57 2017 -0700 Fix various field names and types. Make some names used for multiple fields more stylistically consistent. If a bitfield has more than one bit, it's not Boolean - the extra bits mean it has more values than "true" or "false". If a bitfield has only one bit, and indicates that something is true/false, or enabled/disabled, or present/absent, or on/off, or something else obviously Boolean, it's Boolean. Change-Id: I89d5e214eb5c4e7f72eb59a78757f0f49fec3dbf Reviewed-on: https://code.wireshark.org/review/21579 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=c25b4f2f3fbec3e305638a90c93a90… commit c25b4f2f3fbec3e305638a90c93a90cdce931178 Author: Björn Ruytenberg <bjorn(a)bjornweb.nl> Date: Tue May 9 20:04:27 2017 +0200 debian-setup: Always install Libgcrypt Libgcrypt is mandatory to build. Do not mark as optional. Change-Id: Iafbe1cd93f4185e2a9492bb0a21c30a26d8883ce Reviewed-on: https://code.wireshark.org/review/21575 Reviewed-by: Peter Wu <peter(a)lekensteyn.nl> http://cgit.osmocom.org/wireshark/commit/?id=09209056ce0fd4ef0a4f5ac1ccc2fa… commit 09209056ce0fd4ef0a4f5ac1ccc2fa1b12bba49c Author: Guy Harris <guy(a)alum.mit.edu> Date: Tue May 9 10:51:01 2017 -0700 Set nss and rate_mcs_index in more arms of the Series II PLCP switch. For HT mixed, set it the same way it's set for HT greenfield. For pre-HT, set it to 0. Also, for the "unknown" case, set rate_mcs_index to 0. This should obviate the need to initialize either of those variables, don't initialize them, so that failing to set them in an arm of the switch statement shows up as an error if the compiler's dataflow analysis actually bothers to check this. Change-Id: I92703770dd5000a579b53609fb93a2085fd9fca3 Reviewed-on: https://code.wireshark.org/review/21573 Reviewed-by: Guy Harris <guy(a)alum.mit.edu> http://cgit.osmocom.org/wireshark/commit/?id=2d44f7f3035fd6b44b413d2f2a7416… commit 2d44f7f3035fd6b44b413d2f2a741686d8c190f3 Author: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Date: Tue May 9 14:08:40 2017 +0200 ua3g: fix conflicting entry in its value_string Field 'Audio Env.' (ua3g.ip.set_param_req.parameter.audio_env) has a conflicting entry in its value_string: 2 is at indices 2 (Open Space) and 3 (Meeting Room)) Field 'Audio Env.' (ua3g.ip.set_param_req.parameter.audio_env) has a conflicting entry in its value_string: 2 is at indices 2 (Open Space) and 4 (Noisy)) Field 'Audio Env.' (ua3g.ip.set_param_req.parameter.audio_env) has a conflicting entry in its value_string: 2 is at indices 3 (Meeting Room) and 4 (Noisy)) Change-Id: I495f183dd52e59ec64e6eea0df15a5b243d8b87c Reviewed-on: https://code.wireshark.org/review/21568 Reviewed-by: Nicolas BERTIN <nicolas.bertin(a)al-enterprise.com> Reviewed-by: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Petri-Dish: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply(a)wireshark.org> Reviewed-by: Michael Mann <mmann78(a)netscape.net> http://cgit.osmocom.org/wireshark/commit/?id=0972db2f9d5a82d0545f3b22f03d4c… commit 0972db2f9d5a82d0545f3b22f03d4c8388fda354 Author: Adrian Simionov <daniel.simionov(a)gmail.com> Date: Tue May 9 23:16:16 2017 +1000 docsis: DCC-REQ TLV 2.7 support Change-Id: I40724306d5facd0f4a5a9ca2354330577857d27f Reviewed-on: https://code.wireshark.org/review/21563 Petri-Dish: Michael Mann <mmann78(a)netscape.net> Tested-by: Petri Dish Buildbot <buildbot-no-reply(a)wireshark.org> Reviewed-by: Michael Mann <mmann78(a)netscape.net> http://cgit.osmocom.org/wireshark/commit/?id=5cd7ea6af225e8ed968c7db8cebad0… commit 5cd7ea6af225e8ed968c7db8cebad0fc8c0293af Author: Michael Mann <mmann78(a)netscape.net> Date: Tue May 9 10:11:27 2017 -0400 Don't have epan_dissect_init() return anything. It manipulates the epan_dissect_t structure passed into it and then returns that. Callers can (and have been) just using the passed in epan_dissect_t structure anyway. Change-Id: Ia19d360a7347ff473654eeb553756f59a38f95bd Reviewed-on: https://code.wireshark.org/review/21570 Petri-Dish: Michael Mann <mmann78(a)netscape.net> Reviewed-by: Jaap Keuter <jaap.keuter(a)xs4all.nl> Tested-by: Petri Dish Buildbot <buildbot-no-reply(a)wireshark.org> Reviewed-by: Michael Mann <mmann78(a)netscape.net> http://cgit.osmocom.org/wireshark/commit/?id=8c576051fd3edf3783eeb981a0c83a… commit 8c576051fd3edf3783eeb981a0c83a7f9e18a845 Author: Graham Bloice <graham.bloice(a)trihedral.com> Date: Tue May 9 13:19:55 2017 +0100 More fields, more pre-allocated memory for fields. Change-Id: I36469fad59d28a1fbf15d3fd99544a3a6b0ef015 Reviewed-on: https://code.wireshark.org/review/21569 Petri-Dish: Graham Bloice <graham.bloice(a)trihedral.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply(a)wireshark.org> Reviewed-by: Graham Bloice <graham.bloice(a)trihedral.com> http://cgit.osmocom.org/wireshark/commit/?id=33f3866fb436ae104a5dc63cb2a326… commit 33f3866fb436ae104a5dc63cb2a3262574383421 Author: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Date: Tue May 9 11:36:29 2017 +0200 tshark: Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang Change-Id: I4425be3bbee0e781dff59a07d7de65f5eba81bdd Reviewed-on: https://code.wireshark.org/review/21564 Reviewed-by: Michael Mann <mmann78(a)netscape.net> http://cgit.osmocom.org/wireshark/commit/?id=ea3b356eb2ed8afc9ad370db71067e… commit ea3b356eb2ed8afc9ad370db71067e9451bb4671 Author: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Date: Tue May 9 14:42:18 2017 +0200 nm: Uninitialized argument value Warning found by Clang Analyzer false positive Change-Id: I8aa2537328a04a64b06142a04ee76705d0df5443 Reviewed-on: https://code.wireshark.org/review/21566 Reviewed-by: Michael Mann <mmann78(a)netscape.net> http://cgit.osmocom.org/wireshark/commit/?id=666366e069ab52308139048a705912… commit 666366e069ab52308139048a70591284e97e2095 Author: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Date: Tue May 9 11:41:32 2017 +0200 vwr (wiretap): fix Assigned value is garbage or undefined Warning found by Clang Analyzer Change-Id: I45766c7e89a009176ae56fe9e9f5a8dd63b2995b Reviewed-on: https://code.wireshark.org/review/21567 Petri-Dish: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Tested-by: Petri Dish Buildbot <buildbot-no-reply(a)wireshark.org> Reviewed-by: Anders Broman <a.broman58(a)gmail.com> http://cgit.osmocom.org/wireshark/commit/?id=a7b0674076cdf034cc44fe97572e3a… commit a7b0674076cdf034cc44fe97572e3a8f6e0e837f Author: Alexis La Goutte <alexis.lagoutte(a)gmail.com> Date: Tue May 9 11:37:02 2017 +0200 nm: Fix Dead Store (Dead assignement/Dead increment) Warning found by Clang Change-Id: I7592737477e105870ef43118f40c0ab552c81626 Reviewed-on: https://code.wireshark.org/review/21565 Reviewed-by: Anders Broman <a.broman58(a)gmail.com> ----------------------------------------------------------------------- Summary of changes: acinclude.m4 | 4 +- caputils/capture-pcap-util-unix.c | 18 + caputils/capture-pcap-util.h | 12 + caputils/capture_wpcap_packet.c | 12 + cmake/modules/FindPCAP.cmake | 1 - cmakeconfig.h.in | 6 +- epan/dissectors/Makefile.am | 1 - epan/dissectors/packet-dns.c | 16 +- epan/dissectors/packet-epl.c | 453 +++++++++++- epan/dissectors/packet-gsm_osmux.c | 94 ++- epan/dissectors/packet-gsm_osmux.h | 38 - epan/dissectors/packet-gsm_rlcmac.c | 5 +- epan/dissectors/packet-ixveriwave.c | 1247 +++++++++++++++----------------- epan/dissectors/packet-loop.c | 10 +- epan/dissectors/packet-netgear.c | 16 + epan/dissectors/packet-ua3g.c | 4 +- epan/dissectors/packet-udp-nm.c | 4 +- epan/epan.c | 7 +- epan/epan.h | 2 +- epan/proto.c | 2 +- extcap/androiddump.c | 12 + plugins/docsis/packet-dccreq.c | 23 + plugins/mate/mate.h | 10 + plugins/mate/mate_parser.l | 10 +- radius/dictionary.zte | 1 + ringbuffer.c | 12 + tools/debian-setup.sh | 4 +- trigcap.c | 13 + tshark.c | 2 +- ui/gtk/airpcap_dlg.c | 12 + ui/gtk/main_menubar.c | 34 +- ui/qt/capture_filter_syntax_worker.cpp | 14 + ui/qt/compiled_filter_output.cpp | 12 + ui/qt/main_window.cpp | 2 +- ui/qt/main_window.h | 3 +- ui/qt/main_window_slots.cpp | 9 +- wiretap/vwr.c | 6 +- 37 files changed, 1332 insertions(+), 799 deletions(-) delete mode 100644 epan/dissectors/packet-gsm_osmux.h hooks/post-receive -- wireshark with Osmocom specific dissectors