diff options
| author |  Miek Gieben <miek@miek.nl>
| 2017-06-01 12:33:40 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2017-06-01 12:33:40 +0100 |
| commit | 30ecb83dce02491a22eb64674354dbd9b1fa08c7 (patch) | |
| tree | 98fc9d813a6df9f33b40112b7c2ae1d2b2bfad78 /middleware/proxy/grpc_test.go | |
| parent | e261ac1a6e64077e288ca95fbb805c9ce7aafba0 (diff) | |
| download | coredns-30ecb83dce02491a22eb64674354dbd9b1fa08c7.tar.gz coredns-30ecb83dce02491a22eb64674354dbd9b1fa08c7.tar.zst coredns-30ecb83dce02491a22eb64674354dbd9b1fa08c7.zip | |
middleware/secondary: fix crash with no zone (#680)
When CoreDNS starts up and can't get a zone transfer going the Apex is
empty. This `nil` is then transformed into wireformat, which fails with
a nil pointer dereference in Go DNS.
In this case we should just return SERVFAIL, because we don't have any
info (yet). Note the lookup code returned NXDOMAIN, which is correct
from a lookup standpoint, but also invalidates every name in the future
loaded zone.
Anyway, look for an apex before doing the lookup and return SERVFAIL if
nothing is found.
Fixes #679
Diffstat (limited to 'middleware/proxy/grpc_test.go')
0 files changed, 0 insertions, 0 deletions