Skip to content

fix: string termination in sysctlbyname#212

Open
ohaiibuzzle wants to merge 2 commits intoPlayCover:masterfrom
ohaiibuzzle:fix/string-terminate
Open

fix: string termination in sysctlbyname#212
ohaiibuzzle wants to merge 2 commits intoPlayCover:masterfrom
ohaiibuzzle:fix/string-terminate

Conversation

@ohaiibuzzle
Copy link
Member

@ohaiibuzzle ohaiibuzzle commented Feb 2, 2026

When copying a string, it should include the null termination bytes, else...
image

@TheMoonThatRises
Copy link
Member

Would it be possible to have the machine char array contain the null byte originally rather than appending when copying?

@ohaiibuzzle
Copy link
Member Author

@TheMoonThatRises Actually now that I think about it, why are we even calling the original funcs at all...?

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.

3 participants