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.
_______________________________________________
sssd-devel mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]
