On 12/29/2017 8:25 PM, Carl Eugen Hoyos wrote: > 2017-12-29 20:47 GMT+01:00 Derek Buitenhuis <derek.buitenh...@gmail.com>: >> On 12/29/2017 1:10 AM, Carl Eugen Hoyos wrote: >>> New patch attached, only tested with fate. >> >>> + if (INT_MAX / sizeof(*sc->stts_data) <= entries) > > This is an arbitrary limit...
Yeah, true. I should have pointed that out too. > >>> return AVERROR(ENOMEM); >> >> Should probably be something thing AVERROR(EINVAL), I think. > > ... and it is therefore - imo - not correct to return EINVAL. ENOMEM is arguably even more incorrect, though. > >>> + sc->stts_count = FFMIN(1024 * 1024, entries); > > This is not a limit and therefore not an arbitrary limit. ... what? It is a limit on the size of stts_count. A arbitrary (1mb) one. In any case, I misunderstood this part, and it is OK. Sorry about that - disregard my complaint. FFMIN(sc->stts_count * 2, entries) could in theory end up using more memory for legitimate large files though, due to exponential buffer growth. Is that something worth caring about? Other than that, seems OK. Sorry for the noise. - Derek _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
