[vlc-devel] [PATCH 01/17] android: utils: fix double free

Thomas Guillem thomas at gllm.fr
Wed Jan 13 14:32:51 UTC 2021


Patch set LGTM

On Wed, Jan 13, 2021, at 12:07, Alexandre Janniaux wrote:
> The SurfaceTexture class is freed multiple times whenever the
> libvlc_media_player is restarted.
> ---
>  modules/video_output/android/utils.c | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/modules/video_output/android/utils.c 
> b/modules/video_output/android/utils.c
> index 1cd7469cf2..6e8a6f22b0 100644
> --- a/modules/video_output/android/utils.c
> +++ b/modules/video_output/android/utils.c
> @@ -893,12 +893,6 @@ AWindowHandler_destroy(AWindowHandler *p_awh)
>  
>      if (p_env)
>      {
> -        if (jfields.SurfaceTexture.clazz)
> -            (*p_env)->DeleteGlobalRef(p_env, 
> jfields.SurfaceTexture.clazz);
> -
> -        if (jfields.Surface.clazz)
> -            (*p_env)->DeleteGlobalRef(p_env, jfields.Surface.clazz);
> -
>          JNI_ANWCALL(CallVoidMethod, unregisterNative);
>          AWindowHandler_releaseANativeWindowEnv(p_awh, p_env, 
> AWindow_Video);
>          AWindowHandler_releaseANativeWindowEnv(p_awh, p_env, 
> AWindow_Subtitles);
> -- 
> 2.30.0
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list