From f4090b5c847293e94186e13144d8ac5ce0f9db4d Mon Sep 17 00:00:00 2001 From: Alexander Larsson Date: Mon, 2 Mar 2009 15:09:27 +0000 Subject: [PATCH] =?UTF-8?q?Bug=20573087=20=E2=80=93=20gdkwindow.c:=20{x,y}?= =?UTF-8?q?=5Foffset=20used=20uninitialized?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 2009-03-02 Alexander Larsson Bug 573087 – gdkwindow.c: {x,y}_offset used uninitialized * gdk/gdkwindow.c (gdk_window_clear_backing_rect_redirect): Don't use uninitialized vars. Fix offsets for redirected pixmap, draw to pixmap not window. svn path=/trunk/; revision=22441 --- ChangeLog | 8 ++++++++ gdk/gdkwindow.c | 19 ++++++++++++++----- 2 files changed, 22 insertions(+), 5 deletions(-) diff --git a/ChangeLog b/ChangeLog index da357b2ece..1287e765c7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2009-03-02 Alexander Larsson + + Bug 573087 – gdkwindow.c: {x,y}_offset used uninitialized + + * gdk/gdkwindow.c (gdk_window_clear_backing_rect_redirect): + Don't use uninitialized vars. + Fix offsets for redirected pixmap, draw to pixmap not window. + 2009-03-02 Theppitak Karoonboonyanan Bug 457086 - numpad does not work when the Thai-Lao input method is diff --git a/gdk/gdkwindow.c b/gdk/gdkwindow.c index 66b83ae85d..76a2ec618d 100644 --- a/gdk/gdkwindow.c +++ b/gdk/gdkwindow.c @@ -2116,8 +2116,8 @@ gdk_window_clear_backing_rect_redirect (GdkWindow *window, if (GDK_WINDOW_DESTROYED (window)) return; - paint.x_offset = x_offset; - paint.y_offset = y_offset; + paint.x_offset = 0; + paint.y_offset = 0; paint.pixmap = redirect->pixmap; paint.surface = _gdk_drawable_ref_cairo_surface (redirect->pixmap); @@ -2125,10 +2125,19 @@ gdk_window_clear_backing_rect_redirect (GdkWindow *window, GDK_WINDOW (redirect->redirected), NULL, TRUE, &x_offset, &y_offset); - + /* offset is from redirected window origin to window origin, convert to + the offset from the redirected pixmap origin to the window origin */ + x_offset += redirect->dest_x - redirect->src_x; + y_offset += redirect->dest_y - redirect->src_y; + + /* Convert region and rect to pixmap coords */ + gdk_region_offset (clip_region, x_offset, y_offset); + x += x_offset; + y += y_offset; + method.cr = NULL; method.gc = NULL; - setup_backing_rect_method (&method, window, &paint, 0, 0); + setup_backing_rect_method (&method, window, &paint, -x_offset, -y_offset); if (method.cr) { @@ -2147,7 +2156,7 @@ gdk_window_clear_backing_rect_redirect (GdkWindow *window, g_assert (method.gc != NULL); gdk_gc_set_clip_region (method.gc, clip_region); - gdk_draw_rectangle (window, method.gc, TRUE, x, y, width, height); + gdk_draw_rectangle (redirect->pixmap, method.gc, TRUE, x, y, width, height); g_object_unref (method.gc); }