[Wireshark-commits] master-2.0 b5fa164: OSSP: Retrieve OUI at proper offset in packet

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=b5fa16424280bf5f2df1b0c116ae607b93f833a3
Submitter: Jaap Keuter (jaap.keu...@xs4all.nl)
Changed: branch: master-2.0
Repository: wireshark

Commits:

b5fa164 by Jaap Keuter (jaap.keu...@xs4all.nl):

OSSP: Retrieve OUI at proper offset in packet

Bug: 12801
Change-Id: Ic70f0d93bbffc99ceacf6b2963b04d7477e4914b
Reviewed-on: https://code.wireshark.org/review/17425
Reviewed-by: Jaap Keuter 
Petri-Dish: Jaap Keuter 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Michael Mann 
(cherry picked from commit cb2f8fc8914b89d7455a4a7f5e99b9de64f32556)
Reviewed-on: https://code.wireshark.org/review/17429
(cherry picked from commit 27554219b6db4bd2beb5717aacbcb4aaf5330f37)
Reviewed-on: https://code.wireshark.org/review/17430
Petri-Dish: Michael Mann 


Actions performed:

from  2daf80b   Don't use a no-longer-extant variable.
adds  b5fa164   OSSP: Retrieve OUI at proper offset in packet


Summary of changes:
 epan/dissectors/packet-ossp.c |8 
 1 file changed, 4 insertions(+), 4 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master-2.2 60649ac: erf: fix build against older glib

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=60649ac3a4723f7d22c809a6c7cdd85ebfef94a6
Submitter: Peter Wu (pe...@lekensteyn.nl)
Changed: branch: master-2.2
Repository: wireshark

Commits:

60649ac by Peter Wu (pe...@lekensteyn.nl):

erf: fix build against older glib

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: Ie89432a6cd688b89dd346ce5cdecf96a76e92692
Reviewed-on: https://code.wireshark.org/review/17395
Reviewed-by: João Valverde 
Reviewed-by: Peter Wu 


Actions performed:

from  2755421   OSSP: Retrieve OUI at proper offset in packet
adds  60649ac   erf: fix build against older glib


Summary of changes:
 epan/dissectors/packet-erf.c |1 +
 1 file changed, 1 insertion(+)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master 717215d: E.212: update list to Operational Bulletin No. 1106 (15.VIII.2016)

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=717215df61f7fc4daf596c3cd07b1f4e19b38700
Submitter: Pascal Quantin (pascal.quan...@gmail.com)
Changed: branch: master
Repository: wireshark

Commits:

717215d by Pascal Quantin (pascal.quan...@gmail.com):

E.212: update list to Operational Bulletin No. 1106 (15.VIII.2016)

Also manually add PLMN 460 02 (Chian Mobile) as it is not listed by ITU yet

Bug: 12622
Bug: 12798
Change-Id: I7c6fab9dcb9da90178186e94f624301ef1861421
Reviewed-on: https://code.wireshark.org/review/17428
Petri-Dish: Pascal Quantin 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Pascal Quantin 


Actions performed:

from  cb2f8fc   OSSP: Retrieve OUI at proper offset in packet
adds  717215d   E.212: update list to Operational Bulletin No. 1106 
(15.VIII.2016)


Summary of changes:
 epan/dissectors/packet-e212.c |  424 -
 1 file changed, 289 insertions(+), 135 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] buildbot failure in Wireshark (development) on OSX 10.6 x64

[buildbot-no-reply]

The Buildbot has detected a new failure on builder OSX 10.6 x64 while building 
wireshark. Full details are available at:

http://buildbot.wireshark.org/wireshark-master/builders/OSX%2010.6%20x64/builds/11915

Buildbot URL: http://buildbot.wireshark.org/wireshark-master/

Buildslave for this Build: osx-10.6-x64

Build Reason: The SingleBranchScheduler scheduler named 'Gerrit' triggered this 
build
Build Source Stamp: [branch master] cb2f8fc8914b89d7455a4a7f5e99b9de64f32556
Blamelist: Michael Mann 

