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 738297 - DTMF telephone-event timestamps are bogus
DTMF telephone-event timestamps are bogus
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
1.4.3
Other Linux
: Normal normal
: 1.4.4
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-10-10 13:17 UTC by David Woodhouse
Modified: 2014-10-10 22:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description David Woodhouse 2014-10-10 13:17:30 UTC 
When sending DTMF for example using the pidgin patch at https://developer.pidgin.im/ticket/12617 the RTP timestamps on DTMF packets are wrong. It looks like the clock-base for the DTMF source is not being synchronised with the audio.

This seems to be trivial to reproduce so I won't pollute the bug with my attempts to debug it. If I could would out how all this stuff fitted together in order to coherently describe the problem, I'd be filing a patch...
Comment 1 Olivier Crête 2014-10-10 22:17:42 UTC 
Wim broke it in 1.0 by renaming some caps field in GstRtpBasePayload and it seems he never bothered to check if these were used anywhere else!

Fixed, patches in both the 1.4 and master branches.

Commits in master:

commit 155ed569c3f9c83537ccf70d3bf94bb59cd63958
Author: Olivier Crête <olivier.crete@ocrete.ca>
Date:   Fri Oct 10 18:11:19 2014 -0400

    rtpdtmfsrc: clock-base and seqnum-base -> timestamp-offset and seqnum-offset
    
    These were renamed in GstRTPBasePayload in 1.0

commit b3069634bd04cf2d9183574a953ebd7936b5a5fc
Author: Olivier Crête <olivier.crete@ocrete.ca>
Date:   Fri Oct 10 17:30:24 2014 -0400

    rtpmux: clock-base and seqnum-base -> timestamp-offset and seqnum-offset
    
    These were renamed in GstRTPBasePayload in 1.0