[Bug 54875] make the default account backend store GVariants on disk

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=54875

--- Comment #13 from Guillaume Desmottes <guillaume.desmottes at collabora.co.uk> ---
Comment on attachment 88297
  --> https://bugs.freedesktop.org/attachment.cgi?id=88297
Default account backend: store each account as a  stringified GVariant in a
file

Review of attachment 88297:
 --> (https://bugs.freedesktop.org/page.cgi?id=splinter.html&bug=54875&attachment=88297)
-----------------------------------------------------------------

one minor comment

::: src/mcd-account-manager-default.c
@@ +422,5 @@
> +      for (iter = g_get_system_data_dirs ();
> +          iter != NULL && *iter != NULL;
> +          iter++)
> +        {
> +          gchar *other = account_file_in (*iter, account_name);

'other' is leaked.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.