BUILD FAILED: failed

Sincerely,
 -The Buildbot



___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master-2.2 2755421: OSSP: Retrieve OUI at proper offset in packet

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=27554219b6db4bd2beb5717aacbcb4aaf5330f37
Submitter: Michael Mann (mman...@netscape.net)
Changed: branch: master-2.2
Repository: wireshark

Commits:

2755421 by Jaap Keuter (jaap.keu...@xs4all.nl):

OSSP: Retrieve OUI at proper offset in packet

Bug: 12801
Change-Id: Ic70f0d93bbffc99ceacf6b2963b04d7477e4914b
Reviewed-on: https://code.wireshark.org/review/17425
Reviewed-by: Jaap Keuter 
Petri-Dish: Jaap Keuter 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Michael Mann 
(cherry picked from commit cb2f8fc8914b89d7455a4a7f5e99b9de64f32556)
Reviewed-on: https://code.wireshark.org/review/17429


Actions performed:

from  a075602   2.2.0rc2 → 2.2.0rc3.
adds  2755421   OSSP: Retrieve OUI at proper offset in packet


Summary of changes:
 epan/dissectors/packet-ossp.c |8 
 1 file changed, 4 insertions(+), 4 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master cb2f8fc: OSSP: Retrieve OUI at proper offset in packet

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cb2f8fc8914b89d7455a4a7f5e99b9de64f32556
Submitter: Michael Mann (mman...@netscape.net)
Changed: branch: master
Repository: wireshark

Commits:

cb2f8fc by Jaap Keuter (jaap.keu...@xs4all.nl):

OSSP: Retrieve OUI at proper offset in packet

Bug: 12801
Change-Id: Ic70f0d93bbffc99ceacf6b2963b04d7477e4914b
Reviewed-on: https://code.wireshark.org/review/17425
Reviewed-by: Jaap Keuter 
Petri-Dish: Jaap Keuter 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Michael Mann 


Actions performed:

from  ce2fa7b   NFS: fix indent (remove some trailing spaces)
adds  cb2f8fc   OSSP: Retrieve OUI at proper offset in packet


Summary of changes:
 epan/dissectors/packet-ossp.c |8 
 1 file changed, 4 insertions(+), 4 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master-2.2 a075602: 2.2.0rc2 → 2.2.0rc3.

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a07560271a3e21ec374f55ba4ee02cb1cbab43b9
Submitter: Gerald Combs (ger...@wireshark.org)
Changed: branch: master-2.2
Repository: wireshark

Commits:

a075602 by Gerald Combs (ger...@wireshark.org):

2.2.0rc2 → 2.2.0rc3.

Change-Id: I1c4f6a2857534aff6cfb01d7102edbd8140e759d
Reviewed-on: https://code.wireshark.org/review/17427
Reviewed-by: Gerald Combs 


Actions performed:

from  7670a27   Build 2.2.0rc2.
adds  a075602   2.2.0rc2 → 2.2.0rc3.


Summary of changes:
 docbook/release-notes.asciidoc |2 +-
 version.conf   |   12 ++--
 2 files changed, 7 insertions(+), 7 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] [wireshark] annotated tag wireshark-2.2.0rc2 created (now e705462)

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e70546294bdf4be6fc31cf9a10dd0ee827f01f29
Submitter: Gerald Combs (ger...@wireshark.org)
Changed: annotated tag: wireshark-2.2.0rc2
Repository: wireshark

Commits:

[ log message not found ]


Actions performed:

  at  e705462   (tag)
 tagging  7670a2725da9942f8fef2ec695de25c4e8f06b2d (commit)
  replaces  wireshark-2.2.0rc1
 tagged by  Gerald Combs
on  Wed Aug 31 10:32:37 2016 -0700

- Log -
Build 2.2.0rc2.
-BEGIN PGP SIGNATURE-
Version: GnuPG v2

