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 629522 - [rtpjpegpay] add support for Define Restart Interval (DRI)
[rtpjpegpay] add support for Define Restart Interval (DRI)
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Windows
: Normal enhancement
: 0.10.26
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-09-13 14:13 UTC by American Dynamics
Modified: 2010-09-13 15:45 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch the gstrtpjpegpay.c in git-patch format (5.59 KB, patch)
2010-09-13 14:13 UTC, American Dynamics 		none Details | Review

Description American Dynamics 2010-09-13 14:13:00 UTC 
Created attachment 170150 [details] [review]
patch the gstrtpjpegpay.c in git-patch format

The current git codebase appears to not have support for Define Restart Interval (DRI) detection in the rtp jpeg payloader.  We have encountered some IP-based cameras that deliver MJPEG with the DRI marker encoded in the jpeg stream.  We have implemented the handling of the DRI marker in accordance specified in section 3.1.7 of RFC2435.  This is not a full implementation of the DRI marker feature for we indicate to the receiver that THEY must reassemble the entire frame before decoding it (restart count = 0x3FFFF).  We have also included some defensive code to protect against those streams that DO input a DRI marker but have no actual restart interval.
Comment 1 Wim Taymans 2010-09-13 15:33:43 UTC 
commit 604c6555a49b72cc1165a7af8cfd7328044c3ba8
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Mon Sep 13 17:31:35 2010 +0200

    rtpjpegpay: cleanups for DRI markers
    
    Protect against invalid DRI markers.
    do some cleanups

commit 0f3151c73bbe6708444c0fca4be93c217996fada
Author: American Dynamics <GStreamer-Bugs@tycosp.com>
Date:   Fri Sep 10 11:35:53 2010 -0400

    gstrtpjpegpay: Added Define Restart Interval (DRI) Marker
    
    Added ability to detect and respond to a JPEG-defined DRI marker