Hi Paolo, On 10/06/2020 10:47, Paolo Abeni wrote:
The msk sk_shutdown flag is set by a workqueue, possibly introducing some delay in user-space notification. If the last subflow carries some data with the fin packet, the user space can wake-up before RCV_SHUTDOWN is set. If it executes unblocking recvmsg(), it may return with an error instead of eof.Address the issue explicitly checking for eof in recvmsg(), when no data is found. Fixes: 59832e246515 ("mptcp: subflow: check parent mptcp socket on subflow state change") Signed-off-by: Paolo Abeni <[email protected]>
Thank you for the patch, it looks good to me! Reviewed-by: Matthieu Baerts <[email protected]> Cheers, Matt -- Matthieu Baerts | R&D Engineer [email protected] Tessares SA | Hybrid Access Solutions www.tessares.net 1 Avenue Jean Monnet, 1348 Louvain-la-Neuve, Belgium
