[FFmpeg-devel] [PATCH 3/4] avformat/mpegenc - accept PCM_DVD streams
Michael Niedermayer
michael at niedermayer.cc
Fri Feb 16 23:50:03 EET 2018
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>
More information about the ffmpeg-devel
mailing list