On 07/13/2012 01:47 PM, Janne Grunau wrote: > CC: [email protected] > --- > libavcodec/imgconvert.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c > index 90c9b7b..7a6102e 100644 > --- a/libavcodec/imgconvert.c > +++ b/libavcodec/imgconvert.c > @@ -599,7 +599,8 @@ static enum PixelFormat > avcodec_find_best_pix_fmt1(int64_t pix_fmt_mask, > /* find exact color match with smallest size */ > dst_pix_fmt = PIX_FMT_NONE; > min_dist = 0x7fffffff; > - for(i = 0;i < PIX_FMT_NB; i++) { > + /* test only the first 64 pixel formats to avoid undefined behaviour */ > + for (i = 0; i < 64; i++) { > if (pix_fmt_mask & (1ULL << i)) { > loss = avcodec_get_pix_fmt_loss(i, src_pix_fmt, has_alpha) & > loss_mask; > if (loss == 0) {
Thank you. Patchset ok I guess =) -- Luca Barbato Gentoo/linux http://dev.gentoo.org/~lu_zero _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