iQIcBAABCgAGBQJXxxS5AAoJEIIkSnjm/q7q0hEQAN0I3Vsx41hzT5cKldpd5kQm
8gftdi+nZ8H3fO6CfiQr2fEQn0tHTRbiCaFT1I0WTGC9E1xCVAhJdwQ+FpjtC52K
YzJo5hXLvmrqy0yE3usnMkC3FhTpSz9QMY6RmYnA735RKuCm+MQrFskYc+ZTXBOf
5RhLW6DowdV35FQwxrID5Zy35XxpRNtdUpiaAmv3JtNXu6wAfBWCTi4K9TMrJd+2
01obGIdVpVIemM0IqHaFO9zs9M2J/hwTJhNIecDqqWIK0jsBUGKbRvCHaHgCrukk
7734QorSnToUKsIIyGqec80LlLTAcLbmW8szUgTY5of3tc35F8rZ0k5/nUyInkxt
gjDRhceURtoftbvb+VIv3iCcvM29Pixt5l048xGWPOPJixdO/SRtOmyXCk2SHKj2
cN6lmG6CbjNyUb/BuB07Gr9jhX/tDKFO9rqAk51jKD/DWZMK1Ae8zPCA1f1s4Imw
Ap2fycHic9QsvW+DPhlA7YxEupT6nyY9Xt+wQ14M1LycDeOwJBMBVjWT97DxY2vd
ejtI02fMcQSBJooJ4mbR8CwSxV4CY4VQcHUrnXLvDz9KF7RR6FtnOlCXkPhgUpRr
kl+1S64LiQETvecPjOaEAzikK6+Y9jbGmu/r681qT67mNsqyglhX43R2saN2XefJ
1dBXodTziNSCnDBOtxsd
=ilTN
-END PGP SIGNATURE-
---

___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] [wireshark] annotated tag v2.2.0rc2 created (now 43f95f3)

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=43f95f38a5f042acff9ec9f41a2ea0cd3c9b26de
Submitter: Gerald Combs (ger...@wireshark.org)
Changed: annotated tag: v2.2.0rc2
Repository: wireshark

Commits:

[ log message not found ]


Actions performed:

  at  43f95f3   (tag)
 tagging  7670a2725da9942f8fef2ec695de25c4e8f06b2d (commit)
  replaces  wireshark-2.2.0rc1
 tagged by  Gerald Combs
on  Wed Aug 31 10:32:17 2016 -0700

- Log -
Build 2.2.0rc2.
-BEGIN PGP SIGNATURE-
Version: GnuPG v2

iQIcBAABCgAGBQJXxxSsAAoJEIIkSnjm/q7qfxcP/iX0H7rMcCKDTWlmZbGLlAr/
RJghme4UFv2rqSGcd4WVLHkcrtANaMK3MU4CLiS/LAE7XWUAfUn22vM3nL9gapwQ
SNaHMpRVdwlxe0qLyxYZH+QFR/urzU+bdkTstAIBEG4ajOqwF3lty/37iU+KH8W4
OpBXMVCOp2c2PpjEpX7FPVEJ51a5TMcDI96TBvtD03rJECr0q82WY246/qlIr1rQ
cah+6KGgEeCW2iBZsrlKy7nmEoEYzBvY2oIPVN3GbgHbjQjQpWR3Wa9gaECMrmhQ
iRpmC+qdk5cb0EKv3urHYwsnV3y3ub5vWOIayFdF7+UsSY+UmuF5ec1/0xSwbSlW
3uQcTSG1c65fU5AabUjIXaaLqXdy3dRuKibHfyVOfkGPHd79e083y6w7TwaduLi0
94tNmPWejW2GYTOQQz4E+6j1pbCjcvQete8Lw2TqGtS29INyJ2jxhyg1jsnDkk8N
vBBlmkTC50IHIlebGtjeM7HfcZtUvJOTYyx+g9maXAEgZmw15YQc93o3IEJM1dXq
Xyldr/BuliUW8muXm2+SsIjBbRrPDnaV2XIcvgHh2X8GRCPEJ8QAxsm0Ef/d0ITL
vDBc6eZlYM9XJyzPUpAksB3V+cw6zkLCeAGd2s+VVFdRcyhKUNZHVUx/4US1vNu7
xIx7ujsm4HHZrpaXxGiQ
=RE5B
-END PGP SIGNATURE-
---

