On Wed, 2011-07-20 at 12:49 +0200, Jan Zelený wrote:
> On 07/19/2011 05:15 PM, Jan Zelený wrote:
> >>
https://fedorahosted.org/sssd/ticket/911
> >>
> >> [PATCH 1/2] Do not add a NULL host parsed from LDAP URI
> >> Do not add a server into fail over in case its host name is NULL. Also
> >> fixes a possible memleak in case be_fo_add_server() failed.
> >>
> >> [PATCH 2/2] Only print server address if one is available
> >> The second patch is kinda related. In fail over, it is possible that
> >> fo_get_server_hostent() returns NULL in case we did not find any name
> >> for the server. Therefore we must check its return value before
> >> dereferencing it.
> >
> > Nack, there is a compilation error caused by the second patch.
> >
> > Jan
>
> Oops, that code should have been in a later patch, bad rebasing.
>
> New patches attached.
Ack to both of them
Pushed to master
Simo.
--
Simo Sorce * Red Hat, Inc * New York