Skip to content

fix: Make sure we null terminate and filter file names#426

Merged
toktok-releaser merged 1 commit intoTokTok:masterfrom
JFreegman:process_filename
Jan 27, 2026
Merged

fix: Make sure we null terminate and filter file names#426
toktok-releaser merged 1 commit intoTokTok:masterfrom
JFreegman:process_filename

Conversation

@JFreegman
Copy link
Member

@JFreegman JFreegman commented Jan 27, 2026

The filename is currently null terminated by toxcore before being passed to this callback, but this is a happy accident and should not be relied upon.


This change is Reviewable

@JFreegman JFreegman added the bug Bug fix for the user, not a fix to a build script label Jan 27, 2026
@Green-Sky Green-Sky added this to the v0.16.2 milestone Jan 27, 2026
@JFreegman JFreegman changed the title bug: Make sure we null terminate and filter file names fix: Make sure we null terminate and filter file names Jan 27, 2026
@JFreegman JFreegman force-pushed the process_filename branch 2 times, most recently from ec4297b to 1fdbdc9 Compare January 27, 2026 21:05
The filename is currently null terminated by toxcore before being
passed to this callback, but this is a happy accident and should
not be relied upon.
@toktok-releaser toktok-releaser merged commit cd468b8 into TokTok:master Jan 27, 2026
16 checks passed
@JFreegman JFreegman deleted the process_filename branch January 27, 2026 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug fix for the user, not a fix to a build script

Development

Successfully merging this pull request may close these issues.

4 participants