瀏覽代碼

mount: do pre-mount checks before opening repository

Marian Beermann 8 年之前
父節點
當前提交
fa9cef7abf
共有 1 個文件被更改,包括 6 次插入4 次删除
  1. 6 4
      borg/archiver.py

+ 6 - 4
borg/archiver.py

@@ -559,22 +559,24 @@ class Archiver:
 
 
     def do_mount(self, args):
     def do_mount(self, args):
         """Mount archive or an entire repository as a FUSE filesystem"""
         """Mount archive or an entire repository as a FUSE filesystem"""
+        # Perform these checks before opening the repository and asking for a passphrase.
+
         try:
         try:
             import borg.fuse
             import borg.fuse
         except ImportError as e:
         except ImportError as e:
             self.print_error('borg mount not available: loading fuse support failed [ImportError: %s]' % str(e))
             self.print_error('borg mount not available: loading fuse support failed [ImportError: %s]' % str(e))
             return self.exit_code
             return self.exit_code
 
 
+        if not os.path.isdir(args.mountpoint) or not os.access(args.mountpoint, os.R_OK | os.W_OK | os.X_OK):
+            self.print_error('%s: Mountpoint must be a writable directory' % args.mountpoint)
+            return self.exit_code
+
         return self._do_mount(args)
         return self._do_mount(args)
 
 
     @with_repository()
     @with_repository()
     def _do_mount(self, args, repository, manifest, key):
     def _do_mount(self, args, repository, manifest, key):
         from .fuse import FuseOperations
         from .fuse import FuseOperations
 
 
-        if not os.path.isdir(args.mountpoint) or not os.access(args.mountpoint, os.R_OK | os.W_OK | os.X_OK):
-            self.print_error('%s: Mountpoint must be a writable directory' % args.mountpoint)
-            return self.exit_code
-
         with cache_if_remote(repository) as cached_repo:
         with cache_if_remote(repository) as cached_repo:
             if args.location.archive:
             if args.location.archive:
                 archive = Archive(repository, key, manifest, args.location.archive)
                 archive = Archive(repository, key, manifest, args.location.archive)