commit c532504d17fcedf8fbaad3017635fc4a3ca32fc5
author szager@chromium.org <szager@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Sat Aug 23 02:35:23 2014 +0000
committer szager@chromium.org <szager@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Sat Aug 23 02:35:23 2014 +0000
tree 0fa21a84fafeb83613f9dbdae463f6d8d40dd065
parent 1132f5f896e1efc5ff177bab377e34d5cd1a1ebd

Fix thread lock around cachepath access.

R=cmp@chromium.org, agable@chromium.org
BUG=

Review URL: https://codereview.chromium.org/468253003

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@291571 0039d316-1c4b-4281-b951-d872f2087c98

git_cache.py

diff --git a/git_cache.py b/git_cache.py
index 1ac9a41..d6ed2a8 100755
--- a/git_cache.py
+++ b/git_cache.py
@@ -200,7 +200,9 @@
 
   @classmethod
   def SetCachePath(cls, cachepath):
+    cls.cachepath_lock.acquire()
     setattr(cls, 'cachepath', cachepath)
+    cls.cachepath_lock.release()
 
   @classmethod
   def GetCachePath(cls):
@@ -215,7 +217,7 @@
         cls.cachepath_lock.release()
         raise RuntimeError('No global cache.cachepath git configuration found.')
       setattr(cls, 'cachepath', cachepath)
-      cls.cachepath_lock.release()
+    cls.cachepath_lock.release()
     return getattr(cls, 'cachepath')
 
   def RunGit(self, cmd, **kwargs):