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