On 2015-07-01 14:44:53 +0200, Luca Barbato wrote:
> On 01/07/15 13:36, Janne Grunau wrote:
> > ---
> >  libavcodec/g2meet.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libavcodec/g2meet.c b/libavcodec/g2meet.c
> > index d0b36f0..600e2b2 100644
> > --- a/libavcodec/g2meet.c
> > +++ b/libavcodec/g2meet.c
> > @@ -399,7 +399,7 @@ static inline int log2_ceil(uint32_t x)
> >  /* improved djb2 hash from http://www.cse.yorku.ca/~oz/hash.html */
> >  static int djb2_hash(uint32_t key)
> >  {
> > -    int h = 5381;
> > +    uint32_t h = 5381;
> >  
> >      h = (h * 33) ^ ((key >> 24) & 0xFF); // xxx: probably not needed at all
> >      h = (h * 33) ^ ((key >> 16) & 0xFF);
> > 
> 
> Probably ok, what does it fix?

probably nothing except an ubsan error

Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to