___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master-2.2 7670a27: Build 2.2.0rc2.

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=7670a2725da9942f8fef2ec695de25c4e8f06b2d
Submitter: Gerald Combs (ger...@wireshark.org)
Changed: branch: master-2.2
Repository: wireshark

Commits:

7670a27 by Gerald Combs (ger...@wireshark.org):

Build 2.2.0rc2.

Change-Id: I7b6db7384303aaa41fc164a20da1f634b1d3cbff
Reviewed-on: https://code.wireshark.org/review/17424
Reviewed-by: Gerald Combs 


Actions performed:

from  987fb23   autotools: really disable warnings by default
adds  7670a27   Build 2.2.0rc2.


Summary of changes:
 ChangeLog  | 3749 +++-
 NEWS   |  104 +-
 docbook/asciidoc.conf  |2 +-
 docbook/release-notes.asciidoc |   14 +-
 version.conf   |   12 +-
 5 files changed, 676 insertions(+), 3205 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master ce2fa7b: NFS: fix indent (remove some trailing spaces)

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ce2fa7b30a741929a2d502eaf390c3f238bbe6b9
Submitter: Anders Broman (a.broma...@gmail.com)
Changed: branch: master
Repository: wireshark

Commits:

ce2fa7b by Alexis La Goutte (alexis.lagou...@gmail.com):

NFS: fix indent (remove some trailing spaces)

Change-Id: Ic72d264686c3b37ac15118eec07057c7bb8a2a50
Reviewed-on: https://code.wireshark.org/review/17422
Petri-Dish: Alexis La Goutte 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Anders Broman 


Actions performed:

from  5c2d82a   diam_dict.l: remove use of strdup
adds  ce2fa7b   NFS: fix indent (remove some trailing spaces)


Summary of changes:
 epan/dissectors/packet-nfs.c |   40 
 1 file changed, 20 insertions(+), 20 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master 5c2d82a: diam_dict.l: remove use of strdup

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=5c2d82a6c24a06da0132fb7bd493ce4e58594f0d
Submitter: Guy Harris (g...@alum.mit.edu)
Changed: branch: master
Repository: wireshark

Commits:

5c2d82a by Peter Wu (pe...@lekensteyn.nl):

diam_dict.l: remove use of strdup

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: I34111385c82715de70fb42fe44b99b89e132a374
Reviewed-on: https://code.wireshark.org/review/17423
Reviewed-by: Guy Harris 


Actions performed:

from  a7dd070   Win: switch back to Lua library compiled with MinGW
adds  5c2d82a   diam_dict.l: remove use of strdup


Summary of changes:
 epan/diam_dict.l |   22 ++
 1 file changed, 6 insertions(+), 16 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master-2.2 987fb23: autotools: really disable warnings by default

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=987fb2347f786cc0f22b876753fdcd4a95912212
Submitter: Gerald Combs (ger...@wireshark.org)
Changed: branch: master-2.2
Repository: wireshark

Commits:

987fb23 by Peter Wu (pe...@lekensteyn.nl):

autotools: really disable warnings by default

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.

Change-Id: Ib12f5762e9b30a681b193014ed78dc73bc794b0f
Reviewed-on: https://code.wireshark.org/review/17420
Reviewed-by: Gerald Combs 


Actions performed:

from  50c51c8   ipv6: fix build with old 32-bit glib headers
adds  987fb23   autotools: really disable warnings by default


Summary of changes:
 configure.ac |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master a7dd070: Win: switch back to Lua library compiled with MinGW

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a7dd070d55e41e76da5a3c595d3f547c8cf3837b
Submitter: Pascal Quantin (pascal.quan...@gmail.com)
Changed: branch: master
Repository: wireshark

Commits:

a7dd070 by Pascal Quantin (pascal.quan...@gmail.com):

Win: switch back to Lua library compiled with MinGW

It does not suffer anymore from bug 9957

Change-Id: I871f01db67101b09a21545ecec5473941997a5cb
Reviewed-on: https://code.wireshark.org/review/17416
Petri-Dish: Pascal Quantin 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Pascal Quantin 


Actions performed:

from  bccbae5   rftap: add new dissector
adds  a7dd070   Win: switch back to Lua library compiled with MinGW


Summary of changes:
 tools/win-setup.ps1 |   16 
 1 file changed, 4 insertions(+), 12 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master bccbae5: rftap: add new dissector

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=bccbae58a14c2cf635f5fd3c363b2c355215e0f9
Submitter: Michael Mann (mman...@netscape.net)
Changed: branch: master
Repository: wireshark

Commits:

bccbae5 by Jonathan Brucker (jonathan.bru...@gmail.com):

rftap: add new dissector

The RFtap protocol is a simple metadata header designed to provide
Radio Frequency (RF) metadata about frames.

For official specifications see: https://rftap.github.io/

Signed-off-by: Jonathan Brucker 
Change-Id: I0d008b2baadcc5cc9577113e9795eef2691b961a
Reviewed-on: https://code.wireshark.org/review/17355
Petri-Dish: Alexis La Goutte 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Alexis La Goutte 
Reviewed-by: Michael Mann 


Actions performed:

from  6362183   Revert "Revert "diameter: fix 400kb leaked memory on exit""
adds  bccbae5   rftap: add new dissector


Summary of changes:
 docbook/release-notes.asciidoc |1 +
 epan/dissectors/CMakeLists.txt |1 +
 epan/dissectors/Makefile.am|1 +
 epan/dissectors/packet-rftap.c |  608 
 4 files changed, 611 insertions(+)
 create mode 100644 epan/dissectors/packet-rftap.c
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] buildbot failure in Wireshark (development) on Windows Server 2012 R2 x64

[buildbot-no-reply]

The Buildbot has detected a new failure on builder Windows Server 2012 R2 x64 
while building wireshark. Full details are available at:

http://buildbot.wireshark.org/wireshark-master/builders/Windows%20Server%202012%20R2%20x64/builds/1544

Buildbot URL: http://buildbot.wireshark.org/wireshark-master/

Buildslave for this Build: windows-2012r2-x64

Build Reason: The SingleBranchScheduler scheduler named 'Gerrit' triggered this 
build
Build Source Stamp: [branch master] 6362183825cb4a12fd091b78d88619606dec4554
Blamelist: Guy Harris 

BUILD FAILED: failed compile_5 test.sh

Sincerely,
 -The Buildbot



___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master 6362183: Revert "Revert "diameter: fix 400kb leaked memory on exit""

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=6362183825cb4a12fd091b78d88619606dec4554
Submitter: Guy Harris (g...@alum.mit.edu)
Changed: branch: master
Repository: wireshark

Commits:

6362183 by Guy Harris (g...@alum.mit.edu):

Revert "Revert "diameter: fix 400kb leaked memory on exit""

This reverts commit 5fea2b5f4198f1a36f313ef38532ddffd02ac5b1.

I.e., it puts back the change; the reverted version passed the tests on 
which the versions with this change crashed.

Change-Id: Idcc0eb11588cf14e2fe666de1905ee63917b0fcf
Reviewed-on: https://code.wireshark.org/review/17413
Reviewed-by: Guy Harris 


Actions performed:

from  5fea2b5   Revert "diameter: fix 400kb leaked memory on exit"
adds  6362183   Revert "Revert "diameter: fix 400kb leaked memory on exit""


