क्या यह जावा एन्क्रिप्शन कोड थ्रेड सुरक्षित है?

Question

मैं उच्च-समवर्ती अनुप्रयोग के लिए निम्न कोड का उपयोग करना चाहता हूं जहां कुछ डेटा एन्क्रिप्टेड और डिक्रिप्ट किया जाना चाहिए। तो मुझे पता होना चाहिए कि अप्रत्याशित मुद्दों से बचने के लिए, इस कोड का कौन सा हिस्सा सिंक्रनाइज़ किया जाना चाहिए, यदि कोई है।

public class DesEncrypter { 
    Cipher ecipher; 
    Cipher dcipher; 

    // 8-byte Salt 
    byte[] salt = { 
     (byte)0xA9, (byte)0x9B, (byte)0xC8, (byte)0x32, 
     (byte)0x56, (byte)0x35, (byte)0xE3, (byte)0x03 
    }; 

    int iterationCount = 19; 

    DesEncrypter(String passPhrase) { 
     try { 
      // Create the key 
      KeySpec keySpec = new PBEKeySpec(passPhrase.toCharArray(), salt, iterationCount); 

      SecretKey key = SecretKeyFactory.getInstance("PBEWithMD5AndDES").generateSecret(keySpec); 
      ecipher = Cipher.getInstance(key.getAlgorithm()); 
      dcipher = Cipher.getInstance(key.getAlgorithm()); 

      // Prepare the parameter to the ciphers 
      AlgorithmParameterSpec paramSpec = new PBEParameterSpec(salt, iterationCount); 

      // Create the ciphers 
      ecipher.init(Cipher.ENCRYPT_MODE, key, paramSpec); 
      dcipher.init(Cipher.DECRYPT_MODE, key, paramSpec); 
     } catch (...) 
    } 

    public String encrypt(String str) { 
     try { 
      // Encode the string into bytes using utf-8 
      byte[] utf8 = str.getBytes("UTF8"); 
      // Encrypt 
      byte[] enc = ecipher.doFinal(utf8); 
      // Encode bytes to base64 to get a string 
      return new sun.misc.BASE64Encoder().encode(enc); 

     } catch (...) 
    } 

    public String decrypt(String str) { 
     try { 
      // Decode base64 to get bytes 
      byte[] dec = new sun.misc.BASE64Decoder().decodeBuffer(str); 
      // Decrypt 
      byte[] utf8 = dcipher.doFinal(dec); 
      // Decode using utf-8 
      return new String(utf8, "UTF8"); 
     } catch (...) 
    } 
} 

अगर मैं) एन्क्रिप्ट (में एक नया सिफर बना सकते हैं और प्रत्येक मंगलाचरण के लिए डिक्रिप्ट() पद्धतियों, तो मैं संगामिति समस्याओं से बचने कर सकते हैं, मैं सिर्फ यकीन नहीं है अगर वहाँ एक नया प्राप्त करने में भूमि के ऊपर का एक बहुत है प्रत्येक आमंत्रण के लिए एक सिफर का उदाहरण।

public String encrypt(String str) { 
     try { 
      // Encode the string into bytes using utf-8 
      byte[] utf8 = str.getBytes("UTF8"); 
      // Encrypt 
      //new cipher instance 
      ecipher = Cipher.getInstance(key.getAlgorithm()); 

      byte[] enc = ecipher.doFinal(utf8); 
      // Encode bytes to base64 to get a string 
      return new sun.misc.BASE64Encoder().encode(enc); 

     } catch (...) 

Answer 1

चीजें केवल थ्रेड-सुरक्षित होने की आवश्यकता होती है अगर उन्हें एक साथ कई धागे द्वारा उपयोग किया जाता है। चूंकि इस वर्ग के प्रत्येक उदाहरण का संभवतः केवल एक ही थ्रेड द्वारा उपयोग किया जाएगा, इस बारे में चिंता करने की आवश्यकता नहीं है कि यह थ्रेडसेफ है या नहीं।

एक असंबंधित नोट पर, हार्डकोडेड नमक, नॉन या IV होने पर कभी भी एक अच्छा विचार नहीं है।

Answer 2

मानक नियम है - जब तक जावाडोक राज्यों स्पष्ट है कि जावा पुस्तकालयों में एक वर्ग के धागे की सुरक्षित है, तो आप मान लेना चाहिए कि ऐसा नहीं है।

इस विशेष उदाहरण में:

• विभिन्न कक्षाओं धागा सुरक्षित के रूप में दस्तावेज नहीं हैं।

• Cipher.getInstance(...) और SecretKeyFactory.getInstance(...) विधियों को नई वस्तुओं को वापस करने के रूप में दस्तावेज किया गया है; यानी मौजूदा वस्तुओं के संदर्भ नहीं हैं जिनके अन्य धागे के संदर्भ हो सकते हैं।

  "। एक नया SecretKeyFactory वस्तु पहले प्रदाता का समर्थन करता है कि निर्दिष्ट एल्गोरिथ्म दिया जाता है से SecretKeyFactorySpi कार्यान्वयन encapsulating": -

  अद्यतनjavadoc इस कहते हैं

  इसके अलावा, source code स्पष्ट रूप से पुष्टि की है कि एक नई वस्तु बनाया है और दिया जाता है।

संक्षेप में, इसका मतलब है कि आपके DesEncryptor वर्ग वर्तमान में धागे की सुरक्षित नहीं है, लेकिन आप प्रासंगिक आपरेशन (जैसे encode और decode) सिंक्रनाइज़ किया जा रहा है, और उजागर नहीं द्वारा यह धागा सुरक्षित बनाने के लिए सक्षम होना चाहिए दो सिफर वस्तुओं। यदि सिंक्रनाइज़ किए गए तरीकों को एक बाधा उत्पन्न करने की संभावना है, तो प्रत्येक थ्रेड के लिए DesEncryptor का एक अलग उदाहरण बनाएं।

Answer 3

Cipher ऑब्जेक्ट थ्रेड-सुरक्षित नहीं होने वाला है, क्योंकि यह एन्क्रिप्शन प्रक्रिया के बारे में आंतरिक स्थिति को बरकरार रखता है। यह आपके DesEncrypter वर्ग पर भी लागू होता है - प्रत्येक थ्रेड को DesEncrypter के अपने उदाहरण का उपयोग करने की आवश्यकता होगी, जब तक कि आप encode और decode विधियों को सिंकोननाइज़ न करें।