Wireshark-dev: [Wireshark-dev] issue to set up an RTP conversation

From: Bertin Nicolas <nicolas.bertin@xxxxxxxxxxxxxxxxx>
Date: Fri, 22 Mar 2019 11:52:24 +0000

Hello,

I've a dissector (packet-ua3g) that calls rtp_add_address(...) to set up an RTP conversation.
It was working but now it fails. I've checked on master, 3.0.0 (windows) and 2.6.6 (ubuntu 14.04)

I've found it's due to pinfo->fd->visited being already set (strp_add_address(...) function)  that
prevents the conversation to be added.

    /*
     * If this isn't the first time this packet has been processed,
     * we've already done this work, so we don't need to do it
     * again.
     */
    if ((pinfo->fd->visited) || (rtp_handle == NULL) || (rtp_rfc4571_handle == NULL))
    {
        return;
    }


I can workaround by clearing the "visited" bit before calling rtp_add_address(...) but I don't think

it's correct...


Any comments?


Nicolas