Summary of changes:
 epan/diam_dict.l  |   35 ---
 epan/dissectors/packet-diameter.c |   12 ++--
 2 files changed, 38 insertions(+), 9 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master 5fea2b5: Revert "diameter: fix 400kb leaked memory on exit"

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=5fea2b5f4198f1a36f313ef38532ddffd02ac5b1
Submitter: Guy Harris (g...@alum.mit.edu)
Changed: branch: master
Repository: wireshark

Commits:

5fea2b5 by Guy Harris (g...@alum.mit.edu):

Revert "diameter: fix 400kb leaked memory on exit"

This reverts commit a04b6fcb3db901734ed948134c973996786be8b7.

Temporary revert to see if this prevents the "tshark -G" crashes being seen 
on the 64-bit Windows buildbot.

Change-Id: I561439039ca2667b72d7e2319a6f3f5f97e18d15
Reviewed-on: https://code.wireshark.org/review/17412
Reviewed-by: Guy Harris 


Actions performed:

from  c8869f6   OK, it probably crashed in proto_initialize_all_prefixes().
adds  5fea2b5   Revert "diameter: fix 400kb leaked memory on exit"


Summary of changes:
 epan/diam_dict.l  |   35 +++
 epan/dissectors/packet-diameter.c |   12 ++--
 2 files changed, 9 insertions(+), 38 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master c8869f6: OK, it probably crashed in proto_initialize_all_prefixes().

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c8869f68db6cd1d4a341d57600ecf22627632c9f
Submitter: Guy Harris (g...@alum.mit.edu)
Changed: branch: master
Repository: wireshark

Commits:

c8869f6 by Guy Harris (g...@alum.mit.edu):

OK, it probably crashed in proto_initialize_all_prefixes().

Remove the debugging printouts.

The changes that were committed between the last build that didn't crash
and the first build that did were:

commit 961f743d69b323aa217a6b39184485d6a0bfb2d5
Author: Peter Wu 
Date:   Mon Aug 29 01:34:22 2016 +0200

xml: fix some memleaks

No more memleaks reported for the attachment in bug 12790 :-)

Change-Id: I8472e442143b332edfacdf9ef3b8b893f1ec4386
Ping-Bug: 12790
Reviewed-on: https://code.wireshark.org/review/17365
Reviewed-by: Michael Mann 
Petri-Dish: Michael Mann 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Peter Wu 

commit a04b6fcb3db901734ed948134c973996786be8b7
Author: Peter Wu 
Date:   Sun Aug 28 22:19:29 2016 +0200

diameter: fix 400kb leaked memory on exit

Before:

SUMMARY: AddressSanitizer: 399684 byte(s) leaked in 17208 
allocation(s).

After addressing to-do by calling ddict_free:

SUMMARY: AddressSanitizer: 3024 byte(s) leaked in 256 allocation(s).

After fixing all remaining leaks cases in the flex file for diameter:

SUMMARY: AddressSanitizer: 735 byte(s) leaked in 58 allocation(s).

Not bad huh :-)

Ping-Bug: 12790
Change-Id: I0c730ad77ae15c69390bc6cf0a3a985395a64771
Reviewed-on: https://code.wireshark.org/review/17364
Petri-Dish: Peter Wu 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Michael Mann 
Reviewed-by: Jeff Morriss 
Reviewed-by: Peter Wu 

commit 14312835c63a3e2ec9d311ed1ffee5285141f4f9
Author: Peter Wu 
Date:   Sun Aug 28 19:20:59 2016 +0200

pcapng: do not leak blocks

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.

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).

Ping-Bug: 12790
Change-Id: I081f841addb36f16e3671095a919d357f4bc16c5
Reviewed-on: https://code.wireshark.org/review/17362
Petri-Dish: Peter Wu 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Peter Wu 

*If* one of those is the cause, my guess is that it's the Diameter one,
as the crash happens before any file is read (so it's probably not the
pcapng one) and thus before any dissection is done (so it's probably not
the XML dissector one).

Change-Id: I816c1bbd6078eab251efd02ebb7c3195f6dd1483
Reviewed-on: https://code.wireshark.org/review/17411
Reviewed-by: Guy Harris 


