android/opaque: fix crash on Close with subtitles

The pixels allocated from picture_NewFromFormat were replaced (and leaked) by
pixels allocated by android native windows. This also leaded to a crash on
Close when subtitles_picture was freed.

Use picture_NewFromResource with an empty to resource in order to don't
allocate any pixels.

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

diff --git a/modules/video_output/android/opaque.c b/modules/video_output/android/opaque.c
index 6263b58ac58b7722ab116ea724eed705e00ce0ff..b3016344b8f309be3e7a0a8529f293476ca994a7 100644 (file)
--- a/modules/video_output/android/opaque.c
+++ b/modules/video_output/android/opaque.c
@@ -195,7 +195,9 @@ static int Open(vlc_object_t *p_this)
     video_format_t subpicture_format = sys->fmt;
     subpicture_format.i_chroma = VLC_CODEC_RGBA;
     /* Create a RGBA picture for rendering subtitles. */
-    sys->subtitles_picture = picture_NewFromFormat(&subpicture_format);
+    picture_resource_t rsc;
+    memset(&rsc, 0, sizeof(rsc));
+    sys->subtitles_picture = picture_NewFromResource(&subpicture_format, &rsc);
 
     /* Export the subpicture capability of this vout. */
     vd->info.subpicture_chromas = subpicture_chromas;