URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=7e13a14229c515d52c7485e5aedc27ed8747c765
Submitter: Balint Reczey (balint@xxxxxxxxxxxxxxx)
Changed: branch: lts-1.12.1
Repository: wireshark
Commits:
7e13a14 by Peter Wu (peter@xxxxxxxxxxxxx):
alljoyn: fix signature length adjustments
Ensure that the signature pointer and length always matches, otherwise a
buffer overrun (read) is possible.
Tested with the original captures from bug 12953, the PDML output is
still the same while the fuzzed capture does not crash anymore.
Conflicts:
epan/dissectors/packet-alljoyn.c
Bug: 12953
Change-Id: I8843a5daf98a79fb19906e824326cdf619164484
Reviewed-on: https://code.wireshark.org/review/18347
Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
Reviewed-by: Martin Kaiser <wireshark@xxxxxxxxx>
(cherry picked from commit 7dfaec969e67e3aa14b9763d804802ef614c9ddd)
[Peter: resolved conflicts in context]
Reviewed-on: https://code.wireshark.org/review/18410
Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
Reviewed-on: https://code.wireshark.org/review/18873
Reviewed-by: Balint Reczey <balint@xxxxxxxxxxxxxxx>
Actions performed:
from d101dc0 DCERPC: save strings in wmem file scope
adds 7e13a14 alljoyn: fix signature length adjustments
Summary of changes:
epan/dissectors/packet-alljoyn.c | 24 +++++++++++++++---------
1 file changed, 15 insertions(+), 9 deletions(-)