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

Reply via email to