Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Fix a bad buffer overflow in inv_strs. #162

Merged
merged 1 commit into from
Jun 6, 2021
Merged

Fix a bad buffer overflow in inv_strs. #162

merged 1 commit into from
Jun 6, 2021

Conversation

heiner
Copy link
Contributor

@heiner heiner commented Jun 5, 2021

No description provided.

@heiner heiner requested a review from cdmatters June 5, 2021 09:01
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 5, 2021
@heiner heiner force-pushed the heiner/fix-bufferoverflow branch from 36db803 to 8b927bf Compare June 5, 2021 09:04
cdmatters
cdmatters approved these changes Jun 6, 2021
View reviewed changes
Copy link
Contributor

@cdmatters cdmatters left a comment

Choose a reason for hiding this comment

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

We should check for this elsewhere. My bet is this could be causing problems in screen_descriptions and maybe also message.

@heiner heiner merged commit e464dba into master Jun 6, 2021
@heiner heiner deleted the heiner/fix-bufferoverflow branch July 7, 2021 10:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cdmatters cdmatters cdmatters approved these changes

Assignees
No one assigned
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@heiner @cdmatters @facebook-github-bot