It made sense, but this commit don't touch this part of the code. It can made more sense to add an another path to "prevent bad memory allocation in geq filter". What do you think ?
Regards, Le mer. 22 nov. 2017 à 15:19, Derek Buitenhuis <[email protected]> a écrit : > On 11/22/2017 10:24 AM, Marc-Antoine ARNAUD wrote: > > New patch version which fixe the last remark. > > [...] > > > + char depth_string[8]; > > + snprintf(depth_string, sizeof(depth_string), "%d", > (1<<geq->depth) - 1); > > + geq->expr_str[A] = av_strdup(depth_string); > > Missing return value check for av_strdup. > > - Derek > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
