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

Fix buffer overflow error when bc has 12 characters #192

Merged
merged 1 commit into from
Oct 3, 2023
Merged

Fix buffer overflow error when bc has 12 characters #192

merged 1 commit into from
Oct 3, 2023

Conversation

lkeegan
Copy link
Contributor

@lkeegan lkeegan commented Oct 2, 2023

@lkeegan
Fix buffer overflow error when bc has 12 characters
3c77170 
- mcx_config.bc has type char[12], with no null terminator required
- resolves fangq#191
@fangq fangq merged commit 35170f9 into fangq:master Oct 3, 2023
25 checks passed
@fangq
Copy link
Owner

fangq commented Oct 3, 2023

thanks for reporting this and provide a fix. patch merged.

fangq added a commit that referenced this pull request Oct 3, 2023
@fangq
treat cfg->bc as a null-ending string, #191 #192
2e71a51
fangq added a commit to fangq/mcxcl that referenced this pull request Oct 3, 2023
@fangq
fix bc mcxlab/pmcxcl memory error,fangq/mcx#191,fangq/mcx#192
fa8dbb2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

pmcx buffer overflow error if bc parameter has 12 characters
2 participants
@lkeegan @fangq