On 01/13/2016 05:12 PM, Sumit Bose wrote:

@@ -1018,7 +1019,6 @@ rules_iterator(hash_entry_t *item,

      if (ctx == NULL) {
          DEBUG(SSSDBG_CRIT_FAILURE, "Bug: ctx is NULL\n");
-        ctx->ret = ERR_INTERNAL;
          return false;
      }

@@ -1066,7 +1066,6 @@ cmdgroups_iterator(hash_entry_t *item,

      if (ctx == NULL) {
          DEBUG(SSSDBG_CRIT_FAILURE, "Bug: ctx is NULL\n");
-        ctx->ret = ERR_INTERNAL;
          return false;
      }


it looks like the last 2 hunks are not available in the latest tar ball.

Sorry about that. Added.

and there are some trailing whitespaces in patch 9 at line 41.

Fixed by git.

I also attached two more patches then converts usn into number and then use in filter entryUSN >= (current + 1) instead of (entryUSN >= cur) && (entryUSN != cur) as per Thierry request.

It could probably be squashed and replace some previous patches but I think sending it as new patches is easier for you to review, given the previous patches are basically acked.

Attachment: patches.tgz
Description: GNU Zip compressed data

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

Reply via email to