imap-login: Fix storing ID NIL values into client_id

diff --git a/src/imap-login/imap-login-client.c b/src/imap-login/imap-login-client.c
index a4b1957665f80ccf98491c34dfeba467a3534748..16999f2bb814711adb2b73bdc218132226308be0 100644 (file)
--- a/src/imap-login/imap-login-client.c
+++ b/src/imap-login/imap-login-client.c
@@ -208,12 +208,15 @@ static void cmd_id_handle_keyvalue(struct imap_client *client,
        size_t kvlen = strlen(key) + 2 + 1 +
                       (value == NULL ? 3 : strlen(value)) + 2;
 
-       if (value == NULL) {
-               /* do not try to process NIL value */
-               client_id_str = FALSE;
-       } else if (client->common.trusted && !client->id_logged) {
-               client_id_str = !client_update_info(client, key, value);
-               i_assert(client_id_str == !client_id_reserved_word(key));
+       if (client->common.trusted && !client->id_logged) {
+               if (value == NULL) {
+                       /* do not try to process NIL values as client-info,
+                          but store them for non-reserved keys */
+                       client_id_str = !client_id_reserved_word(key);
+               } else {
+                       client_id_str = !client_update_info(client, key, value);
+                       i_assert(client_id_str == !client_id_reserved_word(key));
+               }
        } else {
                client_id_str = !client_id_reserved_word(key);
        }