From eb2e29c354ceb7ffe3bcc786929006a6cf9b13ea Mon Sep 17 00:00:00 2001 From: Anoop C S Date: Wed, 29 May 2019 11:40:55 +0530 Subject: [PATCH] s3-smbcontrol: Replace && with || to declare variables in do_sleep() MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit --enable-developer internally use --enable-selftest alongside. But when configured only with --enable-selftest the following code block becomes invalid: #if defined(DEVELOPER) && defined(ENABLE_SELFTEST) unsigned int seconds; long input; const long MAX_SLEEP = 60 * 60; /* One hour maximum sleep */ #endif making those variables undeclared for further use. Therefore replace && with || to have those variables declared with --enable-selftest configure option. Signed-off-by: Anoop C S Reviewed-by: Ralph Boehme Reviewed-by: Alexander Bokovoy Autobuild-User(master): Ralph Böhme Autobuild-Date(master): Fri May 31 20:38:09 UTC 2019 on sn-devel-184 --- source3/utils/smbcontrol.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/source3/utils/smbcontrol.c b/source3/utils/smbcontrol.c index 7a761a6ff59..b2aa1dfa2a6 100644 --- a/source3/utils/smbcontrol.c +++ b/source3/utils/smbcontrol.c @@ -415,7 +415,7 @@ static bool do_sleep(struct tevent_context *ev_ctx, const struct server_id pid, const int argc, const char **argv) { -#if defined(DEVELOPER) && defined(ENABLE_SELFTEST) +#if defined(DEVELOPER) || defined(ENABLE_SELFTEST) unsigned int seconds; long input; const long MAX_SLEEP = 60 * 60; /* One hour maximum sleep */ -- 2.47.3