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

geoipbackend: Check GeoIP_id_by_addr_gl and GeoIP_id_by_addr_v6_gl return value #6677

Merged
merged 1 commit into from May 28, 2018

Conversation

Projects
None yet
2 participants
@cmouse
Copy link
Contributor

cmouse commented May 27, 2018

Short description

The return value must be greater than 0, all other indicate lookup failure.

Fixes #6676 (when using non-geoip database, or database mismatch)

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)
geoipbackend: Check GeoIP_id_by_addr_gl and GeoIP_id_by_addr_v6_gl re…
…turn value

The return value must be greater than 0, all other indicate lookup failure.
Fixes signal 11 crash.

@rgacogne rgacogne merged commit f72aa6d into PowerDNS:master May 28, 2018

4 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
lgtm analysis: C/C++ No alert changes
Details
lgtm analysis: JavaScript No alert changes
Details
lgtm analysis: Python No alert changes
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.