[vlc-devel] [sout developers] VLC streaming violation of MPEG-TS discontinuity flag

Ilkka Ollakka ileoo at videolan.org
Tue Oct 13 07:26:49 CEST 2015


On Mon, Oct 12, 2015 at 05:03:58PM +0200, Jean-Paul Saman wrote:


> It took a little longer, but here are patches:

Great, seemed ok what I tested. As patches touches pretty much every
codec-module that there is, I can't test them all and surely some odd
corner cases will raise. Patches are now in.

> The following logic has been used in the patches:

> + decoders :
>    - reset timeline on BLOCK_FLAG_DISCONTINUITY
>    - flush on BLOCK_FLAG_CORRUPTED
> + packetizers:
>    - flush bytestream on  BLOCK_FLAG_DISCONTINUITY and BLOCK_FLAG_CORRUPTED
> + ts-demux:
>    - set BLOCK_FLAG_CORRUPTED on incomplete frame (might also happen on
> seek)
>    - set BLOCK_FLAG_DISCONTINUITY on seeking (time line discontinuity)
>    - pass on ts-header discontinuity as BLOCK_FLAG_DISCONTINUITY

> Here are the patches:
>  0001-vlc_block.h-Clarify-usage-of-BLOCK_FLAG_DI...
> <https://drive.google.com/file/d/0B5sXlFwquWxcY1Z0NlpPbWRhaTA/view?usp=drive_web>
>  0002-avcodec-video.c-do-not-drop-good-data-on-d...
> <https://drive.google.com/file/d/0B5sXlFwquWxcT1VvNGc0dGo0NTA/view?usp=drive_web>
>  0003-ts-demuxer-Passthrough-timeline-discontinu...
> <https://drive.google.com/file/d/0B5sXlFwquWxcUzFtT1dpM1U3U1E/view?usp=drive_web>
>  0004-qt4-discontinuity-stats-signal-timeline-ch...
> <https://drive.google.com/file/d/0B5sXlFwquWxcZjluMnhRRmZwWlE/view?usp=drive_web>
>  0005-codec-don-t-drop-discontinue-blocks.patch
> <https://drive.google.com/file/d/0B5sXlFwquWxceHJWRE13UHRCMlU/view?usp=drive_web>
>  0006-codec-vpx.c-do-not-leak-memory.patch
> <https://drive.google.com/file/d/0B5sXlFwquWxcT0RDZW9FVXM2Tjg/view?usp=drive_web>
>  0007-packetizer-don-t-drop-blocks-with-DISCONTI...
> <https://drive.google.com/file/d/0B5sXlFwquWxcU25KZEY4MkdJSVU/view?usp=drive_web>
>  0008-codec-don-t-drop-blocks-marked-BLOCK_FLAG_...
> <https://drive.google.com/file/d/0B5sXlFwquWxcQW5WU0F3VV9MRWM/view?usp=drive_web>

I think in future it would be better to attach actual files instead of
links to google drive.

-- 
Ilkka Ollakka
You will be awarded some great honor.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20151013/44cfc334/attachment.sig>


More information about the vlc-devel mailing list