mirror of
https://gitlab.winehq.org/wine/wine-gecko.git
synced 2024-09-13 09:24:08 -07:00
Bug 1061021, Part 16: Stop using PLArenaPool in pkixocsp_CreateEncodedOCSPRequest, r=keeler
--HG-- extra : rebase_source : 551d0c0e45d770c2218fb77874737fe23909d6c9
This commit is contained in:
parent
493b0e85e9
commit
fd03fc59d8
@ -83,36 +83,6 @@ private:
|
||||
class pkixocsp_CreateEncodedOCSPRequest : public NSSTest
|
||||
{
|
||||
protected:
|
||||
// These SECItems are allocated in arena, and so will be auto-cleaned.
|
||||
SECItem* unsupportedLongSerialNumber;
|
||||
SECItem* longestRequiredSerialNumber;
|
||||
|
||||
void SetUp()
|
||||
{
|
||||
static const uint8_t UNSUPPORTED_LEN = 128; // must be larger than 127
|
||||
// tag + length + value is 1 + 2 + UNSUPPORTED_LEN
|
||||
unsupportedLongSerialNumber = SECITEM_AllocItem(arena.get(), nullptr,
|
||||
1 + 2 + UNSUPPORTED_LEN);
|
||||
memset(unsupportedLongSerialNumber->data, 0,
|
||||
unsupportedLongSerialNumber->len);
|
||||
unsupportedLongSerialNumber->data[0] = der::INTEGER;
|
||||
// Encoding the length takes two bytes: one byte to indicate that a
|
||||
// second byte follows, and the second byte to indicate the length.
|
||||
unsupportedLongSerialNumber->data[1] = 0x80 + 1;
|
||||
unsupportedLongSerialNumber->data[2] = UNSUPPORTED_LEN;
|
||||
unsupportedLongSerialNumber->data[3] = 0x01; // value is 0x010000...00
|
||||
|
||||
static const uint8_t LONGEST_REQUIRED_LEN = 20;
|
||||
// tag + length + value is 1 + 1 + LONGEST_REQUIRED_LEN
|
||||
longestRequiredSerialNumber = SECITEM_AllocItem(arena.get(), nullptr,
|
||||
1 + 1 + LONGEST_REQUIRED_LEN);
|
||||
memset(longestRequiredSerialNumber->data, 0,
|
||||
longestRequiredSerialNumber->len);
|
||||
longestRequiredSerialNumber->data[0] = der::INTEGER;
|
||||
longestRequiredSerialNumber->data[1] = LONGEST_REQUIRED_LEN;
|
||||
longestRequiredSerialNumber->data[2] = 0x01; // value is 0x010000...00
|
||||
}
|
||||
|
||||
void MakeIssuerCertIDComponents(const char* issuerASCII,
|
||||
/*out*/ ByteString& issuerDER,
|
||||
/*out*/ ByteString& issuerSPKI)
|
||||
@ -138,6 +108,20 @@ protected:
|
||||
// CreateEncodedOCSPRequest to fail.
|
||||
TEST_F(pkixocsp_CreateEncodedOCSPRequest, ChildCertLongSerialNumberTest)
|
||||
{
|
||||
static const uint8_t UNSUPPORTED_LEN = 128; // must be larger than 127
|
||||
|
||||
ByteString serialNumberString;
|
||||
// tag + length + value is 1 + 2 + UNSUPPORTED_LEN
|
||||
// Encoding the length takes two bytes: one byte to indicate that a
|
||||
// second byte follows, and the second byte to indicate the length.
|
||||
serialNumberString.push_back(0x80 + 1);
|
||||
serialNumberString.push_back(UNSUPPORTED_LEN);
|
||||
// value is 0x010000...00
|
||||
serialNumberString.push_back(0x01);
|
||||
for (size_t i = 1; i < UNSUPPORTED_LEN; ++i) {
|
||||
serialNumberString.push_back(0x00);
|
||||
}
|
||||
|
||||
ByteString issuerDER;
|
||||
ByteString issuerSPKI;
|
||||
ASSERT_NO_FATAL_FAILURE(MakeIssuerCertIDComponents("CA", issuerDER,
|
||||
@ -150,8 +134,8 @@ TEST_F(pkixocsp_CreateEncodedOCSPRequest, ChildCertLongSerialNumberTest)
|
||||
ASSERT_EQ(Success, spki.Init(issuerSPKI.data(), issuerSPKI.length()));
|
||||
|
||||
Input serialNumber;
|
||||
ASSERT_EQ(Success, serialNumber.Init(unsupportedLongSerialNumber->data,
|
||||
unsupportedLongSerialNumber->len));
|
||||
ASSERT_EQ(Success, serialNumber.Init(serialNumberString.data(),
|
||||
serialNumberString.length()));
|
||||
|
||||
uint8_t ocspRequest[OCSP_REQUEST_MAX_LENGTH];
|
||||
size_t ocspRequestLength;
|
||||
@ -165,6 +149,18 @@ TEST_F(pkixocsp_CreateEncodedOCSPRequest, ChildCertLongSerialNumberTest)
|
||||
// it's required to support (i.e. 20 octets).
|
||||
TEST_F(pkixocsp_CreateEncodedOCSPRequest, LongestSupportedSerialNumberTest)
|
||||
{
|
||||
static const uint8_t LONGEST_REQUIRED_LEN = 20;
|
||||
|
||||
ByteString serialNumberString;
|
||||
// tag + length + value is 1 + 1 + LONGEST_REQUIRED_LEN
|
||||
serialNumberString.push_back(der::INTEGER);
|
||||
serialNumberString.push_back(LONGEST_REQUIRED_LEN);
|
||||
serialNumberString.push_back(0x01);
|
||||
// value is 0x010000...00
|
||||
for (size_t i = 1; i < LONGEST_REQUIRED_LEN; ++i) {
|
||||
serialNumberString.push_back(0x00);
|
||||
}
|
||||
|
||||
ByteString issuerDER;
|
||||
ByteString issuerSPKI;
|
||||
ASSERT_NO_FATAL_FAILURE(MakeIssuerCertIDComponents("CA", issuerDER,
|
||||
@ -177,8 +173,8 @@ TEST_F(pkixocsp_CreateEncodedOCSPRequest, LongestSupportedSerialNumberTest)
|
||||
ASSERT_EQ(Success, spki.Init(issuerSPKI.data(), issuerSPKI.length()));
|
||||
|
||||
Input serialNumber;
|
||||
ASSERT_EQ(Success, serialNumber.Init(longestRequiredSerialNumber->data,
|
||||
longestRequiredSerialNumber->len));
|
||||
ASSERT_EQ(Success, serialNumber.Init(serialNumberString.data(),
|
||||
serialNumberString.length()));
|
||||
|
||||
uint8_t ocspRequest[OCSP_REQUEST_MAX_LENGTH];
|
||||
size_t ocspRequestLength;
|
||||
|
Loading…
Reference in New Issue
Block a user