diff options
author | Anoop C S <anoopcs@redhat.com> | 2019-05-29 11:40:55 +0530 |
---|---|---|
committer | Ralph Boehme <slow@samba.org> | 2019-05-31 20:38:09 +0000 |
commit | eb2e29c354ceb7ffe3bcc786929006a6cf9b13ea (patch) | |
tree | ec880be0d294704e1064026a637e15943052ad32 /source3/utils | |
parent | b5bf1b66d55f40862ece83ef013793fb303acca7 (diff) | |
download | samba-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.c | 2 |
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 */ |