On 10/07/15 10:39, Anton Khirnov wrote:
> Quoting Luca Barbato (2015-07-09 21:07:27)
>> On 09/07/15 20:22, Anton Khirnov wrote:
>>> ---
>>>  libavcodec/Makefile               |   1 +
>>>  libavcodec/allcodecs.c            |   1 +
>>>  libavcodec/hevc_mp4toannexb_bsf.c | 186 
>>> ++++++++++++++++++++++++++++++++++++++
>>>  3 files changed, 188 insertions(+)
>>>  create mode 100644 libavcodec/hevc_mp4toannexb_bsf.c
>>
>>> +
>>> +    *poutbuf_size = 0;
>>> +    *poutbuf      = NULL;
>>> +
>>> +    bytestream2_init(&gb, buf, buf_size);
>>> +
>>
>> buf_size is validated somewhere else? Beside that looks ok.
> 
> I don't think it needs to be validated, the code will work fine even if
> it is zero.
> 

Fine for me then.

lu
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to