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

Use unsigned integer arithmetic in parser_scan_content() #10

Merged
merged 1 commit into from Jul 31, 2017

Conversation

Projects
None yet
3 participants
@jwilk
Contributor

jwilk commented Jul 31, 2017

mask - 0x01010101 could cause signed integer overflow, which is undefined behavior.

Use unsigned integer arithmetic in parser_scan_content()
"mask - 0x01010101" could cause signed integer overflow, which is undefined
behavior.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jul 31, 2017

Coverage Status

Coverage remained the same at 56.259% when pulling e8dec40 on jwilk-forks:parser-intoverflow into b29c54c on jstedfast:master.

coveralls commented Jul 31, 2017

Coverage Status

Coverage remained the same at 56.259% when pulling e8dec40 on jwilk-forks:parser-intoverflow into b29c54c on jstedfast:master.

@jstedfast jstedfast merged commit 404099a into jstedfast:master Jul 31, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 56.259%
Details
@jstedfast

This comment has been minimized.

Show comment
Hide comment
@jstedfast

jstedfast Jul 31, 2017

Owner

Looks good, thanks!

Owner

jstedfast commented Jul 31, 2017

Looks good, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment