RE: [PATCH] rsocket: Return ECONNRESET when socket in recv is disconnected

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

 



Hi Jason & Sean,

POSIX recv() manual page describes behavior as follows (reproduced here for convenience) -

http://linux.die.net/man/3/recv

==========
Return Value

Upon successful completion, recv() shall return the length of the message in bytes. If no messages are available to be received and the peer has performed an orderly shutdown, recv() shall return 0. Otherwise, -1 shall be returned and errno set to indicate the error.

Errors

The recv() function shall fail if:

ECONNRESET
    A connection was forcibly closed by a peer.
==========

So the current behavior of returning 0 is wrong as there was no orderly shutdown performed on the other end before a close() is issued [ http://stackoverflow.com/questions/5879560/how-can-i-cause-an-econnreset-in-recv-from-a-client ].

Thank You.

- Sreedhar


On 2014-10-09 22:14, Hefty, Sean wrote:
Thanks for the fix.  Now, we are able to see the similar behavior
between tcp/ip and rsockets when the other end is suddenly closed
while waiting on data.

Can you read and respond to Jason's comments on this patch?

That isn't the whole story though - you only get ECONNRESET in some
cases, and it is OS dependent.

stackoverflow suggets:

http://stackoverflow.com/questions/2974021/what-does-econnreset-mean-in-the-context-of-an-af-local-socket

Which looks reasonable to me and matches my experience with socket
programming on Linux. The Steven's book might have some authoritative
clarifications on the subject as well.

Based on this information, it's not clear that the current behavior of
returning 0 is wrong.

- Sean

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux