wolfSSL/wolfssl
1
0
Fork 1
mirror of https://github.com/wolfSSL/wolfssl.git synced 2025-07-30 02:37:28 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3368 from dgarske/zd11057

Browse Source 
Fix for possible malformed encrypted key with DES3 causing negative length
This commit is contained in:
toddouska
2020-10-14 15:32:48 -07:00
committed by GitHub
parent 1c4b15d427 d33d100526
commit 026ba4e750
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
wolfcrypt/src/asn.c
View File

@ -10690,7 +10690,7 @@ int PemToDer(const unsigned char* buff, long longSz, int type,
#ifndef NO_DES3
if (info->cipherType == WC_CIPHER_DES3) {
/* Assuming there is padding:
* (der->length > 0 &&
* (der->length > 0 && der->length > DES_BLOCK_SIZE &&
* (der->length % DES_BLOCK_SIZE) != 0)
* and assuming the last value signifies the number of
* padded bytes IE if last value is 0x08 then there are
@ -10700,9 +10700,10 @@ int PemToDer(const unsigned char* buff, long longSz, int type,
* der->length -= padVal;
*/
if (der->length > 0 &&
der->length > DES_BLOCK_SIZE &&
(der->length % DES_BLOCK_SIZE) != 0) {
padVal = der->buffer[der->length-1];
if (padVal <= DES_BLOCK_SIZE) {
if (padVal < DES_BLOCK_SIZE) {
der->length -= padVal;
}
}