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

add missing null check #395

Merged
merged 1 commit into from
Jan 6, 2021
Merged

add missing null check #395

merged 1 commit into from
Jan 6, 2021

Commits on Jan 6, 2021

  1. add missing null check

    I have a unbound forward zone configured on my router for my $DAYJOB.
    The address associated with the zone is only accessible when the router
    is connected to a VPN. If the VPN connection is absent, trying to
    resolve any domain that must be handled by the zone crashes unbound.
    Turns out there's a missing NULL check in `comm_point_send_udp_msg()`.
    The same routine already has `if (addr) {} else {}` branches so I guess
    protecting the call to `log_addr()` using the same conditional is
    reasonable
    
    I have also committed the same fix to unbound shipped with OpenBSD[1].
    
    [1] https://marc.info/?l=openbsd-cvs&m=160993335615698&w=2
    mptre committed Jan 6, 2021
    Configuration menu
    Copy the full SHA
    422213c View commit details
    Browse the repository at this point in the history