osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

RTPS: APP_ACK_CONF submessage dissection fixed. 

Count field of APP_ACK_CONF submessage was dissected using a signed
integer rather than unsigned. That avoids the dissection to be concluded
due to a wrong type error.

Change-Id: Ie5f85ce5b3d745d74e1b50d96a77560fb854034b
Reviewed-on: https://code.wireshark.org/review/30605
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
This commit is contained in:
ismaelrti 2018-11-13 17:06:37 +01:00 committed by Anders Broman
parent d2227b768e
commit 2ed4743915
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
epan/dissectors/packet-rtps.c
View File

@ -10799,7 +10799,7 @@ void proto_register_rtps(void) {
{ &hf_rtps_param_app_ack_conf_virtual_writer_count,
{ "virtualWriterCount", "rtps.app_ack_conf.virtual_writer_count",
FT_INT32, BASE_DEC, NULL, 0,
FT_UINT32, BASE_DEC, NULL, 0,
NULL, HFILL }
},