Attachment 131015 Details for Bug 98479 – FIXUP: Actually save the changes to the services line

[patch] FIXUP: Actually save the changes to the services line

0001-FIXUP-Actually-save-the-changes-to-the-services-line.patch (text/plain), 1.46 KB, created by Stef Walter on 2017-04-25 08:25:11 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Stef Walter

Created: 2017-04-25 08:25:11 UTC

Size: 1.46 KB

patch

obsolete

>From 82642b3b622ef043ff497709fc1ddb1b2ab5328d Mon Sep 17 00:00:00 2001
>From: Stef Walter <stefw@redhat.com>
>Date: Tue, 25 Apr 2017 10:15:59 +0200
>Subject: [PATCH] FIXUP: Actually save the changes to the services lines
>
>---
> service/realm-sssd-ad.c  | 3 ++-
> service/realm-sssd-ipa.c | 3 ++-
> 2 files changed, 4 insertions(+), 2 deletions(-)
>
>diff --git a/service/realm-sssd-ad.c b/service/realm-sssd-ad.c
>index 5fa81ce..8543ca8 100644
>--- a/service/realm-sssd-ad.c
>+++ b/service/realm-sssd-ad.c
>@@ -207,7 +207,8 @@ configure_sssd_for_domain (RealmIniConfig *config,
> 	                                    "ldap_sasl_authid", authid,
> 	                                    NULL);
> 
>-	realm_ini_config_set_list_diff (config, "sssd", "services", ", ", services, NULL);
>+	if (ret)
>+		ret = realm_ini_config_change_list (config, "sssd", "services", ", ", services, NULL, error);
> 
> 	g_free (authid);
> 	g_string_free (realmd_tags, TRUE);
>diff --git a/service/realm-sssd-ipa.c b/service/realm-sssd-ipa.c
>index 001870d..ff1dc8a 100644
>--- a/service/realm-sssd-ipa.c
>+++ b/service/realm-sssd-ipa.c
>@@ -208,7 +208,8 @@ on_ipa_client_do_restart (GObject *source,
> 		                                 "realmd_tags", realmd_tags,
> 		                                 NULL);
> 
>-		realm_ini_config_set_list_diff (config, "sssd", "services", ", ", services, NULL);
>+		if (error == NULL)
>+			realm_ini_config_change_list (config, "sssd", "services", ", ", services, NULL, &error);
> 
> 		g_free (home);
> 	}
>-- 
>2.7.4
>

Actions: View | Splinter Review

Attachments on bug 98479: 127592 | 131010 | 131011 | 131013 | 131015 | 131017

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.