Skip to content

Address chunk size issue in wh_Client_SheSecureBoot#293

Merged
bigbrett merged 2 commits intowolfSSL:mainfrom
padelsbach:padelsbach/finding-146
Mar 5, 2026
Merged

Address chunk size issue in wh_Client_SheSecureBoot#293
bigbrett merged 2 commits intowolfSSL:mainfrom
padelsbach:padelsbach/finding-146

Conversation

@padelsbach
Copy link
Contributor

@padelsbach padelsbach commented Feb 27, 2026

Fixes finding 146 and adds associated regression test. Also, this adjusts a buffer size constant in the test code.

@padelsbach padelsbach force-pushed the padelsbach/finding-146 branch 16 times, most recently from 9d9d6b6 to 6a06299 Compare March 2, 2026 07:39
@padelsbach padelsbach marked this pull request as ready for review March 2, 2026 17:34
Copy link
Contributor

@bigbrett bigbrett left a comment

Choose a reason for hiding this comment

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

Thanks @padelsbach a few fixes required before we can merge.

Also, can you please run git-clang-format main and commit the formatting changes as a separate step?

@padelsbach padelsbach force-pushed the padelsbach/finding-146 branch 4 times, most recently from 69cedcf to 4110b9b Compare March 3, 2026 21:17
@padelsbach padelsbach force-pushed the padelsbach/finding-146 branch from 4110b9b to a018af2 Compare March 3, 2026 21:36
@padelsbach padelsbach marked this pull request as draft March 3, 2026 21:54
@padelsbach padelsbach marked this pull request as ready for review March 3, 2026 23:49
@padelsbach
Copy link
Contributor Author

@bigbrett, I think I addressed all of the comments. I ran git-clang-format main and added as a second commit. Let me know if this is what you're looking for.

@padelsbach padelsbach requested a review from bigbrett March 3, 2026 23:50
@bigbrett bigbrett merged commit 83cfd06 into wolfSSL:main Mar 5, 2026
51 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.

3 participants