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.