[FFmpeg-devel] [PATCH] mpegts: add the judgement if a new program is ctreated successfully

Michael Niedermayer michaelni at gmx.at
Thu Oct 16 04:23:37 CEST 2014


On Wed, Oct 15, 2014 at 11:05:29PM +0200, Marton Balint wrote:
> 
> On Wed, 15 Oct 2014, di1028.wu at samsung.com wrote:
> 
> >From: Di Wu <di1028.wu at samsung.com>
> >
> >Add the judement after create a new program to avoid segment fault.
> >
> >Signed-off-by: Di Wu <di1028.wu at samsung.com>
> >---
> >libavformat/mpegts.c |    6 ++++--
> >1 file changed, 4 insertions(+), 2 deletions(-)
> >
> >diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c
> >index 8808269..cc0ce79 100644
> >--- a/libavformat/mpegts.c
> >+++ b/libavformat/mpegts.c
> >@@ -1925,8 +1925,10 @@ static void pat_cb(MpegTSFilter *filter, const uint8_t *section, int section_len
> >        } else {
> >            MpegTSFilter *fil = ts->pids[pmt_pid];
> >            program = av_new_program(ts->stream, sid);
> >-            program->program_num = sid;
> >-            program->pmt_pid = pmt_pid;
> >+            if (program) {
> >+                program->program_num = sid;
> >+                program->pmt_pid = pmt_pid;
> >+            }
> >            if (fil)
> >                if (   fil->type != MPEGTS_SECTION
> >                    || fil->pid != pmt_pid
> 
> LGTM, thanks.

applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141016/c7c0de92/attachment.asc>


More information about the ffmpeg-devel mailing list