On 8/23/19 3:46 PM, Aman Gupta wrote:
On Wed, Aug 14, 2019 at 3:53 AM Pavel Koshevoy <pkoshe...@gmail.com> wrote:

On Tue, Aug 6, 2019 at 8:50 PM Pavel Koshevoy <pkoshe...@gmail.com> wrote:
vtctx->cached_hw_frames_ctx is unref'd in videotoolbox_uninit,
but videotoolbox_hevc used ff_videotoolbox_uninit which
doesn't unref cache_hw_frames_ctx.
---
  libavcodec/videotoolbox.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/videotoolbox.c b/libavcodec/videotoolbox.c
index c718e82cc5..acaeef77dd 100644
--- a/libavcodec/videotoolbox.c
+++ b/libavcodec/videotoolbox.c
@@ -1143,7 +1143,7 @@ const AVHWAccel ff_hevc_videotoolbox_hwaccel = {
      .end_frame      = videotoolbox_hevc_end_frame,
      .frame_params   = videotoolbox_frame_params,
      .init           = videotoolbox_common_init,
-    .uninit         = ff_videotoolbox_uninit,
+    .uninit         = videotoolbox_uninit,
      .priv_data_size = sizeof(VTContext),
  };

--
2.16.4


Ping.  It's a 1-line leak fix ... is there any reason this should not
be applied?

LGTM. Feel free to push it.


applied and pushed,


Thank you,
    Pavel.



_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to