Hi,

as cvs log shows, accept(2) can error out with errno set to
ECONNABORTED, and it is easy to forget that this shouldn't be a fatal
error.  I suggest the following diff or something similar.

Index: accept.2
===================================================================
RCS file: /cvs/src/lib/libc/sys/accept.2,v
retrieving revision 1.25
diff -u -p -r1.25 accept.2
--- accept.2    21 Apr 2013 07:44:59 -0000      1.25
+++ accept.2    30 Apr 2013 16:18:38 -0000
@@ -147,7 +147,7 @@ int retcode;
 
 retcode = accept(s, (struct sockaddr *)&addr, &len);
 if (retcode == -1)
-       err(1, "accept");
+       err(1, "accept"); /* Probably too simplistic. */
 .Ed
 .Sh ERRORS
 .Fn accept
@@ -211,3 +211,5 @@ Thus considerable care is required in
 and
 .Xr poll 2
 loops.
+.Er ECONNABORTED
+is often overlooked and should be handled appropriately.

Reply via email to