diff options
| author | Ray Strode <rstrode@redhat.com> | 2018-05-09 19:34:19 -0400 |
|---|---|---|
| committer | Ray Strode <rstrode@redhat.com> | 2018-05-09 19:34:19 -0400 |
| commit | c17b57e1ffc3d6086cb4bbe17ff40a091662d1be (patch) | |
| tree | 80295072ee29501fdbdc2b8a87f08d1622286063 | |
| parent | 9d14729519d1d33768cd2df139d44eaabe56ce68 (diff) | |
daemon: move g_autofree out of if block
The admin group variable is getting prematurely freed.
This commit fixes that.
| -rw-r--r-- | src/daemon.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/daemon.c b/src/daemon.c index a12b71b..66406c6 100644 --- a/src/daemon.c +++ b/src/daemon.c @@ -1061,6 +1061,7 @@ daemon_create_user_authorized_cb (Daemon *daemon, CreateUserData *cd = data; User *user; g_autoptr(GError) error = NULL; + g_autofree gchar *admin_groups = NULL; const gchar *argv[9]; if (getpwnam (cd->user_name) != NULL) { @@ -1075,8 +1076,6 @@ daemon_create_user_authorized_cb (Daemon *daemon, argv[2] = "-c"; argv[3] = cd->real_name; if (cd->account_type == ACCOUNT_TYPE_ADMINISTRATOR) { - g_autofree gchar *admin_groups = NULL; - admin_groups = g_strdup (ADMIN_GROUP); if (EXTRA_ADMIN_GROUPS != NULL && EXTRA_ADMIN_GROUPS[0] != '\0') admin_groups = g_strconcat (admin_groups, ",", |