Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Static analysis fixes #7616

Merged
merged 1 commit into from
Jun 12, 2024
Merged

Static analysis fixes #7616

merged 1 commit into from
Jun 12, 2024

Conversation

embhorn
Copy link
Member

@embhorn embhorn commented Jun 4, 2024 •
edited
Loading

Description

  • Fix InitHandshakeHashesAndCopy for unchecked return in call to InitHandshakeHashes
  • Cleaned up some whitespace further down in the function

Fixes zd17762

Testing

How did you test?

Checklist

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

@embhorn embhorn self-assigned this Jun 4, 2024
@embhorn embhorn marked this pull request as ready for review June 11, 2024 21:37
@embhorn embhorn requested a review from philljj June 11, 2024 21:39
@embhorn embhorn assigned philljj and unassigned embhorn Jun 11, 2024
@philljj philljj assigned wolfSSL-Bot and unassigned philljj Jun 11, 2024
@douzzer douzzer merged commit d2f4cc9 into wolfSSL:master Jun 12, 2024
107 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@douzzer douzzer douzzer approved these changes

@philljj philljj philljj approved these changes

Assignees

@wolfSSL-Bot wolfSSL-Bot

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@embhorn @douzzer @philljj @wolfSSL-Bot