On Sat, Dec 30, 2017 at 8:34 AM Derek Buitenhuis <[email protected]> wrote:
> On 12/30/2017 4:31 PM, Nicolas George wrote: > > Does it really matter? If av_opt_get() fails for any reason, > > http_multiple will just be false, which would let the processing > > continue, only in a slightly degraded manner that was the norm a few > > months ago. > > I contend that checking errors should *always* be done, as a matter of > good practice, not whenever it "seems needed". Especially when it can > be a alloc failure, leading, in this case to calling: > > strncmp((const char *)NULL, http_version_opt, "1.1", 3) > > if it fails. There is already a check in place to prevent strncmp from being called with NULL. Aman > > - 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
