Advertisement
vliaskov

set_window_handle on vaapisink Badaccess

Sep 15th, 2015
111
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
  1. Without this patch, I get http://pastebin.com/JJ9spwqH
  2.  
  3. diff --git a/gst/vaapi/gstvaapisink.c b/gst/vaapi/gstvaapisink.c
  4. index b148cac..b603782 100644
  5. --- a/gst/vaapi/gstvaapisink.c
  6. +++ b/gst/vaapi/gstvaapisink.c
  7. @@ -438,7 +434,7 @@ gst_vaapisink_x11_pre_start_event_thread (GstVaapiSink * sink)
  8. GstVaapiDisplayX11 *const display =
  9. GST_VAAPI_DISPLAY_X11 (GST_VAAPI_PLUGIN_BASE_DISPLAY (sink));
  10. static const int x11_event_mask = (KeyPressMask | KeyReleaseMask |
  11. - ButtonPressMask | ButtonReleaseMask | PointerMotionMask |
  12. + //ButtonPressMask | ButtonReleaseMask | PointerMotionMask |
  13. ExposureMask | StructureNotifyMask);
  14.  
  15.  
  16. Also: Even when calling gst_video_overlay_set_window_handle, vaapisink creates its own X11 display. Specifically gst_vaapi_create_display is called, and not gst_vaapi_create_display_from_handle. So even though we pass an X11window handle in, vaapisink will create its own X11 display.. not sure this works.
Advertisement
Advertisement
Advertisement
RAW Paste Data Copied
Advertisement