[FFmpeg-devel] [PATCH 3/4] avformat/mpegenc - accept PCM_DVD streams

[Michael Niedermayer]

On Thu, Feb 15, 2018 at 06:23:06PM +0530, Gyan Doshi wrote:
> 
> On 2/15/2018 8:57 AM, Michael Niedermayer wrote:
> >On Tue, Feb 13, 2018 at 12:33:56AM +0530, Gyan Doshi wrote:
> 
> >>+    if (st->codecpar->codec_id == AV_CODEC_ID_PCM_DVD) {
> >>+        if (size < 3) {
> >>+            av_log(ctx, AV_LOG_ERROR, "Invalid packet size %d\n", size);
> >
> >>+            return -1;
> >
> >This should be a AVERROR code
> 
> Revised patch attached.
> 
> Gyan

>  mpegenc.c |   23 +++++++++++++++++++++--
>  1 file changed, 21 insertions(+), 2 deletions(-)
> 5cf30a6ffe273094e380f19299812d63987a0e2e  v4-0001-avformat-mpegenc-accept-PCM_DVD-streams.patch
> From 12c5c7e5307aae14cbe401d376d2d8c0c096310c Mon Sep 17 00:00:00 2001
> From: Gyan Doshi <gyandoshi at gmail.com>
> Date: Wed, 7 Feb 2018 18:05:08 +0530
> Subject: [PATCH v4] avformat/mpegenc - accept PCM_DVD streams
> 
> PCM_S16BE stream packets in MPEG-PS have a 3-byte header and
> are recognized as PCM_DVD by the demuxer which prevents their
> correct remuxing in MPEG-1/2 PS.
> ---
>  libavformat/mpegenc.c | 23 +++++++++++++++++++++--
>  1 file changed, 21 insertions(+), 2 deletions(-)

will apply

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180216/0144ecea/attachment.sig>