AdguardTeam / dnsproxy

Simple DNS proxy with DoH, DoT, DoQ and DNSCrypt support

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

DNS64 config option causes segfault

suffieldacademy opened this issue · comments

Was trying out the DNS64 configuration and noticed the process kept crashing on startup. Thought it was an issue with my YAML but realized it was the dns64 config option. Even the command-line argument causes it to crash:

dnsproxy --upstream=::1 --bootstrap=::1 --dns64
2023/09/06 15:25:54 [info] Starting dnsproxy v0.54.0
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x8 pc=0x8c3546]

goroutine 1 [running]:
main.initDNS64(0xc00013f628, 0xc0000bb8c0?)
	github.com/AdguardTeam/dnsproxy/main.go:569 +0x46
main.createProxyConfig(_)
	github.com/AdguardTeam/dnsproxy/main.go:344 +0x22d
main.run(0xc0000bb8c0)
	github.com/AdguardTeam/dnsproxy/main.go:258 +0x1c7
main.main()
	github.com/AdguardTeam/dnsproxy/main.go:237 +0x44f
commented

@suffieldacademy, hello again. Could you please try the master version now? It should fix this exact validation step, additionally requiring the --private-rdns-upstream option, see the README.md.

Just compiled and tried it, looks much better! Not sure of the ticket etiquette here (do you close issues or do I?) so happy to close this if you're ready.

Thanks for the quick fix.

commented

@suffieldacademy, closing the issue you've opened is appreciated. Thanks.