Skip to content
New issue

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

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

Already on GitHub? # to your account

correct wrong use of sizeof #347

Merged
merged 1 commit into from
Feb 3, 2024
Merged

correct wrong use of sizeof #347

merged 1 commit into from
Feb 3, 2024

Conversation

baentsch
Copy link
Member

@baentsch baentsch commented Feb 2, 2024

Kudos to the team of RadicallyOpenSecurity for pointing out this possible memory overrun error.

@baentsch baentsch marked this pull request as ready for review February 2, 2024 10:49
@baentsch baentsch requested a review from a user February 2, 2024 12:09
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Nice catch!
Thanks RadicallyOpenSecurity :)

@baentsch baentsch merged commit 81eae6d into main Feb 3, 2024
44 checks passed
@baentsch baentsch deleted the mb-fixsizeofuse branch February 3, 2024 16:04
@baentsch
Copy link
Member Author

baentsch commented Feb 8, 2024

Tagging @pcwizz for review

Copy link

@pcwizz pcwizz left a comment

Choose a reason for hiding this comment

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

Nice fix.

feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 13, 2024
Signed-off-by: Felipe Ventura <felipe.ventura@entrust.com>
feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 16, 2024
Signed-off-by: Felipe Ventura <felipe.ventura@entrust.com>
feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 17, 2024
Signed-off-by: Felipe Ventura <felipe.ventura@entrust.com>
feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 17, 2024
Signed-off-by: Felipe Ventura <felipe.ventura@entrust.com>
feventura pushed a commit to EntrustCorporation/oqs-provider that referenced this pull request Mar 17, 2024
Signed-off-by: Felipe Ventura <felipe.ventura@entrust.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants