feedc0de/wolfssl
1
0
Fork 0
forked from wolfSSL/wolfssl
Code Pull Requests Activity

Merge pull request #4236 from kabuobeid/serialSz

Browse Source 
Fix signed comparison issue with serialSz.
This commit is contained in:
JacobBarthelmeh
2021-07-27 13:54:38 +07:00
committed by GitHub
parent 8c63701577 8bd304e4c5
commit 3ecd7262b7
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
wolfcrypt/src/asn.c
View File

@ -10696,13 +10696,13 @@ int GetSerialNumber(const byte* input, word32* inOutIdx,
if (ret != 0) if (ret != 0)
return ret; return ret;
if (*serialSz > EXTERNAL_SERIAL_SIZE) { if (*serialSz > EXTERNAL_SERIAL_SIZE || *serialSz <= 0) {
WOLFSSL_MSG("Serial size bad"); WOLFSSL_MSG("Serial size bad");
return ASN_PARSE_E; return ASN_PARSE_E;
} }
/* return serial */ /* return serial */
XMEMCPY(serial, &input[*inOutIdx], *serialSz); XMEMCPY(serial, &input[*inOutIdx], (size_t)*serialSz);
*inOutIdx += *serialSz; *inOutIdx += *serialSz;
return result; return result;