diff --git a/daemon/unbound.c b/daemon/unbound.c index ceb3da6f5..3f96603c4 100644 --- a/daemon/unbound.c +++ b/daemon/unbound.c @@ -534,6 +534,8 @@ perform_setup(struct daemon* daemon, struct config_file* cfg, int debug_mode, LOGIN_SETALL & ~LOGIN_SETUSER & ~LOGIN_SETGROUP) != 0) log_warn("unable to setusercontext %s: %s", cfg->username, strerror(errno)); +#else + (void)pwd; #endif /* HAVE_SETUSERCONTEXT */ } #endif /* HAVE_GETPWNAM */ diff --git a/doc/Changelog b/doc/Changelog index d7fb1e298..f3450e075 100644 --- a/doc/Changelog +++ b/doc/Changelog @@ -1,3 +1,7 @@ +16 July 2020: Wouter + - Fix check conf test for referencing installation paths. + - Fix unused variable warning for clang analyzer. + 16 July 2020: George - Introduce 'include-toplevel:' configuration option. diff --git a/testdata/04-checkconf.tdir/bad.badfwd b/testdata/04-checkconf.tdir/bad.badfwd index 27d58c5ea..139d31238 100644 --- a/testdata/04-checkconf.tdir/bad.badfwd +++ b/testdata/04-checkconf.tdir/bad.badfwd @@ -2,6 +2,7 @@ server: # to make sure the check doesn't fail on username or chrootdir. username: "" chroot: "" + directory: "" forward-zone: name: "example.com" diff --git a/testdata/04-checkconf.tdir/bad.user b/testdata/04-checkconf.tdir/bad.user index 40631abb4..ffa556e44 100644 --- a/testdata/04-checkconf.tdir/bad.user +++ b/testdata/04-checkconf.tdir/bad.user @@ -1,2 +1,4 @@ server: username: blabla_noexist_user + chroot: "" + directory: "" diff --git a/testdata/04-checkconf.tdir/good.include-toplevel b/testdata/04-checkconf.tdir/good.include-toplevel index 91dc6980a..e88df7d56 100644 --- a/testdata/04-checkconf.tdir/good.include-toplevel +++ b/testdata/04-checkconf.tdir/good.include-toplevel @@ -10,3 +10,7 @@ server: identity: "top 2" include-toplevel: include.includetop.withclauses.* include-toplevel: include.include.withclauses.* include-toplevel: include.include.withclauses.* +server: + chroot: "" + directory: "" + username: ""