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

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

[debbugs-tracker] bug#23615: closed (25.1.50; Which platforms can safely


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#23615: closed (25.1.50; Which platforms can safely use getsockopt(,,SO_ERROR,,)?)
Date: Fri, 10 Jun 2016 12:39:02 +0000

Your message dated Fri, 10 Jun 2016 08:38:36 -0400
with message-id <address@hidden>
and subject line Re: bug#23615: 25.1.50; Which platforms can safely use 
getsockopt(,,SO_ERROR,,)?
has caused the debbugs.gnu.org bug report #23615,
regarding 25.1.50; Which platforms can safely use getsockopt(,,SO_ERROR,,)?
to be marked as done.

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


-- 
23615: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=23615
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 25.1.50; Which platforms can safely use getsockopt(,,SO_ERROR,,)? Date: Tue, 24 May 2016 20:26:13 -0400 User-agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.1.0
There are two places in process.c where getsockopt(,,SO_ERROR,,) is
used to check the status of a socket connection attempt.  The first is
at line 3289, where it is done on all platforms except MS Windows.  The
second is at line 5500, where it is done only on GNU/Linux:

#ifdef GNU_LINUX
              /* getsockopt(,,SO_ERROR,,) is said to hang on some systems.
                 So only use it on systems where it is known to work.  */
              {
                socklen_t xlen = sizeof (xerrno);
                if (getsockopt (channel, SOL_SOCKET, SO_ERROR, &xerrno, &xlen))
                  xerrno = errno;
              }
#else
              {
                struct sockaddr pname;
                socklen_t pnamelen = sizeof (pname);

                /* If connection failed, getpeername will fail.  */
                xerrno = 0;
                if (getpeername (channel, &pname, &pnamelen) < 0)
                  {
                    /* Obtain connect failure code through error slippage.  */
                    char dummy;
                    xerrno = errno;
                    if (errno == ENOTCONN && read (channel, &dummy, 1) < 0)
                      xerrno = errno;
                  }
              }
#endif

It would be better to use it on as many platforms as possible, since
it's much more likely to give the real reason for a connection failure
than the "error slippage" method.

Ken



--- End Message ---
--- Begin Message --- Subject: Re: bug#23615: 25.1.50; Which platforms can safely use getsockopt(,,SO_ERROR,,)? Date: Fri, 10 Jun 2016 08:38:36 -0400 User-agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 I see that you've made the change I suggested, so my test program is no longer relevant. I'm closing the bug.

Ken


--- End Message ---

reply via email to

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