GNU bug report logs - #58288
29.0.50; (flyspell-correct-word-before-point) followed by <down> crashes emacs

Previous Next

Full log

View this message in rfc822 format

From: Robert Pluim <rpluim <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 58288 <at> debbugs.gnu.org, larsi <at> gnus.org, ajd2195 <at> columbia.edu
Subject: bug#58288: 29.0.50; (flyspell-correct-word-before-point) followed by <down> crashes emacs
Date: Wed, 05 Oct 2022 12:29:18 +0200

>>>>> On Wed, 05 Oct 2022 13:06:52 +0300, Eli Zaretskii <eliz <at> gnu.org> said:
    >> void
    >> save_menu_items (void)
    >> {
    >> Lisp_Object saved = list4 (menu_items_inuse ? menu_items : Qnil,
    >> make_fixnum (menu_items_used),
    >> make_fixnum (menu_items_n_panes),
    >> make_fixnum (menu_items_submenu_depth));
    >> record_unwind_protect (restore_menu_items, saved);
    >> menu_items_inuse = false;
    >> menu_items = Qnil;
    >> }
    >> 
    >> This fixes it, but it seems a bit dodgy to me, why has no other
    >> platform ever run into this?

    Eli> How come a menu is in use, but menu_items_inuse is zero?

`menu_items_inuse' is not zero, itʼs false. `menu_items_used' is
non-zero. I guess something called `unuse_menu_items', and the
assumption is then that `menu_items' will not be accessed (but then
why save away `menu_items_used' etc?

menu.c says:

    /* Whether the global vars defined here are already in use.
       Used to detect cases where we try to re-enter this non-reentrant code.  */
    bool menu_items_inuse;

so should we be adding "if (menu_items_inuse)" guards everywhere?

Robert
-- 

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.