Skip to content
This repository was archived by the owner on May 13, 2022. It is now read-only.

Issue #16 - xxHash Deviations - #17

Merged
brandondahler merged 1 commit into
masterfrom
bugfix/xxHash
Sep 27, 2015
Merged

brandondahler merged 1 commit into
masterfrom
bugfix/xxHash

Conversation

@brandondahler

Copy link
Copy Markdown
Owner

xxHash calculated incorrect valuex when in 64-bit mode and the input satisfies: length % 8 == 4.
xxHash calculated incorrect values when in either mode and the input length was greater than or equal to 2^31 characters in length.

Both of these issues cause breaking changes to any current users of xxHash given the value(s) could satisfy those constraints.

xxHash calculated incorrect valuex when in 64-bit mode and the input satisfies: length % 8 == 4.
xxHash calculated incorrect values when in either mode and the input length was greater than or equal to 2^31 characters in length.

Both of these issues cause breaking changes to any current users of xxHash given the value(s) could satisfy those constraints.
@brandondahler

Copy link
Copy Markdown
Owner Author

test this please

@DataHashFunctionBot

Copy link
Copy Markdown
Collaborator

Refer to this link for build results (access rights to CI server needed):
http://jenkins.dahler.org//job/Data.HashFunction/job/Data.HashFunction%20Pull%20Requests/34/

brandondahler added a commit that referenced this pull request Sep 27, 2015
@brandondahler
brandondahler merged commit 9a464b9 into master Sep 27, 2015
@brandondahler
brandondahler deleted the bugfix/xxHash branch September 27, 2015 01:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

⌂ 换网址