Revert "Test proper GCM IV length handling" 12/298712/1
authorKrzysztof Jackiewicz <k.jackiewicz@samsung.com>
Tue, 12 Sep 2023 10:36:35 +0000 (12:36 +0200)
committerKrzysztof Jackiewicz <k.jackiewicz@samsung.com>
Tue, 12 Sep 2023 15:41:42 +0000 (17:41 +0200)
This reverts commit 61b910797b706b3e8494eb5841e4462bf1356125.

Change-Id: Ifb7e276f1dee253c606800049ef97aea5c3bec77

src/manager/crypto/generic-backend/crypto-params.h
unit-tests/test_sw-backend.cpp

index 41a4461..ae23fba 100644 (file)
@@ -27,7 +27,6 @@ class Params
 {
 public:
        static const size_t DEFAULT_AES_IV_LEN = 16; // max acceptable size of IV
-       static const size_t DEFAULT_AES_GCM_IV_LEN = 12; // default size of IV in GCM mode
        static const int DEFAULT_AES_GCM_TAG_LEN_BYTES = 16; // length of AES GCM tag
        static const int DEFAULT_AES_GCM_TAG_LEN_BITS = DEFAULT_AES_GCM_TAG_LEN_BYTES * 8;
        static const int DERIVED_KEY_LENGTH = 16; // length of AES key derived from password in bytes
index c4ae72c..ae0154e 100644 (file)
@@ -645,17 +645,9 @@ NEGATIVE_TEST_CASE(symmetricEncryptDecryptGcm)
 
        // wrong iv
        auto wrongIv = iv;
-       wrongIv[iv.size() - 1] ^= 0x1;
+       wrongIv[0] ^= 0x1;
        ca2.setParam(ParamName::ED_IV, wrongIv);
        BOOST_REQUIRE_THROW(key->decrypt(ca2, encrypted), Exc::Crypto::InputParam);
-
-       // shortened iv
-       auto shortenedIv = iv;
-       static_assert(Params::DEFAULT_AES_GCM_IV_LEN < Params::DEFAULT_AES_IV_LEN);
-       shortenedIv.resize(Params::DEFAULT_AES_GCM_IV_LEN);
-       ca2.setParam(ParamName::ED_IV, shortenedIv);
-       BOOST_REQUIRE_THROW(key->decrypt(ca2, encrypted), Exc::Crypto::InputParam);
-
        ca2.setParam(ParamName::ED_IV, iv);
 
        // wrong ciphertext