Hi Greg, I am wondering if it is possible to include exact stable kernel version (e.g. 4.20.3 in this case) in the email. This would help to quickly understand which kernel version should be installed in order to get the fix.
Best regards, Pavel Shilovsky -----Original Message----- From: Greg Kroah-Hartman <[email protected]> Sent: Tuesday, January 15, 2019 8:36 AM To: [email protected] Cc: Greg Kroah-Hartman <[email protected]>; [email protected]; Pavel Shilovskiy <[email protected]>; Jeff Layton <[email protected]>; Steven French <[email protected]> Subject: [PATCH 4.20 14/57] CIFS: Do not hide EINTR after sending network packets 4.20-stable review patch. If anyone has any objections, please let me know. ------------------ From: Pavel Shilovsky <[email protected]> commit ee13919c2e8d1f904e035ad4b4239029a8994131 upstream. Currently we hide EINTR code returned from sock_sendmsg() and return 0 instead. This makes a caller think that we successfully completed the network operation which is not true. Fix this by properly returning EINTR to callers. Cc: <[email protected]> Signed-off-by: Pavel Shilovsky <[email protected]> Reviewed-by: Jeff Layton <[email protected]> Signed-off-by: Steve French <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]> --- fs/cifs/transport.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/fs/cifs/transport.c +++ b/fs/cifs/transport.c @@ -385,7 +385,7 @@ smbd_done: if (rc < 0 && rc != -EINTR) cifs_dbg(VFS, "Error %d sending data on socket to server\n", rc); - else + else if (rc > 0) rc = 0; return rc;
