From 5ef50a826296707af689d21cd8a782e26ae745dd Mon Sep 17 00:00:00 2001 From: Fabian Ebner Date: Mon, 15 Nov 2021 13:37:56 +0100 Subject: [PATCH] cifs: check connection: bubble up NT_STATUS_LOGON_FAILURE in the same manner as NT_STATUS_ACCESS_DENIED. It can be assumed to be a configuration error, so avoid showing the generic "storage is not online". Reported in the community forum: https://forum.proxmox.com/threads/storage-is-not-online-cifs.99201/post-428858 Signed-off-by: Fabian Ebner --- PVE/Storage/CIFSPlugin.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/Storage/CIFSPlugin.pm b/PVE/Storage/CIFSPlugin.pm index a3f9ebe..d5efa5f 100644 --- a/PVE/Storage/CIFSPlugin.pm +++ b/PVE/Storage/CIFSPlugin.pm @@ -280,7 +280,7 @@ sub check_connection { if (my $err = $@) { die "$out_str\n" if defined($out_str) && - ($out_str =~ m/NT_STATUS_ACCESS_DENIED/); + ($out_str =~ m/NT_STATUS_(ACCESS_DENIED|LOGON_FAILURE)/); return 0; }