Skip to content

ASN original, OCSP: fix maximum size calc to GetDateInfo#6339

Merged
JacobBarthelmeh merged 1 commit intowolfSSL:masterfrom
SparkiDev:asn_orig_ocsp_fix
Apr 26, 2023
Merged

ASN original, OCSP: fix maximum size calc to GetDateInfo#6339
JacobBarthelmeh merged 1 commit intowolfSSL:masterfrom
SparkiDev:asn_orig_ocsp_fix

Conversation

@SparkiDev
Copy link
Copy Markdown
Contributor

Description

GetDateInfo called with a local index (reset to 0) but size is of original buffer. Subtract the index into original buffer from size to get size available for parsing.

Fixes zd#16034

Testing

POC from fuzzing

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@SparkiDev SparkiDev self-assigned this Apr 25, 2023
GetDateInfo called with a local index (reset to 0) but size is of
original buffer. Subtract the index into original buffer from size to
get size available for parsing.
@JacobBarthelmeh
Copy link
Copy Markdown
Contributor

retest this please

@JacobBarthelmeh JacobBarthelmeh merged commit ca4a17e into wolfSSL:master Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants