Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Also disallow statnames with slashes in them #53

Closed
wants to merge 1 commit into from
Closed

Also disallow statnames with slashes in them #53

wants to merge 1 commit into from

Conversation

sloppyfocus
Copy link
Contributor

No description provided.

@eklitzke
Copy link
Contributor

This uses BSD-specific functions. If you update this to use standard C/POSIX extensions I will accept this.

@sloppyfocus
Copy link
Contributor Author

No

@kobolog
Copy link

kobolog commented Feb 12, 2016

JFYI, memmem(3) is POSIX-compliant and doesn't rely on null-termination.

yashsarma referenced this pull request in lyft/statsrelay Feb 8, 2017
@lyft-buildnotify-3
version bump (#53)
5049009 
NOT PASSING TESTS
theatrus referenced this pull request in lyft/statsrelay Feb 13, 2017
@lyft-buildnotify-3
version bump (#53)
dbcae09 
NOT PASSING TESTS
theatrus referenced this pull request in lyft/statsrelay May 1, 2020
@lyft-buildnotify-3
version bump (#53)
993cd2c 
NOT PASSING TESTS
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.

None yet
3 participants
@sloppyfocus @eklitzke @kobolog