URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=2193bea3212d74e2a907152055e27d409b59485e
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark
Commits:
2193bea by Pascal Quantin (pascal.quantin@xxxxxxxxx):
WAP: check that tvb_get_guintvar does not overflow
Bug: 12661
Change-Id: I2ef857d6be6595fd89f3dbb8d41c1c70d550ad93
Reviewed-on: https://code.wireshark.org/review/16665
Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
Tested-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
Actions performed:
from 5670642 proto.c: add bounds check to proto_tree_add_text(_valist)_internal
adds 2193bea WAP: check that tvb_get_guintvar does not overflow
Summary of changes:
epan/dissectors/packet-mmse.c | 59 +++++++++++-------
epan/dissectors/packet-wap.c | 6 +-
epan/dissectors/packet-wap.h | 3 +-
epan/dissectors/packet-wbxml.c | 132 ++++++++++++++++++++--------------------
epan/dissectors/packet-wsp.c | 56 +++++++++--------
5 files changed, 139 insertions(+), 117 deletions(-)