Merge lp:~seb128/unity-control-center/users-no-debug-output into lp:unity-control-center

[Sebastien Bacher]

users: don't display the ecryptfs-verify output on stdout

users: don't display the ecryptfs-verify output on stdout

[Robert Ancell (robert-ancell)]

Does it leak anything important of just annoying?

review: Approve

[Robert Ancell (robert-ancell)]

Shouldn't you free nostdout?

review: Needs Fixing

[Sebastien Bacher (seb128)]

> Shouldn't you free nostdout?

indeed, thanks for catching that, should teach me to want to push a change when I'm already late rather than doing it properly later!

[Sebastien Bacher (seb128)]

(sorry pushed by mistake to trunk, I reverted that and I'm changing to status back)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:12757
http://jenkins.qa.ubuntu.com/job/unity-control-center-ci/78/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-trusty-amd64-ci/78
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-trusty-armhf-ci/78
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-trusty-i386-ci/79

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-control-center-ci/78/rebuild

review: Approve (continuous-integration)

[lp:~seb128/unity-control-center/users-no-debug-output]

12759. By Sebastien Bacher on 2014-03-18

correct the spacing

Unmerged revisions

=== modified file 'panels/user-accounts/um-utils.c'
--- panels/user-accounts/um-utils.c	2014-02-21 15:48:18 +0000
+++ panels/user-accounts/um-utils.c	2014-03-18 19:45:42 +0000
@@ -777,12 +777,13 @@
         int status;
         gchar *prog;
         gchar *cmd;
+        gchar *nostdout = NULL;
 
         prog = g_find_program_in_path ("ecryptfs-verify");
         if (prog != NULL) {
                 gchar *cmd = g_strdup_printf("'%s' -h -u '%s'", prog, name);
 
-                if (g_spawn_command_line_sync (cmd, NULL, NULL, &status, NULL)) {
+                if (g_spawn_command_line_sync (cmd, &nostdout, NULL, &status, NULL)) {
                         if (status == 0)
                                 using_ecryptfs = TRUE;
                 }