After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 711495 - srtpenc: new streams remove any existing streams
srtpenc: new streams remove any existing streams
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other All
: Normal normal
: 1.2.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-11-05 15:38 UTC by Patrik Oldsberg
Modified: 2013-11-20 01:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Patrik Oldsberg 2013-11-05 15:38:08 UTC 
When a new stream is added to the SRTP encoder it overrides the existing stream.
It seems as though all added streams are handled as if they were the first stream.

This can be reproduced by requesting multiple sink pads from the SRTP encoder and then sending RTP packets with different SSRC on the pads.

The resulting error from libsrtp is err_status_no_ctx, which indicates that there is no stream present for the SSRC of the packet that is being protected.
Comment 1 Olivier Crête 2013-11-20 01:21:33 UTC 
Fixed

commit b58ca88fb9c3c3e1efd5a19c0c339cbe56987655
Author: Olivier Crête <olivier.crete@collabora.com>
Date:   Tue Nov 19 20:18:48 2013 -0500

    srtpenc: Set the first session flag to false once one is create
    
    Otherwise it always thinks it's the first session.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=711495
Comment 2 Olivier Crête 2013-11-20 01:35:26 UTC 
Pushed to 1.2 branch too