* xfns.c (make_invisible_cursor): Don't return garbage

if XCreateBitmapFromData fails.

diff --git a/src/ChangeLog b/src/ChangeLog
index 17cc06bfe32ccc3544e83e867b860f5c2a3484fc..b2a337b187858602843e7c617c9e2e54b4852a9a 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,5 +1,8 @@
 2011-04-02  Paul Eggert  <eggert@cs.ucla.edu>
 
+       * xfns.c (make_invisible_cursor): Don't return garbage
+       if XCreateBitmapFromData fails.
+
        * xselect.c (x_get_local_selection, x_handle_property_notify):
        Remove vars that are set but not used.
 

diff --git a/src/xfns.c b/src/xfns.c
index 39c77b8c8c7b258dc0c45e1c9bc546988ffa2bea..8e5639681dfb74df1ce4a465fdfae640e3a5131c 100644 (file)
--- a/src/xfns.c
+++ b/src/xfns.c
@@ -855,19 +855,20 @@ make_invisible_cursor (struct frame *f)
   static char const no_data[] = { 0 };
   Pixmap pix;
   XColor col;
-  Cursor c;
+  Cursor c = 0;
 
   x_catch_errors (dpy);
   pix = XCreateBitmapFromData (dpy, FRAME_X_DISPLAY_INFO (f)->root_window,
                                no_data, 1, 1);
   if (! x_had_errors_p (dpy) && pix != None)
     {
+      Cursor pixc;
       col.pixel = 0;
       col.red = col.green = col.blue = 0;
       col.flags = DoRed | DoGreen | DoBlue;
-      c = XCreatePixmapCursor (dpy, pix, pix, &col, &col, 0, 0);
-      if (x_had_errors_p (dpy) || c == None)
-        c = 0;
+      pixc = XCreatePixmapCursor (dpy, pix, pix, &col, &col, 0, 0);
+      if (! x_had_errors_p (dpy) && pixc != None)
+        c = pixc;
       XFreePixmap (dpy, pix);
     }