Actions performed:

from  1178ee6   GTP: patch for Target identification for non-compliance 
workaround
adds  c8869f6   OK, it probably crashed in proto_initialize_all_prefixes().


Summary of changes:
 epan/proto.c |2 --
 1 file changed, 2 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] wireshark-win64-libs rev 449: /tags/2016-08-31/

[pascal]

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev=Wireshark-win64-libs=449

User: pascal
Date: 2016/08/31 07:14 AM

Log:
 Add Lua 5.2.4 generated with MinGW

___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] wireshark-win64-libs rev 448: / /trunk/packages/: lua-5.2.4_Win64_dllw4_lib.zip /trunk/: README.txt

[pascal]

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev=Wireshark-win64-libs=448

User: pascal
Date: 2016/08/31 07:12 AM

Log:
 Add Lua 5.2.4 generated with MinGW

Directory: /trunk/packages/
  ChangesPath Action
  +0 -0  lua-5.2.4_Win64_dllw4_lib.zipAdded

Directory: /trunk/
  ChangesPath  Action
  +7 -0  README.txtModified

___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] wireshark-win32-libs rev 496: /tags/2016-08-31/

[pascal]

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev=Wireshark-win32-libs=496

User: pascal
Date: 2016/08/31 07:08 AM

Log:
 Add Lua 5.2.4 generated with MinGW

___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] wireshark-win32-libs rev 495: / /trunk/packages/: lua-5.2.4_Win32_dllw4_lib.zip /trunk/: README.txt

[pascal]

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev=Wireshark-win32-libs=495

User: pascal
Date: 2016/08/31 07:06 AM

Log:
 Add Lua 5.2.4 generated with MinGW

Directory: /trunk/packages/
  ChangesPath Action
  +0 -0  lua-5.2.4_Win32_dllw4_lib.zipAdded

Directory: /trunk/
  ChangesPath  Action
  +7 -0  README.txtModified

___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master 1178ee6: GTP: patch for Target identification for non-compliance workaround

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1178ee68fbbea6dc096df0d69f776a2158de630e
Submitter: Anders Broman (a.broma...@gmail.com)
Changed: branch: master
Repository: wireshark

Commits:

1178ee6 by Binh Trinh (bea...@gmail.com):

GTP: patch for Target identification for non-compliance workaround

bug 3974

Change-Id: I2faa473c725a803056d6ffd0cb34b46d75121061
Reviewed-on: https://code.wireshark.org/review/17410
Petri-Dish: Anders Broman 
Tested-by: Petri Dish Buildbot 
Reviewed-by: Anders Broman 


Actions performed:

from  5be39d9   ieee80211: Fix the BAR Ack policy values
adds  1178ee6   GTP: patch for Target identification for non-compliance 
workaround


Summary of changes:
 epan/dissectors/packet-gtp.c |7 +++
 1 file changed, 7 insertions(+)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe

[Wireshark-commits] master-2.2 50c51c8: ipv6: fix build with old 32-bit glib headers

[Wireshark code review]

URL: 
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=50c51c8a32d76a8629cd562c6d0e57653379d64a
Submitter: João Valverde (j...@v6e.pt)
Changed: branch: master-2.2
Repository: wireshark

Commits:

50c51c8 by Peter Wu (pe...@lekensteyn.nl):

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:

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

Change-Id: I1b25060ed329ce05e4f1ab75e32a56feda8e3435
Reviewed-on: https://code.wireshark.org/review/17401
Reviewed-by: Michael Mann 
Petri-Dish: Michael Mann 
Tested-by: Petri Dish Buildbot 
Reviewed-by: João Valverde 


Actions performed:

from  8b7444e   Use "ecatf" as the protocol name, to parallel everything 
else.
adds  50c51c8   ipv6: fix build with old 32-bit glib headers


Summary of changes:
 epan/dissectors/packet-ipv6.c |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
___
Sent via:Wireshark-commits mailing list 
Archives:https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
 mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe