GNU bug report logs - #49496
28.0.50; process-tests/fd-setsize-no-crash/make-network-process hangs on Cygwin

Previous Next

Full log

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ken Brown <kbrown <at> cornell.edu>
To: bug-gnu-emacs <at> gnu.org
Subject: 28.0.50; process-tests/fd-setsize-no-crash/make-network-process hangs
 on Cygwin
Date: Fri, 9 Jul 2021 14:01:33 -0400

The hang can actually be triggered by evaluating the following, extracted from 
the test in the subject:

(setq socket-name "/tmp/socktest/socket")
(delete-file socket-name)
(make-network-process :name "server"
                      :server 10
                      :buffer nil
                      :service socket-name
                      :family 'local
                      :coding 'no-conversion
                      :noquery t)
(make-network-process :name "client"
		      :service socket-name
		      :family 'local
		      :coding 'no-conversion
		      :noquery t)

The reason for the hang is that Cygwin does a blocking handshake in 
accept/connect to exchange credentials.  This can hang in certain situations in 
which connect is called before a listening socket is ready to accept connections.

It is possible to work around this problem by disabling the credential exchange 
as in the following patch:

diff --git a/src/process.c b/src/process.c
index b8c3e4ecfb..01548c8ab5 100644
--- a/src/process.c
+++ b/src/process.c
@@ -3355,6 +3355,12 @@ connect_network_socket (Lisp_Object proc, Lisp_Object 
addrinfos,
 	      xerrno = EMFILE;
 	      continue;
 	    }
+#ifdef CYGWIN
+	  /* Avoid possible hang in connect/accept.  */
+	  if (family == AF_LOCAL && p->socktype == SOCK_STREAM
+	      && setsockopt (s, SOL_SOCKET, SO_PEERCRED, 0, 0) < 0)
+	    report_file_error ("Cannot disable credential exchange", Qnil);
+#endif
 	}
        if (p->is_non_blocking_client && ! (SOCK_NONBLOCK && socket_to_use < 0))

But this seems too drastic just to fix a potential problem that doesn't seem to 
occur in practice, but only in the contrived situation of 
process-tests/fd-setsize-no-crash/make-network-process.  So my preference is to 
simply skip that test on Cygwin.

OK?

In GNU Emacs 28.0.50 (build 9, x86_64-pc-cygwin, GTK+ Version 3.22.28, cairo 
version 1.17.4)
 of 2021-07-09 built on moufang2
Repository revision: 68276f6d30bbdc09cc26fb49d7f0c3aa4bce35f2
Repository branch: master
Windowing system distributor 'The Cygwin/X Project', version 11.0.12011000
Configured using:
 'configure 'CFLAGS=-g -O0''

Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GSETTINGS HARFBUZZ JPEG JSON
LCMS2 LIBOTF LIBXML2 M17N_FLT MODULES NOTIFY GFILENOTIFY PDUMPER PNG
RSVG SOUND THREADS TIFF TOOLKIT_SCROLL_BARS X11 XDBE XIM XPM GTK3 ZLIB

Previous Next

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