Rilascio di un mutex

Question

Ho un'applicazione Web che deve utilizzare una cache dell'applicazione per archiviare i dati (a causa dell'elevato sovraccarico di ottenere tali dati su una richiesta per richiesta). Vedi post precedente a https://stackoverflow.com/a/16961962/236860

Questo approccio sembra funzionare bene, ma sto vedendo i seguenti errori occasionali errori del sito web:

System.ApplicationException: Object synchronization method was called from an 
unsynchronized block of code. 
at System.Threading.Mutex.ReleaseMutex() 
at InboxInsight.Web.Web_Controls.Twitter.TwitterFeed.GetTwitterData(HttpContext context) 
at InboxInsight.Web.Web_Controls.Twitter.TwitterFeed.ProcessRequest(HttpContext context) 
at System.Web.HttpApplication.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute() 
at System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously) 

Per riferimento, ecco il blocco di codice:

public string GetData(HttpContext context) 
{ 
    var cache  = context.Cache; 
    Mutex mutex = null; 
    string data = (string)cache[CacheKey]; 

    // Start check to see if available on cache 
    if (data == null) 
    { 
     try 
     { 
      // Lock base on resource key 
      // (note that not all chars are valid for name) 
      mutex = new Mutex(true, CacheKey); 

      // Wait until it is safe to enter (someone else might already be 
      // doing this), but also add 30 seconds max. 
      mutex.WaitOne(30000); 

      // Now let's see if some one else has added it... 
      data = (string)cache[CacheKey]; 

      // They did, so send it... 
      if (data != null) 
      { 
       return data; 
      } 


      // Still not there, so now is the time to look for it! 
      data = GetSlowFeed(context); 

      cache.Remove(CacheKey); 
      cache.Add(CacheKey, data, null, GetExpiryDate(), 
       TimeSpan.Zero, CacheItemPriority.Normal, null); 
     } 
     finally 
     { 
      // Release the Mutex. 
      if (mutex != null) 
      { 
       mutex.ReleaseMutex(); 
      } 
     } 
    } 

    return data; 
} 

Da quello che ho cercato, suggerisce che questo problema è causato da un thread di processo che tenta di rilasciare un Mutex che non ha creato, ma non capisco come ciò potrebbe accadere.

Qualcuno può suggerire come posso ri-strutturare il codice per evitare questo problema?

Answer 1

Non si sta gestendo il caso in cui il mutex.WaitOne restituisce false ovvero timeout. Se WaitOne restituisce false, non si possiede il mutex, quindi non è necessario rilasciarlo.

bool iOwnTheMutex; 

try { 
    // set up mutex here... 
    iOwnTheMutex = mutex.WaitOne(2000); 
    if (iOwnTheMutex) { 
     // do what you need to do 
    } 
} 
finally { 
    if (mutex != null && iOwnTheMutex) { 
     mutex.ReleaseMutex(); 
    } 
}