On Sun, 2016-10-09 at 15:34 +0300, Mantas Mikulėnas wrote: > + s = g_hash_table_lookup(attributes, "user"); > + if (s) { > + g_free(c->username); > + c->username = g_strdup(s); > + } This always overwrites c->username, the original gnome-keyring version only does that when the username isn't set. Other than that it looks good to me. Reviewed-by: Dennis Kaarsemaker <dennis@xxxxxxxxxxxxxxx> Tested-by: Dennis Kaarsemaker <dennis@xxxxxxxxxxxxxxx> D.