Fix infinite loop when replacing No Identity card
authorKevin Wasserman <krwasserman@painless-security.com>
Thu, 12 Mar 2015 15:22:54 +0000 (11:22 -0400)
committerKevin Wasserman <krwasserman@painless-security.com>
Thu, 12 Mar 2015 15:22:54 +0000 (11:22 -0400)
Don't use the card list accessor that auto-adds 'No Identity' when
adding a card.

src/moonshot-identities-manager.vala

index a0506d8..292fb0e 100644 (file)
@@ -118,7 +118,7 @@ public class IdentityManagerModel : Object {
     {
         if (&candidate != null)
           candidate = null;
-        foreach (IdCard id_card in this.get_card_list())
+        foreach (IdCard id_card in this.store.get_card_list())
         {
           if (id_card.display_name == name)
           {
@@ -145,7 +145,7 @@ public class IdentityManagerModel : Object {
         bool duplicate_found = false;
         bool found = false;
         do {
-           var cards = get_card_list();
+           var cards = this.store.get_card_list();
            found = false;
            foreach (IdCard id_card in cards) {
                if ((card != id_card) && (id_card.nai == card.nai)) {