lib: libsmbclient: Ensure cli_rename() always sets cli->raw_status.

Identical change as used in cli_unlink(), cli_mkdir(), cli_rmdir()
cli_chkpath() to ensure SMB2 calls correctly set raw_status for
libsmbclient uses.

Remove knownfail.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14938

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed Feb  2 21:50:31 UTC 2022 on sn-devel-184

diff --git a/selftest/knownfail.d/libsmbclient_rename b/selftest/knownfail.d/libsmbclient_rename
deleted file mode 100644 (file)
index 88343b7..0000000
--- a/selftest/knownfail.d/libsmbclient_rename
+++ /dev/null
@@ -1 +0,0 @@
-^samba4.libsmbclient.rename.SMB3.rename\(nt4_dc\)

diff --git a/source3/libsmb/clifile.c b/source3/libsmb/clifile.c
index f1d0a9483f6e802929c876d576c18301df598968..3c3f44923fcf48d2620d1f24c910009b3af25517 100644 (file)
--- a/source3/libsmb/clifile.c
+++ b/source3/libsmb/clifile.c
@@ -1489,6 +1489,7 @@ NTSTATUS cli_rename(struct cli_state *cli,
        }
 
        status = cli_rename_recv(req);
+       cli->raw_status = status; /* cli_smb2_rename_recv doesn't set this */
 
  fail:
        TALLOC_FREE(frame);