On Thu, 2011-07-21 at 08:41 -0400, Simo Sorce wrote:
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
Pushed to 1-5 too.
Simo.
--
Simo Sorce * Red Hat, Inc * New York