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 562455 - [farsight rtpmux] : get ssrc from src peer pad
[farsight rtpmux] : get ssrc from src peer pad
Status: RESOLVED NOTGNOME
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 549645
Blocks:
 
 
Reported: 2008-11-27 14:21 UTC by Aurelien Grimaud
Modified: 2009-02-11 00:09 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
rtpmux : get ssrc from src peer caps (1.39 KB, patch)
2008-11-27 14:22 UTC, Aurelien Grimaud 		reviewed Details | Review

Description Aurelien Grimaud 2008-11-27 14:21:17 UTC 
I have got the following pipeline

  +-------+    +--------+
  |pcmapay+---->        |
  +-------+    |        |    +---------+
               |dtmfmux +---->gstrtpbin|
  +-------+    |        |    +---------+
  |dtmfsrc+---->        |
  +-------+    +--------+

My problem is related to ssrc.
gstrtpbin is complaining about ssrc which has to be updated. (rtpsource.c:1129)
 
gstrtpbin manages its private ssrc to apply to sent packets and expects upstream payloader to build packets with this ssrc.
To sync ssrc between rtpbin and payloader, gstbasertppayload gets ssrc in src pad peer caps.

The problem is that dtmfmux does not.
Here is a patch to fix rtpmux.
Comment 1 Aurelien Grimaud 2008-11-27 14:22:27 UTC 
Created attachment 123552 [details] [review]
rtpmux : get ssrc from src peer caps
Comment 2 Wim Taymans 2008-11-27 14:30:16 UTC 
oh, please file bugs for farsight on bugs.freedesktop.org.
Comment 3 Aurelien Grimaud 2008-11-27 14:47:39 UTC 
Sorry !
What about migration from farsight to plugins-bad for dtmfsrc and dtmfmux ?
http://bugzilla.gnome.org/show_bug.cgi?id=549645
Comment 4 Wim Taymans 2008-11-27 14:56:23 UTC 
yes, merge would be fine. I didn't remember a request for that was already in bugzilla. you could make this bug depend on Bug #549645 too.