On Sun, Mar 10, 2013 at 1:24 PM, David Arnold <dav...@pobox.com> wrote:
> On 10/03/2013, at 5:13 PM, Evan Huus wrote:
>
> Hi Evan,
>
> Thanks for your help.
>
> <...>
>
>> I'm not sure I follow this. Can't you simply store the most recent
>> sequence number in the conversation struct, and increment it for each
>> PDU received?
>
> I tried to do this, and perhaps I just got it (slightly) wrong.
>
> In my foo_dissect() function, I looked up the conversation, grabbed the
> latest sequence number, incremented it, and wrote it back to the
> conversation. I displayed it in the tree_item for the PDU, and in the
> COL_INFO, which was basically what I was after.
>
> The issues I had seemed to be related to when my dissection function was
> called: if I scrolled from start to end of the capture, it was fine. If I
> jumped to the final frame, the sequence number wasn't incremented for the
> intervening PDUs.
>
> Perhaps I need it to be done outside my if (tree) { ... } ?
Yes, definitely. In general, the only things done in an if (tree)
block should be adding things to the tree. Any other tasks (setting
column information, reassembly, conversation management, etc.) should
be done regardless.
___________________________________________________________________________
Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org>
Archives: http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe
