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

Restore port 53 of dnsmasq after the smartdns is stopped. #1738

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zxlhhyccc
Copy link
Contributor

No description provided.

@zxlhhyccc
Copy link
Contributor Author

@TDXCHAIN 何意思?

@@ -119,8 +119,10 @@ stop_main_dns()
uci -q set dhcp.@dnsmasq[0].dns_redirect=1
uci -q delete dhcp.@dnsmasq[0].old_dns_redirect
}
uci -q delete dhcp.@dnsmasq[0].port
[ "$dnsmasq_port" = "0" ] && {
uci -q set dhcp.@dnsmasq[0].port="53"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不设置的话应该是有默认值的。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里选择删除端口信息的话,dnsmasq应该会创建默认的53端口配置。

是遇到什么问题么?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里选择删除端口信息的话,dnsmasq应该会创建默认的53端口配置。

是遇到什么问题么?

根据我实际使用情况来看,dnsmasq默认配置上端口号可能为空值(此时确实是53端口)。在此情况下,smartdns将其改写为53正确但不合理(别动我的dnsmasq!)。
顺便说一下当openwrt只有一个接口且配置为dhcp client时,配置文件中的接口ip也可能是空值,也会导致smartdns的一些问题。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里选择删除端口信息的话,dnsmasq应该会创建默认的53端口配置。

是遇到什么问题么?

uci配置和实际运行配置不一样。dnsmasq创建53端口的运行配置不会改变uci的配置,此时读uci配置的端口号可能是空值。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uci配置和实际运行配置不一样。dnsmasq创建53端口的运行配置不会改变uci的配置,此时读uci配置的端口号可能是空值。

实际情况是:
1、在smartdns未启动时,dnsmasq默认配置上端口号实际是空值,如事前dnsmasq默认配置上端口号设置为53,端口即为53。
2、在smartdns启动后,smartdns将dnsmasq默认配置上端口号改写为0。
3、停用smartdns后,smartdns会将dnsmasq默认配置上端口号变为空值,而不会恢复为53端口(假设第1项smartdns未启动情况且已设置了dnsmasq默认配置上端口号为53)。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants