Skip to content

Commit 78abd54

Browse files
Enforce max size of responses array in SendCertificateStatus.
Thanks to Zou Dikai for the report.
1 parent 5d0e050 commit 78abd54

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

src/internal.c

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25925,6 +25925,10 @@ int SendCertificateStatus(WOLFSSL* ssl)
2592525925

2592625926
if (idx > chain->length)
2592725927
break;
25928+
if ((i + 1) >= MAX_CERT_EXTENSIONS) {
25929+
ret = MAX_CERT_EXTENSIONS_ERR;
25930+
break;
25931+
}
2592825932
ret = CreateOcspRequest(ssl, request, cert, der.buffer,
2592925933
der.length, &ctxOwnsRequest);
2593025934
if (ret == 0) {
@@ -25953,6 +25957,11 @@ int SendCertificateStatus(WOLFSSL* ssl)
2595325957
else {
2595425958
while (ret == 0 &&
2595525959
NULL != (request = ssl->ctx->chainOcspRequest[i])) {
25960+
if ((i + 1) >= MAX_CERT_EXTENSIONS) {
25961+
ret = MAX_CERT_EXTENSIONS_ERR;
25962+
break;
25963+
}
25964+
2595625965
request->ssl = ssl;
2595725966
ret = CheckOcspRequest(SSL_CM(ssl)->ocsp_stapling,
2595825967
request, &responses[++i], ssl->heap);

0 commit comments

Comments
 (0)