[libdvbpsi-devel] Section CRC not being checked

[Jean-Paul Saman]

Brad,

On Wed, Oct 2, 2013 at 7:03 PM, Brad Bitterman <bitter at vtilt.com> wrote:

> The patch works.
>
> On another note, in atsc_mgt.c I think there might be an issue in
> dvbpsi_AddSectionMGT. At line 387 there is a check to see if
> p_decoder->p_building_mgt gets created from the call to dvbpsi_atsc_NewMGT.
> The check is as follows:
>
> if (p_decoder->p_building_mgt)
>          return false;
>
> Shouldn't it be this:
>
> if (!p_decoder->p_building_mgt)
>           return false;
>
>
indeed, I quickly checked 2 other ATSC table  and they have the same
problem.
I guess it refers to all ATSC tables. Could you prepare a patch for all
ATSC tables and properly test it?

I do not have ATSC sources, so cannot test.

Kind regards,

Jean-Paul Saman.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/libdvbpsi-devel/attachments/20131002/c5e9b347/attachment-0001.html>