[Bug 722345] directfb: video sink crashes when used with ext_surface

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=722345
  GStreamer | gst-plugins-bad | 1.2.2

--- Comment #19 from Eric <etrousset at awox.com> 2014-01-17 16:11:19 UTC ---
(In reply to comment #18)
> (In reply to comment #17)
> > yes, but with DFB_BYTES_PER_LINE(meta->pixel_format,
> > meta->width); we can avoid Lock and Unlocking the surface just to get the
> > pitch.
> 
> DFB_BYTES_PER_LINE() is the minimum stride allowed for the format. It fine if
> you need to allocate a surface from system memory, or for backend
> implementation, but on DirectFB application side I don't see why you would ever
> want to do that. I'll need to have a bigger look at the context.

I think you are right, it would be better to use the actual pitch of the
surface, in stead of guessing... I'll change the patch accordingly

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.