[SSSD] [PATCHES] SSH: Fix several client bugs

Jakub Hrozek jhrozek at redhat.com
Tue May 7 08:30:16 UTC 2013


On Fri, Apr 26, 2013 at 02:49:05PM +0200, Jan Cholasta wrote:
> Hi,
> 
> the attached patches fix
> <https://fedorahosted.org/sssd/ticket/1897>. See commit messages for
> more info. Backward compatibility with older clients is maintained.
> 
> Honza
> 
> -- 
> Jan Cholasta

[PATCH 1/4] UTIL: Add function sss_names_init_static
The functionality is OK, but I would prefer a different name..maybe
something like sss_names_init_ex or sss_names_init_pattern ? _static
sounds to me like the patterns are hardcoded internally.

[PATCH 2/4] SSH: Fix parsing of names from client requests
Can you please set is_user=false in sss_ssh_cmd_get_host_pubkeys()
explicitly so that a search for is_user makes it clear what it is set to
in different functions?

> @@ -675,6 +676,7 @@ static errno_t
>  ssh_cmd_parse_request(struct ssh_cmd_ctx *cmd_ctx)
>  {
>      struct cli_ctx *cctx = cmd_ctx->cctx;
> +    struct ssh_ctx *ssh_ctx = (struct ssh_ctx *)cctx->rctx->pvt_ctx;

Please use talloc_get_type here so that if there was a type mismatch, we
would get an error message in the core file.

[PATCH 3/4] SSH: Use separate field for domain name in client
Looks good to me and Honza confirmed that the changes are
backwards-compatible.

[PATCH 4/4] SSH: Do not skip domains with use_fully_qualified_names
Ack
Why were the domains skipped in the first place?



More information about the sssd-devel mailing list