Wireshark mailing list archives

issue to set up an RTP conversation


From: Bertin Nicolas <nicolas.bertin () al-enterprise com>
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
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe

Current thread: