On Mon, Apr 25, 2016 at 08:04:44PM +0100, Derek Buitenhuis wrote:
> Signed-off-by: Derek Buitenhuis <derek.buitenh...@gmail.com>
> ---
> grep reveals no other usages.
> ---
> libavcodec/qsvenc_hevc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/qsvenc_hevc.c b/libavcodec/qsvenc_hevc.c
> index 30fde72..e0bead9 100644
> --- a/libavcodec/qsvenc_hevc.c
> +++ b/libavcodec/qsvenc_hevc.c
> @@ -68,7 +68,7 @@ static int generate_fake_vps(QSVEncContext *q,
> AVCodecContext *avctx)
> }
>
> /* parse the SPS */
> - ret = ff_hevc_extract_rbsp(NULL, avctx->extradata + 4,
> avctx->extradata_size - 4, &sps_nal);
> + ret = ff_hevc_extract_rbsp(avctx->extradata + 4, avctx->extradata_size -
> 4, &sps_nal);
> if (ret < 0) {
> av_log(avctx, AV_LOG_ERROR, "Error unescaping the SPS buffer\n");
> return ret;i cant easily test this but does this need a av_freep of skipped_bytes_pos somewhere ? otherwise it should be ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many things microsoft did are stupid, but not doing something just because microsoft did it is even more stupid. If everything ms did were stupid they would be bankrupt already.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
