Hi all, Would like to bring to attention the undefined left shift of negative numbers that are found by latest clang 3.7.0 (-Wshift-negative-value). I send this because I have no idea important they are, but clearly they are of the type that interests static analyzers/fuzzers: e5aa6f702130f1afd53c61d058c2ebc48e61e9ea
Anyway, I am not submitting a patch for one of them myself (libavcodec/apedec) as I do not even know whether current code is correct, e.g does it actually implicitly assume res (of type int) is 32 bits, and the shift by 30/31 is for high order bit stuff? In that case, additional change to int32_t is needed. More can be found e.g http://fate.ffmpeg.org/log.cgi?time=20150917122742&log=compile&slot=x86_64-darwin-clang-3.7-O3 Regards, Ganesh _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
