Section CRC not being checked

Brad Bitterman <[email protected]>
Newsgroups gmane.comp.video.videolan.libdvbpsi.devel
Message-ID <[email protected]>
We were testing libdvbpsi 1.1.0 with an ATSC tuner and found that we were getting corrupted data sometimes for the VCT. After looking at the code we found that the CRC was not being checked. 

In the function dvbpsi_packet_push around line 400 the boolean variable is declared and initialized by calling dvbpsi_has_crc( p_section ). This function uses p_section->b_syntax_indicator to check to see if the CRC should be validated. The p_section->b_syntax_indicator is not initialized until a few lines down. This cause the CRC to never be check. 

I moved the call to dvbpsi_has_crc down after b_syntax_indicator gets initialized and I now see the CRC check happening. Is this correct or is there a reason that the code is the way it is?

Thanks,

Brad Bitterman
[email protected]

_______________________________________________
libdvbpsi-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/libdvbpsi-devel
signature.asc (application/pgp-signature, 235 B)
-----BEGIN PGP SIGNATURE-----
Comment: GPGTools - http://gpgtools.org

iF4EAREIAAYFAlJMO1YACgkQoT6fpi0w1Tf25QD/QjJLi7udb/LfZtKM69tohrXi
8xubIuzYDNpg0VJiPvUA/33Qyrue5azGaJF/TQn/Rt1hyJsDw6/PvA2iCzElvcyu
=ccLS
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.