Re: cursor for display segv

Torsten Schoenfeld <kaffeetisch gmx de> writes:

GdkCursor seems to simply assume that the GdkDisplay is valid
throughout the cursor's lifetime, but it doesn't take a reference on

Ah yeah.

I don't think it's the bindings' responsibility to do something
about this.

What about a few words below, only because it's one of those things were
seemingly innocent perl code can crash.

GdkCursor should just take a reference on the display.

Hmm.  Sounds likely.  Maybe someone thought it made destruction easier
not to.

diff --git a/xs/GdkCursor.xs b/xs/GdkCursor.xs
index c79846f..e6525de 100644
--- a/xs/GdkCursor.xs
+++ b/xs/GdkCursor.xs
@@ -1,5 +1,5 @@
- * Copyright (c) 2003-2005 by the gtk2-perl team (see the file AUTHORS)
+ * Copyright (c) 2003-2005, 2010 by the gtk2-perl team (see the file AUTHORS)
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ -23,6 +23,22 @@
 MODULE = Gtk2::Gdk::Cursor     PACKAGE = Gtk2::Gdk::Cursor     PREFIX = gdk_cursor_
+=for position DESCRIPTION
+For reference, cursors are a per-display resource and can only be used
+with the display they were created on.
+As of Gtk 2.22 a cursor doesn't keep a reference to its
+C<Gtk2::Gdk::Display> and if the display object is destroyed before
+the cursor then a later destroy of the cursor may get a segv.
+Perl-Gtk2 doesn't try to do anything about this.  Care may be needed
+if keeping a cursor separate from a widget or window.  (Closing the
+display is fine, but not destroying it.)
 gdk_cursor_type (cursor)
        GdkCursor *cursor

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]