[SSSD] [PATCH] backend: add PAC to the list of known clients

Sumit Bose sbose at redhat.com
Mon Nov 12 17:31:16 UTC 2012


On Mon, Nov 12, 2012 at 04:47:29PM +0100, Pavel Březina wrote:
> This patch hides following message:
> [client_registration] (0x0020): Unknown client! [PAC]

> From 0a0437a444e473a3b5b62dc133723a87925a6e2b Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Pavel=20B=C5=99ezina?= <pbrezina at redhat.com>
> Date: Mon, 12 Nov 2012 16:44:37 +0100
> Subject: [PATCH] backend: add PAC to the list of known clients
> 
> ---
>  src/providers/data_provider_be.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/providers/data_provider_be.c b/src/providers/data_provider_be.c
> index 0ac83f43be797555388d2a2dc1c41a25ddd53c05..253dc46fb3010b2270ef454bb97e47243a18ceed 100644
> --- a/src/providers/data_provider_be.c
> +++ b/src/providers/data_provider_be.c
> @@ -1663,6 +1663,8 @@ static int client_registration(DBusMessage *message,
>          becli->bectx->autofs_cli = becli;
>      } else if (strcasecmp(cli_name, "SSH") == 0) {
>          becli->bectx->ssh_cli = becli;
> +    } else if (strcasecmp(cli_name, "PAC") == 0) {
> +        /* no need to set becli */

I wonder if those becli->bectx->*_cli members are still used somewhere?
The only place I see is be_client_destructor() to print a debug message
with the type of the client. If there is no other use, would it make
sense to open a ticket to remove those member from struct be_ctx?

bye,
Sumit

>      } else {
>          DEBUG(1, ("Unknown client! [%s]\n", cli_name));
>      }
> -- 
> 1.7.11.7
> 

> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel




More information about the sssd-devel mailing list