On 23.05.2015 00:42, Michael Niedermayer wrote: > On Fri, May 22, 2015 at 11:36:17PM +0200, Andreas Cadhalpun wrote: >> If x is 2 it is not initialized before it is used. >> >> This fixes 'Conditional jump or move depends on uninitialised value(s)' >> valgrind warnings. >> >> Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com> >> --- >> libavformat/nutdec.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/libavformat/nutdec.c b/libavformat/nutdec.c >> index f1cece8..935adea 100644 >> --- a/libavformat/nutdec.c >> +++ b/libavformat/nutdec.c >> @@ -690,6 +690,7 @@ static int find_and_decode_index(NUTContext *nut) >> GET_V(syncpoint_count, tmp < INT_MAX / 8 && tmp > 0); >> syncpoints = av_malloc_array(syncpoint_count, sizeof(int64_t)); >> has_keyframe = av_malloc_array(syncpoint_count + 1, sizeof(int8_t)); >> + has_keyframe[0] = 0; > > ive added a check for x, this should fix it
Yes, that fixes it. > btw above also might segfault as its before the NULL check Right... Best regards, Andreas _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel