[dvblast-devel] Coverity Scan

[Christophe Massiot]

Hello,

I have fixed the most serious flaws already, thanks !

Le 21 oct. 2013 à 21:23, JULIAN GARDNER <joolzg at btinternet.com> a écrit :

> scan.coverity.com
> 
> Ive added dvblast as a project and it has come up with interesting interesting errors, please log in and see what it has found.
> 
> Also bitstream has a subtle error
> 
> desc_69.h
> static inline uint8_t desc69_get_day(const uint8_t *p_desc)
> {
>     uint8_t i_day = ((p_desc[2] & 0x0f) << 1) | ((p_desc[3] & 0x7f) >> 7); // rrrr1111 1xxxxxxx
>     return i_day + 1;
> }
> 
> should be
> 
> static inline uint8_t desc69_get_day(const uint8_t *p_desc)
> {
>     uint8_t i_day = ((p_desc[2] & 0x0f) << 1) | ((p_desc[3] & 0x80) >> 7); // rrrr1111 1xxxxxxx
>     return i_day + 1;
> }
> 
> 
> joolz
> 
> _______________________________________________
> dvblast-devel mailing list
> dvblast-devel at videolan.org
> https://mailman.videolan.org/listinfo/dvblast-devel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/dvblast-devel/attachments/20131023/7e44e8e0/attachment-0001.html>