[PATCH] IPA: fix override with the same name

[Sumit Bose]

Hi,

this patch fixes an issue in the IPA override code described in
https://fedorahosted.org/sssd/ticket/2874. If the user name is
overridden with itself the name is written to the alias attribute twice
causing an ldb error.

bye,
Sumit

[Jakub Hrozek]

On Fri, Nov 20, 2015 at 12:58:29PM +0100, Sumit Bose wrote:
> Hi,
>
> this patch fixes an issue in the IPA override code described in
> https://fedorahosted.org/sssd/ticket/2874. If the user name is
> overridden with itself the name is written to the alias attribute twice
> causing an ldb error.
>
> bye,
> Sumit

ACK

the login overrides also still work fine.

I'll push the patch when CI finishes.

[Jakub Hrozek]

On Fri, Nov 20, 2015 at 02:42:51PM +0100, Jakub Hrozek wrote:
> On Fri, Nov 20, 2015 at 12:58:29PM +0100, Sumit Bose wrote:
> > Hi,
> >
> > this patch fixes an issue in the IPA override code described in
> > https://fedorahosted.org/sssd/ticket/2874. If the user name is
> > overridden with itself the name is written to the alias attribute twice
> > causing an ldb error.
> >
> > bye,
> > Sumit
>
> ACK
>
> the login overrides also still work fine.
>
> I'll push the patch when CI finishes.

CI: http://sssd-ci.duckdns.org/logs/job/33/63/summary.html

[Jakub Hrozek]

On Fri, Nov 20, 2015 at 02:48:07PM +0100, Jakub Hrozek wrote:
> On Fri, Nov 20, 2015 at 02:42:51PM +0100, Jakub Hrozek wrote:
> > On Fri, Nov 20, 2015 at 12:58:29PM +0100, Sumit Bose wrote:
> > > Hi,
> > >
> > > this patch fixes an issue in the IPA override code described in
> > > https://fedorahosted.org/sssd/ticket/2874. If the user name is
> > > overridden with itself the name is written to the alias attribute twice
> > > causing an ldb error.
> > >
> > > bye,
> > > Sumit
> >
> > ACK
> >
> > the login overrides also still work fine.
> >
> > I'll push the patch when CI finishes.
>
> CI: http://sssd-ci.duckdns.org/logs/job/33/63/summary.html

* master: aedc71fe8360a51785933523f14bb5c4e7e2c38b
* sssd-1-13: 957ec390252128d89479606341ebd7f4f35d785f

[Lukas Slebodnik]

On (20/11/15 14:55), Jakub Hrozek wrote:
> On Fri, Nov 20, 2015 at 02:48:07PM +0100, Jakub Hrozek wrote:
> > On Fri, Nov 20, 2015 at 02:42:51PM +0100, Jakub Hrozek wrote:
> > > On Fri, Nov 20, 2015 at 12:58:29PM +0100, Sumit Bose wrote:
> > > > Hi,
> > > >
> > > > this patch fixes an issue in the IPA override code described in
> > > > https://fedorahosted.org/sssd/ticket/2874. If the user name is
> > > > overridden with itself the name is written to the alias attribute twice
> > > > causing an ldb error.
> > > >
> > > > bye,
> > > > Sumit
> > >
> > > ACK
> > >
> > > the login overrides also still work fine.
> > >
> > > I'll push the patch when CI finishes.
> >
> > CI: http://sssd-ci.duckdns.org/logs/job/33/63/summary.html
> master: aedc71fe8360a51785933523f14bb5c4e7e2c38b
> sssd-1-13: 957ec390252128d89479606341ebd7f4f35d785f

and also

sssd-1-12:
* cd126cd5dd82b62b6303bfbc5ab3cbf859fc5245

LS