dbcheck: Correctly initialise keep_transaction in missing_parent test
authorAndrew Bartlett <abartlet@samba.org>
Thu, 3 Nov 2016 00:30:56 +0000 (13:30 +1300)
committerGarming Sam <garming@samba.org>
Fri, 4 Nov 2016 03:41:19 +0000 (04:41 +0100)
Otherwise there is no point to this variable, we are trying to work out
if the subsequent modify succeded

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Garming Sam <garming@catalyst.net.nz>
BUG: https://bugzilla.samba.org/show_bug.cgi?id=9954

python/samba/dbchecker.py

index cc8116a04a8eb9c50c4f8d8a63070ec1698708b7..294c3200bd3f6361a3e47cc6ad2be6db4f2a6eb5 100644 (file)
@@ -682,7 +682,7 @@ newSuperior: %s""" % (str(from_dn), str(to_rdn), str(to_base)))
             self.report('Not moving object %s into LostAndFound' % (obj.dn))
             return
 
-        keep_transaction = True
+        keep_transaction = False
         self.samdb.transaction_start()
         try:
             nc_root = self.samdb.get_nc_root(obj.dn);