emacs-bug-tracker
[Top][All Lists]
Advanced

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

[debbugs-tracker] bug#9754: closed (emacs -nv fails on glib 2.31)


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#9754: closed (emacs -nv fails on glib 2.31)
Date: Sat, 19 May 2012 21:58:02 +0000

Your message dated Sat, 19 May 2012 17:56:33 -0400
with message-id <address@hidden>
and subject line Re: bug#9754: Issue with Emacs 23.4
has caused the debbugs.gnu.org bug report #9754,
regarding emacs -nv fails on glib 2.31
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
9754: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=9754
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: emacs -nv fails on glib 2.31 Date: Fri, 14 Oct 2011 12:18:18 -0400
Packages: emacs
Version: 23.3

When running "emacs -nw" with the latest glib version, emacs gets stuck
in this infinite loop:

in xg_select at /usr/src/debug/emacs-23.3/src/xgselect.c:59

58              while (n_gfds > gfds_size) 
59                gfds_size *= 2;

This code is buggy in the case that gfds_size is zero (since clearly, no
matter how many times you multiply by 2, you're not going to increase
it).

Further down in the same file, you see:

155     void
156     xgselect_initialize ()
157     {
158     #if defined (USE_GTK) || defined (HAVE_GCONF)
159       gfds_size = 128;
160       gfds = xmalloc (sizeof (*gfds)*gfds_size);


So it's clear that xgselect_initialize() is not being called in the
"-nw" case.  That makes sense -- why initialise GTK when not using it?

The problem is that xg_select() is used even in the "-nw" case, without
_initialize() having been called.  This worked before because an unused
GMainContext used to have 0 fds in it, so n_gfds would be zero and
gfds_size would not need to be increased, causing the bug to be skipped
over.

Recent changes in glib have introduced one fd to every GMainContext to
deal with the inherent race introduced by signal delivery (closing a
longstanding glib bug).  This means that the untouched GMainContext no
longer has 0 fds -- but 1.  This is what is triggering the problem in
the buggy code above.


The solution to this problem is one of:

 - ensure xgselect_initialize() is always called
 - don't use xg_select in -nw case
 - fix the code to deal with the array being zero-sized and nuke
   xgselect_initialize()




--- End Message ---
--- Begin Message --- Subject: Re: bug#9754: Issue with Emacs 23.4 Date: Sat, 19 May 2012 17:56:33 -0400 User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0) Gecko/20120327 Thunderbird/11.0.1
Version: 24.2

I've committed the change as bzr revision 108316, and I'm closing the bug.

Ken





--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]