Browse Source

fix concurrency issue in base app host

LukePulverenti 12 years ago
parent
commit
26536c22ad
1 changed files with 5 additions and 1 deletions
  1. 5 1
      MediaBrowser.Common.Implementations/BaseApplicationHost.cs

+ 5 - 1
MediaBrowser.Common.Implementations/BaseApplicationHost.cs

@@ -381,6 +381,7 @@ namespace MediaBrowser.Common.Implementations
                 if (disposable != null)
                 {
                     Logger.Info("Registering " + disposable.GetType().Name);
+
                     DisposableParts.Add(disposable);
                 }
             }
@@ -458,7 +459,10 @@ namespace MediaBrowser.Common.Implementations
 
             if (manageLiftime)
             {
-                DisposableParts.AddRange(parts.OfType<IDisposable>());
+                lock (DisposableParts)
+                {
+                    DisposableParts.AddRange(parts.OfType<IDisposable>());
+                }
             }
 
             return parts;