Attachment 77872 Details for Bug 62518 – Use a dollar sign as our simple_allow_users placeholder

[patch] Use a dollar sign as our simple_allow_users placeholder

Use-a-dollar-sign-as-our-simpleallowusers-placehol.patch (text/plain), 2.09 KB, created by Stef Walter on 2013-04-12 12:22:18 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Stef Walter

Created: 2013-04-12 12:22:18 UTC

Size: 2.09 KB

patch

obsolete

>From 0bb8f0a94dc89c7e56f49e9baf02275f99f7b323 Mon Sep 17 00:00:00 2001
>From: Stef Walter <stefw@redhat.com>
>Date: Fri, 12 Apr 2013 13:46:29 +0200
>Subject: [PATCH] Use a dollar sign as our simple_allow_users placeholder
>
>We have to use a placeholder to get sssd to recognize that the
>simple_allow_users is to allow no users to login.
>
>Simo recommended using a dollar sign instead of a comma.
>
>https://bugs.freedesktop.org/show_bug.cgi?id=62518
>---
> service/realm-sssd.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/service/realm-sssd.c b/service/realm-sssd.c
>index a91dc79..65ba49f 100644
>--- a/service/realm-sssd.c
>+++ b/service/realm-sssd.c
>@@ -90,12 +90,12 @@ sssd_config_change_login_policy (RealmIniConfig *config,
> 
> 	/*
> 	 * HACK: Work around for sssd problem where it allows users if
>-	 * simple_allow_users is empty. Set it to a comma in this case.
>+	 * simple_allow_users is empty. Set it to a dollar in this case.
> 	 */
> 	allow = realm_ini_config_get (config, section, "simple_allow_users");
> 	if (allow != NULL) {
> 		g_strstrip (allow);
>-		if (g_str_equal (allow, "") || g_str_equal (allow, ",")) {
>+		if (g_str_equal (allow, "") || g_str_equal (allow, "$") || g_str_equal (allow, ",")) {
> 			g_free (allow);
> 			allow = NULL;
> 		}
>@@ -103,7 +103,7 @@ sssd_config_change_login_policy (RealmIniConfig *config,
> 
> 	if (allow == NULL) {
> 		if (g_str_equal (access_provider, "simple"))
>-			realm_ini_config_set (config, section, "simple_allow_users", ",");
>+			realm_ini_config_set (config, section, "simple_allow_users", "$");
> 		else
> 			realm_ini_config_set (config, section, "simple_allow_users", NULL);
> 	}
>@@ -320,7 +320,7 @@ update_login_policy (RealmSssd *self)
> 		values = realm_ini_config_get_list (self->pv->config, self->pv->section,
> 		                                    "simple_allow_users", ",");
> 		for (i = 0; values != NULL && values[i] != NULL; i++) {
>-			if (!g_str_equal (values[i], ""))
>+			if (!g_str_equal (values[i], "") && !g_str_equal (values[i], "$"))
> 				g_ptr_array_add (permitted, realm_kerberos_format_login (kerberos, values[i]));
> 		}
> 		g_strfreev (values);
>-- 
>1.8.1.4

Actions: View | Splinter Review

Attachments on bug 62518: 77866 | 77867 | 77868 | 77872

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.