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

From Lei Chen: 

pDescr should not step on until proto_tree_add_text() has been called.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6151

svn path=/trunk/; revision=38164
This commit is contained in:
Anders Broman 2011-07-22 08:10:25 +00:00
parent be8667d7ad
commit bdc9f65888
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
epan/dissectors/packet-csn1.c
View File

@ -195,8 +195,7 @@ csnStreamDissector(proto_tree *tree, csnStream_t* ar, const CSN_DESCR* pDescr, t
if (remaining_bits_len > 0)
{
pui8 = pui8DATA(data, pDescr->offset);
pDescr++;
*pui8 = tvb_get_bits8(tvb, bit_offset, 1);
proto_tree_add_text(tree, tvb, bit_offset>>3, 1, "%s %s",
decode_bits_in_field(bit_offset, 1, tvb_get_bits8(tvb, bit_offset, 1)),
@ -204,7 +203,12 @@ csnStreamDissector(proto_tree *tree, csnStream_t* ar, const CSN_DESCR* pDescr, t
/* end add the bit value to protocol tree */
}
else
{
return ProcessError(tree, tvb, bit_offset,"csnStreamDissector", CSN_ERROR_NEED_MORE_BITS_TO_UNPACK, pDescr);
}
pDescr++;
remaining_bits_len--;
bit_offset++;
break;