diff options
| author | Luke Pulverenti <luke.pulverenti@gmail.com> | 2013-05-10 13:51:10 -0400 |
|---|---|---|
| committer | Luke Pulverenti <luke.pulverenti@gmail.com> | 2013-05-10 13:51:10 -0400 |
| commit | 196f3eb574d6a5275eb99450ddca06e6837c68dc (patch) | |
| tree | 93a5221fa4baab8c8689d3d66ceff8bf54c7496d | |
| parent | 88ff3e4a9bb6ed0db94be7d53f520b224908348b (diff) | |
better disposal around using
| -rw-r--r-- | MediaBrowser.Server.Implementations/Providers/ProviderManager.cs | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/MediaBrowser.Server.Implementations/Providers/ProviderManager.cs b/MediaBrowser.Server.Implementations/Providers/ProviderManager.cs index 840e8265d..5dfa31b09 100644 --- a/MediaBrowser.Server.Implementations/Providers/ProviderManager.cs +++ b/MediaBrowser.Server.Implementations/Providers/ProviderManager.cs @@ -468,17 +468,17 @@ namespace MediaBrowser.Server.Implementations.Providers try { - // If the file already exists but is hidden, the below save will throw an UnauthorizedAccessException - var existingFileInfo = new FileInfo(path); - - if (existingFileInfo.Exists && existingFileInfo.Attributes.HasFlag(FileAttributes.Hidden)) + using (dataToSave) { - existingFileInfo.Delete(); - } + // If the file already exists but is hidden, the below save will throw an UnauthorizedAccessException + var existingFileInfo = new FileInfo(path); - using (var fs = new FileStream(path, FileMode.Create, FileAccess.Write, FileShare.Read, StreamDefaults.DefaultFileStreamBufferSize, FileOptions.Asynchronous)) - { - using (dataToSave) + if (existingFileInfo.Exists && existingFileInfo.Attributes.HasFlag(FileAttributes.Hidden)) + { + existingFileInfo.Delete(); + } + + using (var fs = new FileStream(path, FileMode.Create, FileAccess.Write, FileShare.Read, StreamDefaults.DefaultFileStreamBufferSize, FileOptions.Asynchronous)) { await dataToSave.CopyToAsync(fs, StreamDefaults.DefaultCopyToBufferSize, cancellationToken).ConfigureAwait(false); } |
