Skip to content

ctap: size response and HID buffers for 1200-byte CBOR messages - #104

Merged
jthornblad merged 1 commit into
mainfrom
buffer_sizes
Jul 10, 2026
Merged

ctap: size response and HID buffers for 1200-byte CBOR messages#104
jthornblad merged 1 commit into
mainfrom
buffer_sizes

Conversation

@jthornblad

@jthornblad jthornblad commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Description

Reduce the CTAP response buffer to 1200 bytes to match the advertised maxMsgSize value returned by GetInfo.

Shrink the CTAPHID reassembly buffer accordingly and document the packet-size calculation for a 1200-byte CBOR payload.

Type of change

Please tick any that are relevant to this PR and remove any that aren't.

  • Bugfix (non breaking change which resolve an issue)
  • Feature (non breaking change which adds functionality)
  • Breaking Change (a change which would cause existing functionality to not work as expected)
  • Documentation (a change to documentation)

Submission checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my changes
  • I have tested and verified my changes on target
  • My changes are well written and CI is passing
  • I have squashed my work to relevant commits and rebased on main for linear history
  • I have added a "Co-authored-by: x" if several people contributed, either pair programming or by squashing commits from different authors.
  • I have updated the documentation where relevant (readme, dev.tillitis.se etc.)
  • QEMU is updated to reflect changes

@jthornblad
jthornblad requested a review from dehanj July 10, 2026 12:36
Reduce the CTAP response buffer to 1200 bytes to match the advertised
`maxMsgSize` value returned by GetInfo.

Shrink the CTAPHID reassembly buffer accordingly and document the
packet-size calculation for a 1200-byte CBOR payload.
@jthornblad
jthornblad merged commit b012934 into main Jul 10, 2026
3 of 5 checks passed
@jthornblad
jthornblad deleted the buffer_sizes branch July 10, 2026 13:03
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