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 453037 - rtpilbcpayloader encoding-name field mismatch
rtpilbcpayloader encoding-name field mismatch
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal normal
: 0.10.7
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-07-02 08:33 UTC by Tommi Myöhänen
Modified: 2007-07-03 10:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gstrtpilbcpay.patch (601 bytes, patch)
2007-07-02 08:34 UTC, Tommi Myöhänen
committed Details | Review

Description Tommi Myöhänen 2007-07-02 08:33:07 UTC
"encoding-name" field was changed in rtpilbc(de)payloader caps template from "iLBC" to "ILBC":
http://webcvs.freedesktop.org/gstreamer/gst-plugins-good/gst/rtp/gstrtpilbcpay.c?r1=1.4&r2=1.5

However in gst_rtpilbcpay_setcaps the encoding-name field is still set to "iLBC", which doesn't seem to match the rtpilbcdepayloader caps template anymore.
Comment 1 Tommi Myöhänen 2007-07-02 08:34:04 UTC
Created attachment 91011 [details] [review]
gstrtpilbcpay.patch
Comment 2 Wim Taymans 2007-07-03 10:00:08 UTC
        Patch by: Tommi Myöhänen  <ext-tommi dot myohanen at nokia dot com>

        * gst/rtp/gstrtpilbcpay.c: (gst_rtpilbcpay_setcaps):
        Set the encoding-name in the rtp caps to all uppercase, as required by
        the caps spec.
        Some small cleanups in the error paths. Fixes #453037.