diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2014-07-25 17:39:54 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-07-29 14:03:32 +0200 |
| commit | 1b72f6377e997bbadabad9e5e43998dddfe38156 (patch) | |
| tree | 13997225a3bc866e148fb15d8f68c5383307ccc5 /src/providers/ad/ad_gpo.c | |
| parent | 210bc14aac190bbb043272821aeb3342c995a4f6 (diff) | |
| download | sssd-1b72f6377e997bbadabad9e5e43998dddfe38156.tar.gz sssd-1b72f6377e997bbadabad9e5e43998dddfe38156.tar.xz sssd-1b72f6377e997bbadabad9e5e43998dddfe38156.zip | |
AD: Increment som_index when advancing to the next GPO
When the GPO processing tried to advance to the next GPO on encountering
an error, the som_index was not incremented, causing an infinite loop.
Reviewed-by: Yassir Elley <yelley@redhat.com>
Diffstat (limited to 'src/providers/ad/ad_gpo.c')
| -rw-r--r-- | src/providers/ad/ad_gpo.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/providers/ad/ad_gpo.c b/src/providers/ad/ad_gpo.c index 7c05afa39..68a612e76 100644 --- a/src/providers/ad/ad_gpo.c +++ b/src/providers/ad/ad_gpo.c @@ -2000,6 +2000,7 @@ ad_gpo_get_som_attrs_done(struct tevent_req *subreq) } if ((num_results < 1) || (results == NULL)) { DEBUG(SSSDBG_OP_FAILURE, "no attrs found for SOM; try next SOM.\n"); + state->som_index++; ret = ad_gpo_get_som_attrs_step(req); goto done; } else if (num_results > 1) { @@ -2020,6 +2021,7 @@ ad_gpo_get_som_attrs_done(struct tevent_req *subreq) if ((ret == ENOENT) || (el->num_values == 0)) { DEBUG(SSSDBG_OP_FAILURE, "no attrs found for SOM; try next SOM\n"); + state->som_index++; ret = ad_gpo_get_som_attrs_step(req); goto done; } |