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.

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

key hashing incorrectly uses 1 for empty strings #124

Open
ddelbondio opened this issue Jun 24, 2017 · 0 comments
Open

key hashing incorrectly uses 1 for empty strings #124

ddelbondio opened this issue Jun 24, 2017 · 0 comments

Comments

@ddelbondio
Copy link

ddelbondio commented Jun 24, 2017

The cmemcache_hash function currently returns 1 for empty strings (or hash values which would result in 0).
Looking at the libmemcache implementation, this selects a different server than libmemcache would.
As far as i can see the correct version should remove the "or 1" from the code.

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

No branches or pull requests

1 participant