[libdvbpsi-devel] [PATCH] SEGV fixed in dvbpsi_decoder_psi_section_add

[Roberto Corno]

Hi Jean-Paul,
    it seems that your patch is the right one. I tested it and it seems to
correctly fix the problem.
Regards
Roberto

On 4 September 2012 13:34, Jean-Paul Saman <jpsaman at videolan.org> wrote:

> Robert,
>
> On Mon, Sep 3, 2012 at 3:07 PM, Roberto Corno <corno.roberto at gmail.com>
> wrote:
> > SEGV happens within dvbpsi_decoder_psi_section_add when p->i_number ==
> > p_section->i_number  and  p_prev == NULL.
> > It's the first branch of the while(p) loop.
> > I got it with a heavily degraded transport stream, where a lot of CC
> error
> > occur.
> > I've added the check for p_prev==NULL
> > Patch attached.
>
> Your patch ignores the new section instead of replacing the section
> with the same number.
> The caller could therefor leak the section pointer. Please try this
> patch instead.
>
> Kind regards,
>
> Jean-Paul Saman
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/libdvbpsi-devel/attachments/20120905/336151b8/attachment.html>