Skip to content

add missing WOLFSSL_QUIC_MAX_RECORD_CAPACITY check on the early-data#10201

Merged
SparkiDev merged 2 commits intowolfSSL:masterfrom
gasbytes:quic_record_cap
Apr 15, 2026
Merged

add missing WOLFSSL_QUIC_MAX_RECORD_CAPACITY check on the early-data#10201
SparkiDev merged 2 commits intowolfSSL:masterfrom
gasbytes:quic_record_cap

Conversation

@gasbytes
Copy link
Copy Markdown
Contributor

@gasbytes gasbytes commented Apr 13, 2026

Description

add missing WOLFSSL_QUIC_MAX_RECORD_CAPACITY check on the early-data path in quic_record_make() and added a unit test for it.

Fixes zd#21595

Testing

added regression test test_quic_record_cap in the unit tests testsuitee. tested against that and the original reproducer that was shared.

Checklist

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

@gasbytes gasbytes self-assigned this Apr 13, 2026
…path in quic_record_make() and added a unit test for it
julek-wolfssl
julek-wolfssl previously approved these changes Apr 15, 2026
@julek-wolfssl julek-wolfssl removed their assignment Apr 15, 2026
SparkiDev
SparkiDev previously approved these changes Apr 15, 2026
Copy link
Copy Markdown

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fenrir Automated Review — PR #10201

Scan targets checked: wolfssl-bugs, wolfssl-compliance, wolfssl-consttime, wolfssl-defaults, wolfssl-mutation, wolfssl-proptest, wolfssl-src, wolfssl-zeroize

Findings: 2
2 finding(s) posted as inline comments (see file-level comments below)

This review was generated automatically by Fenrir. Findings are non-blocking.

Comment thread src/quic.c
Comment thread tests/quic.c
…1 to catch off-by-onee mutations in the early data capacity check
@gasbytes gasbytes dismissed stale reviews from SparkiDev and julek-wolfssl via 1576cf9 April 15, 2026 12:39
@github-actions
Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

@gasbytes gasbytes requested a review from SparkiDev April 15, 2026 14:47
@SparkiDev SparkiDev assigned gasbytes and unassigned gasbytes Apr 15, 2026
@SparkiDev SparkiDev merged commit f286f62 into wolfSSL:master Apr 15, 2026
425 of 426 checks passed
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.

6 participants