dnsmasq: mark global ubus context as closed after fork
authorSven Eckelmann <sven@narfation.org>
Sat, 18 Nov 2023 15:29:09 +0000 (16:29 +0100)
committerHauke Mehrtens <hauke@hauke-m.de>
Mon, 18 Mar 2024 20:55:47 +0000 (21:55 +0100)
commitaf22a169c1e480b991eb8c4e69674bf5846d6215
tree9ce1ff595f69dfc1643fbd5f75195d688bb8db5c
parent0a571c9e77b381ff4ea4beb42113b5cd43144f12
dnsmasq: mark global ubus context as closed after fork

If the dnsmasq process forks to handle TCP connections, it closes the ubus
context. But instead of changing the daemon wide pointer to NULL, only the
local variable was adjusted - and this portion of the code was even dropped
(dead store) by some optimizing compilers.

It makes more sense to change the daemon->ubus pointer because various
functions are already checking it for NULL. It is also the behavior which
ubus_destroy() implements.

Fixes: d8b33dad0bb7 ("dnsmasq: add support for monitoring and modifying dns lookup results via ubus")
Signed-off-by: Sven Eckelmann <sven@narfation.org>
(cherry picked from commit 711dcb77630e96e75413b5cdbe3ddb5432f394f6)
package/network/services/dnsmasq/patches/200-ubus_dns.patch