Nuo Mi (12024-08-25): > Thank you for the patch. > Perhaps we can consider removing the assert, as other processes guarantee > the range, correct?"
Uh, what?!? The point of an assert is precisely to check that the “other processes” that are supposed to guarantee something are really doing it. What happens if a bug is introduced in these “other processes” that lets invalid value fall through? With the assert, the code crashes immediately. Without the assert, it causes a memory corruption, leading to silent corruption of data or exploitable security issue. Leave the asserts. Add asserts where there are none. Regards, -- Nicolas George _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".