[lwip-devel] [bug #28818] New connection probably not marked writable on accept()

[Alexander Y]

URL:
  <http://savannah.nongnu.org/bugs/?28818>

                 Summary: New connection probably not marked writable on
accept()
                 Project: lwIP - A Lightweight TCP/IP stack
            Submitted by: binturong
            Submitted on: Fri 05 Feb 2010 09:06:13 AM GMT
                Category: sockets
                Severity: 3 - Normal
              Item Group: Faulty Behaviour
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 
            lwIP version: CVS Head

    _______________________________________________________

Details:

This recent modification to sockets.c looks problematic:

   227        /* TCP sendbuf is empty, but not connected yet, so not yet
writable */
   228        sockets[i].sendevent  = (newconn->type == NETCONN_TCP ? 0 : 1);


However, sendevent appears to be only set by NETCONN_EVT_SENDPLUS, and that
is only generated from sent_tcp(), err_tcp(), do_close_internal(),
do_connected().

Shouldn't sendevent be raised on accept() as well? What should a server do if
it listens and accepts a connection and then wants to write to the socket?





    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?28818>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/