Skip to content

Fix setting and cleanup around flags weOwnKey and weOwnAltKey - #11200

Open
padelsbach wants to merge 1 commit into
wolfSSL:masterfrom
padelsbach:weown-fixes
Open

Fix setting and cleanup around flags weOwnKey and weOwnAltKey#11200
padelsbach wants to merge 1 commit into
wolfSSL:masterfrom
padelsbach:weown-fixes

Conversation

@padelsbach

Copy link
Copy Markdown
Contributor

Description

Address potential double-free cases by adding missing paths where the flags were not set.

Testing

Added unit tests.

Checklist

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

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fenrir Automated Review — PR #11200

Scan targets checked: wolfssl-bugs, wolfssl-src

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

This review was generated automatically by Fenrir. Blocking findings require changes before merge.

Comment thread src/internal.c
Comment thread tests/api.c Outdated
Comment thread tests/api.c
Comment thread src/internal.c
Comment thread tests/api.c
@padelsbach

Copy link
Copy Markdown
Contributor Author

jenkins retest this please

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