Perché SHA1.ComputeHash non riesce con un carico elevato con molti thread?

Question

Sto riscontrando un problema con un codice che mantengo. Il codice seguente ha un membro private static SHA1 (che è un IDisposable ma poiché è static, non dovrebbe mai essere finalizzato). Tuttavia, in condizioni di stress questo codice genera un'eccezione che suggerisce che è stato chiuso:

Caught exception. Safe handle has been closed" 
Stack trace: Call stack where exception was thrown 
at System.Runtime.InteropServices.SafeHandle.DangerousAddRef(Boolean& success) 
at System.Security.Cryptography.Utils.HashData(SafeHashHandle hHash, Byte[] data, Int32 cbData, Int32 ibStart, Int32 cbSize) 
at System.Security.Cryptography.Utils.HashData(SafeHashHandle hHash, Byte[] data, Int32 ibStart, Int32 cbSize) 
at System.Security.Cryptography.HashAlgorithm.ComputeHash(Byte[] buffer) 

Il codice in questione è:

internal class TokenCache 
{ 
    private static SHA1 _sha1 = SHA1.Create(); 

    private string ComputeHash(string password) 
    { 
     byte[] passwordBytes = UTF8Encoding.UTF8.GetBytes(password); 
     return UTF8Encoding.UTF8.GetString(_sha1.ComputeHash(passwordBytes)); 
    } 

La mia domanda è, ovviamente, quello che potrebbe causare questo problema. La chiamata a SHA1.Create può fallire in silenzio (quante risorse di crittografia sono disponibili)? Ciò potrebbe essere causato dall'appunta- mento della appdomain?

Altre teorie?

Answer 1

Come per the documentation per la classe di base HashAlgorithm

Any public static (Shared in Visual Basic) members of this type are thread safe. Any instance members are not guaranteed to be thread safe.

Si consiglia di non condividere queste classi tra i thread in cui diversi thread cercano e chiamano ComputeHash nella stessa istanza, allo stesso tempo.

EDIT Questo è ciò che sta causando il tuo errore. Lo stress test sottostante produce una varietà di errori dovuti a più thread che chiamano ComputeHash sulla stessa istanza dell'algoritmo di hash. Il tuo errore è uno di questi.

Specificamente, ho visto i seguenti errori con questo test di stress:

• System.Security.Cryptography.CryptographicException: Hash non valido per l'utilizzo nello stato specificato.
• System.ObjectDisposedException: maniglia di sicurezza è stato chiuso

stress codice di test di esempio:

const int threadCount = 2; 
var sha1 = SHA1.Create(); 
var b = new Barrier(threadCount); 
Action start =() => { 
        b.SignalAndWait(); 
        for (int i = 0; i < 10000; i++) 
        { 
         var pwd = Guid.NewGuid().ToString(); 
         var bytes = Encoding.UTF8.GetBytes(pwd); 
         sha1.ComputeHash(bytes); 
        } 
       }; 
var threads = Enumerable.Range(0, threadCount) 
         .Select(_ => new ThreadStart(start)) 
         .Select(x => new Thread(x)) 
         .ToList(); 
foreach (var t in threads) t.Start(); 
foreach (var t in threads) t.Join();