Panagiotis Issaris <[EMAIL PROTECTED]> added the comment:
Michael Niedermayer schreef:
> Michael Niedermayer <[EMAIL PROTECTED]> added the comment:
>
> [...]
>> @@ -3417,6 +3418,13 @@
>> }
>> printf("\n");
>>
>> + printf("Bitstream filters:\n");
>> + for(bsf = first_bitstream_filter; bsf != NULL; bsf = bsf->next) {
>> + if(bsf->name)
>> + printf(" %s", bsf->name);
>
> why the NULL check?
Because I figured it wasn't guaranteed to not be NULL. But, I just
noticed that av_bitstream_filter_init() also assumes that bsf->name is
non-NULL and indeed, one can see it as a requirement to set name when
creating a bitstreamfilter.
Updated patch attached.
Takis
______________________________________________________
FFmpeg issue tracker <[EMAIL PROTECTED]>
<https://roundup.mplayerhq.hu/roundup/ffmpeg/issue295>
______________________________________________________
