URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=49b6523c6cd4f8c56f428797283e150e63a52aad
Submitter: "Stig Bjørlykke <stig@xxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark
Commits:
49b6523 by Jonas Jonsson (jonas@xxxxxxxxxxx):
btle: Correctly detect l2cap fragment start
The first L2CAP PDU fragment starts with the 4 octet long L2CAP header
consisting of the Length and the CID fields. The Length field doesn't
include the header itself. Thus the Length field in the BLE Data header
will be 4 octets larger than the L2CAP PDU header Length field if the
packet wouldn't be fragmented.
The current implementation doesn't correctly detect the start fragment
causing reassembly to fail as it compares the BLE Data Length with the
L2CAP Length without compensating for the header.
By increasing the L2CAP PDU Length field with the header length the
reassembly works.
Rename the variable to better reflect what length it actually
represents.
Bug: 15807
Change-Id: Idcb6bdccc4daae756a63a9bae0839fe25ae99f23
Reviewed-on: https://code.wireshark.org/review/33428
Petri-Dish: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
Reviewed-by: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
Actions performed:
from 860e73c WSDG: Don't make hyperlinks of Gerrit git access links
add 49b6523 btle: Correctly detect l2cap fragment start
Summary of changes:
epan/dissectors/packet-btle.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)