From: Douglas Bagnall Date: Thu, 27 Jun 2019 04:57:21 +0000 (+1200) Subject: s4/scripting/mymachinepw: print usage with bad arguments X-Git-Tag: samba-4.11.0rc1~87 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=ffdb0ca8b099ce638854dbb974093b2021f7ed50;p=thirdparty%2Fsamba.git s4/scripting/mymachinepw: print usage with bad arguments Also, use sys.exit() function, not exit(), because sys.exit() reliably exists. Signed-off-by: Douglas Bagnall Reviewed-by: Andrew Bartlett --- diff --git a/selftest/knownfail.d/usage b/selftest/knownfail.d/usage index 193e1bbb7c5..c5028e1d31f 100644 --- a/selftest/knownfail.d/usage +++ b/selftest/knownfail.d/usage @@ -1,7 +1,6 @@ samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_chgtdcpass.none. samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_demodirsync_py.none. samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_findprovisionusnranges.none. -samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_mymachinepw.none. samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_rebuildextendeddn.none. samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_renamedc.none. samba.tests.usage.samba.tests.usage.PythonScriptUsageTests.test_repl_cleartext_pwd_py.none. diff --git a/source4/scripting/bin/mymachinepw b/source4/scripting/bin/mymachinepw index 921cd53bffc..5ad9c7e96dc 100755 --- a/source4/scripting/bin/mymachinepw +++ b/source4/scripting/bin/mymachinepw @@ -30,7 +30,8 @@ for o, v in optlist: if o == "-s": if not conf.load(v): print(v + " not found") - exit(1) + print("\nUsage: mymachinepw [-s SMBCONF]") + sys.exit(1) loaded = True if not loaded: @@ -50,9 +51,11 @@ if not msg: print("Error:") print("Password for host[%s] not found in path[%s]." % (netbios, path)) print("You may want to pass the smb.conf location via the -s option.") - exit(1) + print() + print("Usage: mymachinepw [-s SMBCONF]") + sys.exit(1) password=msg[0]['secret'][0] print(password) -exit(0) +sys.exit(0)