[PATCH v4 1/2] dnsmasq: Update to version 2.86test3

Etan Kissling etan.kissling at gmail.com
Sun Jun 27 00:47:45 PDT 2021



On 27.06.21, 09:36, "Oldřich Jedlička" <oldium.pro at gmail.com> wrote:

> The crash is caused by a conflict in UBUS service name (the root name
> of UBUS path). Previously only warning was logged when you tried to
> use multiple Dnsmasq instances with UBUS enabled - only the first one
> can register to the UBUS with the same name. Dying is a new behaviour
> caused by a recent commit [1].
> 
> For the long run, I think the best would be to fix this in the init
> script, I already added support to specify different UBUS names both
> in Dnsmasq (commit from last year [2]) and in Openwrt (see Pull
> Request [3]). The solution would be to pass a specific name for each
> Dnsmasq instance (maybe matching the section name?), not only the
> enable/disable flag.

Agree that it would be better to fix the init script to only request
Ubus on one of the instances, or to request different names for each
instance. 

However, as I only want to update the Dnsmasq version and because the 
recent commit that introduces the dying behaviour does not seem to have 
done this intentionally I think that fixing the unintended change in
behaviour is okay for now, as this introduces fewer changes / less risk.

Eventually, the init script should still be adjusted, though, regardless
of whether dnsmasq spits out warnings or crashes.

Etan




More information about the openwrt-devel mailing list