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