diff options
author | Ralph Boehme <slow@samba.org> | 2021-02-16 14:24:05 +0100 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2021-02-22 10:53:26 +0000 |
commit | 1f520ec35e151bcde8103ba6cfbdced67b242d0a (patch) | |
tree | 6d0485c2a894dce6ec745536b09ed93566e334e2 | |
parent | b30a1950e260fb7fd098da2deff608b7c505ef0d (diff) | |
download | samba-1f520ec35e151bcde8103ba6cfbdced67b242d0a.tar.gz |
printing: use correct error out in get_correct_cversion() when openat_pathref_fsp() fails
Fixes a regression introduced by a74f0af1a91fe0bbc68e4d41d65f43ec383ae8bf: if
there's no existing file, openat_pathref_fsp() will fail with
NT_STATUS_OBJECT_NAME_NOT_FOUND which must be handled the same way it is done by
the SMB_VFS_CREATE_FILE() call below.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=14635
RN: Printerdriver upload 4.14rc1 not working
Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Guenther Deschner <gd@samba.org>
Autobuild-User(master): Günther Deschner <gd@samba.org>
Autobuild-Date(master): Wed Feb 17 19:53:00 UTC 2021 on sn-devel-184
(cherry picked from commit 718f7b1a84f1c6eb35d52232a8573370f45add56)
-rw-r--r-- | source3/printing/nt_printing.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/source3/printing/nt_printing.c b/source3/printing/nt_printing.c index 6fbe6780370..64033043675 100644 --- a/source3/printing/nt_printing.c +++ b/source3/printing/nt_printing.c @@ -1104,7 +1104,10 @@ static uint32_t get_correct_cversion(const struct auth_session_info *session_inf nt_status = openat_pathref_fsp(conn->cwd_fsp, smb_fname); if (!NT_STATUS_IS_OK(nt_status)) { - return false; + DBG_NOTICE("Can't open file [%s], errno =%d\n", + smb_fname_str_dbg(smb_fname), errno); + *perr = WERR_ACCESS_DENIED; + goto error_exit; } nt_status = SMB_VFS_CREATE_FILE( |