From cb73a6e792fa12257573a9ad47489d92f48b52ab Mon Sep 17 00:00:00 2001 From: =?utf8?q?Tim-Philipp=20M=C3=BCller?= Date: Thu, 3 May 2007 16:29:10 +0000 Subject: [PATCH] sys/ximage/ximagesink.c: When XShm is not available, we might get row strides that are not rounded up to multiples of... Original commit message from CVS: * sys/ximage/ximagesink.c: (gst_ximagesink_ximage_new): When XShm is not available, we might get row strides that are not rounded up to multiples of four; this is bad, because virtually every RGB-processing element in GStreamer assumes rowstrides are rounded up to multiples of four, so let's allocate at least enough memory to avoid crashes in this case. The image will still be displayed distorted though if this happens, so that still needs fixing (maybe by allocating a bigger image with an 'even' width and then clipping it appropriately when rendering - something for Xlib aficionados in any case). --- ChangeLog | 13 +++++++++++++ sys/ximage/ximagesink.c | 22 +++++++++++++++++++++- 2 files changed, 34 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 4253064..56e4599 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,16 @@ +2007-05-03 Tim-Philipp Müller + + * sys/ximage/ximagesink.c: (gst_ximagesink_ximage_new): + When XShm is not available, we might get row strides that are not + rounded up to multiples of four; this is bad, because virtually + every RGB-processing element in GStreamer assumes rowstrides are + rounded up to multiples of four, so let's allocate at least enough + memory to avoid crashes in this case. The image will still be + displayed distorted though if this happens, so that still needs + fixing (maybe by allocating a bigger image with an 'even' width + and then clipping it appropriately when rendering - something for + Xlib aficionados in any case). + 2007-05-03 Michael Smith * gst/audiorate/gstaudiorate.c: (gst_audio_rate_chain): diff --git a/sys/ximage/ximagesink.c b/sys/ximage/ximagesink.c index af50cc0..321a1fc 100644 --- a/sys/ximage/ximagesink.c +++ b/sys/ximage/ximagesink.c @@ -500,6 +500,8 @@ gst_ximagesink_ximage_new (GstXImageSink * ximagesink, GstCaps * caps) } else #endif /* HAVE_XSHM */ { + guint allocsize; + ximage->ximage = XCreateImage (ximagesink->xcontext->disp, ximagesink->xcontext->visual, ximagesink->xcontext->depth, @@ -519,9 +521,27 @@ gst_ximagesink_ximage_new (GstXImageSink * ximagesink, GstCaps * caps) goto beach; } + /* upstream will assume that rowstrides are multiples of 4, but this + * doesn't always seem to be the case with XCreateImage() */ + if ((ximage->ximage->bytes_per_line % 4) != 0) { + GST_WARNING_OBJECT (ximagesink, "returned stride not a multiple of 4 as " + "usually assumed"); + } + /* we have to use the returned bytes_per_line for our image size */ ximage->size = ximage->ximage->bytes_per_line * ximage->ximage->height; - ximage->ximage->data = g_malloc (ximage->size); + + /* alloc a bit more for unexpected strides to avoid crashes upstream. + * FIXME: if we get an unrounded stride, the image will be displayed + * distorted, since all upstream elements assume a rounded stride */ + allocsize = + GST_ROUND_UP_4 (ximage->ximage->bytes_per_line) * + ximage->ximage->height; + ximage->ximage->data = g_malloc (allocsize); + GST_LOG_OBJECT (ximagesink, + "non-XShm image size is %" G_GSIZE_FORMAT " (alloced: %u), width %d, " + "stride %d", ximage->size, allocsize, ximage->width, + ximage->ximage->bytes_per_line); XSync (ximagesink->xcontext->disp, FALSE); } -- 2.7.4