On (14/04/16 15:28), Pavel Březina wrote:
>On 04/14/2016 02:36 PM, Lukas Slebodnik wrote:
>>On (14/04/16 14:07), Pavel Březina wrote:
>>>Hi,
>>>good catch.
>>>
>>>On 04/14/2016 10:27 AM, Luka
>>>> DLIST_FOR_EACH(listitem, list) {
>>>> cmdgroup = ipa_sudo_conv_lookup(conv->cmdgroups, listitem->dn);
>>>>+ if (cmdgroup == NULL) {
>>>>+ DEBUG(SSSDBG_CRIT_FAILURE,
>>>>+ "ipa_sudo_conv_lookup failed for DN:%s\n", listitem->dn);
>>>>+ continue;
>>>
>>>If you consider it a critical failure we should return here, so I'd recommend
>>>using SSSSDBG_MINOR_FAILURE if we will just skip it. I'm fine either way.
>>SSSSDBG_MINOR_FAILURE should be enough.
>>
>>updated version is attached.
>>
>>LS
>>
>Ack.
http://sssd-ci.duckdns.org/logs/job/41/92/summary.html
master:
* e547eb597ade731f49b679ce264bbfd907363ff8
LS
_______________________________________________
sssd-devel mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]
