lazysodium-java中cryptoSecretBoxOpenEasy解密结果不一致问题求助

问题描述 #

使用libsodium的Java封装库lazysodium-java进行加解密时，同一参数下偶尔会出现解密失败的情况，预期输入不变时解密应始终成功，但实际测试中失败的迭代次数不固定（cryptoSecretBoxOpenEasy返回false时不会更新传入的message数组）。

复现用Scala测试用例：

test("Decrypt you fool") {
    // encrypt a message
    val nonceSize = SecretBoxModule.NONCEBYTES
    val secret = SecretKey("secret")
    val nonce = lazySodium.nonce(nonceSize)
    val secretMessage = "catcher"
    val messageToEncrypt = secretMessage.getBytes(StandardCharsets.UTF_8)
    val cipher = new Array[Byte](messageToEncrypt.length + SecretBoxModule.MACBYTES)
    lazySodium.cryptoSecretBoxEasy(cipher, messageToEncrypt, messageToEncrypt.length, nonce, secret.keyBytes)
    val encrypted = nonce ++ cipher 

    // loop 1000 times to decrypt message
    for (i <- 1 to 1000) {
      println(s"Running $i")

      val message = new Array[Byte](encrypted.length - SecretBoxModule.MACBYTES - nonceSize)
      val (nonce, cipherText) = encrypted.splitAt(nonceSize)
      secretBoxModule.cryptoSecretBoxOpenEasy(message, cipherText, cipherText.length, nonce, secret.keyBytes)
      val response = new String(message, StandardCharsets.UTF_8)

      scala.Predef.assert(response == secretMessage)
    }
    assertCompletes
  
}

问题原因 #

1. 密钥不符合算法要求：SecretKey("secret")的用法错误。crypto_secretbox要求密钥必须是32字节（对应SecretBoxModule.KEYBYTES常量）的随机字节序列，直接用短字符串"secret"生成的密钥长度不足，内部会用不确定的方式补全或截断，导致实际使用的密钥不符合规范，这是偶发解密失败的核心原因。

2. 未处理解密失败的场景：cryptoSecretBoxOpenEasy返回false时说明解密验证失败，此时message数组的内容是未初始化的垃圾数据，直接用它构造字符串必然和原消息不一致，但代码里没有判断返回值就直接断言，导致偶发断言失败。

3. 变量名冲突：解密循环内的nonce变量覆盖了外层的同名变量，虽然逻辑上拆分出来的nonce是正确的，但这种命名容易引发逻辑混淆，增加出错概率。

修复方案 #

修正后的测试用例 #

test("Decrypt correctly") {
    val nonceSize = SecretBoxModule.NONCEBYTES
    val keySize = SecretBoxModule.KEYBYTES
    // 生成符合要求的32字节随机密钥
    val secret = lazySodium.cryptoSecretBoxKeygen()
    val nonce = lazySodium.nonce(nonceSize)
    val secretMessage = "catcher"
    val messageToEncrypt = secretMessage.getBytes(StandardCharsets.UTF_8)
    val cipher = new Array[Byte](messageToEncrypt.length + SecretBoxModule.MACBYTES)
    // 加密前先确认加密成功
    val encryptSuccess = lazySodium.cryptoSecretBoxEasy(cipher, messageToEncrypt, messageToEncrypt.length, nonce, secret)
    assert(encryptSuccess, "Encryption failed")
    
    val encrypted = nonce ++ cipher 

    for (i <- 1 to 1000) {
      println(s"Running $i")
      val message = new Array[Byte](encrypted.length - SecretBoxModule.MACBYTES - nonceSize)
      val (decryptNonce, cipherText) = encrypted.splitAt(nonceSize)
      // 解密时必须检查返回值
      val decryptSuccess = secretBoxModule.cryptoSecretBoxOpenEasy(message, cipherText, cipherText.length, decryptNonce, secret)
      assert(decryptSuccess, s"Decryption failed at iteration $i")
      
      val response = new String(message, StandardCharsets.UTF_8)
      assert(response == secretMessage, s"Mismatched message at iteration $i")
    }
    assertCompletes
}

关键修复点 #

• 使用lazySodium.cryptoSecretBoxKeygen()生成符合要求的32字节随机密钥，确保密钥符合算法规范。
• 加密和解密时都检查方法的返回值，只有操作成功时才继续后续逻辑。
• 重命名循环内的nonce变量为decryptNonce，避免变量名冲突。

内容的提问来源于stack exchange，提问作者lior