netcmd: models: SDDLField move line down where it gets used

Signed-off-by: Rob van der Linde <rob@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jo Sutton <josutton@catalyst.net.nz>

diff --git a/python/samba/netcmd/domain/models/fields.py b/python/samba/netcmd/domain/models/fields.py
index 9a3a07569c778ed794bc2ddfa99d0a19e84014d1..d2a3b5e9d9e2408fe08cf2c28c53262d6126205b 100644 (file)
--- a/python/samba/netcmd/domain/models/fields.py
+++ b/python/samba/netcmd/domain/models/fields.py
@@ -402,7 +402,6 @@ class SDDLField(Field):
             return ndr_unpack(security.descriptor, value[0])
 
     def to_db_value(self, ldb, value, flags):
-        domain_sid = security.dom_sid(ldb.get_domain_sid())
         if value is None:
             return
         elif isinstance(value, list):
@@ -411,6 +410,8 @@ class SDDLField(Field):
                 flags,
                 self.name)
         else:
+            domain_sid = security.dom_sid(ldb.get_domain_sid())
+
             # If this is a SDDL string convert it to a descriptor.
             if isinstance(value, str):
                 desc = security.descriptor.from_sddl(