[Ffmpeg-devel] [RFC] Fraps v2 support

Kostya kostya.shishkov
Mon Nov 6 07:10:08 CET 2006


On Sun, Nov 05, 2006 at 04:24:09PM +0100, Michael Niedermayer wrote:
[...] 
> 
> > Thus, is the sum does not overflow (check is done with int64_t sum, so when
> > when something will appear in bits > 31 it will be catched), all nodes' counts
> > are <= sum so overflow won't happen.
> 
> i indeed have misread your code somehow ... sorry
> 
> but your code still doesnt look safe, as you add signed int32, so large and
> small counts can cancell each other out and then intermediats in the final
> calculation can overflow

I'll add check for count < 0.

> 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> In the past you could go to a library and read, borrow or copy any book
> Today you'd get arrested for mere telling someone where the library is
> 




More information about the ffmpeg-devel mailing list