Skip to content

Add fencing to ClientSessionToSession()#7096

Merged
SparkiDev merged 3 commits intowolfSSL:masterfrom
julek-wolfssl:zd/17219
Jan 4, 2024
Merged

Add fencing to ClientSessionToSession()#7096
SparkiDev merged 3 commits intowolfSSL:masterfrom
julek-wolfssl:zd/17219

Conversation

@julek-wolfssl
Copy link
Copy Markdown
Member

Prevent memory access before clientSession->serverRow and clientSession->serverIdx are sanitized.

Fixes ZD17219

Co-authored-by: Daniele Lacamera dan@danielinux.net

Prevent memory access before clientSession->serverRow and clientSession->serverIdx are sanitized.

Fixes ZD17219

Co-authored-by: Daniele Lacamera <dan@danielinux.net>
danielinux
danielinux previously approved these changes Dec 28, 2023
Copy link
Copy Markdown
Member

@danielinux danielinux left a comment

Choose a reason for hiding this comment

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

LGTM

@julek-wolfssl
Copy link
Copy Markdown
Member Author

Retest this please

Comment thread wolfssl/wolfcrypt/wc_port.h
Comment thread wolfssl/wolfcrypt/wc_port.h
Comment thread wolfssl/wolfcrypt/wc_port.h
@dgarske dgarske assigned julek-wolfssl and unassigned dgarske and wolfSSL-Bot Dec 29, 2023
Copy link
Copy Markdown
Member

@dgarske dgarske left a comment

Choose a reason for hiding this comment

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

Over to @SparkiDev

@dgarske dgarske assigned SparkiDev and unassigned dgarske Jan 4, 2024
@SparkiDev SparkiDev merged commit 9e468a9 into wolfSSL:master Jan 4, 2024
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.

5 participants