From c91f4b44421fd4262bb487ff0cc2d8fe8615b105 Mon Sep 17 00:00:00 2001 From: Qstick Date: Sun, 29 Oct 2017 00:49:47 -0400 Subject: [PATCH] Fixed: Logging error when accessing mount point --- src/NzbDrone.Mono/Disk/DiskProvider.cs | 36 ++++++-------------------- 1 file changed, 8 insertions(+), 28 deletions(-) diff --git a/src/NzbDrone.Mono/Disk/DiskProvider.cs b/src/NzbDrone.Mono/Disk/DiskProvider.cs index 27bdaf60c..bc47532fb 100644 --- a/src/NzbDrone.Mono/Disk/DiskProvider.cs +++ b/src/NzbDrone.Mono/Disk/DiskProvider.cs @@ -1,4 +1,4 @@ -using System; +using System; using System.Collections.Generic; using System.IO; using System.Linq; @@ -40,24 +40,15 @@ namespace NzbDrone.Mono.Disk { Ensure.That(path, () => path).IsValidPath(); - try - { - var mount = GetMount(path); - - if (mount == null) - { - Logger.Debug("Unable to get free space for '{0}', unable to find suitable drive", path); - return null; - } + var mount = GetMount(path); - return mount.AvailableFreeSpace; - } - catch (InvalidOperationException ex) + if (mount == null) { - Logger.Error(ex, "Couldn't get free space for {0}", path); + Logger.Debug("Unable to get free space for '{0}', unable to find suitable drive", path); + return null; } - return null; + return mount.AvailableFreeSpace; } public override void InheritFolderPermissions(string filename) @@ -100,20 +91,9 @@ namespace NzbDrone.Mono.Disk { Ensure.That(path, () => path).IsValidPath(); - try - { - var mount = GetMount(path); - - if (mount == null) return null; - - return mount.TotalSize; - } - catch (InvalidOperationException e) - { - Logger.Error(e, "Couldn't get total space for {0}", path); - } + var mount = GetMount(path); - return null; + return mount?.TotalSize; } public override bool TryCreateHardLink(string source, string destination)