BUG/MINOR: server: inherit from netns in srv_settings_cpy()

When support for 'namespace' keyword was added for the 'default-server'
directive in 22f41a2 ("MINOR: server: Make 'default-server' support
'namespace' keyword."), we forgot to copy the attribute from the parent
to the newly created server.

This resulted in the 'namespace' keyword being parsed without errors when
used from a 'default-server' directive, but in practise the option was
simply ignored.

There's no need to duplicate the netns struct because it is stored in
a shared list, so copying the pointer does the job.

This patch partially fixes GH #2038 and should be backported to all
stable versions.
This commit is contained in:
Aurelien DARRAGON 2023-06-14 09:53:32 +02:00 committed by Christopher Faulet
parent 4d56b725fb
commit 19b5a7c7a5

View File

@ -2404,6 +2404,7 @@ void srv_settings_cpy(struct server *srv, const struct server *src, int srv_tmpl
if (srv_tmpl) if (srv_tmpl)
srv->srvrq = src->srvrq; srv->srvrq = src->srvrq;
srv->netns = src->netns;
srv->check.via_socks4 = src->check.via_socks4; srv->check.via_socks4 = src->check.via_socks4;
srv->socks4_addr = src->socks4_addr; srv->socks4_addr = src->socks4_addr;
} }