|
 |
bf3537 |
From 79381295b788a8196ccbf2ff378268286d7782d5 Mon Sep 17 00:00:00 2001
|
|
|
bf3537 |
From: Jeremy Allison <jra@samba.org>
|
|
|
bf3537 |
Date: Fri, 8 Sep 2017 16:20:34 -0700
|
|
|
bf3537 |
Subject: [PATCH] libsmbclient: Allow server (NetApp) to return
|
|
|
bf3537 |
STATUS_INVALID_PARAMETER from an echo.
|
|
|
bf3537 |
|
|
|
bf3537 |
It does this if we send a session ID of zero. The server still replied.
|
|
|
bf3537 |
|
|
|
bf3537 |
BUG: https://bugzilla.samba.org/show_bug.cgi?id=13007
|
|
|
bf3537 |
|
|
|
bf3537 |
Signed-off-by: Jeremy Allison <jra@samba.org>
|
|
|
bf3537 |
Reviewed-by: Andreas Schneider <asn@samba.org>
|
|
|
bf3537 |
|
|
|
bf3537 |
Autobuild-User(master): Jeremy Allison <jra@samba.org>
|
|
|
bf3537 |
Autobuild-Date(master): Sat Nov 11 08:44:37 CET 2017 on sn-devel-144
|
|
|
bf3537 |
|
|
|
bf3537 |
(cherry picked from commit a0f6ea8dec1ab3d19bc93da12a9b0a1c0ccf6142)
|
|
|
bf3537 |
---
|
|
|
bf3537 |
source3/client/client.c | 8 +++++++-
|
|
|
bf3537 |
source3/libsmb/libsmb_server.c | 11 ++++++++++-
|
|
|
bf3537 |
2 files changed, 17 insertions(+), 2 deletions(-)
|
|
|
bf3537 |
|
|
|
bf3537 |
diff --git a/source3/client/client.c b/source3/client/client.c
|
|
|
bf3537 |
index b4a6c7d0389..9c57375881d 100644
|
|
|
bf3537 |
--- a/source3/client/client.c
|
|
|
bf3537 |
+++ b/source3/client/client.c
|
|
|
bf3537 |
@@ -5900,7 +5900,13 @@ static void readline_callback(void)
|
|
|
bf3537 |
/* Ping the server to keep the connection alive using SMBecho. */
|
|
|
bf3537 |
memset(garbage, 0xf0, sizeof(garbage));
|
|
|
bf3537 |
status = cli_echo(cli, 1, data_blob_const(garbage, sizeof(garbage)));
|
|
|
bf3537 |
- if (NT_STATUS_IS_OK(status)) {
|
|
|
bf3537 |
+ if (NT_STATUS_IS_OK(status) ||
|
|
|
bf3537 |
+ NT_STATUS_EQUAL(status, NT_STATUS_INVALID_PARAMETER)) {
|
|
|
bf3537 |
+ /*
|
|
|
bf3537 |
+ * Even if server returns NT_STATUS_INVALID_PARAMETER
|
|
|
bf3537 |
+ * it still responded.
|
|
|
bf3537 |
+ * BUG: https://bugzilla.samba.org/show_bug.cgi?id=13007
|
|
|
bf3537 |
+ */
|
|
|
bf3537 |
return;
|
|
|
bf3537 |
}
|
|
|
bf3537 |
|
|
|
bf3537 |
diff --git a/source3/libsmb/libsmb_server.c b/source3/libsmb/libsmb_server.c
|
|
|
bf3537 |
index b0e5926fa65..2d41f2facf3 100644
|
|
|
bf3537 |
--- a/source3/libsmb/libsmb_server.c
|
|
|
bf3537 |
+++ b/source3/libsmb/libsmb_server.c
|
|
|
bf3537 |
@@ -61,7 +61,16 @@ SMBC_check_server(SMBCCTX * context,
|
|
|
bf3537 |
1,
|
|
|
bf3537 |
data_blob_const(data, sizeof(data)));
|
|
|
bf3537 |
if (!NT_STATUS_IS_OK(status)) {
|
|
|
bf3537 |
- return 1;
|
|
|
bf3537 |
+ /*
|
|
|
bf3537 |
+ * Some NetApp servers return
|
|
|
bf3537 |
+ * NT_STATUS_INVALID_PARAMETER.That's OK, they still
|
|
|
bf3537 |
+ * replied.
|
|
|
bf3537 |
+ * BUG: https://bugzilla.samba.org/show_bug.cgi?id=13007
|
|
|
bf3537 |
+ */
|
|
|
bf3537 |
+ if (!NT_STATUS_EQUAL(status,
|
|
|
bf3537 |
+ NT_STATUS_INVALID_PARAMETER)) {
|
|
|
bf3537 |
+ return 1;
|
|
|
bf3537 |
+ }
|
|
|
bf3537 |
}
|
|
|
bf3537 |
server->last_echo_time = now;
|
|
|
bf3537 |
}
|
|
|
bf3537 |
--
|
|
|
bf3537 |
2.15.0.448.gf294e3d99a-goog
|
|
|
bf3537 |
|