Re: crash in xdisp.c: show_mouse_face

[YAMAMOTO Mitsuharu]

>>>>> On Sat, 1 Apr 2006 00:41:29 +0100, David Reitter <address@hidden> said:

> A user reported a crash under the circumstances described below.
> This is the Carbon port of GNU Emacs from March 28, with some
> patches (none to xdisp.c).  Please see the stack trace below, and
> ask Phil directly in case there are questions - I don't have more
> information.

I'm not sure if it is related, but the following change seems to have
introduced the use of the uninitialized variable `f'.

2006-03-24  Kim F. Storm  <address@hidden>

        * macterm.c (XTread_socket): Don't let key-press clear mouse face
        on in toolbar window if mouse-highlight is an integer.

I've installed the following fix to the CVS.  Please try it.  Phil, do
you possibly set the variable `mouse-highlight' to some integer?

                                     YAMAMOTO Mitsuharu
                                address@hidden

===================================================================
RCS file: /sources/emacs/emacs/src/macterm.c,v
retrieving revision 1.162
retrieving revision 1.163
diff -c -r1.162 -r1.163
*** emacs/src/macterm.c 2006/03/24 15:22:44     1.162
--- emacs/src/macterm.c 2006/04/03 06:28:39     1.163
***************
*** 10455,10460 ****
--- 10455,10462 ----
  
            ObscureCursor ();
  
+           f = mac_focus_frame (dpyinfo);
+ 
            if (!dpyinfo->mouse_face_hidden && INTEGERP (Vmouse_highlight)
                && !EQ (f->tool_bar_window, dpyinfo->mouse_face_window))
              {
***************
*** 10500,10506 ****
          inev.modifiers |= (extra_keyboard_modifiers
                             & (meta_modifier | alt_modifier
                                | hyper_modifier | super_modifier));
!         XSETFRAME (inev.frame_or_window, mac_focus_frame (dpyinfo));
          break;
  
        case kHighLevelEvent:
--- 10502,10508 ----
          inev.modifiers |= (extra_keyboard_modifiers
                             & (meta_modifier | alt_modifier
                                | hyper_modifier | super_modifier));
!         XSETFRAME (inev.frame_or_window, f);
          break;
  
        case kHighLevelEvent: