The migration performance patch added a crash bug if no entries were successfully migrated. This should fix it.

rob
>From 3724f1ee1fb4e231586d253c922c7dcf3e26f459 Mon Sep 17 00:00:00 2001
From: Rob Crittenden <rcrit...@redhat.com>
Date: Fri, 8 Feb 2013 09:42:34 -0500
Subject: [PATCH] Prevent a crash when no entries are successfully migrated.

It would fail in _update_default_group() because migrate_cnt wasn't
defined in context.

https://fedorahosted.org/freeipa/ticket/3386
---
 ipalib/plugins/migration.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/ipalib/plugins/migration.py b/ipalib/plugins/migration.py
index ccb484e4ae46826a29ffecae472a20198cadeb07..1ba21ece0cad7db35168b5ec088a267fcb337ef2 100644
--- a/ipalib/plugins/migration.py
+++ b/ipalib/plugins/migration.py
@@ -750,6 +750,7 @@ can use their Kerberos accounts.''')
             valid_gids = []
             invalid_gids = []
             migrate_cnt = 0
+            context['migrate_cnt'] = 0
             for (dn, entry_attrs) in entries:
                 context['migrate_cnt'] = migrate_cnt
                 s = datetime.datetime.now()
-- 
1.8.1

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to