[LEDE-DEV] [PATCH] dnsmasq: fix error in call of list_contains()

Hans Dedecker dedeckeh at gmail.com
Tue Jun 13 00:47:38 PDT 2017


On Mon, Jun 12, 2017 at 11:11 PM, Paul Oranje <por at xs4all.nl> wrote:
> Fixes the first parameter to the call of list_contains() in dnsmasq_ismain()
>
> Signed-off-by: Paul Oranje <por at xs4all.nl>
> ---
> Commit a53f8ba6771de64c9c82a2e6867791226f3003cb introduces an error that has not
> shown up in my tests of that commit (testing sh script is tough).
> Thanks to Hartmut (e9hack).
> ---
>  package/network/services/dnsmasq/files/dnsmasq.init | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/package/network/services/dnsmasq/files/dnsmasq.init b/package/network/services/dnsmasq/files/dnsmasq.init
> index fda11401db..8dc11288af 100644
> --- a/package/network/services/dnsmasq/files/dnsmasq.init
> +++ b/package/network/services/dnsmasq/files/dnsmasq.init
> @@ -716,7 +716,7 @@ dnsmasq_ismain()
>         [ $port = "53" ] || return 1
>
>         config_get notinterfaces "$cfg" notinterface ""
> -       [ -n $notinterfaces ] && list_contains $notinterfaces "loopback" || return 1
> +       [ -n $notinterfaces ] && list_contains notinterfaces "loopback" || return 1
>
>         # dnsmasq instance is designated to listen on 127.0.0.1#53.
>         return 0
> --
> 2.13.1
>
Increase PKG_RELEASE as well in the patch.

Hans



More information about the Lede-dev mailing list