Fixes a trio of concurrency issues found by Coverity

2 files changed, 13 insertions, 4 deletions

diff --git a/LibGit2Sharp/Core/Handles/SafeHandleBase.cs b/LibGit2Sharp/Core/Handles/SafeHandleBase.cs
index b4ad9804..e16e3091 100644
--- a/LibGit2Sharp/Core/Handles/SafeHandleBase.cs
+++ b/LibGit2Sharp/Core/Handles/SafeHandleBase.cs
@@ -66,7 +66,15 @@ namespace LibGit2Sharp.Core
         /// </summary>
         public static IEnumerable<string> TypeNames
         {
-            get { return _typeNames.ToArray(); }
+            get
+            {
+                string[] result = null;
+                lock (_lockpad)
+                {
+                    result = _typeNames.ToArray();
+                }
+                return result;
+            }
         }
     }
 }
diff --git a/LibGit2Sharp/Core/LazyGroup.cs b/LibGit2Sharp/Core/LazyGroup.cs
index 3c82fa3a..da6fed92 100644
--- a/LibGit2Sharp/Core/LazyGroup.cs
+++ b/LibGit2Sharp/Core/LazyGroup.cs
@@ -24,18 +24,19 @@ namespace LibGit2Sharp.Core
 
         public void Evaluate()
         {
-            if (evaluated)
-                return;
-
             lock (@lock)
             {
                 if (evaluated)
+                {
                     return;
+                }
 
                 EvaluateInternal(input =>
                                  {
                                      foreach (var e in evaluators)
+                                     {
                                          e.Evaluate(input);
+                                     }
                                  });
                 evaluated = true;
             }