From 8e25e2f3408bda87e2acffa3fc464c32173bb814 Mon Sep 17 00:00:00 2001 From: David Mulder Date: Wed, 11 Apr 2018 12:40:18 -0600 Subject: [PATCH] libgpo: gpo_copy_file() shouldn't explicitly call smb1 Don't call cli_openx directly to open a file this calls smb1 code explicitly, which fails if we did a multi-protocol negotiate and negotiated smb2+. Use the higher level cli_open() instead. Signed-off-by: David Mulder Reviewed-by: Douglas Bagnall Reviewed-by: Andrew Bartlett --- libgpo/gpo_filesync.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libgpo/gpo_filesync.c b/libgpo/gpo_filesync.c index 6e3efdaf6c1..bf0bb5381fc 100644 --- a/libgpo/gpo_filesync.c +++ b/libgpo/gpo_filesync.c @@ -50,7 +50,7 @@ NTSTATUS gpo_copy_file(TALLOC_CTX *mem_ctx, int read_size = io_bufsize; off_t nread = 0; - result = cli_openx(cli, nt_path, O_RDONLY, DENY_NONE, &fnum); + result = cli_open(cli, nt_path, O_RDONLY, DENY_NONE, &fnum); if (!NT_STATUS_IS_OK(result)) { goto out; }