octoleo/restic
2
2
Fork 0
mirror of https://github.com/octoleo/restic.git synced 2024-11-23 13:17:42 +00:00
Code Issues Releases Wiki Activity

Gracefully handle invalid prepared volume names

Browse Source
This commit is contained in:
aneesh-n 2024-08-11 01:48:25 -06:00 committed by Michael Eischer
parent 111490b8be
commit 51fad2eecb
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
internal/restic/node_windows.go
View File

@ -417,11 +417,13 @@ func checkAndStoreEASupport(path string) (isEASupportedVolume bool, err error) {
// First check if the manually prepared volume name is already in the map // First check if the manually prepared volume name is already in the map
eaSupportedValue, exists := eaSupportedVolumesMap.Load(volumeName) eaSupportedValue, exists := eaSupportedVolumesMap.Load(volumeName)
if exists { if exists {
// Cache hit, immediately return the cached value
return eaSupportedValue.(bool), nil return eaSupportedValue.(bool), nil
} }
// If not found, check if EA is supported with manually prepared volume name // If not found, check if EA is supported with manually prepared volume name
isEASupportedVolume, err = fs.PathSupportsExtendedAttributes(volumeName + `\`) isEASupportedVolume, err = fs.PathSupportsExtendedAttributes(volumeName + `\`)
if err != nil { // If the prepared volume name is not valid, we will next fetch the actual volume name.
if err != nil && !errors.Is(err, windows.DNS_ERROR_INVALID_NAME) {
return false, err return false, err
} }
} }
@ -434,6 +436,7 @@ func checkAndStoreEASupport(path string) (isEASupportedVolume bool, err error) {
// If the actual volume name is different, check cache for the actual volume name // If the actual volume name is different, check cache for the actual volume name
eaSupportedValue, exists := eaSupportedVolumesMap.Load(volumeNameActual) eaSupportedValue, exists := eaSupportedVolumesMap.Load(volumeNameActual)
if exists { if exists {
// Cache hit, immediately return the cached value
return eaSupportedValue.(bool), nil return eaSupportedValue.(bool), nil
} }
// If the actual volume name is different and is not in the map, again check if the new volume supports extended attributes with the actual volume name // If the actual volume name is different and is not in the map, again check if the new volume supports extended attributes with the actual volume name