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

Hash function issues in 32 bit environment #43

Merged
merged 2 commits into from Aug 8, 2019
Merged

Hash function issues in 32 bit environment #43

merged 2 commits into from Aug 8, 2019

Conversation

@AndriusA
Copy link
Collaborator

AndriusA commented Aug 8, 2019

Fixes an issue where in 32-bit environment the used hash function was yielding a different (64-bit) hash value, resulting in filters not matching.

Adds more tests and example scripts to generate and use serialized dat files, including a known good version of the full rule set pulled from the browser.

AndriusA added 2 commits Aug 7, 2019
…th the library in 32-bit environment
@AndriusA AndriusA requested a review from bbondy Aug 8, 2019
@bbondy
bbondy approved these changes Aug 8, 2019
@AndriusA AndriusA merged commit 01b4df0 into master Aug 8, 2019
0 of 4 checks passed
0 of 4 checks passed
Travis CI - Branch Build Errored
Details
Travis CI - Pull Request Build Errored
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.