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.

Sign up for GitHub

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

Jump to bottom

Fix memcmp size argument off by one #10

Merged

realnc merged 1 commit into realnc:master from disconnect3d:patch-1

Feb 10, 2020

Contributor

disconnect3d commented Feb 10, 2020

The "true" string has 4 bytes so memcmp(p2, "true", 3) should be memcmp(p2, "true", 4)


          Fix memcmp size argument off by one

36b6ebc

The `"true"` string has 4 bytes so `memcmp(p2, "true", 3)` should be `memcmp(p2, "true", 4)`

realnc merged commit e952092 into realnc:master

Owner

realnc commented Feb 10, 2020

Thanks! Merged in frobtads as well: realnc/frobtads@52c09ea

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

Labels

None yet