diff options
author | Rob van der Linde <rob@catalyst.net.nz> | 2023-02-23 15:54:37 +1300 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2023-02-23 23:33:46 +0000 |
commit | 5cb8805811ee5e5a880c1c2d42f4fd9b195abe75 (patch) | |
tree | 0592489f394d1e044a2f3befefc4c055d81e72ff /python/samba/netcmd/gpo.py | |
parent | e7c87b1d9bd10280bff69f1acaf292364b79d496 (diff) | |
download | samba-5cb8805811ee5e5a880c1c2d42f4fd9b195abe75.tar.gz |
python: fix mutable default arguments
Signed-off-by: Rob van der Linde <rob@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Feb 23 23:33:46 UTC 2023 on atb-devel-224
Diffstat (limited to 'python/samba/netcmd/gpo.py')
-rw-r--r-- | python/samba/netcmd/gpo.py | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/python/samba/netcmd/gpo.py b/python/samba/netcmd/gpo.py index 340f8f55e5e..9b00a9016c3 100644 --- a/python/samba/netcmd/gpo.py +++ b/python/samba/netcmd/gpo.py @@ -761,9 +761,13 @@ class cmd_load(GPOCommand): ] def run(self, gpo, H=None, content=None, - machine_exts=['{35378EAC-683F-11D2-A89A-00C04FBBCFA2}'], - user_exts=['{35378EAC-683F-11D2-A89A-00C04FBBCFA2}'], + machine_exts=None, + user_exts=None, replace=False, sambaopts=None, credopts=None, versionopts=None): + if machine_exts is None: + machine_exts = ['{35378EAC-683F-11D2-A89A-00C04FBBCFA2}'] + if user_exts is None: + user_exts = ['{35378EAC-683F-11D2-A89A-00C04FBBCFA2}'] if content is None: policy_defs = json.loads(sys.stdin.read()) elif os.path.exists(content): @@ -848,8 +852,12 @@ class cmd_remove(GPOCommand): help="A user extension name to remove from gPCUserExtensionNames") ] - def run(self, gpo, H=None, content=None, machine_exts=[], user_exts=[], + def run(self, gpo, H=None, content=None, machine_exts=None, user_exts=None, sambaopts=None, credopts=None, versionopts=None): + if machine_exts is None: + machine_exts = [] + if user_exts is None: + user_exts = [] if content is None: policy_defs = json.loads(sys.stdin.read()) elif os.path.exists(content): |