Wireshark-commits: [Wireshark-commits] master-1.12 1c1733e: WaveAgent - Use tvb_get_guint8 instead
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 22 Jul 2015 07:15:16 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1c1733e4be25c1e125e02f9eae2c7dc2e0206bf4
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

1c1733e by Michael Mann (mmann78@xxxxxxxxxxxx):

    WaveAgent - Use tvb_get_guint8 instead of tvb_get_ptr to walk a packet and protect against a really big tag value
    
    Ping-Bug: 11358
    Change-Id: I9ecb5fe6bcd7f25c763d968bf56fb2d9bce2180c
    Reviewed-on: https://code.wireshark.org/review/9639
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    (cherry picked from commit 5523726e6960fe9d7e301376fd7a94599f65fd42)
    Reviewed-on: https://code.wireshark.org/review/9653
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  bc26f2e   WCCP - Only populate address table once and presume first address table type is the one that should populate it.
    adds  1c1733e   WaveAgent - Use tvb_get_guint8 instead of tvb_get_ptr to walk a packet and protect against a really big tag value


Summary of changes:
 epan/dissectors/packet-waveagent.c |   13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)