diff options
author | Juanjo Martin <juanjo@rti.com> | 2016-06-02 13:15:03 +0200 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2016-06-05 10:54:42 +0000 |
commit | 96783ea83980fbb529ce2c7956d7bcf1c4743d75 (patch) | |
tree | 2bb1ad0f98688162c28d184dfbc59a9e62f9c2e5 /services | |
parent | ab82d4519f69572b581ccf04a91d305b6a10f6eb (diff) | |
download | wireshark-96783ea83980fbb529ce2c7956d7bcf1c4743d75.tar.gz |
RTPS: Fixed the scenario where discovery contains PID_EXTENDED
Before there was a wrong dissection of parameters that wasn't
predictable because of the wrong offset. This could lead to anything.
Now, it shows an expert warning indicating that the RTPS packet
doesn't contain all the bytes specified by the parameter but it shows
properly the parameterId and the length. In order to parse properly
the whole PID_EXTENDED parameter we'd need to handle fragments. That
is not the purpose of this commit.
Change-Id: I0f3f0f1d309d43a530047f510169fb07983a9fb6
Reviewed-on: https://code.wireshark.org/review/15698
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'services')
0 files changed, 0 insertions, 0 deletions