Also fixed several other return codes for other sub-commands. Fixes bug #7615 --- src/bin/secstate | 16 ++++++++-------- src/secstate/main.py | 3 +++ 2 files changed, 11 insertions(+), 8 deletions(-) diff --git a/src/bin/secstate b/src/bin/secstate index b4c484d..2962ad4 100644 --- a/src/bin/secstate +++ b/src/bin/secstate @@ -87,28 +87,28 @@ def main(): return remove_content(sys.argv[arg_num:]) elif subcommand == 'select': - select(sys.argv[arg_num:], True) + return select(sys.argv[arg_num:], True) elif subcommand == 'deselect': - select(sys.argv[arg_num:], False) + return select(sys.argv[arg_num:], False) elif subcommand == 'audit': - audit(sys.argv[arg_num:]) + return audit(sys.argv[arg_num:]) elif subcommand == 'search': - search(sys.argv[arg_num:]) + return search(sys.argv[arg_num:]) elif subcommand == 'list': - list_content(sys.argv[arg_num:]) + return list_content(sys.argv[arg_num:]) elif subcommand == 'show': - show(sys.argv[arg_num:]) + return show(sys.argv[arg_num:]) elif subcommand == 'remediate': - remediate(sys.argv[arg_num:]) + return remediate(sys.argv[arg_num:]) elif subcommand == 'save': - save_profile(sys.argv[arg_num:]) + return save_profile(sys.argv[arg_num:]) else: sys.stderr.write("Uknown subcommand: %(command)s" % {'command':subcommand}) diff --git a/src/secstate/main.py b/src/secstate/main.py index 9a77b16..712878b 100644 --- a/src/secstate/main.py +++ b/src/secstate/main.py @@ -752,6 +752,9 @@ class Secstate: return True + self.log.error("Item '%(id)s' could not be found in the imported content" % {'id':item_id}) + return False + def sublist(self, content, arg, recurse, show_all, tabs=0): tabstr = "\t" * tabs selected = "" -- 1.7.2.1