summaryrefslogtreecommitdiff
path: root/source3/utils
diff options
context:
space:
mode:
authorAnoop C S <anoopcs@redhat.com>2019-05-29 11:40:55 +0530
committerRalph Boehme <slow@samba.org>2019-05-31 20:38:09 +0000
commiteb2e29c354ceb7ffe3bcc786929006a6cf9b13ea (patch)
treeec880be0d294704e1064026a637e15943052ad32 /source3/utils
parentb5bf1b66d55f40862ece83ef013793fb303acca7 (diff)
downloadsamba-eb2e29c354ceb7ffe3bcc786929006a6cf9b13ea.tar.gz
s3-smbcontrol: Replace && with || to declare variables in do_sleep()
--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 <anoopcs@redhat.com> Reviewed-by: Ralph Boehme <slow@samba.org> Reviewed-by: Alexander Bokovoy <ab@samba.org> Autobuild-User(master): Ralph Böhme <slow@samba.org> Autobuild-Date(master): Fri May 31 20:38:09 UTC 2019 on sn-devel-184
Diffstat (limited to 'source3/utils')
-rw-r--r--source3/utils/smbcontrol.c2
1 files changed, 1 insertions, 1 deletions
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 */