Merge pull request #1857 from k8s-infra-cherrypick-robot/cherry-pick-1855-to-release-1.28

[release-1.28] cleanup: refine extensiveMountCheck usage in unmount

Author: andyzhangx

pkg/azurefile/azure_common_linux.go

@@ -33,12 +33,12 @@ func SMBMount(m *mount.SafeFormatAndMount, source, target, fsType string, option
 	return m.MountSensitive(source, target, fsType, options, sensitiveMountOptions)
 }
 
-func SMBUnmount(m *mount.SafeFormatAndMount, target string, _, _ bool) error {
-	return mount.CleanupMountPoint(target, m.Interface, true /*extensiveMountPointCheck*/)
+func SMBUnmount(m *mount.SafeFormatAndMount, target string, extensiveMountCheck, _ bool) error {
+	return mount.CleanupMountPoint(target, m.Interface, extensiveMountCheck)
 }
 
-func CleanupMountPoint(m *mount.SafeFormatAndMount, target string, _ bool) error {
-	return mount.CleanupMountPoint(target, m.Interface, true /*extensiveMountPointCheck*/)
+func CleanupMountPoint(m *mount.SafeFormatAndMount, target string, extensiveMountCheck bool) error {
+	return mount.CleanupMountPoint(target, m.Interface, extensiveMountCheck)
 }
 
 func preparePublishPath(_ string, _ *mount.SafeFormatAndMount) error {

pkg/mounter/safe_mounter_host_process_windows.go

@@ -143,8 +143,7 @@ func (mounter *winMounter) Unmount(target string) error {
 	remoteServer, err := smb.GetRemoteServerFromTarget(target)
 	if err == nil {
 		klog.V(2).Infof("remote server path: %s, local path: %s", remoteServer, target)
-		hasDupSMBMount, err := smb.CheckForDuplicateSMBMounts(driverGlobalMountPath, target, remoteServer)
-		if err == nil {
+		if hasDupSMBMount, err := smb.CheckForDuplicateSMBMounts(driverGlobalMountPath, target, remoteServer); err == nil {
 			if !hasDupSMBMount {
 				remoteServer = strings.Replace(remoteServer, "UNC\\", "\\\\", 1)
 				if err := smb.RemoveSmbGlobalMapping(remoteServer); err != nil {