Skip to content

Fix wcsdup malloc size #641

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.

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

Merged
merged 1 commit into from
Oct 2, 2022
Merged

Fix wcsdup malloc size #641

merged 1 commit into from
Oct 2, 2022

Conversation

izissise
Copy link
Contributor

Other wcs* function expect a full sizeof(wchar_t) NUL char at the end of the string.

A quick grep -R "sizeof(wchar_t)" ./ shows no other obvious candidate for this change

Other wcs* function expect a full sizeof(wchar_t) NUL char at the end
of the string.
Copy link
Owner

@jart jart left a comment

Choose a reason for hiding this comment

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

Thank you for spotting this and taking the time to send a PR.

Welcome as a new contributor. We need to go through one quick hurdle before we can proceed. Could you please email Justine Tunney [email protected] and say that you intend to assign her the copyright to the changes you contribute to Cosmopolitan?
Please send the email under your legal name rather than anonymously. Please use a real email account associated with your identity.
See CONTRIBUTING.md for further details. We make the ask because it helps us ship the tiniest most legally permissive prim and proper binaries.
You also only need to do it one time, so any future changes you send us can be quickly merged.

Thanks!

@jart jart merged commit 304cf88 into jart:master Oct 2, 2022
@izissise izissise mentioned this pull request May 21, 2025
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.

2 participants