Revert "socket/netconn recv: FIN should only be reported once (as '0' for sockets, as 'ERR_CLSD' for netconns)"

This reverts commit db3a4e31586f8614b0a27b482406b0c212a4e08a.

See description of regression in bug #52417
This commit is contained in:
Joel Cunningham 2017-11-20 13:54:11 -06:00
parent 6af3b4accc
commit ebcae98ae6

View File

@ -646,8 +646,8 @@ netconn_recv_data_tcp(struct netconn *conn, struct pbuf **new_buf, u8_t apiflags
#endif /* LWIP_TCP */ #endif /* LWIP_TCP */
if (!sys_mbox_valid(&conn->recvmbox)) { if (!sys_mbox_valid(&conn->recvmbox)) {
/* This only happens when calling this function more than once *after* receiving FIN */ /* This happens when calling this function after receiving FIN */
return ERR_CONN; return sys_mbox_valid(&conn->acceptmbox) ? ERR_CONN : ERR_CLSD;
} }
if (!(apiflags & NETCONN_NOAUTORCVD)) { if (!(apiflags & NETCONN_NOAUTORCVD)) {