sunrpc: handle -ENOTCONN in xs_tcp_setup_socket()

BugLink: https://bugs.launchpad.net/bugs/2100292

[ Upstream commit 10f0740234f0b157b41bdc7e9c3555a9b86c1599 ]

xs_tcp_finish_connecting() can return -ENOTCONN but the switch statement
in xs_tcp_setup_socket() treats that as an unhandled error.

If we treat it as a known error it would propagate back to
call_connect_status() which does handle that error code.  This appears
to be the intention of the commit (given below) which added -ENOTCONN as
a return status for xs_tcp_finish_connecting().

So add -ENOTCONN to the switch statement as an error to pass through to
the caller.

Link: https://bugzilla.suse.com/show_bug.cgi?id=1231050
Link: https://access.redhat.com/discussions/3434091
Fixes: 01d37c428a ("SUNRPC: xprt_connect() don't abort the task if the transport isn't bound")
Signed-off-by: NeilBrown <neilb@suse.de>
Reviewed-by: Benjamin Coddington <bcodding@redhat.com>
Signed-off-by: Anna Schumaker <anna.schumaker@oracle.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Signed-off-by: Koichiro Den <koichiro.den@canonical.com>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>

This commit is contained in:

NeilBrown

2025-02-26 22:49:14 +09:00

committed by

Stefan Bader

parent 9c54a14746

commit 17cdba58b5

1 changed files with 1 additions and 0 deletions

									
										net/sunrpc/xprtsock.c
									
		+1
		
												View File
												
				@@ -2441,6 +2441,7 @@ static void xs_tcp_setup_socket(struct work_struct *work)

					case -EHOSTUNREACH:

					case -EADDRINUSE:

					case -ENOBUFS:

					case -ENOTCONN:

						break;

					default:

						printk("%s: connect returned unhandled error %d\n",