[SSSD] [PATCH] SSH: Add dp_get_host_send to common responder code

Stephen Gallagher sgallagh at redhat.com
Tue May 1 18:47:34 UTC 2012


On Wed, 2012-04-25 at 14:52 +0200, Jakub Hrozek wrote:
> On Tue, Apr 24, 2012 at 11:31:10AM +0200, Jakub Hrozek wrote:
> > [PATCH] Rename split_service_name_filter
> > The function was used outside services code which was confusing due to
> > its name. This patch renames it to sound more netrual.
> > 

Ack.

> > [PATCH] SSH: Add dp_get_host_send to common responder code
> > Instead of using account_info request, creates a new ssh specific
> > request. This improves code readability and will make the code more
> > flexible in the future.
> > 
> > https://fedorahosted.org/sssd/ticket/1176
> > 
> 

Nack.

I don't like the construct you added to the "Invalid Filter" section of
be_host_handler(). For one thing, ret is checked without a guarantee it
has been set (it's set coincidentally further up, but we cannot rely on
that). For another, it's identical to the handling for when the
strncmp() fails. Please combine them and make sure that ret is
initialized properly.

You're not freeing filter in sss_dp_get_ssh_host_msg() if
dbus_message_new_method_call() returns NULL.



> Attached patches are rebased on top of the subdomains patches.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20120501/d1589d2a/attachment.sig>


More information about the sssd-devel mailing list