Merge lp:~aacid/unity-greeter/check_remote_fields_before_use into lp:unity-greeter

[Albert Astals Cid]

Check the fields are there, it'd be rare but it can happen if r-l-s sends us wrong data

Check the fields are there, it'd be rare but it can happen if r-l-s sends us wrong data

[Michael Terry (mterry)]

Looks fine.

review: Approve

=== modified file 'src/user-list.vala'
--- src/user-list.vala	2012-08-22 14:24:18 +0000
+++ src/user-list.vala	2012-08-23 12:55:23 +0000
@@ -333,6 +333,17 @@
 
         DashEntry password_field = current_remote_fields.get("password") as DashEntry;
         Gtk.Entry email_field = current_remote_fields.get("email") as Gtk.Entry;
+        if (password_field == null)
+        {
+            debug("Something wrong happened in remote_directory_respond_cb. There was no password field");
+            return;
+        }
+        if (email_field == null)
+        {
+            debug("Something wrong happened in remote_directory_respond_cb. There was no email field");
+            return;
+        }
+
         RemoteServer[] server_list = {};
         string email = email_field.text;
         bool email_valid = false;