summaryrefslogtreecommitdiff
path: root/python/samba/netcmd/gpo.py
diff options
context:
space:
mode:
authorRob van der Linde <rob@catalyst.net.nz>2023-02-23 15:54:37 +1300
committerAndrew Bartlett <abartlet@samba.org>2023-02-23 23:33:46 +0000
commit5cb8805811ee5e5a880c1c2d42f4fd9b195abe75 (patch)
tree0592489f394d1e044a2f3befefc4c055d81e72ff /python/samba/netcmd/gpo.py
parente7c87b1d9bd10280bff69f1acaf292364b79d496 (diff)
downloadsamba-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.py14
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):