[PATCH] IPA: Don't fail the request when BE doesn't find the object

[Jakub Hrozek]

Hi,

see the attached patch. To reproduce, just run 'getent group
nosuchgroup(a)ad.trusted.domain' and make sure that with the patch
applied, groups_get_done() or users_get_done() also run and attempt to
delete the nonexistant user or group.

Additionally, there is a variable called user_dom in the codepath which
handles both users and group. Shall I rename it to obj_dom or similar?

[Pavel Reichl]

On 10/29/2014 09:26 PM, Jakub Hrozek wrote:
>  Hi,
>
>  see the attached patch. To reproduce, just run 'getent group
>  nosuchgroup(a)ad.trusted.domain' and make sure that with the patch
>  applied, groups_get_done() or users_get_done() also run and attempt to
>  delete the nonexistant user or group. 
ACK
>  Additionally, there is a variable called user_dom in the codepath which
>  handles both users and group. Shall I rename it to obj_dom or similar? 
Renaming
sounds OK to me.
>  _______________________________________________
>  sssd-devel mailing list
>  sssd-devel(a)lists.fedorahosted.org
>  https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

[Jakub Hrozek]

On Thu, Oct 30, 2014 at 04:19:16PM +0100, Pavel Reichl wrote:
>  
>  On 10/29/2014 09:26 PM, Jakub Hrozek wrote:
>  >Hi,
>  >
>  >see the attached patch. To reproduce, just run 'getent group
>  >nosuchgroup(a)ad.trusted.domain' and make sure that with the patch
>  >applied, groups_get_done() or users_get_done() also run and attempt to
>  >delete the nonexistant user or group.
>  ACK 
* master: 7b5a077bd162cb2340d5d4e9f5dcc0dd00709f1a

>  >
>  >Additionally, there is a variable called user_dom in the codepath which
>  >handles both users and group. Shall I rename it to obj_dom or similar?
>  Renaming sounds OK to me. 
OK, I'll send another patch.

>  >
>  >
>  >_______________________________________________
>  >sssd-devel mailing list
>  >sssd-devel(a)lists.fedorahosted.org
>  >https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>   
>  _______________________________________________
>  sssd-devel mailing list
>  sssd-devel(a)lists.fedorahosted.org
>  https://lists.fedorahosted.org/mailman/listinfo/sssd-devel