PR_Accept() should retry after getting ECONNABORTED.

RESOLVED FIXED in 4.2

Status

defect
P2
normal
RESOLVED FIXED
18 years ago
18 years ago

People

(Reporter: wtc, Assigned: wtc)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Assignee

Description

18 years ago
The ECONNABORTED error of accept() means that a pending
connect request in the listen queue was aborted by the
peer before we could accept the connection.  This is not
a fatal error.  If the listening socket is blocking,
PR_Accept() should handle ECONNABORTED the same way it
handles EAGAIN or EWOULDBLOCK and loop back to retry
accept().
Assignee

Comment 1

18 years ago
Assignee

Comment 3

18 years ago
I checked in the proposed patch v2 on the trunk.

I only fixed the Unix implementation of PR_Accept().
I don't know whether Winsock's accept() would also
fail with the WSAECONNABORTED error under the same
condition.  I guess we will fix it when such errors
are observed or reported by a customer.
Status: NEW → RESOLVED
Closed: 18 years ago
Priority: -- → P2
Resolution: --- → FIXED
Target Milestone: --- → 4.2
Assignee

Updated

18 years ago
OS: Windows 98 → All
Hardware: PC → All
You need to log in before you can comment on or make changes to this bug.