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